יש לי בעיה עם תנאי בלולאת FOR ואשמח לקבל עזרה:

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

תודה מראש
יעקב

קוד:

#include <iostream>
using namespace std;
int main()

{
	int i;
	int num[10];

	cout << "Enter 10 netural numbers between 1 to 1000:\n";

	for (i=0 ; i<10 ; i++) {
		cin >> num[i];
	}
		
	for (i=0 ; i<10 ; i++) {
		cout << "  ";
		cout << num[i];
	}

	for(i=0 ; num[i] < num[++i] ; i++){
		cout << "sorted list ";
	}

	cout << " End";
return 0;
}

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

אתה מתעסק פעמיים ב-i באותו משפט. זה אסור. התוצאה של זה לא מוגדרת - כל דבר יכול לקרות.
(אין לי כח להסביר בדיוק את הפרטים הטכניים. חפש בגוגל sequence point.)

מעבר לזה, אפילו אם הקוד היה חוקי הוא לא היה הגיוני, כי אתה מגדיל את i פעמיים בכל איטרציה, וזה לא מה שכתבת שאתה רוצה. הדבר הנכון לכתוב הוא for (size_t i = 0; num[i] < num[i+1]; ++i)‎.

כשאתה כותב num[i] < num[++i]‎ זה big no-no. כאמור, חפש sequence point להסבר מפורט. אתה יכול לראות בערך מה קורה אם תוסיף שורה בסגנון מחוץ לאחת הלולאות, סתם כך:

i=0; cout << endl << num[i] << " " << num[++i] << endl;

לי הודפס במקרה הזה הפלט הבא:

Enter 10 netural numbers between 1 to 1000:
1 2 3 4 5 6 7 8 9 10
2 2
End

GCC מתריע על זה ואומר:

prog.cpp:20:28: warning: operation on ‘i’ may be undefined [-Wsequence-point]
for(i=0 ; num[i] < num[++i] ;++i){

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

אתה בכלל לא נכנס ל for האחרון, אתה מתייחס אליו בתור if וזו טעות (במקרים כאלה צריך לעבוד עם ה debugger).

תנסה לחשוב על זה בצורה קצת אחרת: איך אתה עובר בלולאה על כל איברי המערך ומשווה איבור במקום ה i עם האיבר במקום ה- i+1.


עריכה: וכל זה כמובן בתוספת לפרטים הטכנים שפסטן כתב מעליי.

Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning. -Rick Cook

תודה על התגובות המחכימות.

הבעיה הייתה באמת שהתעסקתי עם ה i פעמיים, ועשיתי פעולה של הצבה פעמיים.

הפתרון לזה היה לעשות בתנאי i+1 במקום i++:

קוד:

for(i=0 ; num[i] < num[i+1] ; i++){		
	}

	if(i==9)
			cout << " sorted list ";
		else
			cout << " not sorted list ";

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

מה מונע ממך לצאת מגבולות המערך?

לא הבנתי את כוונתך, תסביר את השאלה

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

ציטוט:

במקור נכתב על ידי ycsycs תודה על התגובות המחכימות. הבעיה הייתה באמת שהתעסקתי עם ה i פעמיים, ועשיתי פעולה של הצבה פעמיים. הפתרון לזה היה לעשות בתנאי i+1 במקום i++: קוד: for(i=0 ; num[i] < num[i+1] ; i++){ } if(i==9) cout << " sorted list "; else cout << " not sorted list ";

בהינתן שהמספרים בזיכרון ממויינים, הלולאה הזאת ממשיכה עד אינסוף (או עד שהתוכנה קורסת...).
התנאי צריך להיות ‎(i < ARR_SIZE) && (num[i] < num[i+1])‎.

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

עושים ככה:

קוד:

bool isSorted = true;

// notice i skiped the first cell
for(i=1 ; i < SIZE  ; i++)
{
    if (array[i] < array[i-1])
    {
        isSorted = false;
    }
}

אם אתה רוצה שזה יעצור באמצע, תכתוב לולאת WHILE. אפילו BREAK עדיף על FOR שעוצר באמצע. בשפות תכנות נורמאליות אי אפשר לשים תנאי עצירה ב FOR