Wyświetalnie Tak lub Nie na podstawie Czasu - JavaScript

Question

Witam,
Na stronie sklepu chciałbym umieścić skrypt który pokazuje że sklep jest zamknięty bądź otwarty w zależności od aktualnej godziny i dnia tygodnia.

<script>
        function zegarek()
        {
            var godzina = data.getHours();
            var dzien = data.getDay();
            var otwarcie;
            var stan;
            if (godzina > 7 <18) otwarcie = 0 + 1;
            if (dzien => 0 < 6) otwarcie + 1;
            if (otwarcie == 2) stan = "otwarty"
            if (otwarcie =! 2) stan = "zamknięty"

           var pokazDate = "Aktualnie sklep jest " + stan;

Niestety skrypt nie działa. Ktoś ma pomysł jak to lepiej napisać? Poniżej daję zdjęcie w których dniach i godzianch miało by pokazywać "otwarte"

Comments

Masz gdzieś jakąś linię modyfikującą DOM? Bo jedyne co widzę to zapisywanie do zmiennej i nic więcej.

---

@nielot, na marginesie dodam, że taki przełącznik warto mieć również (a raczej przede wszystkim) po stronie serwera, żeby nie było sytuacji, że na froncie coś się wykrzaczy, albo ktoś to ograniczenie obejdzie i złoży zamówienie, gdy sklep rzekomo będzie zamknięty.

Answer 1

Zalecałbym korzystanie ze zmiennej "Let". Moim zdaniem "Var" jest troszkę przestarzałe

if (godzina > 7 <18) otwarcie = 0 + 1;
if (dzien => 0 < 6) otwarcie + 1;

Powyższe linijki są napisane niepoprawnie, nie ma takich warunków. Ja zapisałem to np tak. Wiem, że to może być skomplikowane ale po co pisać nie wiadomo ile linijek warunków

const box = document.querySelector('#box');    //U mnie ten element ma id "box"!

// const to stała w JavaScripcie (Możesz zamiast const wpisać var jak chcesz)

if ((godzina > 7) && (godzina < 18) && ((dzien >= 0) && (dzien < 6))) {


                box.innerHTML += '<b>Otwarty</b>';    //Oczywiście biorąc pod uwagę podłączenie do elementu HTML

            }

            else {
                box.innerHTML += '<b>Zamknięty</b>';    //Oczywiście biorąc pod uwagę podłączenie do elementu HTML
            }

 

Answer 2

Wyrażenie godzina > 7 <18 nie jest poprawne W jednym warunku możesz porównac zmienną tylko z jedną liczbą! Musisz to rozbić na dwa warunki i połączyć je operatorem &&. 

Czyli powinno być coś w ten deseń

if (godzina>=7 && godzina < 18)

To samo musisz z robić z warunkiem odnośnie dzień