Moin,
display_check_bild();
anderer Funktionsname als definiert
function display_bild() {var now = new Date(); hour = now.getHours(); minutes = now.getMinutes();
Warum speicherst du dies in extra Variablen? Nutze sie direkt!
var display_hour = 15; var display_min = 30;
Hier wäre besser du erstellst die Funktion mit 2 Argumenten und übergibst diese von "außen"... damit kannst du die Funktion beliebig of wiederverwenden
if (hour = display_hour && minutes = display_min)
Hier muss du Vergleichen, nicht zuordnen (== oder === statt =)
{document.getElementById('bild').style.display="";}
die Style-Eigenschaft display darf nicht leer sein. Nutze hierzu doch bitte den Standard (inline oder block je nach Element)
Ganz klasse wäre es, wenn die Möglichkeit bestehen würde, dass die Grafik zu diesem Zeitpunkt ausgetauscht wird. Also bis 15:29 Uhr wird Bild1 angezeigt, um 15:30 Uhr Bild2 und ab 15:31 wieder Bild1. Daran habe ich mich allerdings noch nicht versucht.
Behebe erstmal die anderen Fehler. Dann würde ich an deiner Stelle erst diese Problematik angehen.
Damit du den Effekt siehst, dein Script mal optimiert:
display_bild(15,30);
function display_bild(hour,minute)
{
var now = new Date();
if (now.getHours() == hour && now.getMinutes() == minute) {
document.getElementById('bild').style.display="inline";
}
}
Gruß Bobby
-> Für jedes Problem gibt es eine Lösung, die einfach, sauber und falsch ist! <- ### Henry L. Mencken ### -> Nicht das Problem macht die Schwierigkeiten, sondern unsere Sichtweise! <- ### Viktor Frankl ### ie:{ br:> fl:{ va:} ls:< fo:) rl:( n4:( de:> ss:) ch:? js:( mo:} sh:) zu:)