נכנסתי קצת לנושא ה HTML dom, מההתלהבות הראשונית מהפונקציה scrollTo קפץ לי לראש הרעיון להכין auto scroller.

וזה הקוד:

קוד:

<script type="text/javascript">
var pix = 0;
function autoScroller(mode) {
   pix++;
   
   window.scrollTo(pix, 50);
   
   SetTimeout(autoScroller(), 10000);
 }
</script>

אבל כשאני מריץ את הקוד, קופץ alert שאומר:

קוד:

out of memory at line:9

line 9:

קוד:

window.scrollTo(pix, 50);

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


תודה מראש.

קוד מתוקן:

קוד:

<script type="text/javascript">
var pix = 0;
function autoScroller(mode) {
   ++pix; // העברתי את הפלוסים להתחלה
   window.scrollTo(50, pix);
   window.setTimeout(autoScroller, 1000);
 }
</script>

עדיין אותה שגיאה.

באיזה דפדפן?

IE7, ובFF זה אומנם לא מוציא שגיאה, אבל זה גם לא עובד.

עבד לי באופרה, לא היה לי זמן לבדוק בדפדפנים אחרים

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

קוד:

setInterval(function(){
	window.scrollBy(0, 1);
}, 50);

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

תודה! עובד מעולה

אגב, סתם שאלה.

למה כשאני משנה את זה לזה, זה לא עובד?

קוד:

function scroll() {

window.scrollBy(0, 1);
}
setInterval(scroll(), 100)

קוד:

Error:‏‏הארגומנט אינו חוקי.

בעקרון, אתה אמור להעביר את ההפונקציה כמחרוזת ל-setInterval (מה שנקרא eval - רע מאוד למשאבי המערכת).
בנוסף, בשיטה שלך יצרת רקורסיה שבזבזה משאבים מיותרים.

הכנסתי את זה למחרוזת, וכעת זה עובד, תודה!

והאמת, אני לא רואה הבדל משמעותי בין שתי השיטות, אולי לא שמת לב שהsetInterval מחוץ לפונקציה?

1. תמנע תמיד משמוש ב-eval (כל קוד ושלוקח מחרוזת ומנסה להריץ אותה).
2. התכוונתי לקוד הראשוני שהצגת.

אם תעביר את הפונקציה כמצביע (כלומר ללא הסוגריים), זה כן יעבוד.

צודק, תודה גם לך