Countdown fehler

[Boulder Dash]

habe hier ein countdown für eine schule, im großen und ganzen funktioniert er, aber bei einigen zeiten schätze ich mal das sich die bedingungen überschneiden. Und zwar bei 9:00-9:30 und 10:00-11:15 friert die uhr ein.
Und bei 13:00-13:15 zählt der countdown die seit bis zu 15:00 runter

/*
Stundenzeiten:
  8:00 -   9:30
  9:45 - 11:15
11:45 - 13:15
13:30 - 15:00
*/
myDate = new Date();
sek = myDate.getSeconds();
min = myDate.getMinutes();
std = myDate.getHours();
tag = myDate.getDay();
 
sek2 = 59 - sek;
 
    erg = (std2 * 60) + min2
    length(erg) ==2
 
if (std >= 15 || std < 7 || tag == 0 || tag ==6) {
    if (length(min) == 1) {
        min = "0" + min;
    }
    if (length(sek) == 1) {
        sek = "0" + sek;
    }
    anzeige = std + ":" + min + ":" + sek;
    }   
                         else{
// zeit bis zur 1. stunde   8:00
    if (std >= 7 && std <8) {
        min2 = 59 - min;
        std2 = 7 - std;
    
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur 1. Stunde" ;
    }
        else{
// zeit bis zur 1. Pause    8:00 - 9:30
        if (std >= 8 && std <= 9 && erg <90) {
            min2 = 29 - min;
            std2 = 9 - std;
 
            anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur Pause" ;
        }
    }
// zeit bis zur 2. stunde   9:30 - 9:45
    if (std == 9 && min >= 30 && min < 45) {
        min2 = 44 - min;
        std2 = 9 - std;
 
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur 2. Stunde" ;
    }
    else{
// zeit bis zur 2. Pause    9:45 - 11:15
        if (std >= 9 && std <= 11 && erg <=90) {
            min2 = 14 - min;
            std2 = 11 - std;
 
            anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur Pause" ;
        }
    }
// zeit bis zur 3. stunde   11:15 - 11:45
    if (std == 11 && min >= 15 && min < 45) {
        min2 = 44 - min;
        std2 = 11 - std;
 
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur 3. Stunde" ;
    }
    else{
// zeit bis zur 3. Pause    11:45 - 13:15
        if (std >= 11 && std <= 13 && erg <=90) {
            min2 = 14 - min;
            std2 = 13 - std;
 
            anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur Pause" ;
        }
    }
// zeit bis zur 4. stunde   13:15 - 13:30
    if (std == 13 && min >= 15 && min < 30) {
        min2 = 29 - min;
        std2 = 13 - std;
 
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur 4. Stunde" ;
    }
    else{
// zeit bis zum schluss 13:30 - 15:00
        if (std >= 13 && std <= 15) {
            min2 = 59 - min;
            std2 = 14 - std;
        
            anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zum Schluss" ;
        }
    }
} 

bedanke mich für Lösungen

[Holmes]

ich hatte keine Lust mir dein Skript durchzulesen. Dafür möchte ich mich mal entschuldigen.

Aber versuch mal eine Variable für minuten zu finden, die nicht gleichzeitig den kleineren Wert von zweien auswertet. (min als Variable dürfte nicht gut gehen)

[Boulder Dash]

Zitat von Holmes

ich hatte keine Lust mir dein Skript durchzulesen. Dafür möchte ich mich mal entschuldigen.

Aber versuch mal eine Variable für minuten zu finden, die nicht gleichzeitig den kleineren Wert von zweien auswertet. (min als Variable dürfte nicht gut gehen)

ok zugegeben, min als var is nen bissel unglücklich gewählt, aber daran liegt es nicht.
und das mit den auswerten von 2 werten müsste eigendlich ok sein. ich denke nur das die if bedingung anderst geschieben werden muss, das da keine überschneidungen (oder was auch immer) mehr drin is

[hgseib]

sehr fleisig! 6 - setzen ;-)
a) rechne nur mit einer bezugsgrösse. das wären hier z.b. die minuten.
b) samstag, sonntag kannst du auch vorher rausrechnen. an diesen tagen ist die urzeit wohl egal.
c) wehe dir, die zeiten ändern sich. dann musst du dein ganzes programm umstricken. besser ist es, sowas in einem array zu verwalten. da kann man locker mal ne stunde zugeben ;-)

schulzeit = new Array();
schulzeit.push({bez:'zur 1. Stunde', ab:8*60+00});
schulzeit.push({bez:'zur 1. Pause', ab:9*60+30});
schulzeit.push({bez:'zur 2. Stunde', ab:9*60+45});
schulzeit.push({bez:'zur 2. Pause', ab:11*60+15});
schulzeit.push({bez:'zur 3. Stunde', ab:11*60+45});
schulzeit.push({bez:'zur 3. Pause', ab:13*60+15});
schulzeit.push({bez:'zur 4. Stunde', ab:13*60+30});
schulzeit.push({bez:'zum Unterrichtsende', ab:15*60+00});
//
function fN(zeit, text) {
    var n = zeit == 1 ? '' : 'n';
    return zeit ? zeit+' '+text+n+' ' : '';
}
//
//var jetzt = new Date();
//var min = jetzt.getMinutes();
//var std = jetzt.getHours();
// zum testen:
var std = 10;
var min = 1;
//
var testmin = std*60+min;
var Meldung = 'Feierabend!';
for (var i = 0; i<schulzeit.length; i++) {
    if (testmin<schulzeit[i].ab) {
        min = schulzeit[i].ab-testmin;
        std = int(min/60);
        min -= std*60;
        var Meldung = 'Noch '+fN(std, 'Stunde')+fN(min, 'Minute')+'bis '+schulzeit[i].bez+'.';
        break;
    }
}
trace(Meldung); 

fN ergänzt. weil: das heisst nicht 1 Stunden ;-)

[Holmes]

sehr sehr schön!!!
Das hätte ich in Tagen nicht hinbekommen ... :-)

[Boulder Dash]

Zitat von hgseib

sehr fleisig! 6 - setzen ;-)
a) rechne nur mit einer bezugsgrösse. das wären hier z.b. die minuten.
b) samstag, sonntag kannst du auch vorher rausrechnen. an diesen tagen ist die urzeit wohl egal.
c) wehe dir, die zeiten ändern sich. dann musst du dein ganzes programm umstricken. besser ist es, sowas in einem array zu verwalten. da kann man locker mal ne stunde zugeben ;-)

sorry hat nen bissel länger mit der antwort gedauert...
möchte dir erstmal ganz herzlich danken, is nen super code, aber da ich noch anfänger bin, bleib ich mal bei meiner variante, deinen hebe ich mir mal für später auf .
denke mal das ich das problem jetzt gelöst habe. habe deinen rat mit nur einer bezugsgröße mal zu herzen genommen. und zwar lasse ich die stunden mit einer variablen "zeit2" überprüfen, guckst du hier:

zeit2 = std add min 

so kann keine überschneidung mehr eintreffen... aber obs wirklich funktioniert wird der praxistest zeigen

ps. weiß nicht ob ich nen fehler gemacht habe. aber bei deinen code wird bei jeder zeit "Noch 1 Stunde 14 Minuten" ausgegeben...

[hgseib]

"..aber bei deinen code wird bei jeder zeit "Noch 1 Stunde 14 Minuten" ausgegeben.."
scharfsinnig beobachtet!
muss wohl daher kommen, das ich die zeiterfassung auskommentiert habe und ZUM TESTEN einen festen wert vorgebe:
//var jetzt = new Date(); <-- die // löschen
//var min = jetzt.getMinutes(); <-- die // löschen
//var std = jetzt.getHours(); <-- die // löschen
// zum testen: <-- löschen
var std = 10; <-- löschen
var min = 1; <-- löschen

"..obs wirklich funktioniert wird der praxistest zeigen.."
aaah ja. du wartest zum testen also immer bis es 12 uhr schlägt? na malzeit :-)))


??? hauptsache du weisst, was du da machst
zeit2 = std add min
also
2 std + 10 min = 12 was?

mein vorschlag war gewesen mit nur einer bezugsgrösse zu arbeiten. also nicht stunden und minuten getrennt behandeln, was solche konstrukte benötigt:
if (std == 11 && min >= 15 && min < 45)
sondern alle zeiten komplett in z.b. minuten umrechnen. also
zeit2 = std*60 + min
2 std *60 + 10 min = 130 minuten

---------------
hab nochmal einen blick auf deinen code geworfen. da tränen mir die augen ;-) als ich nach drei stunde wieder halbwegs klar sehen konnte:
schmeiss diese kontrukte weg!!!
if (..) {
...
} else {
if (...) {
...
}
}

schreib's so
if (..) {...}
if (...) {...}
if (...) {...}
...

oder so
if (..) {...}
else if (...) {...}
else if (...) {...}
else if (...) {...}
...


und es gibt bei dir kein std2??
erg = (std2 * 60) + min2

und was soll das da sein???
length(erg) ==2

dieses erg ist bei dir auch mehr ein 'ärg'ernis als eine sinnvolle funktion? :-)

[Boulder Dash]

dank dir noch mal für deine antwort, und tut mir leid dass ich dir solche Kopfschmerzen verursache . Habe das schon öfters zu hören bekommen das meine code´s verwirrend unlogisch und weiß der Geier was noch…

also hatte vor ner Woche mit deinem Code rumgespielt, und selbstverständlich die auskommentierten Sachen wech genommen, aber irgendwie hab ich´s dann doch nicht hinbekommen, aber wie gesagt ich hebe mir den für später auf, momentan is das noch zu viel Stoff für mich… (wollte auch auf gar keinen fall sagen das dass falsch oder so is)

"aaah ja. du wartest zum testen also immer bis es 12 uhr schlägt? na malzeit :-)))"
nein mache ich bestimmt nicht, ich meine nur das man den Rechner in der schule sowie so im blick hat und wenn da was falsch geht dann sieht man das ja, deswechen "Praxistest"…

"??? hauptsache du weisst, was du da machst
zeit2 = std add min
also
2 std + 10 min = 12 was?"
da muss ich dich korrigieren „+“ und „add“ is nicht dasselbe:
nehmen wir 10:12
+ --> 22
add --> 1012
somit kann es nicht mehr zu Überschneidungen durch die min. kommen da man die zeit komplett eingrenzen kann

"und es gibt bei dir kein std2??
erg = (std2 * 60) + min2"
is auch nicht richtig, std2 wird jeweils in der if klammer deklariert, klingt komisch funktioniert aber...

hier nochmal der aktuelle code, is zwar auch nicht das wahre vom ei, aber naja guck einfach selbst, wenn du denkst du braucht ne brille um deine augen vor augenkrebs zu schützen tu dir kein zwang an

myDate = new Date();
sek = myDate.getSeconds();
min = myDate.getMinutes();
std = myDate.getHours();
tag = myDate.getDay();
    
    
if (length(min) == 1) {
    min = "0" + min;
}
 
zeit2 = std add min
sek2 = 59 - sek;
erg = (std2 * 60) + min2
 
if (zeit2 < 700 || zeit2 >= 1500 || tag == 0 || tag == 6) {
 
    if (length(sek) == 1) {
        sek = "0" + sek;
    }
    anzeige = std + ":" + min + ":" + sek;
    }
else{
// zeit bis zur 1. stunde   8:00
    if (zeit2 >= 700 && zeit2 < 800) {
        min2 = 59 - min;
        std2 = 7 - std;
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur 1. Stunde" ;
    }
// zeit bis zur 1. Pause    8:00 - 9:30
    if (zeit2 >= 800 && zeit2 < 930) {
        min2 = 29 - min;
        std2 = 9 - std;
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur Pause" ;
    }
// zeit bis zur 2. stunde   9:30 - 9:45
    if (zeit2 >= 930 && zeit2 < 945) {
        min2 = 44 - min;
        std2 = 9 - std;
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur 2. Stunde" ;
    }
// zeit bis zur 2. Pause    9:45 - 11:15
    if (zeit2 >= 945 && zeit2 < 1115) {
        min2 = 14 - min;
        std2 = 11 - std;
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur Pause" ;
    }
// zeit bis zur 3. stunde   11:15 - 11:45
    if (zeit2 >= 1115 && zeit2 < 1145) {
        min2 = 44 - min;
        std2 = 11 - std;
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur 3. Stunde" ;
    }
// zeit bis zur 3. Pause    11:45 - 13:15
    if (zeit2 >= 1145 && zeit2 < 1315) {
        min2 = 14 - min;
        std2 = 13 - std;
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur Pause" ;
    }
// zeit bis zur 4. stunde   13:15 - 13:30
    if (zeit2 >= 1315 && zeit2 < 1330) {
        min2 = 29 - min;
        std2 = 13 - std;
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zur 4. Stunde" ;
    }
// zeit bis zum schluss 13:30 - 15:00
    if (zeit2 >= 1330 && zeit2 < 1500) {
        min2 = 59 - min;
        std2 = 14 - std;
        anzeige ="noch " + erg + " min. und " + sek2 + " sek. bis zum Schluss" ;
    }
} 

achso das mit length(erg) ==2
war nen überbleibsel von nem versuch, den ich vergessen hatte wieder zu löschen...