Einrichten eines JavaScript, das schreibt, ob der Store geöffnet oder geschlossen ist

Question

Ich versuche, ein einfaches JavaScript einzurichten. Wenn der Tag Sonntag oder Samstag ist, wird im HTML "Closed" geschrieben, oder wenn es später ist 17:30 Uhr, oder es ist früher als 9:00 Uhr, wird es auch geschlossen schreiben.

Mein Hauptproblem ist, dass die if-Anweisung immer wahr zu sein scheint. Ich habe mehrere Dinge ausprobiert, um das zu beheben, aber ohne Erfolg. Jede Hilfe würde sehr geschätzt werden.

Code: http://pastebin.com/R5kE6fX6

Answer 1

Dies ist keine vollständige Antwort, es ist eher ein fortgeschrittener Kommentar zum Code-Styling.

Ihre if-Anweisungen sehen kompliziert aus, ich verstehe sie immer noch nicht. Um Ihren Code zu verbessern, machen Sie ihn lesbarer. Das erleichtert es Ihnen und uns, es zu verbessern. Zum Beispiel:

status="opened"; 

if(date=="wednesday"){ 
    if(time>closetime){ 
     status="closed. Its too late"; 
    } 
    if(time<opentime){ 
     status="closed. youre to early"; 
    } 
} else { 
    //code for other dates 
} 

//echo code 
document.write(status); 

Answer 2

Dieser Code Versuchen:

// Place back slashes in front of any grammar, used for checking open or closed based on day and time. 
var dayornight = new Date(); 
var opened = 9; 
var closed = [23, 59]; 
var closedThurs = 19; 
var closedWed = 16; 
var day = dayornight.getDay(); 

alert(day); 

if(day == 0 || day == 6 || dayornight.getHours() < opened) { 
    if(day == 4 && dayornight.getHours() >= closedThurs || day == 3 && dayornight.getHours() >= closedWed) { 
    if(dayornight.getHours() >= closed[0] && dayornight.getMinutes() >= closed[1]){ 

     document.write("<span class=\"Closed\">Closed</span>"); 
     alert("Closed is working"); 

    } 
    } 
} else { 
    document.write("<span class=\"Open\">Open</span>"); 
    alert("Opened else is working"); 
} 

Answer 3

Sie testen, ob ein String mit Inhalt hat:

if("day == 0 || day == 6 || dayornight.getHours() < opened") {} 

In diesem Beispiel wird die ganze Bedingung ist eine Zeichenfolge (gekapselt von "). Und if("some string") wird true zurückgeben, solange die Zeichenfolge mindestens ein Zeichen enthält.

Sie müssen Bedingungen als JS-Code verwenden - was bedeutet, alle überflüssigen Anführungszeichen zu entfernen.

sollte diese Arbeit:

if(day == 0 || day == 6 || dayornight.getHours() < opened) {} 

By the way: Bitte starten Sie einen Code-Editor mit Syntax-Hervorhebung verwendet wird, während der Entwicklung. Wie Sie in den obigen zwei Schnipsel sehen können, ist eine Zeichenfolge vollständig in einer Farbe, während verschiedene Teile des Codes in einem Editor mit Syntaxhervorhebung unterschiedliche Farben erhalten können. Dies ist sehr hilfreich, wenn man solchen Fehlern gegenübersteht.