שלום חבר'ה,

אתם בטח מכירים את התופעה בשפת C בה אתם מדפיסים מחרוזת שקלטתם
ולפתע אחרי ההדפסה בתוך ממשק הקונסול מופיע בהמשך מחרוזת של סימני ג'יבריש.
איך נפתרים או מונעים את זה ?
קראתי משהו לגבי buffer flash משהו כזה לגבי ניקוי זכרון ולא הבנתי כל כך למישהו יש
מושג לגבי כל המניפולציה הזו של ג'יבריש בסטרינגים בחלון הקונסול ?
תודה מראש.

כי ביתי בית תפילה יקרא לכל העמים

צריכה להיות סיבה שזה קורה... זה לא סתם .

כשאתה כותב char string[20] אז המחשב מקצה לך 20 מקומות בזיכרון אבל הוא לא מאתחל אותם כלומר יש בהם תוכן. אם אתה קולט רק 10 מתוך ה 20 מקומות אז נשארו לך עוד 10 מקומות לא מאותחלים ושאתה מדפיס את כל המחרוזת אתה מדפיס גם את 10 המקומות הלא מאותחלים וזה יוצר גיבריש.

קיצר הפתרון הוא לאפס את כל המחרוזת לפני שאתה מבצע אליה קלט. כלומר לשים בכל תו ותו במחרוזת את הערך : ' '
מקווה שהבנת.

http://myspace.com/streetanimal

הלהקה הכי טובה בארץ!

נסה להריץ את הקוד הבא (מ visual studio 2005 ומעלה):
תוכנית קונסול. התוכנית מבצעת הקצאה דינאמית של הכנסת נתוני סטודנטים לטבלה
ע"י הקשת קוד סטודנט, רווח, שם סטודנט ואז לחיצה על Enter
לאחר שאני מזין למשל שדה ראשון ולאחר מכן בוחר בתפריט 2 שזה להציג
את נתוני הטבלה עד עכשיו אני מקבל את מה שהזנתי בנוסף לג'יבריש.
אשמח מאוד אם מישהו ימצא לזה פתרון.
הנה יש לכם אתגר מאחר והפתרון הקודם לא עוזר !
אני יודע שיש איזו שורת קוד שעושה סוג של ניקוי מאגר זכרון אשר אמור למנוע
תופעה זו אך שכחתי מה הקוד ואם יש לכם דרך אחרת אשמח לדעת.
תודה מראש.

קוד:

#include "stdafx.h"
#include "string.h"
#include "malloc.h"

struct stu
{
	char id[10];
	char name[10];
};

struct stu *table;
int size;

int menu()
{
	int choice;

	printf("Menu:\n0.Exit\n1.Add new\n2.Show table\n");
	scanf("%d",&choice);
	
	return choice;
}

void add()
{
	char id[10],name[10];
		
	printf("Please type student id and student name: ");
	scanf("%s%s",id,name);
		
	table=(struct stu*)realloc(table,sizeof(struct stu)*size);
	strcpy(table[size-1].id,id);
	strcpy(table[size-1].name,name);
	size++;
}

void print()
{
	int i;

	for(i=0;i<size;i++)
		printf("%s %s\n",table[i].id,table[i].name);
}

void main()
{
	int i;
	
	table=NULL;
	size=1;
	i=menu();

	while(i)
	{
		if(i==1) add();
		else
		if(i==2) print();		
		
		i=menu();
	}
}

כי ביתי בית תפילה יקרא לכל העמים

לא נראה לי שזה הגיוני לעשות scanf("%s%s")
איך הפונקציה תדע מתי להפסיק את המחרוזת הראשונה ולהתחיל את השנייה? היא לא קוראת את מחשבות המשתמש..
תשים רווח ביניהם, או פסיק או משהו להפריד, ואז אולי לא תקבל בעיות בהדפסה.

לא עברתי על כל הקוד שלך, אבל אני לא דיבאגר. תעבור עם דיבאגר ותראה איפה נכנס זבל למחרוזת שלך, ואיפה אתה לא סוגר את המחרוזת עם \0

!!אזהרה!!

ההודעה עלולה להכיל שברי אגוזים ו/או איברי דגים כלשהם

!!אזהרה!!

ציטוט:

במקור נכתב על ידי yoavmatchulsky לא נראה לי שזה הגיוני לעשות scanf("%s%s") איך הפונקציה תדע מתי להפסיק את המחרוזת הראשונה ולהתחיל את השנייה? היא לא קוראת את מחשבות המשתמש.. תשים רווח ביניהם, או פסיק או משהו להפריד, ואז אולי לא תקבל בעיות בהדפסה. לא עברתי על כל הקוד שלך, אבל אני לא דיבאגר. תעבור עם דיבאגר ותראה איפה נכנס זבל למחרוזת שלך, ואיפה אתה לא סוגר את המחרוזת עם \0

מותר לעשות scanf %s%s מאחר וכאשר אתה משתמש בפקודת scanf לקליטת מחרוזת ואח"כ
אתה עושה רווח זה נחשב כאילו שעשית ENTER וסיימת לקלוט מחרוזת ראשונה ואח"כ אותו
דבר אתה קולט את השניה רק שבסוף אתה מקיש ENTER.
מהתשובה שאתה מציג לא הרצת בכלל את הקוד. נכון שמומלץ יותר להשתמש בפקודת gets
שבה אתה יכול גם להקיש רווח במחרוזת וכך יצא משפט. אך כאן לא הייתי צריך gets מאחר
וה s% הראשון קולט ברצף מספרים ללא רווחים ביניהם ואח"כ אתה לוחץ רווח שזה כאילו ENTER
ואתה קולט את שם הסטודנט שזה ה s% השני.
תריצו את הקוד לפני שאתם באים להתעמת איתי ותראו שהתוכנית אכן עובדת אך מדוע גם
נוסף הג'יבריש ?
תודה.

כי ביתי בית תפילה יקרא לכל העמים

זה קורה לך גם כשאתה מזין שם ומזהה באורך קצר מ 10 או רק כשהם ארוכים יותר?
בכל מקרה יש לך כאן בעיית overflow חמורה (אם מכניסים לאחד המשתנים ב scanf יותר מ 10 תווים ההתנהגות של התוכנית תהפוך לבלתי צפוייה).
כדי למנוע את גלישת הזיכרון אתה יכול לבקש מ scanf לקרוא לכל היותר 10 תווים עבור כל משתנה (ע"י שימוש ב %10s).
בנוסף, מאותה סיבה מומלץ להשתמש ב strncpy ולא ב strcpy.

הראל

ציטוט:

במקור נכתב על ידי rlsf זה קורה לך גם כשאתה מזין שם ומזהה באורך קצר מ 10 או רק כשהם ארוכים יותר? בכל מקרה יש לך כאן בעיית overflow חמורה (אם מכניסים לאחד המשתנים ב scanf יותר מ 10 תווים ההתנהגות של התוכנית תהפוך לבלתי צפוייה). כדי למנוע את גלישת הזיכרון אתה יכול לבקש מ scanf לקרוא לכל היותר 10 תווים עבור כל משתנה (ע"י שימוש ב %10s). בנוסף, מאותה סיבה מומלץ להשתמש ב strncpy ולא ב strcpy.

אני בכלל לא מנסה להזין יותר מ 10 מאחר ואני יודע שזה יחרוג מגבולות המחרוזת.
תודה על העזרה אך גם אתה לא הרצת את הקוד ע"פי התשובה.

כי ביתי בית תפילה יקרא לכל העמים

ולכן אני אענה יותר לגופו של עניין...

הבעיה שלך היא בשורת ה-for..
שים לב, זה מה שעשית:

קוד:

for(i=0;i<size;i++)

כאשר בכל איטרציה אתה מגדיל את size ב-1 בלי כל קשר, משמע לאחר שנקלט האדם הראשון ה-size כבר גדל ל-2 אנשים וכך נוצר המצב בו אתה בא להדפיס מקום שכלל אינו אותחל, כלומר המקום של האדם השני שמעולם לא נקלט.

הפיתרון:

קוד:

for(i=0;i<size - 1;i++)

מסקנה:
אני ממליץ בחום לאתחל את כל המשתנים שלך בתחילת התוכנית כך שלא יכילו זבל לא ברור. מה שקורה זה שהמקום לא אותחל. ה"סטרינגים" שלך הם למעשה פוינטרים למקום כלשהו בזיכרון שהוא char, אבל המחשב לא שומר בשום מקום את אורכו ולכן הדרך היחידה לזהות את סופו היא לרוץ בלולאה עד שמגיעים ל-NULL הראשון. התוצאה היא אינה "גיבריש", אלא רצף בתים אקראי ששכב בזיכרון שהתחלת לקרוא ממנו עד לאיזה NULL שנמצא גם הוא ממש במקרה איפשהו בזיכרון. לכן אם המקום היה מאותחל ל-0-ים כבר בהתחלה אז אומנם הייתה עדיין קיימת השגיאה אבל היה מודפס סטרינג ריק שזה קצת פחות נורא..

תודה רבה דורון פתרת לי את הבעיה.
איך באמת לא שמתי לב שהייתי צריך לעשות size-1 הרי זה כל כך ברור.
מאחל לכולכם שנה טובה ואל תשכחו:
The future is software
יש סוד גדול שטמון במשפט הזה. ביל גייטס זרק פצצה לאויר כשאמר אותו !

כי ביתי בית תפילה יקרא לכל העמים

סתם צוחק
על לא דבר ושנה טובה ומתוקה לכולם .