[שאלה -Javascript] מה לא תקין בקוד הרנדומלי?

[Homie]

שלום לכל המשתמשים היקרים,

לפני דקות ספורות יצרתי לי סקריפט שמציג לוגו רנדומלי.

אבל למרות זאת, הוא אינו מציג את הלוגו כראוי.

הקוד:

var logo = new Array()

logo[1] = '/style/logos/logo1.swf'
logo[2] = '/style/logos/logo2.swf'
logo[3] = '/style/logos/logo3.swf'
logo[4] = '/style/logos/logo4.swf'

var j = 0
var p = logo.length;

var preBuffer = new Array()
for (i = 0; i < p; i++){
   preBuffer[i] = new Image()
   preBuffer[i].src = logo[i]
}

var whichLogo = Math.round(Math.random()*(p-1));
function showLogo(){
document.write('<object classid="clsid:D27CDB6E-AE6D-11CF-96B8-444553540000" id="obj1" ');
document.write('codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=6,0,40,0" ');
document.write('border="0" width="780" height="161">');
document.write('"<param name="movie" value="+logo[whichLogo]+">');
document.write('<param name="quality" value="High">');
document.write('<embed src="+logo[whichLogo]+" ');
document.write('pluginspage="http://www.macromedia.com/go/getflashplayer" ');
document.write('type="application/x-shockwave-flash" name="obj1" width="780" ');
document.write('height="161" quality="High"></object>');
}

showLogo();

מה הבעיה פה?

כשהלוגו מוצג, רואים רק לבן, וכתוב Movie not loaded, מה לא נטען? אני גם ניסיתי קישור מלא.

תודה מראש, אלי.

(חזרתי מהטיול)

עריכה: ניסיתי ככה:

var Logo = new Array()

Logo[1] = "/style/logos/logo1.swf";
Logo[2] = "/style/logos/logo2.swf";
Logo[3] = "/style/logos/logo3.swf";
Logo[4] = "/style/logos/logo4.swf";

var L = Logo.length;
var whichLogo = Math.round(Math.random()*(L-1));
function showLogo(){
document.write('<object classid="clsid:D27CDB6E-AE6D-11CF-96B8-444553540000" id="obj1" ');
document.write('codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=6,0,40,0" ');
document.write('border="0" width="780" height="161">');
document.write('"<param name="movie" value="+Logo[whichLogo]+">');
document.write('<param name="quality" value="High">');
document.write('<embed src="+Logo[whichLogo]+" ');
document.write('pluginspage="http://www.macromedia.com/go/getflashplayer" ');
document.write('type="application/x-shockwave-flash" name="obj1" width="780" ');
document.write('height="161" quality="High"></object>');
}

showLogo();

וגם לא עובד, אין שגיאת JS, אבל הלוגו לא מוצג, רואים רק לבן..

תודה מראש.

[BlueNosE]

document.write('"<param name="movie" value="+Logo[whichLogo]+">');

איפה מצויין המשתנה Logo['whichLogo']?
זה whichLogo ולא מערך במשתנה Logo...

אותה טעות גם בקוד הראשון.

עריכה:
אני חושב שהבנתי.
אתה רוצה לקרוא ללוגו שהמפתח שלו הוא whichLogo..
אממ.. תנסה להפוך את whichLogo לקטע במערך עצמו.. זאת אומרת להפוך אותו לLogo לעשות השמה לתוך whichLogo

[Homie]

נכתב במקור על ידי BlueNosE

document.write('"<param name="movie" value="+Logo[whichLogo]+">');

איפה מצויין המשתנה Logo['whichLogo']?
זה whichLogo ולא מערך במשתנה Logo...

אותה טעות גם בקוד הראשון.

עריכה:
אני חושב שהבנתי.
אתה רוצה לקרוא ללוגו שהמפתח שלו הוא whichLogo..
אממ.. תנסה להפוך את whichLogo לקטע במערך עצמו.. זאת אומרת להפוך אותו לLogo לעשות השמה לתוך whichLogo

הבנתי, אבל זה עדיין לא מערך במשתנה.

עומר, יש לך מושג איך לתקן את זה? זה קצת דחוף.

תודה מראש.

[MasterNir]

var whichLogo = Math.round(Math.random()*(L-1));

אמור להיות חילוק, לא כפל...
ככה:

var whichLogo = Math.round(Math.random()/(L-1));

היגיון, יש לך מספר לא ידוע (random), נסמן ב-y, ואתה רוצה שייכנס לטווח בין 0 ל-X... אם תכפיל את y (בהנחה שהוא שלם וחיובי [במילים אחרות גדול מ-1]) ב-x תקבל תוצאה הגדולה מ-x ומ-y.
אתה צריך לחלק, רמז לכך יכול להיות round שמטרתה לעגל (כפל של שני מספרים שלמים יתן שלם, חילוק לעומת זאת יכול להחזיר מספר עשרוני) ולמעשה במקרה של כפל הפונקציה מיותרת, בעוד שבמקרה של חילוק היא נחוצה.

[Homie]

נכתב במקור על ידי MasterNir

var whichLogo = Math.round(Math.random()*(L-1));

אמור להיות חילוק, לא כפל...
ככה:

var whichLogo = Math.round(Math.random()/(L-1));

היגיון, יש לך מספר לא ידוע (random), נסמן ב-y, ואתה רוצה שייכנס לטווח בין 0 ל-X... אם תכפיל את y (בהנחה שהוא שלם וחיובי [במילים אחרות גדול מ-1]) ב-x תקבל תוצאה הגדולה מ-x ומ-y.
אתה צריך לחלק, רמז לכך יכול להיות round שמטרתה לעגל (כפל של שני מספרים שלמים יתן שלם, חילוק לעומת זאת יכול להחזיר מספר עשרוני) ולמעשה במקרה של כפל הפונקציה מיותרת, בעוד שבמקרה של חילוק היא נחוצה.

לא עובד

תוכל בבקשה להביא לי את הקוד כולו מתוקן.

והבנתי מה הבעיה פחות היותר, הוא לא לוקח את ה-+Logo[whichLogo]+

יעני בדקתי במקור של העמוד HTML, ואני רואה שהוא קורא לשם של הקובץ:

+Logo[whichLogo]+.swf

[MasterNir]

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

לדוגמה את השורה הזאת:

document.write('"<param name="movie" value="+Logo[whichLogo]+">');

אתה אמור לשנות שתראה ככה:

document.write('"<param name="movie" value=" ' +Logo[whichLogo]+ ' ">');

בהצלחה

[Homie]

נכתב במקור על ידי MasterNir

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

לדוגמה את השורה הזאת:

document.write('"<param name="movie" value="+Logo[whichLogo]+">');

אתה אמור לשנות שתראה ככה:

document.write('"<param name="movie" value=" ' +Logo[whichLogo]+ ' ">');

בהצלחה

שוב לא מציג את הלוגו,

הוא שינה את שם הלוגו ל-undefined.

....

[MasterNir]

תעשה document.write ל-whichLogo ותכתוב מה התוצאה...

[BlueNosE]

וואלה שמתי לב למשהו לא בסדר בשורה ולא הבנתי מה אז חשבתי שזה הקריאה למשתנה..
חח
אממ, אני לא ממש מבין.

var whichLogo = Math.round(Math.random() / (L-1));

נניח שקיבלנו את המספר.. 63.
L שווה ל4. L-1 שווה ל3.
אוקיי, מבצע:
63 / 3
נותן 21.
כלומר יקרא לLogo[21].
לא נדמה לי שקיים..
עדיף כבר לעשות פונקצייה רנדומלית שבוחרת מספרים בטווח של בין 0 ל3..
לא זוכר אותה כרגע מצטער אני יכול לחפש באיציק1 (סט אפ.. שכחתי.. S: )