scanf הוא רצח :/
הוא פשוט הורג אותי, אני לא מצליח להבין את ההגיון בו, ואם אני כן מבין את ההגיון אז הוא טפשי
נתון הקוד הבא:

קוד:

do
	{
		printf ("Please select the number of turning on and off which causes the led to burn: \n");
		input=scanf ("%d",&k);
		if ((input<1)||(k<1))
			printf ("Please enter a postive (>0) integer only!\n");
	}
	while ((input<1)||(k<1));

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

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

אתה צריך להבין שה-scanf מוציא את המידע ממקום כלשהו. המקום הזה, הוא לא ההתקן של המקלדת עצמו אלא חוצץ (buffer) שמספקת מערכת ההפעלה (אשר אכן קוראת מהמקלדת ישירות ומציבה את המידע אל תוך החוצץ).

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

קוד PHP:

 void ClearBuff(void) {
   char c;
   do {
      scanf("%c", &c);
   } while (c != '\n');
} 


מכיוון שהתו האחרון בחוצץ הוא בד"כ תו ירידת השורה (שנכנס כשלוחצים על ENTER) ואשר סימונו הוא n\ ניתן להניח כי זה יהיה התו האחרון בחוצץ ולכן למעשה הפונקציה הזו קוראת את כל התווים בחוצץ.
זה לא באמת פונקצית ניקוי חוצץ תקינה ותקנית, אבל זה עושה את העבודה ברב תרגילי הבית...

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

תודה, בסוף עשיתי משהו דומה רק עם fflush (stdin) אני יודע שזה לא אלגנטי במיוחד ואפילו יש שיאמרו לא מומלץ...אבל עד שלא תהיה לי שיטה יותר טובה אני אשתמש בזה.

קוד:

int get_input (int down_lim,int up_lim)
{
	int k,input;
		printf ("Please select the number of turning ON and OFF which causes the led to burn: \n");
		input=scanf ("%d",&k);
		if ((input<1)||(k
		{
			printf ("Please enter a postive integer only!\n");
			fflush(stdin);
			k=get_input(1,MAX_INT); //Recursion
		}
	return k;
}