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

הנה הקישור לתמונה:
http://up193.siz.co.il/up1/t522eny5u2yi.jpg

תודה רבה ..

אני חושב שכדאי שתציין באיזה שפה...

באיזה שפה אתה משתמש? ובמה אתה בעצם מתקשה?
יש לך פונקציה rand או random כמעט בכל שפה. אם מדובר בשפת C למשל, היא תחזיר לך ערך בין 0 ל-32,767
שוב, ב-C לצורך העניין ע"מ להגריל מספר בין -4 ל-num אתה צריך לעשות:

קוד:

int r_num = -4 + (rand()%(num+5));

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

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

צודק,ערכתי,
וציינתי שזה בשפה C++

תוכל בבקשה להראות לי דרך מלאה? אני הסתבכתי איך לבדוק וכל זה .. S:
תודה..

http://www.fresh.co.il/vBulletin/sh...557&postcount=9

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

קוד PHP:

 int* fillRand (int size, int topBound)
{
    int tempNum, *array;
    bool flag = true;
    array = new int[size];
    for (int i = 0 ; i < size ; i++)
    {
        tempNum = rand() % (topBound+4);
        tempNum = tempNum - 4;
        for (int j = 0 ; j < i ; j++)
        {
            if (tempNum == array[j])
            {
                i--;
                j = i;
                flag = false;
            }
        }
        if (flag)
        {
            array[i] = tempNum;
        }
        flag = true;
    }
    return array;
} 


עריכה: תיקנתי את הקוד - עכשיו הוא עובד בוודאות.
בדקתי עם ה main הבא:

קוד PHP:

 #include <iostream>
using namespace std;

int* fillRand (int size, int topBound);

int main()
{
    int size = 10, topBound = 60;
    int *array = fillRand (size, topBound);
    for (int i = 0 ; i < size ; i++)
    {
        cout << array[i] << " ";
    }
    free (array);
    cin >> size;
    return 0;
} 


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

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

כדי להסביר את עצמך יותר טוב הייתי מציע לתת קוד משלך..
למה לשנות מונה של לולאת for זה "לא טוב"?

https://www.youtube.com/watch?v=0HRGczvZINQ&noembed

אתה צריך עוד סיבות?

המשמעות הבסיסית של לולאת FOR היא שעוברים פעם אחת על כל אלמנט באוסף מסוים.

בעריכה, החלטתי להשתעשע:

קוד:

int *fillRand (const int size, const int topBound) {
	int *array = new int[size];
	for (int i = 0 ; i < size ; i++) array[i] = i-4;
	return array;
}

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

קוד PHP:

 int* Nums, MoreNums; 


2. תן שמות משמעותיים למשתנים "דגל" זה לא משמעותי, נהוג לנסח בוליאנים כשאלה.
3. נהוג (בימינו המודרניים, והסעיף הזה זה המלצה) להגדיר משתנים הכי קרוב לשימוש, ולא מראש.
4. לא רושמים סתם מספרים בקוד, צריך להגדיר קבוע או מאקרו.
5. כל הFOR השני צריך להיות WHILE. (אני גם חושב שאתה צריך -1 אבל אני לא בטוח ואין לי כח להתעמק)

6. אסור(!) לשנות איטרטור של FOR זה ברמה של GOTO. מכות כאילו...
7. אין כמעט אף פעם היגיון בהשמה של ערך סתם לתוך בוליאני. זה כמעט תמיד תוצאה של פונקציה לוגית.

8. כל המשחק עם הדגל בסוף קצת לא ברור....

1. בחיים לא שמעתי על זה... זה מתקמפל ועובד מצויין. מעולם לא ראיתי הערות מהסוג הזה, אבל אני מבין מדוע מתכנתים צעירים יתבלבלו בשורה מהסוג int* nums, nums2

2. כן, האמת שרציתי, אבל זה כבר היה בסוף הכתיבה ולא היה לי כבר כוח לתת שמות למשתנים. אתה צודק ואני בד"כ משתמש בשמות משמעותיים.

3. ממתי?

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

5. ה for לא צריך להיות while, הוא צריך להיות for.
אפשר היה ליצור בוליאני נוסף ולבצע השוואה נוספת ולטחון את הזיכרון ולנפח את התוכנית עבור השוואה נוספת ועוד משתנים, אבל אם זה כל כך חשוב לך, פשוט תשנה את ה for ל:

קוד PHP:

 for (int j = 0 ; j < i && isOkay; j++) 


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

6. אסור זה שם של דג
גם GOTO מותר לעשות... אל תספר לאף אחד שגיליתי לך

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

8. אבל בכל מקרה, תודה רבה - לקחתי את ההערות לתשומת ליבי

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

ציטוט:

במקור נכתב על ידי Narxx 1. בחיים לא שמעתי על זה... זה מתקמפל ועובד מצויין. מעולם לא ראיתי הערות מהסוג הזה, אבל אני מבין מדוע מתכנתים צעירים יתבלבלו בשורה מהסוג int* nums, nums2 2. כן, האמת שרציתי, אבל זה כבר היה בסוף הכתיבה ולא היה לי כבר כוח לתת שמות למשתנים. אתה צודק ואני בד"כ משתמש בשמות משמעותיים. 3. ממתי? 4. חחח, טוב, אז אני משנה במקום איפה שכתוב 4- למשתנה בשם lowBound שהערך שלו מינוס 4 5. ה for לא צריך להיות while, הוא צריך להיות for. אפשר היה ליצור בוליאני נוסף ולבצע השוואה נוספת ולטחון את הזיכרון ולנפח את התוכנית עבור השוואה נוספת ועוד משתנים, אבל אם זה כל כך חשוב לך, פשוט תשנה את ה for ל: קוד PHP:  for (int j = 0 ; j < i && isOkay; j++)  כאשר הבוליאני isOkay יאותחל ל true וישונה ל false במידה ומצאנו איבר במערך עם אותו מספר אקראי נוכחי. שוה, הדרך שלי היתה בשביל לחסוך במשתנים והשוואה אבל תמיד אפשר לכתוב עם יותר 'שאלות לוגיות' כדי שיהיה יותר קריא. זה נכון. 6. אסור זה שם של דג גם GOTO מותר לעשות... אל תספר לאף אחד שגיליתי לך 7. הפעם יש הגיון... נסה לקרוא שוב ולנסות להבין את הקטע של הדגל.. נסה להסיר את החלק הזה ולראות מה קורה (או לא קורה). 8. אבל בכל מקרה, תודה רבה - לקחתי את ההערות לתשומת ליבי

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

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

3. שנות ה80-90 מאז שאפשר להגדיר משתנים איפה שבא לך, אבל זה מין ויכוח קבוע בין אנשים שאין להם על מה להתווכח.
4. לקומפיילר לא, למי שקורא את הקוד זה משנה, וגם אם אותו מספר מופיע כמה פעמים וצריך לשנות.
5. הוא צריך להיות WHILE בגלל שהוא שובר את המשמעות של FOR, שזה לרוץ על כל התאים במערך, או על סדרה פשוטה של מספרים. גם בגלל התנאי אבל זה מאוד משני ודיי מקובל. אל תגיד לאף אחד אבל אפילו BREAK אפשר לעשות....

6. שוב זה משנה אתה המשמעות האינטואיטיבית של FOR כשמישהו אחר ניגש לקוד שלך

אה ותקרא את זה , זה של פרופסור אדגר דייקסטרה, בתור סטודנט אני חושב שתכיר את השם
http://www.cs.utexas.edu/users/EWD/ewd02xx/EWD249.PDF

7. אין היגיון, אפשר לכתוב את זה אחרת.... בכלל אתה לא צריך דגל...

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

רק הערה קטנה לגבי 3 - זה אפשרי כל עוד הקומפיילר שלך מאפשר את זה (תומך ב C99) אם אתה עובד עם VS אתה לא יכול לעשות את זה כי ב C89/90 הכרזה של משתנה מותרת רק בפתיחת בלוק (מסולסליים).

הראל

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

תודה רבה רבה מתן ..!!!
אין מילים בפי להודות לך (:

קוד PHP:

 #define LOW_BOUND 4

int* MakeRndArray(const int Size, const int Num)
{
    int* arrNums = new int[Size];
    
    for(int nI = 0; nI < Size; nI++)
    {
        int nNewRnd = GetRnd(LOW_BOUND, Num);
        
        // you can use do{}while();
        bool bContains = DoesArrContain(arrNums, 0, nI - 1, nNewRnd);
        while(bContains)
        {
            nNewRnd = GetRnd(LOW_BOUND, Num);
            bContains = DoesArrContain(arrNums, 0, nI - 1, nNewRnd);
        }
        
        arrNums[nI] = nNewRnd;
    }

    return arrNums;
}

int GetRnd( const int Min, const int Max)
{
    // you should use srand....
    return ( rand( ) % ( Max - Min ) + Min );
}

bool DoesArrContain( const int* const Array, const int Start, const int End,
    const int value)
{
    for(int nI = Start; nI <= End ; nI++)
    {
        if (Array[nI] == value)
        {
            return true;
        }

    }

    return false;
} 


1. לא משתמשים ב-‎#define להגדרת קבועים ב-CPP.

אופס

ציטוט:

במקור נכתב על ידי פסטן 1. לא משתמשים ב-‎#define להגדרת קבועים ב-CPP.

אולי יש משהו שאני לא יודע כי אני לא מתכנת C++ אבל כן מכיר את השפה ותחביר, למה לא משתמשים ב #define?

כבר עניתי על השאלה הזו טריליון פעם. חפש בגוגל. או בפורום הזה.
קודם כל, ‎#define הוא לא type-safe.

ציטוט:

במקור נכתב על ידי hellfrost קוד PHP:  bool bContains = DoesArrContain(arrNums, 0, nI - 1, nNewRnd);         while(bContains)         {             nNewRnd = GetRnd(LOW_BOUND, Num);             bContains = DoesArrContain(arrNums, 0, nI - 1, nNewRnd);         }

אפשר לכתוב את זה ככה

while( DoesArrContain(arrNums, 0, nI - 1, nNewRnd) )
{
nNewRnd = GetRnd(LOW_BOUND, Num);

}
בלי בוליאני בכלל....

פסטן
תמיד חשבתי מC, שלקבועים ממש עושים CONST
ולסתם מספר מונפץ עושים דפיין, אבל מה לי ולCPP, אני בכלל מתכנת ב C#

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

יש תמיד איזה-שהוא trade-off. מצד אחד, יש רצון לכתוב 'by the book', בצורה מסודרת כדי שכולם יבינו וכדי שתהיה אחידות כלשהי.. מצד שני, זה לא נגיש עבור סטודנטים שרק נוגעים בפעם הראשונה בקוד.
אם אפשר לייצר את המספר האקראי בתוך הפונקציה, בתוך הלולאה... למה לא לעשות את זה?
אם אפשר לרוץ בתוך לולאה פנימית בכל פעם שרוצים לבדוק האם המספר שהוגרל קיים כבר - למה ליצור עוד פונקציה חיצונית (פונקציית עזר) שתעשה בדיוק את אותה הפעולה?

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

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

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

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

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

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

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

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

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

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

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

אה! אני אוהב BREAK כשצריך, ושילכו כולם להזדיין.

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

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

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

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

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

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

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

אני שמח על האשכול הזה, שהתפתח למה שהוא התפתח

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