03-11-2008, 03:41
|
 |
מנהל משבראש, בלשנות, תכנות ויהדות
|
|
חבר מתאריך: 04.06.06
הודעות: 33,133
|
|
|
|
כמה דברים שראיתי אצלך שדורשים תיקון
1. הרקורסיה אצלך לא מוגדרת טוב
ברגע שהשורה השנייה ב-StartTimer() מתבצעת הפונקציה חוזרת להתחלה ומתעלמת מהשורה האחרונה (המעדכנת את שדה הטקסט)
2. ב-JS יש הבחנה בין אותיות קטנות לגדולות ולכן timer ו-Timer אלו 2 משתנים שונים מבחינתה
3. הגדרת את המשתנה t בתוך הפונקציה StartTimer() ולכן אתה לא יכול לפנות אליו מבחוץ, מה שאומר שאתה לא יכול להפסיק אותו דרך StopIt()
4. לא הבנתי למה השתמשת ב-2 פונק' בשביל התחלת התהליך (גם StartIt וגם StartTimer)
ועוד כמה דברים שלא קשורים ישירות לסקריפט:
5. תג ה-form לא סגור, כדאי לשים לב לדברים כאלה
6. גם תג הטייטל לא סגור היטב
7. כתבת langeage במקום language
גם לטעויות כתיב צריך לשים לב היטב, לפעמים הם יכולות לשגע ולדפוק את הכל
8. onmouseover ו-onmouseout עובדות גם על תג img כך שלא היית זקוק לתג העוגן
9. לא הבנתי גם למה הגדרת אירוע onclick שלא עושה כלום..
בה"כ לאחר כל התיקונים (ועוד פה ושם שיפצורים) זה אמור להיראות כך:
קוד:
<html>
<head>
<title></title>
</head>
<body>
<form name="form1">
<script language="JavaScript">
var timer = 0;
var t;
function StartIt() {
timer++;
window.document.form1.text1.value = timer;
t = setTimeout("StartIt()", 1000);
}
function StopIt() {
clearTimeout(t);
alert(timer);
}
</script>
<input type="text" name="text1" value="">
<img src="http://ks.hinuch.net/CmsSchool/Rsrc/common/icon-print.jpg" onmouseover="StartIt();" onmouseout="StopIt();">
</form>
</body>
</html>
ובקובץ
מקווה שעזרתי 
|
ראשי
צ'אט
מצב כלאיים
