מה לא בסדר בקוד שלי?

[ekotalk]

היי,
חיפשתי בגוגל קוד לתמונה מתחלפת ב-JAVA.
אז אוקיי, עשיתי את הקוד, שיניתי לתמונות שלי, וזה מתרענן ובפעם מציג את תמונה 1, פעם 2, פעם 3, זה חוזר על עצמו, אבל לפעמים במקום תמונה כתוב "undefined".

הנה הקוד שלי:

קוד: בחר הכל

<P align=center><script language="javascript">
<!--
var baner = new Array () ;
tmuna1="-תמונה צונזרה"
tmuna2="-תמונה צונזרה-"
tmuna3="-תמונה צונזרה-"
baner[1]="<img src=" + tmuna1 + ">"
baner[2]="<img src=" + tmuna2 + ">"
baner[3]="<img src=" + tmuna3 + ">"
function makeArray (n) {
this.length=n ;
for (var t=0;t<n;t++) this[t] = '' ;
}

mispar_habanerim = 3
var baner1=Math.round(Math.random(1)*mispar_habanerim);
document.write(baner[baner1]);
-->
</script></P>

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

[Shlomi1]

תן קישור לדף שבו הקוד רץ בפועל.

[ekotalk]

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

[hagai303]

כידוע - כנראה שלא לך - מערך מתחיל מאינדקס '0'.
שנה את האינדקסים של המערך כך שיתחילו מ-0 ובנוסף החלף את המשתנה mispar_habanerim במספר הכי גדול שאליו הגעת במערך... במקרה שלך -> 2

[ekotalk]

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

[hagai303]

לא...

[ekotalk]

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

[golsa]

לא...

[ekotalk]

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

[golsa]

ואם בא לי?

[amit221]

^golsa ההודעה הזו לדעתי ספאם
אה ואני ינסה לתקן את הקוד

[RUBHRUBH]

קוד: בחר הכל

<script language="javascript">

/*

ניסיון * ניסיון * ניסיון *



כל הסקריפט נכנס ל- head עד איפה שצויין אחרת

*/





var ImgNum = 0;

var MyImgSrc=new Array()





MyImgSrc[0] = "http://www.scripter.co.il/Pics/Logo.jpg";    //כתובת התמונה הראשונה

MyImgSrc[1] = "http://www.theocean.co.il/Pics/Logo.jpg";     //כתובת התמונה השניה

MyImgSrc[2] = "http://www.noplex.co.il/Pics2/Logo_Hover.jpg";     //כתובת התמונה השלישית



/*

אם אתם רוצים תמונה רביעית רשמו

MyImgSrc[3] = "http://כתובת-התמונה.co.il"

ואם אתם רוצים תמונה חמישית בדיוק אותו הדבר, רק שנו את

MyImgSrc[3] ל- MyImgSrc[4]

וכו...

*/



function GoForward()

{

        ImgNum++;

        if (ImgNum == MyImgSrc.length)

                ImgNum = 0;

        document.getElementById("MyImg").src = MyImgSrc[ImgNum];

}



function GoBack()

{

        ImgNum--;

        if (ImgNum < 0)

                ImgNum = MyImgSrc.length - 1;

        document.getElementById("MyImg").src = MyImgSrc[ImgNum];

}





</script>



<!-- חלק זה נכנס ל- body -->



<img id="MyImg" alt="" /><br />

<input type="button" value="<<" onclick="GoForward()" /> <!-- כפתור שמעביר תמונה קדימה -->

<input type="button" value=">>" onclick="GoBack()" /> <!-- כפתור שמעביר תמונה אחורה -->



<script language="javascript">

document.getElementById("MyImg").src = MyImgSrc[0];

</script>

קוד אחר, אבל סבבה..
דמו:
http://www.scripter.co.il/RunScript.php?id=88

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

[hagai303]

קוד: בחר הכל

<script language="javascript">

/*

ניסיון * ניסיון * ניסיון *



כל הסקריפט נכנס ל- head עד איפה שצויין אחרת

*/





var ImgNum = 0;

var MyImgSrc=new Array()





MyImgSrc[0] = "http://www.scripter.co.il/Pics/Logo.jpg";    //כתובת התמונה הראשונה

MyImgSrc[1] = "http://www.theocean.co.il/Pics/Logo.jpg";     //כתובת התמונה השניה

MyImgSrc[2] = "http://www.noplex.co.il/Pics2/Logo_Hover.jpg";     //כתובת התמונה השלישית



/*

אם אתם רוצים תמונה רביעית רשמו

MyImgSrc[3] = "http://כתובת-התמונה.co.il"

ואם אתם רוצים תמונה חמישית בדיוק אותו הדבר, רק שנו את

MyImgSrc[3] ל- MyImgSrc[4]

וכו...

*/



function GoForward()

{

        ImgNum++;

        if (ImgNum == MyImgSrc.length)

                ImgNum = 0;

        document.getElementById("MyImg").src = MyImgSrc[ImgNum];

}



function GoBack()

{

        ImgNum--;

        if (ImgNum < 0)

                ImgNum = MyImgSrc.length - 1;

        document.getElementById("MyImg").src = MyImgSrc[ImgNum];

}





</script>



<!-- חלק זה נכנס ל- body -->



<img id="MyImg" alt="" /><br />

<input type="button" value="<<" onclick="GoForward()" /> <!-- כפתור שמעביר תמונה קדימה -->

<input type="button" value=">>" onclick="GoBack()" /> <!-- כפתור שמעביר תמונה אחורה -->



<script language="javascript">

document.getElementById("MyImg").src = MyImgSrc[0];

</script>

קוד אחר, אבל סבבה..
דמו:
http://www.scripter.co.il/RunScript.php?id=88

כל הקשר בינך לבין השאלה מקרי בהחלט!

[ekotalk]

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

[hagai303]

אנחנו נשמח אם תלמד קצת JS לפני שאתה רץ ושואל "מה לא בסדר?" בקוד מוכן שהורדת מאיזה אתר...

ובלי קשר:

כאן זה פורום תמיכה, לא פורום "עושים זאת בשבילך"