שלום,
כתבתי תוכנית שמקבלת 50 מספרים (שנגמרת אחרי 50 מספרים או אחרי שהוקש 1-) ובתוך המערך ניסיתי לבדוק את הסידרה היורדת הארוכה ביותר ולהגיד מאיזה מספר התחילה סידרה זו.

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

קוד:

#include <iostream.h>
void main()
{
	int arr[50];
	int i=0,counter=0,countermax=0,flag=0;
	cout<<"Enter 50 numbers\n";
	while ((i<50) && (arr[i-1]!=-1))
	{
		cin>>arr[i];
		if (arr[i]==-1)
			flag=i;
		i++;
	}
	for (i=0;i<50;i++)
	{
		if (arr[i]>arr[i+1])
			counter++;
		else
		{
			if (counter>countermax)
				countermax=counter;
			counter=0;
		}
		if (counter>countermax)
			countermax=counter;
	}
	if (flag==0)
		cout<<"the biggest down series starts with the number:"<<arr[i-countermax-1]<<"\n";
	else
		cout<<<<arr[flag-countermax];
}

תודה מראש

למה יש לך שם 4 >>>> ?

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

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

טוב, נתחיל קצת באיכות הקוד, אתה כותב ב-++C, נכון?

אז ככה: כשאתה מצרף קובץ ספרייה לקוד שלך, ב-++C אין סיומת .h.
את הסיומת הזו ירש ה-++C מה-C, וזה שם רק כדי ליצור תאימות מסויימת בין השפות.
לכן רצוי להשתמש ב:

קוד:

#include <iostream>

דבר שני, הספריות של ++C עצמן מכילות חלוקה שנקראת namespaces.
לפונק' ב-++C יש לא רק שם, אלא גם שם משפחה.
רב הפונק' בספריות הסטנדרטיות כולן נמצאות במשפחת ה-std.
כעקרון, רצוי להכניס כל דבר שאתה עומד להשתמש בו בנפרד:

קוד:

using std::cout;
using std::cin;

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

קוד:

using namespace std;

---

חוץ מהעובדה שהספריות של ++C קצת שונות מ-C משהו אחד זהה: פונק' ה-main היא פונק' מטיפוס INT, לא VOID!
היא מחזירה ערך למערכת ההפעלה (בד"כ 0).

קוד:

int main()

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

קוד:

int i=0,counter=0,countermax=0,flag=0;

אתה סתם מעביד את המחשב בקוד הזה.
++C עושה הפרדה ברורה בין איתחול משתנים, לבין השמה למשתנים.
מה שאתה עושה פה זה השמה!
זה אמור היה להראות ככה:

קוד:

int i(0),counter(0),countermax(0),flag(0);

זה איתחול.
=========================

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

קוד:

flag=i-1; // was flag=i;

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

קוד:

			if (counter>countermax) {
				countermax=counter;
				maxindex = i;
			

*אל תשכח להגדיר את המשתנה הנוסף maxindex...

בנוסף, יש לך משום מה 2 IFs שעושים בדיוק אותו הדבר, בזה אחר זה... בשביל מה?!

קוד:

else
        {
			if (counter>countermax) {
				countermax=counter;
				maxindex = i;
			}
            counter=0;
        }
        if (counter>countermax) // Why is this if here?!
            countermax=counter; // Delete it...

לבסוף, יש לך IF לחלוטין מיותר:

קוד:

if (flag==0)

זה אומר שהמשפט יודפס רק אם יקלטו כל 50 המספרים?
למה..?

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

הבנתי תודה רבה
ואנחנו משום מה לומדים C בסינטקס של ++C , אולי בגלל זה הקוד נראה לך קצת מוזר?

וואהו!