שלום, רציתי לדעת איך אפשר לקלוט משפט בשפת C?
תודה לעוזרים!!

http://www.fresh.co.il/vBulletin/sh...anf#post2880880

הראל

אתה מתכוון לקלוט משפט מקובץ או מהמקלדת?
אם זה מקובץ, אז הלינק למעלה של הראל - סוכר
אם סתם מהמקלדת, אתה יכול לקלוט עם gets

char str[50]
printf ("enter a string: \n");
gets (str);

hזה יכניס לך את המשפט לתוך המחרוזת str.
בהצלחה

בברכה, מתן.
www.MatanNarkiss.com

וזה גם מתכון מושלם ל buffer overflow, להשתמש ב gets. ציטוט מה man page של משפחת הפונקציות בלינוקס:

קוד:

BUGS
       Never use gets().  Because it is impossible to tell without knowing the
       data in advance how many  characters  gets()  will  read,  and  because
       gets() will continue to store characters past the end of the buffer, it
       is extremely dangerous to use.  It has  been  used  to  break  computer
       security.  Use fgets() instead.

כמובן, אם אתה מתכנת בחברת מיקרוסופט, אתה רשאי להתעלם מהאזהרה הנ"ל...

נמאס לכם לזכור סיסמאות? לחצו כאן!

איך אתה משתמש ב fgets בקליטה מהמקלדת?

בברכה, מתן.
www.MatanNarkiss.com

מחפש בגוגל man fgets בתור צעד ראשון.

קוד:

enum {
	size = 103
}
char buf[size];

fgets(buf, size, stdin);

ובצעד השני, נמנע, למרות הקושי הרב, מפרצות אבטחה מביכות בקוד שלך.

האמת שעוד לא עבדנו עם enum... אני מבין שהוא עובד בצורה דומה לקובץ לצורך העניין?
או שיכלתי להשתמש במחרוזת כלשהי מבלי להשתמש ב enum?

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

בעריכה: הבנתי מה עשית... אבל למה השתמשת ב enum?
אם הייתי מגדיר פשוט את size בתור משתנה int רגיל, זה לא היה תופס?

בברכה, מתן.
www.MatanNarkiss.com

קוד:

 unsigned int size = 103;
char *buf = new char[size];

fgets(buf, size, stdin);

delete buf;

שזה, כמובן, לא משנה לעצם השימוש ב-fgets.

רק אני הייתי משתמש פשוט בפונקציה sizeof על המשתנה שלתוכו אני קורא כדי לקבוע את הפרמטר של מספר הבתים לקריאה?

נמאס לכם לזכור סיסמאות? לחצו כאן!

ואגב, sizeof הוא אופרטור ולא פונקציה.

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

לגבי sizeof, אתה כמובן צודק. (השעה, השעה... )

נמאס לכם לזכור סיסמאות? לחצו כאן!

sizeof לא יפעל לגבי פרמטרים שקיבלת, כי בניגוד למערכים סטטיים שיצרת במקום, הפרמטרים הם פויינטרים, ו-sizeof עליהם יחזיר גודל של פוינטר, וזה לא עוזר לנו. זה מה שלא מאפשר לכתוב משהו כזה:

קוד:

#define COOL_FGETS(buf) fgets(buf, sizeof(buf), stdin)

במקרה הכללי יש שני דברים שאפשר לעשות - מה שעשיתי למעלה, שהוא ליצור קבוע, ובעזרתו לייצר את המשתנה, ואותו לספק כגודל של המשתנה (אבל מה יקרה אם אני אטעה?), או הפיתרון הטוב יותר, שמאפשרת לי CPP (וגם C בצורות מסוימות), להשתמש במחלקה שמכמסת את המחרוזת ואת הגודל שלה. גם ב-C אפשר לעשות משהו כזה:

קוד:

typedef _coolstr {
	char *str;
	unsigned long int size;
} coolstr;

coolstr *allocstring(unsigned long int size);
void freestring(coolstr *str);

bool coolfgets(coolstr *str, FILE *stream) {
	return (fgets(str->str, str->size, stream) != NULL);
}

וכאן כבר אי-אפשר לטעות.

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

בברכה, מתן.
www.MatanNarkiss.com

new זו הדרך לבצע הקצאה דינאמית ב ++C, ושחרור הקצאה נעשה עם delete (במקום free). זה הרבה יותר נוח...

נמאס לכם לזכור סיסמאות? לחצו כאן!

וואו, כן.. .זה נראה הרבה יותר נוח...
אם אני משתמש ב new ב C, זה לא יעבוד?

האם ב cpp אפשר להשתמש ב malloc או שה- new בא בעצם לבטל את כל הבלאגן שביצירת הקצאה בצורה המפרכת:

קוד:

int size = 103;
char* ptr;
ptr = (char*) malloc (size * sizeof(char));

בברכה, מתן.
www.MatanNarkiss.com

ניתן לקמפל קוד C באמצעות קומפיילר ++C...

אם תנסה לקמפל קוד עם new בקומפיילר C הוא יגיד לך ש new הוא undeclared...

נמאס לכם לזכור סיסמאות? לחצו כאן!

הבנתי, ברור

בברכה, מתן.
www.MatanNarkiss.com

new עלול לזרוק exceptions במקום פשוט להחזיר NULL מה שעושה את הקוד יותר מסורבל ופחות אופטימלי (אבל למי אכפת מאופטימליות בימנו).

[התמונה הבאה מגיעה מקישור שלא מתחיל ב https ולכן לא הוטמעה בדף כדי לשמור על https תקין: http://www.fresh.co.il/scripts/birthday.php?date=27/04/1986]

חפש בגוגל new nothrow וכנראה תלמד משהו חדש.

מצד שני, new ו-delete קוראים ל-constructor-ים ול-destructor-ים, ו-xalloc ו-free לא.

מה עדיף?

קוד:

C *p = new(std::nothrow) C;
if (p == NULL) {
	whatever;
}
...
delete p;

או הקוד הנפלא שאתה מציע לנו:

קוד:

C *p = static_cast<C*>(malloc(sizeof(C)));
if (p == NULL) {
	whatever;
}
p->C::C();
...
p->~C();
free(p);

וזה אפילו לא קוד תקני, אלא טריק שעובד ב-VCPP7. כדי לקרוא ל-constructor על זיכרון שהוקצה ידנית, תצטרך להשתמש ב-placement new:

קוד:

C *p = static_cast<C*>(malloc(sizeof(C)));
if (p == NULL) {
	whatever;
}
C *p2 = new(p) C();
...
p->~C();
free(p);

אם אתה רוצה לקרוא ל-constructor, אתה חייב להשתמש ב-new. מצד שני, new לא חייב לזרוק exception אם ההקצאה נכשלה. אתה יכול לבחור איזו התנהגות שלו אתה מעדיף. ברכות.

למה להסתבך כ"כ, תגדיר את size כ define ויש לך קוד הרבה פחות מסובך (וגם ללא הקצאה דינמית מיותרת):

קוד PHP:

 #define MAX_CHARS 103
char buf[MAX_CHARS] = {0};
fgets(buf, size, stdin); 


הראל

אבל תודה על הטיפ.

באגים בהגדרת קבועים כ define? אשמח להרחבה בנושא.

הראל

הבעיה ה"קלה" היא שהקומפיילר לא מודע לקיומם של מאקרואים, ולכן הם לא נכנסים ל-symbol table, וזה יהיה ממש כיף לדבג קוד שמלא ב-magic numbers שאין לך מושג מה המשמעות שלהם, גם אם מדובר בבילד debug.

אבל הבעיה האמיתית חמורה יותר - define הוא לא חלק מהשפה מבחינת הקומפיילר: ל-define-ים אין טיפוס, הם לא מצייתים לחוקי scoping, ועוד ועוד.

בגוגל ניתן למצוא באגים משעשים כמו זה:

קוד:

const int NUMBER = -42;
int main()
{
	int x = -NUMBER;
}

בנוסף לבאגים קלאסיים כמו זה:

קוד:

#define NUMBER 5+2;

int x = 3 * NUMBER;

וסתם קטעים מעצבנים כמו:

קוד:

#define MACRO 1.653

const double CONST = 1.653;

double Square(double const *p)
{
	return (*p) * (*p);
}

int main()
{
	double a = Square(&MACRO);  // won't compile

	double b = Square(&CONST);  // compiles OK

	return 0;
}

ואת אלה הוצאתי מאשכול אחד בלבד מחמישה שפתחתי מגוגל.

define זה נורא - אל תשתמש בזה.

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

בברכה, מתן.
www.MatanNarkiss.com

אני רואה את הבעייתיות בדוגמאות שנתת, אבל זאת בעיתיות שקל (יחסית) להמנע ממנה, וספציפית עבור המקרה של Narxx נראה לי שהיא מפשטת מאוד את הפיתרון.
יש מקומות בהם define הוא לא רעיון טוב, במיוחד כשהערך שלו הוא חלק מחישוב, אבל עבור מחרוזות קבועות או "מספרי קסם" שחוזרים הרבה בקוד define הוא פיתרון די שימושי לדעתי.

הראל

ההצעה שלך מוזרה מאוד: להשתמש ב-const/enum בכל מקום חוץ מבמחרוזות ובמספרים שלא משמשים בחישובים. רע מאוד. לא יודע מאיפה אתה מביא רעיונות כאלה: אתה גם יוצר קוד לא אחיד ללא שום סיבה מוצדקת, גם מציע חוק נורא מסובך כדי כן להבדיל בקושי בין המקרים (שלא יפעל, ובעולם האמיתי קונבנציות זה דבר חשוב), וחמור עוד יותר - מניח שאתה יודע מה המשתמש יעשה עם קבצי ה-H והקבועים שלך. מאיפה לך לדעת האם הקבוע ישמש בחישובים או לא? אולי תכתוב קוד שפועל תמיד ולא רק לפעמים? נשמע לי כמו רעיון לא רע.

חחחח

#Narxx:

אני לא מבין גדול ב-C, אבל יש כאן דוגמה:
http://www.cplusplus.com/reference/...tdio/fgets.html

לעבוד עם fgets אני יודע... בדוגמא שלך יש סריקה מקובץ.
אני שאלתי איך עובדים עם fgets כאשר רוצים לסרוק מהמקלדת, ולא מקובץ (כמו שמאבטח ענה, אבל לא לגמרי הבנתי את הפתרון. כשיענה על השאלה שלי, כנראה שאבין ).

בברכה, מתן.
www.MatanNarkiss.com

קודם כל, הקוד שלך לא עובר קומפילציה בכלל כי אין לך נקודה פסיק אחרי ההגדרה של str.

דבר שני, אחרי שהוספנו נקודה פסיק במקום הדרוש, אם הקומפיילר שבו אתה משתמש לא עושה מה שהקומפיילר שאני משתמש בו עושה בעקבות הקוד שלך, תחליף קומפיילר:

קוד:

$ cat narxx.c
#include <stdio.h>

int main ()
{
char str[50];
printf ("enter a string: \n");
gets (str);
return 0;
}
$ gcc -Wall narxx.c -o narxx
/tmp/ccfYk93k.o: In function `main':
narxx.c:(.text+0x17): warning: the `gets' function is dangerous and should not be used.

נמאס לכם לזכור סיסמאות? לחצו כאן!

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

בברכה, מתן.
www.MatanNarkiss.com