Kolejność wykonywania kodu

Question

Dlaczego w tym kodzie gdy `fuel` otrzymuje wartość większą niż 0 ,wykonuje się 

"Najpierw musisz włączyć silnik."

a nie ten drugi `if`

drive: function() {                 
                if (this.started) {
                    if (this.fuel > 0) {
                        alert(this.make + " " +
                              this.model + " robi: brum wrrrr!");
                        this.fuel = this.fuel - 1;
                    } else {
                        alert("Osz..., brak paliwa.");
                        this.stop();
                    } 
                } else {
                    alert("Najpierw musisz włączyć silnik.");
                }
            },

 

Update. Ale metoda `start` wygląda tak:

start: function() {                
                if (this.fuel == 0) {
                    alert("Bak jest pusty, zatankuj przed jazdą!");
                } else {
                    this.started = true;
                }
            },

Czyli `started` jest `true`

 

Cały kod

var fiat = {                   
            make: "Fiat",
            model: "500",
            year: 1957, 
            color: "szaroniebieski",
            passengers: 2,
            convertible: false,
            mileage: 88000,
            fuel: 0,
            started: false,
        
            start: function() {               
                if (this.fuel == 0) {
                    alert("Bak jest pusty, zatankuj przed jazdą!");
                } else {
                    this.started = true;
                }
            },
        
            stop: function() {        
                this.started = false;
            },
        
            drive: function() {                
                if (this.started) {
                    if (this.fuel > 0) {
                        alert(this.make + " " +
                              this.model + " robi: brum wrrrr!");
                        this.fuel = this.fuel - 1;
                    } else {
                        alert("Osz..., brak paliwa.");
                        this.stop();
                    } 
                } else {
                    alert("Najpierw musisz włączyć silnik.");
                }
            },
        
            addFuel: function(amount) {           
                this.fuel = this.fuel + amount;
            }
        };
        
        fiat.start();        
        fiat.drive();        
        fiat.addFuel(2);    
        fiat.start();       
        fiat.drive();       
        fiat.drive();        
        fiat.drive();       
        fiat.stop();        
        

 

Answer 1

Ponieważ wartość this.started nie zwraca true.

Comments

No ale przecież sam napisałeś sobie nawet odpowiedź w zadaniu :D.

Najpierw musisz włączyć silnik.

Czyli started wynosi false.

---

@DeBos123, zerknij proszę, uaktualniłem kod.

---

@kenjiro244, Zerknij proszę na uaktualnienie postu.

---

@Kubs, pokaż kod, który używa tych funkcji.

Może nie wywołujesz metody start albo wywołujesz ją mając 0 paliwa, czyli this.started nie jest ustawiane na true.

---

Umieściłem cały kod. Zerknij proszę.

---

Twój kod jakby go zapisać jako pseudokod to wyglądałby tak:

Stwórz samochód
Uruchom samochód (nie masz paliwa)
Jedź (nie uruchomiłeś samochodu z powodu braku paliwa)
Zatankuj
Uruchom samochód
Jedź
Jedź
Jedź
Zatrzymaj samochód

Napisałem ci w nawiasach w czym prawdopodobnie leży problem.

---

No ale przecież mam bo  2>0 a wykonuje się ten else podczas gdy powinien ten `if`

---

Dodajesz paliwo dopiero po pierwszej próbie uruchomienia i ruszenia samochodem.

---

Tak , potem wykonuje się start gdzie started przyjmuje true i potem wykonuje się drive gdzie zamiast if wykonuje sie else.

---

Uruchomiłem twój kod i działa prawidłowo.

Pierwsze co robisz po stworzeniu samochodu to próba uruchomienia go.

this.fuel==0, więc dostajesz komunikat o braku paliwa.

this.started==false, ponieważ samochód się nie uruchomił z powodu braku paliwa.

Następnie próbujesz ruszyć tym samochodem.

this.started==false, więc dostajesz informację, że musisz uruchomić samochód.

---

Przekompilowałem kod w głowie i wszystko wydaje się być ok. Poszukaj czy gdzieś nawiasu nie urwało.

 

Daj więcej paliwa !!! Tak z 15 w tamtej lini

---

W jego kodzie się nawet nie uruchamia ta funkcja przed komunikatem o włączeniu silnika.

Objaśniłem to wyżej.

---

        fiat.start();        //started=false paliwo=0
//Bak jest pusty.
        fiat.drive();        //started=false paliwo=0
        fiat.addFuel(2);    //paliwo=2
        fiat.start();       //started=true
        fiat.drive();       //paliwo=1
        fiat.drive();        //paliwo=0
        fiat.drive();       
        fiat.stop();  

 

---

Autor pytania pyta się czemu dostaje komunikat "Najpierw musisz włączyć silnik."

Komunikat ten pojawia się jeżeli przy wywołaniu funkcji drive, this.started==false, co się dzieje w linii nr 45 (w kawałku kodu, który wkleiłeś jest to linia nr 2), więc funkcja addFuel nie jest wywołana przed wyświetleniem tego komunikatu.

---

@DeBos123, No ale this.started==true zaraz po dodaniu paliwa i wykonaniu metody start

---

Ale dodajesz paliwo w linii nr 46.

Samochód uruchamiasz w linii nr 44, a próbujesz nim ruszyć w kolejnej.

Czyli:

Uruchamiasz samochód
Jedziesz
Tankujesz paliwo

Robisz to w złej kolejności.

---

Ok, ok rozumiem teraz o co Ci chodzi :) Po prostu ten komunikat "Najpierw...." wykonuje się w tym pierwszym wywołaniu metody drive.

Ja cały czas referowałem do tej metody drive zaraz po addFuel.

Czy to o to chodzi?

Czasami takie nieuważnie niedopatrzenia mogą być bardzo czasochłonne :)

DZIĘKUJE SERDECZNIE.

---

Czy to o to chodzi?

Tak. 

---

Naprawdę doceniam, że mi pomogłeś i byłeś wytrwały.

---

Czyli zamknięte?

---

Napisał, że pomogłem, więc chyba tak.