Optymalizacja kodu

Thread View: pl.comp.lang.javascript

4 messages

[T] Back to Threads [V] Tree View [G] Group Articles

4 total messages Started by Stefan Sat, 13 Feb 2010 19:36

#49813

Author: Stefan
Date: Sat, 13 Feb 2010 19:36

33 lines
1500 bytes

Jest sobie taki kod: http://pastebin.ca/1795545

==
var cosTam = {
         lastRefresh : 0,

         autoRefresh : function(){
             var dateobj=new Date();

             if (dateobj.getTime()-cosTam.lastRefresh>10000) {
                 cosTam.lastRefresh=dateobj.getTime();
                 cosTam.refresh();
             }
	},
	refresh : function () {
		// ....
	}
}


$(document).focus(cosTam.autoRefresh);
==

Jako ze event focus jest wywolywany przy kazdym kliknieciu mysza, 
podpiety pod to zdarzenie autoRefresh pilnuje aby nie dokonywac dzialan 
czesciej niz co 10 sekund.

Pytanie jest o linie

var dateobj=new Date()

Czy inicjalizacja powinna byc przeniesiona 2 linie wyzej, do deklaracji 
obiektu cosTam?

Re: Optymalizacja kodu

#49816

Author: Ghost
Date: Sun, 14 Feb 2010 09:48

40 lines
2081 bytes

Użytkownik "Stefan" <stefan@costam.wp.pl> napisał w wiadomości 
news:hl6uvr$niu$1@inews.gazeta.pl...
> Jest sobie taki kod: http://pastebin.ca/1795545
>
> ==
> var cosTam = {
>         lastRefresh : 0,
>
>         autoRefresh : function(){
>             var dateobj=new Date();
>
>             if (dateobj.getTime()-cosTam.lastRefresh>10000) {
>                 cosTam.lastRefresh=dateobj.getTime();
>                 cosTam.refresh();
>             }
> },
> refresh : function () {
> // ....
> }
> }
>
>
> $(document).focus(cosTam.autoRefresh);
> ==
>
> Jako ze event focus jest wywolywany przy kazdym kliknieciu mysza, podpiety 
> pod to zdarzenie autoRefresh pilnuje aby nie dokonywac dzialan czesciej 
> niz co 10 sekund.
>
> Pytanie jest o linie
>
> var dateobj=new Date()
>
> Czy inicjalizacja powinna byc przeniesiona 2 linie wyzej, do deklaracji 
> obiektu cosTam?

Raczej nie. Dwie linijki wczesniej mialbys pamietana date z momentu 
rejestracji, a tak masz z ostatniego momentu gdy byl robiony refresh. Innymi 
slowy przesuniecie skutkowaloby szybkim zablokowaniem wywolywania refresha.

Re: Optymalizacja kodu

#49814

Author: keemor
Date: Sun, 14 Feb 2010 11:06

55 lines
2740 bytes

On 13 Lut, 20:36, Stefan <ste...@costam.wp.pl> wrote:
> Jest sobie taki kod:http://pastebin.ca/1795545
>
> ==
> var cosTam = {
>          lastRefresh : 0,
>
>          autoRefresh : function(){
>              var dateobj=new Date();
>
>              if (dateobj.getTime()-cosTam.lastRefresh>10000) {
>                  cosTam.lastRefresh=dateobj.getTime();
>                  cosTam.refresh();
>              }
>         },
>         refresh : function () {
>                 // ....
>         }
>
> }
>
> $(document).focus(cosTam.autoRefresh);
> ==
>
> Jako ze event focus jest wywolywany przy kazdym kliknieciu mysza,
> podpiety pod to zdarzenie autoRefresh pilnuje aby nie dokonywac dzialan
> czesciej niz co 10 sekund.
>
> Pytanie jest o linie
>
> var dateobj=new Date()
>
> Czy inicjalizacja powinna byc przeniesiona 2 linie wyzej, do deklaracji
> obiektu cosTam?

Mozesz z dateobj zrobic atrybut klasy costam i pozbyc sie var w ogole.

 var cosTam = {
          lastRefresh : 0,
         dateobj: null

          autoRefresh : function(){
              cosTam.dateobj = new Date()

              if (cosTam.dateobj.getTime()-cosTam.lastRefresh>10000) {
                  cosTam.lastRefresh=cosTam.dateobj.getTime();
                  cosTam.refresh();
              }
         },
         refresh : function () {
                // ....
         }

pozdrawiam
keemor

Re: Optymalizacja kodu

#49815

Author: Ghost
Date: Sun, 14 Feb 2010 12:19

10 lines
1418 bytes

Uzytkownik "keemor" <keemor@gmail.com> napisal w wiadomosci 
news:8c2a1e5a-0497-4425-bb4e-9949a4a7ede9@o30g2000yqb.googlegroups.com...
On 13 Lut, 20:36, Stefan <ste...@costam.wp.pl> wrote:
>> Czy inicjalizacja powinna byc przeniesiona 2 linie wyzej, do deklaracji
>> obiektu cosTam?

>Mozesz z dateobj zrobic atrybut klasy costam i pozbyc sie var w ogole.

Tylko po co?

Thread Navigation

This is a paginated view of messages in the thread with full content displayed inline.

Messages are displayed in chronological order, with the original post highlighted in green.

Use pagination controls to navigate through all messages in large threads.

Back to All Threads