שאלה קטנה בJavascript

Netta · 20/4/20

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

yitzhak123 · 20/4/20

אפשר לראות את הקוד?

Netta · 20/4/20

קוד:

<html>
        <button id="btn1" onclick="showGame()">Start Game</button>
        <button type="button" id="button" onclick="checkNums()">Go!</button>
</html>

<script>
        var arr = new Array()
        function showGame() {
          arr = [1,2,3,4]
        }
  
        function checkNums (){
    console.log(arr[0])
            if (arr[0] == 3)
                {var c = false}
        }
</script>

@yitzhak123
זה הרעיון.
שיניתי קצת את הקוד הפנימי כי הוא לא רלוונטי.

devMode · 21/4/20

הקוד שהבאת עובד מעולה, תביא קוד עם שגיאה
(חוץ מזה שכתוב
arr[0] = 3
במקום
arr[0] == 3
)

yitzhak123 · 21/4/20

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

Netta · 21/4/20

נכתב ע"י devMode:
הקוד שהבאת עובד מעולה, תביא קוד עם שגיאה
(חוץ מזה שכתוב
arr[0] = 3
במקום
arr[0] == 3
)

אז זהו, כשאני עושה console.log() לarr בפונקציה השניה, הוא כותב undefined כמה פעמים.

Netta · 21/4/20

נכתב ע"י yitzhak123:
המשתנה c אינו נגיש מחוץ לפונקציה שיצרה אותו, כך שאם ניסית לגשת עליו לאחר ההקשה על הכפתור תקבלי שגיאה, אלא אם כן הצהרת עליו לפני שאז מיותר המילה var.
ועוד דבר כמו שכבר נכתב מעלי השוואה נעשית עם === או ==, לא עם = שבה עושים הקצאה.

תודה, תיקנתי את ההשוואה.
ואני לא צריכה גישה למשתנה c
ברגע שאני רוצה לראות את הערכים של arr ע"י console.log בפונקציה השניה, הוא כותב לי שהם undefined
(הוספתי את זה בקוד)

devMode · 21/4/20

פה זה עובד

Edit fiddle - JSFiddle - Code Playground

Test your JavaScript, CSS, HTML or CoffeeScript online with JSFiddle code editor.

jsfiddle.net

קרן שמש · 21/4/20

נכתב ע"י Netta:

קוד:

<html>
        <button id="btn1" onclick="showGame()">Start Game</button>
        <button type="button" id="button" onclick="checkNums()">Go!</button>
</html>

<script>
        var arr = new Array()
        function showGame() {
          arr = [1,2,3,4]
        }
 
        function checkNums (){
    console.log(arr[0])
            if (arr[0] == 3)
                {var c = false}
        }
</script>

@yitzhak123
זה הרעיון.
שיניתי קצת את הקוד הפנימי כי הוא לא רלוונטי.

הקוד הזה תקין.
יש פה דברים מיותרים. לדוגמה לתגית button את לא צריכה להוסיף גם type=button
אפשר להגדיר רק arr בלי לעשות לו new array()

yitzhak123 · 21/4/20

אצלי זה עובד, רק כשאני קורא לפונקציה השניה לפני הראשונה זה מחזיר undefined, יתכן שזה הבעיה?

מיכל. · 21/4/20

באיזה סביבה את כותבת את זה?
יכול להיות שהכפתור עושה SUBMIT ומרענן את הדף?
תנסי להחליף את הBUTTON לINPUT מסוג BUTTON.

רחל1 · 21/4/20

נכתב ע"י Netta:
אז זהו, כשאני עושה console.log() לarr בפונקציה השניה, הוא כותב undefined כמה פעמים.

תנסי להכניס את הערך מתוך הarr לalert במקום הקונסלוג לוג

תא חזי · 21/4/20

@Netta
לכאו' להוסיף RETURN בפונקציות:

HTML:

<html>
        <button id="btn1" onclick="showGame()">Start Game</button>
        <button type="button" id="button" onclick="checkNums()">Go!</button>
</html>

<script>
        var arr = new Array()
        function showGame() {
          return arr = [1,2,3,4]
        }
 
        function checkNums (){
            console.log(arr[0])
            if (arr[0] == 3){
                var c = false
                return c
            }
        }
</script>

@shaye · 21/4/20

נכתב ע"י קרן שמש:
יש פה דברים מיותרים. לדוגמה לתגית button את לא צריכה להוסיף גם type=button

נכתב ע"י מיכל.:
יכול להיות שהכפתור עושה SUBMIT ומרענן את הדף?
תנסי להחליף את הBUTTON לINPUT מסוג BUTTON.

אם מצהירים בכפתור type=button, הכפתור אינו מבצע sumbit, וזאת גם הסיבה שהיא אינה מיותרת.

נכתב ע"י Netta:
הוא מתייחס למערך כריק.

()checkNums יחזיר undefined רק אם קוראים אותה לפני showGame(), מכיון שעד אז arr[0] עדיין לא מוגדר.

Untitled - JSitor

var arr = new Array() function showGame() { arr = [1, 2, 3, 4] } function checkNums() { console.log(arr[0]) if (arr[0] == 3) { var c = false } } <button id=\"btn1\" onclick=\"showGame()\">Start Game</button> <button type=\"button\" id=\"button\" onclick=\"checkNums()\">Go!</button>

jsitor.com

Netta · 21/4/20

נכתב ע"י yitzhak123:
אצלי זה עובד, רק כשאני קורא לפונקציה השניה לפני הראשונה זה מחזיר undefined, יתכן שזה הבעיה?

נכתב ע"י מיכל.:
באיזה סביבה את כותבת את זה?
יכול להיות שהכפתור עושה SUBMIT ומרענן את הדף?
תנסי להחליף את הBUTTON לINPUT מסוג BUTTON.

נכתב ע"י @shaye:
אם מצהירים בכפתור type=button, הכפתור אינו מבצע sumbit, וזאת גם הסיבה שהיא אינה מיותרת.

()checkNums יחזיר undefined רק אם קוראים אותה לפני showGame(), מכיון שעד אז arr[0] עדיין לא מוגדר.

Untitled - JSitor

var arr = new Array() function showGame() { arr = [1, 2, 3, 4] } function checkNums() { console.log(arr[0]) if (arr[0] == 3) { var c = false } } <button id=\"btn1\" onclick=\"showGame()\">Start Game</button> <button type=\"button\" id=\"button\" onclick=\"checkNums()\">Go!</button>

jsitor.com

אני קוראת קודם לראשונה ואח"כ לשניה, כלומר, אחרי שהוא מוגדר...
אני כותבת את הקוד בAtom ובchrome הוא פשוט לא מזהה אותו

נכתב ע"י רחל1:
תנסי להכניס את הערך מתוך הarr לalert במקום הקונסלוג לוג

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

נכתב ע"י תא חזי:
לכאו' להוסיף RETURN בפונקציות:

ולאן הוא מחזיר לי אותו? איך אני מקבלת אותו בפונקציה השניה?

devMode · 21/4/20

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

תא חזי · 21/4/20

נכתב ע"י Netta:
ולאן הוא מחזיר לי אותו? איך אני מקבלת אותו בפונקציה השניה?

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

devMode · 21/4/20

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

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