שימוש בRandon בJava - בלי שתהיה חזרתיות

[מישהו12]

כתבתי קטע קוד לא ארוך בJava, שזמין כאן:

SimpleProjects.java/LearnLang at main · AshiVered/SimpleProjects.java

Simple Java projects. Contribute to AshiVered/SimpleProjects.java development by creating an account on GitHub.

github.com

הקוד עוזר בלימוד שפות חדשות - מכניסים מאגר מילים בשפה הנלמדת ובשפת המקור, והוא מציג בצורה רנדומלית מילה, וצריך לכתוב את התרגום...
הבעיה היא שבגלל שהרשימה קצרה (10 מילים) אז יש חזרתיות.
הסבר קצר לרנדומיליות - מורכב משני שלבים:
אחד, המשתנה SelectArray.
אם הוא 0 - מציגים מילה בשפת המקור, והמשתמש צריך להקליד את התרגום לשפה הנלמדת.
אם 1 - ההפך.
פה בודאי שצריכה להיות חזרתיות - הקוד מציג 10 מילים, חלק ככה וחלק ככה.
הבעיה בposition - שבוחר את המיקום בarray של המילה שתוצג.
פה, יש חזרתיות, ואני לא רוצה שתהיה.
למשל, הפלט מהפעם האחרונה שהרצתי. יש הרבה יותר מידי כפילויות.

the word in English: grandfather
Enter translate
סבא
success!
the word in Hebrew: סבא
Enter translate
grandfather
success!
the word in Hebrew: אבא
Enter translate
father
success!
the word in English: grandmother
Enter translate
סבתא
success!
the word in English: uncle
Enter translate
דוד
success!
the word in English: nephew
Enter translate
אחיין
success!
the word in English: uncle
Enter translate
דוד
success!
the word in English: nephew
Enter translate
אחיין
success!
the word in English: grandfather
Enter translate
סבא
success!
the word in English: aunt
Enter translate
דודה
success!
Your score: 10/10

Process finished with exit code 0

 

[java]

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

 

[מישהו12]

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

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

 

[RGS]

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

 

[java]

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

אני כבר לא חזקה בjava (השם קצת ישן...), אז כתבתי בjs
אבל זה משהו כזה:

const assigned = {
   
}

//Instead of int position = r.nextInt(9);
let found = false;
let position;
while (!found) {
    position = r.nextInt(9);
    if (!assigned[position]) {//Found new word
        found = true;
        assigned[position] = true;
    }
    // Use position
}

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

 

[nhfk]

מערך של דגלים מתכוונים למערך בוליאנים
ודלוק משמעו true
אתה מייצר בהתחלה מערך של פאלסים וכל פעם שאתה מקבל position חדש אתה בודק אם באותו מקום במערך דגלים יש פאלס או טרו
אם טרו אתה מחפש position חדש ואם פאלס אתה משתמש ומשנה לטרו במערך דגלים

 

[יוסייוסייוס]

למה כתבת
int position = r.nextInt(9);
צריך לכתוב
int position = r.nextInt(10);
עד 10 זה לא עד בכלל ואחרון ברשימה הוא [9]

 

[java]

למה כתבת
int position = r.nextInt(9);
צריך לכתוב
int position = r.nextInt(10);
עד 10 זה לא עד בכלל ואחרון ברשימה הוא [9]

זה 0-9, 10 מקומות.

 

[יוסייוסייוס]

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

 

[יוסייוסייוס]

זה 0-9, 10 מקומות.

int position = r.nextInt(9);
זה 0 עד 8 [כולל 8].

 

[יוסייוסייוס]

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

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

 

[java]

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

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

 

[מישהו12]

int position = r.nextInt(9);
זה 0 עד 8 [כולל 8].

בטוח? למיטב זכרוני זה 0-9.

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

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

לעניינו:
מצורף הקוד החדש.
הוספתי עוד מילים, כדי שלא כל המילים שהוכנסו מראש בהכרח יוצגו.
שאלותי:
א. עושה לי רושם של קוד מסורבל. אני יכול לשפר אותו במסגרת הידע שלי?
ב. משום מה הוא כל פעם מריץ מספר מילים אחר, ולא 10. למה?
תודה

/* LearnLang.java
   Author: Ashi Vered
   Version: 1.0
   Date: 23.8.5784
 */

import java.util.Random;
import java.util.Scanner;

public class Main {
    public static void main(String[] args) {
        int score = 0;
        Scanner in = new Scanner(System.in);
        Random r = new Random();
        //array to mark which positions are used
        boolean[] used = new boolean[20];
        used[0] = false;
        used[1] = false;
        used[2] = false;
        used[3] = false;
        used[4] = false;
        used[5] = false;
        used[6] = false;
        used[7] = false;
        used[8] = false;
        used[9] = false;
        used[10] = false;
        used[11] = false;
        used[12] = false;
        used[13] = false;
        used[14] = false;
        used[15] = false;
        used[16] = false;
        used[17] = false;
        used[18] = false;
        used[19] = false;


        //array for original words
        String[] words_orig = new String[20];
        //list of original words
        words_orig[0] = "אבא";
        words_orig[1] = "אמא";
        words_orig[2] = "אח";
        words_orig[3] = "אחות";
        words_orig[4] = "סבא";
        words_orig[5] = "סבתא";
        words_orig[6] = "דוד";
        words_orig[7] = "דודה";
        words_orig[8] = "אחיין";
        words_orig[9] = "אחיינית";
        words_orig[10] = "יהלום";
        words_orig[11] = "חבר";
        words_orig[12] = "משפחה";
        words_orig[13] = "טוב";
        words_orig[14] = "רע";
        words_orig[15] = "אמת";
        words_orig[16] = "שקר";
        words_orig[17] = "מחשב";
        words_orig[18] = "יפה";
        words_orig[19] = "מכוער";

        //array for new words
        String[] words_new = new String[20];
        //list of new words
        words_new[0] = "father";
        words_new[1] = "mother";
        words_new[2] = "brother";
        words_new[3] = "sister";
        words_new[4] = "grandfather";
        words_new[5] = "grandmother";
        words_new[6] = "uncle";
        words_new[7] = "aunt";
        words_new[8] = "nephew";
        words_new[9] = "niece";
        words_new[10] = "diamond";
        words_new[11] = "friend";
        words_new[12] = "family";
        words_new[13] = "good";
        words_new[14] = "bad";
        words_new[15] = "true";
        words_new[16] = "false";
        words_new[17] = "computer";
        words_new[18] = "beautiful";
        words_new[19] = "ugly";


        for (int i = 0; i < 10; i++) {
            int selectArray = r.nextInt(2); // Randomly decide whether to prompt for a translation from original lang to new lang.
            int position = r.nextInt(19); // Randomly choose a position in the arrays (i.e. choose which word to display).
            String answer;
            while (!used[position]) {
                used[position] = true;
                if (selectArray == 0) {
                    // show word in original language.
                    String word_orig = words_orig[position];
                    String word_new = words_new[position];
                    System.out.println("the word in Hebrew: " + word_orig);
                    System.out.println("Enter translate");
                    answer = in.next();
                    if (answer.equals(word_new)) {
                        System.out.println("success!");
                        score++;
                    } else {
                        System.out.println("Error!");
                        System.out.println("Real translate is: " + word_new);
                    }
                } else {
                    // show word in new language.
                    String word_orig = words_orig[position];
                    String word_new = words_new[position];
                    System.out.println("the word in English: " + word_new);
                    System.out.println("Enter translate");
                    answer = in.next();
                    if (answer.equals(word_orig)) {
                        System.out.println("success!");
                        score++;
                    } else {
                        System.out.println("Error!");
                        System.out.println("Real translate is: " + word_orig);
                    }
                }
            }
        }
                System.out.println("Your score: " + score + "/10");



        in.close();
    }
}

 

[RGS]

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

 

[מישהו12]

דבר ראשון שקפץ לי,
מערך בגאווה מאותחל בFALSE

כלומר, בברירת מחדל המערך used הוא false, ולא הייתי צריך להגדיר תא אחרי תא?
תודה...

 

[RGS]

כלומר, בברירת מחדל המערך used הוא false, ולא הייתי צריך להגדיר תא אחרי תא?
תודה...

בדיוק.

 

[commit]

כלומר, בברירת מחדל המערך used הוא false, ולא הייתי צריך להגדיר תא אחרי תא?
תודה...

גם אם רוצים למלא את כל המערך בערכים זהים, אפשר להשתמש בArrays.fill, או לפחות בfor.

יש בתוך הלולאה הרבה קוד כפול, שאפשר להוציא מתוך הif-else

 

[מישהו12]

בדיוק.

תודה רבה.
אין ספק שזה כבר מוריד את רמת הסרבול של הקוד.
יש לי גם תחושה שהשימוש גם בwhile וגם בfor מיותר.
אבל כשהשארתי רק את הwhile - זה רץ כל פעם רק פעם אחת.
אז שמתי את הwhile בתוך הfor כדי שירוץ 10 פעמים.
אז רץ יותר מפעם אחת, אבל כל פעם מספר אחר של פעמים... מה הבעיה?

 

[java]

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

 

[מישהו12]

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

בחינמי בתקופה האחרונה רק אכלתי איתו קש, לא הועיל לי באמת (בעבר הועיל).
את משתמשת בתשלום אולי?