12-05-2007, 01:11
|
|
|
|
חבר מתאריך: 05.08.06
הודעות: 2,860
|
|
הערות(מהפחות חשוב ליותר חשוב):
-כאשר מדובר בpublic functions, ניתן לרשום function f כרגיל, וזה סתם מיותר כל ה this.f=function (לא רשמתי באף מקום סוגריים כי הן קופצות לצד השני)
-אין צורך להוציא את i מחוץ להגדרת המחלקה. i אמור להיות משתנה סטטי פרטי של המחלקה.
-במקום לקבל כל פעם את האובייקטים שלך בעזרת getElementById, פשוט תשמור אותם. זה יאיץ בהרבה את המהירות של הסקריפט.
-
קוד:
this.startClient[x/y]
מי אלה?
-אל תשתמש בwrite. הסיבה העיקרית לכך היא שזה מאוד מגביל את המחלקה שלך, משום שלא ניתן יהיה ליצור חלונות נוספים לאחר שהעמוד ייטען, אבל יש עוד מספר סיבות.
-הevent handlers שלך לא טובים, וזה הבעיה לדעתי הכי חמורה בסקריפט שלך:
אתה לוקח אירועים יותר מדי רחבים, וגם בדרך לא טובה.
השימוש בthis זה עניין עדין, במיוחד כשמדובר במחלקות ואירועים. בצורה שבה אתה מטפל באירועים, this כבר לא מצביע על האובייקט שלך, אלא על האובייקט של האלמנט שבו התרחש האירוע, ולכן אתה לא יכול לפנות יותר למשתנים של האובייקט שלך, ככה שמה שרשמת לא יעבוד. תקרא פה על this וההבדלים בין הסוגים השונים של טיפול באירועים ומה קורה לthis בכל אחת מהן.
אני מקווה שלא פספסתי כלום
|