wywołanie funkcji Opcje

[SaMi]

Witam, mam mały problem i najprawdopodobniej jest to banał. Mianowicie gdy tworzę zewnetrzny plik js poprzez utworzenie tagu <script> w DOMie i wywoluję zawartą w nim funkcje uzyskuję komunikat ReferenceError: user is not defined. natomiast jeśli załaduję klasycznie w htmlu ten sam plik wszystko działa bez problemu.

[HTML] pobierz, plaintext 
<script type="text/javascript">
(function(d,s){
    var js,jsr = d.getElementsByTagName(s)[0];
    js = d.createElement(s);
    js.src = '//www.mojadomena.pl/js/iframe.js';
    jsr.parentNode.insertBefore(js,jsr);
}(document,'script'));
 
user.setAccount(['12345']);	
</script>
[HTML] pobierz, plaintext 

iframe.js w wielkim uproszczeniu

[HTML] pobierz, plaintext 
// (function(w){ - tego tu miało tu nie być 
var user = user || (function(){
_args = {};
 
		return {
			setAccount: function(a){
				_args = a;
			},
			key: function(){
				return _args[0];
			}					
		};
}());
[HTML] pobierz, plaintext 

Ten post edytował SaMi 30.12.2013, 10:30:00

[lukasz1985]

Odpowiedź kryje się w tej całej ceregieli, którą wykonujesz żeby oszczędzić globalną przestrzeń nazw.
Zmienna "user" to zmienna lokalna anonimowej funkcji w iframe.js. Domyślam się, ze chciałeś przesłać do tej funkcji zmienną window (po literce "w" w argumencie) i tam przypisać ją w następujący sposób na końcu:

[JAVASCRIPT] pobierz, plaintext 
w.user = user
[JAVASCRIPT] pobierz, plaintext 

Poza tym, po co takie kombinacje? Po co do funkcji anonimowej przesyłać zmienną document, skoro to zmienna globalna zawsze widoczna w każdym zasięgu?

Przerost formy nad treścią. Proponowałbym zacząć od importowania skryptów zwykłym znacznikiem <script> , a potem interesować się jakimis tam formami usprawniania tego procesu.

Sprawdziłbym jeszcze czy nie lepiej byłoby użyć append zamiast insertBefore. Tutaj specjalistą nie jestem jednak myślę, że problemem może być to, że skrypty są zaimportowane w złej kolejności.

[SaMi]

zakradł się mały błąd który skorygowałem.
Jest to implementacja wyszukiwarki którą pośrednicy zamieszczają na swoich stronach. Na dzień dzisiejszy eksperymentuję i uczę się, także proszę o wyrozumiałość i ewentualne rady

[lukasz1985]

[JAVASCRIPT] pobierz, plaintext 
function(d,s){
 
    var js,jsr = d.getElementsByTagName(s)[0];
 
    js = d.createElement(s);
 
    js.src = '//www.mojadomena.pl/js/iframe.js';
 
    jsr.parentNode.insertBefore(js,jsr);
 
}(document,'script')); // Zupełnie nie potrzebne
 
[JAVASCRIPT] pobierz, plaintext 

[JAVASCRIPT] pobierz, plaintext 
function(){
 
    var js,jsr = document.getElementsByTagName(s)[0];
 
    js = document.createElement("script");
 
    js.src = '//www.mojadomena.pl/js/iframe.js';
 
    jsr.parentNode.insertBefore(js,jsr);
 
}(); 
 
[JAVASCRIPT] pobierz, plaintext 

Poza tym proponowałbym - zamiast używania anonimowych funkcji do korzystania z "przestrzeni nazw":

[JAVASCRIPT] pobierz, plaintext 
var mojmodul = {};
mojmodul.zrobCos = function(){
 
    var js,jsr = document.getElementsByTagName(s)[0];
 
    js = document.createElement("script");
 
    js.src = '//www.mojadomena.pl/js/iframe.js';
 
    jsr.parentNode.insertBefore(js,jsr);
 
}; 
 
mojmodul.zrobCos(); 
 
[JAVASCRIPT] pobierz, plaintext 

Anonimowe funkcje są fajne - dla małych fragmentów kodu - przynajmniej takie moje zdanie po tym jak potrafią one namieszać w kodzie, który w ostateczności staje się jednym wielkim śmietnikiem. Moduły w dużym stopniu elminują konieczność ich używania jeśli chodzi o moduły. W kodzie czysto obiektowym potrzebne jest już zupełnie inne podejście do tematu.

[SaMi]

Zmieniłem na przestrzenie nazw faktycznie łatwiej to ogarnać. Aczkolwiek moje pytanie z pierwszego posta jest nadal aktualne.

[lukasz1985]

Odpowiedź na Twoje pytanie jest dość prosta. Otóż tworząc z poziomu JavaScript tag "script" i ustawiając jego źródło informujesz tym samym przeglądarkę, że ma wykonać kolejne zapytanie pod podane źródło, a więc robiąc tak:

[JAVASCRIPT] pobierz, plaintext 
 
  var js = ddocument.createElement("script");
  js.src = "iframe.js";
  document.body.appendChild(js);
  user.setAccount("213");
[JAVASCRIPT] pobierz, plaintext 

przeglądarka zacznie wgrywać nowy adres, w tym przypadku plik skryptu znajdującego się pod ścieżką "iframe.js".

Tak się składa, że nie dzieje się to natychmiast, a ładowanie następuje asynchronicznie, w tle, tak jak w przypadku XHR (XmlHttpRequest, czy krócej "AJAX"). Oznacza to, że zmienna "user" zdefiniowana w tamtym pliku nie pojawi się natychmiast, a dopiero po wykonaniu całego procesu: pobrania pliku js i wrzucenia go w dokument.

Aby wykonać kod po załadowaniu tego dokumentu należy się posłużyć funkcją zwrotną ("callback'iem"):

[JAVASCRIPT] pobierz, plaintext 
 
  var js = ddocument.createElement("script");
  js.src = "iframe.js";
  document.body.appendChild(js);
 
  js.onload = function() {
    user.setAccount("213");
  }
 
[JAVASCRIPT] pobierz, plaintext 

Ten post edytował lukasz1985 30.12.2013, 22:38:33