Batcher Gesucht

[mark13]

Hallo,
Kann jemand der mehr Ahnung hat als ich mein Batch Programm durchschaun?
Es sollte ein Wecker sein.
----------------------------------------------------------------
@echo off

echo Wecker
:Question
set /p a=Geben sie die gewünschte Zeit (hh:tt:ss,ms)ein:
:time
if %time%==%a% goto wecker
goto time

:wecker
Es ist %time%
:: Ton hier einfügen!
pause
set /p b=Neuer wecker erstellen (a) ausschalten (b):
if %b%==a goto qustion
exit
------------------------------------------------------------------

Funtzt nicht wirklich , Warum?

 

[Ram-Brand]

Hmm ist schon zu lange her.
Mir sind aber folgende Tippfehler aufgefallen:

Hallo,

@echo off

echo Wecker
:Question
set /p a=Geben sie die gewünschte Zeit (hh:tt:ss,ms)ein:
:time
if %time%==%a% goto wecker
___ goto time

:wecker
Es ist %time%
:: Ton hier einfügen!
pause
set /p b=Neuer wecker erstellen (a) ausschalten (b):
if %b%==a goto qustion
exit

Muß bei ___ nicht ein else hin. Heißt doch immer IF THEN ELSE.

 

[mark13]

Danke, ich selbst finde fast nie Tippfehler bei mir

Muß bei ___ nicht ein else hin. Heißt doch immer IF THEN ELSE.

Nein weil der vergleich [if] nicht zutrifft (und ich ja nicht verneine) springt das Programm zur nächsten Zeile
Bei anderen Batch Programmen funktioniert das aber auch

Hast aber recht bei anderen Programmsprachen wie c+ muss da ein 'else' hin

 

[Tibor]

@echo off

echo Wecker
:Question
set /p a=Geben sie die gewünschte Zeit (hh:tt:ss,ms)ein:
:time
if "%time%"=="%a%" goto wecker
goto time

:wecker
Es ist %time%
REM:: Ton hier einfügen!
pause
set /p b=Neuer wecker erstellen (a) ausschalten (b):
if "%b%"=="a" goto question
exit

So funktioniert's.
Sobald 'a' nicht nur zusammenhängende Zeichen enthält, sollte 'a' mit Hochkomma eingeschlossen sein.
Gleiches für %time%, bei der zweiten Abfrage ist das nicht ungedingt nötig, der User kann aber auch sowas wie 'weiss ich doch nicht' antworten, was einen Syntaxfehler bei der Verarbeitung ergibt. '::' gibt auch einen Syntaxfehler, besser den Kommentar mit 'REM' anfangen.

Nach den ':wecker' könnte man noch folgendes einfügen :
if "%date:~3,6%"=="01.01." exit
if "%date:~3,6%"=="02.01." exit
if "%date:~3,6%"=="03.01." exit
if "%date:~3,6%"=="04.01." exit

 

[mark13]

Hey danke