Wat is er mis in mijn bat code?

[Djesse8999]

Ik was voor de lol even bezig in bat, daarvoor had ik een topic gemaakt wat bijna direct weer werd gesloten zonder dat ik kans had voor verbetering. hierbij mijn goede topic.

Batchfile: Test.bat

 @echo off
title IT Toolpack
color 02
echo                +----------+
echo                [ Toolpack ]
echo                +----------+
echo.
echo             Wich tool do you want to use? (X to exit)
echo.
echo             1 (Ipconfig /all) | 2 (Ping)
echo             3 (Tree)          | 4 (Shutdown -i)
echo.
pause
set /p keuze=
if %keuze%==1 goto a
if %keuze%==2 goto b
if %keuze%==3 goto c
if %keuze%==4 goto d
if %keuze%==X goto x

:a
cls
ipconfig /all
pause
goto begin

:b
cls
echo Wich ip to ping?
set /p IP
pause
cls
ping %IP%
pause
goto begin

:c
cls
Tree
pause
goto begin

:d
cls
shutdown -i
pause
goto begin

:x
cls
exit

Toen ik deze code probeerde te runnen sloot deze gelijk weer. Nadat ik de code had doorzocht had ik nog steeds niks gevonden ik heb ook nog geprobeerd als Administrator te runnen, Ook heb ik op internet rondgezocht en daar stond dat het waarschijnlijk een fout was Goh dat had ik ook nog niet bedacht

Ik kreeg een DM van Raven (Dank hiervoor) die zei dat het zo moet:

Batchfile: Selectiemenu.bat

ECHO 1 ga naar label1
ECHO 2 ga naar label2
ECHO 3 ga naar label3
ECHO.
CHOICE /C:123 /N /M "What would you like to do?:"

if "%errorlevel%"=="1" GOTO label1
if "%errorlevel%"=="2" GOTO label2
if "%errorlevel%"=="3" GOTO label3

Ik kom er niet helemaal uit, Hoe moet ik dat in mijn code inbrengen. Ik ben (nog) niet heel goed in bat dus hulp word gewardeerd.

Ik heb deze site gevonden maar snap er niet heel veel van http://www.robvanderwoude.com/choice.php

Alvast bedankt voor de hulp

( Als mijn vorige topic open bleef had ik hem aangepast naar deze maar die is gesloten zonder waarschuwing ofzo )

[Nielson]

Djesse8999 schreef op maandag 19 augustus 2013 @ 21:35:
Toen ik deze code probeerde te runnen sloot deze gelijk weer.

Het is handiger om ipv test.bat te dubbelklikken eerst een cmd prompt te openen en daarin vervolgens test.bat uit te voeren. Dan blijft je venster gewoon open en kun je zien waar het fout gaat. Evt kun je @echo off ook nog weghalen.

[Djesse8999]

Nielson schreef op maandag 19 augustus 2013 @ 21:41:
[...]
Het is handiger om ipv test.bat te dubbelklikken eerst een cmd prompt te openen en daarin vervolgens test.bat uit te voeren. Dan blijft je venster gewoon open en kun je zien waar het fout gaat. Evt kun je @echo off ook nog weghalen.

Ga ik proberen (gewoon run [pad van bat bestand] toch?)

[M2M]

nee, run, CMD, enter, navigeren naar pad van bat file, xxxx.bat, enter

[Nielson]

Gewoon in cmd navigeren naar de directory waar test.bat staat en vervolgens test(.bat) intypen en enter. Overigens gebruik je het pipe symbool (die |) als scheidingsteken en daar gaat het in eerste instatie al mis.

[Djesse8999]

Het is gelukt er staat

Batchfile:

C:\Users\Djesse>C:\Users\Djesse\Desktop\test.bat
                                +-------------+
                                 [ Toolpack ]
                                +-------------+

                           Wich tool do you want to use? (X tp exit)

2 word niet herkend als een interne
of externe opdracht, programma of batchbestand

Ik weet nu waar het probleem zit

Bij het keuze menu had ik eerst | gebruikt dat niet goed, nadat ik het had veranderd naar een - was het goed.

echo 1 (Ipconfig /all) | 2 (Ping)
echo 3 (Tree) | 4 (Shutdown -i)

Moest zijn

echo 1 (Ipconfig /all) - 2 (Ping)
echo 3 (Tree) - 4 (Shutdown -i)

Batchfile:

@echo off
title IT Toolpack
color 02
echo                +----------+
echo                [ Toolpack ]
echo                +----------+
echo.
echo             Wich tool do you want to use? (X to exit)
echo.
echo             1 (Ipconfig /all) - 2 (Ping)
echo             3 (Tree)             - 4 (Shutdown -i)
echo.
set /p keuze=
if %keuze%==1 goto a
if %keuze%==2 goto b
if %keuze%==3 goto c
if %keuze%==4 goto d
if %keuze%==X goto x

:a
cls
ipconfig /all
pause
goto begin

:b
cls
echo Wich ip to ping?
set /p IP
pause
cls
ping %IP%
pause
goto begin

:c
cls
Tree
pause
goto begin

:d
cls
shutdown -i
pause
goto begin

:x
cls
exit

Dank jullie nu red ik me verder!

-EDIT

eigenlijk wat nielson zei

[The Eagle]

Tip: maak van je .bat even een .cmd en gebruik @echo ipv echo en @pause ipv pause. Ziet er een stuk netter uit

[Djesse8999]

The Eagle schreef op maandag 19 augustus 2013 @ 21:57:
Tip: maak van je .bat even een .cmd en gebruik @echo ipv echo en @pause ipv pause. Ziet er een stuk netter uit

Wat is het verschil? ik zie het niet (Ik heb wel @echo off aan het begin.)

[Djesse8999]

Mijn uiteindelijke code:

Batchfile: Test.bat

@echo off
title IT Toolpack
color 02
:begin
cls
echo                +----------+
echo                [ Toolpack ]
echo                +----------+
echo.
echo             Wich tool do you want to use? (5 to exit)
echo.
echo             1 (Ipconfig /all) - 2 (Ping)
echo             3 (Tree)          - 4 (Shutdown -i)
echo.
set /p keuze=
if %keuze%==1 goto a
if %keuze%==2 goto b
if %keuze%==3 goto c
if %keuze%==4 goto d
if %keuze%==5 goto e

:a
cls
ipconfig /all
pause
goto begin

:b
cls
echo Wich ip to ping?
set /p IP=
pause
cls
ping %IP%
pause
goto begin

:c
cls
Tree
pause
goto begin

:d
cls
shutdown -i
pause
goto begin

:e
exit

Bedankt voor alle hulp, Weer wat geleerd

[Grijze Vos]

Leer dan ook meteen even hoe je 'which' moet spellen.

[Daos]

Geinig dat je voor retro-groen kiest. Sommigen van ons hebben dat nog meegemaakt (ik op de middelbare school; ze liepen een beetje achter).

Het was wel feller in mijn beleving. Geen color 02, maar meer color 0a:


Ook is de laatste exit niet nodig. Je hebt er zelfs last van: als je nu vanuit een Opdrachtprompt je script draait, sluit je deze prompt bij keuze 5. Dat wil je niet.

Ik zou ook de kleuren weer goed zetten (hoe je de titel kan herstellen kan ik niet vinden):

Batchfile: Test.bat

:e
color

[ Voor 0% gewijzigd door Daos op 19-08-2013 23:19 . Reden: color 02 ziet er ook mooi uit ]

[Djesse8999]

@grijse vos
ohja da wich was n spelfout, t was dan ook al laat en het was maar ff om te proberen

@daoz

Ik wou ook dat ie bij 5 sloot
5 to exit staat er.

qua titel denk ik dat ik de :begin net onder de @echo off ga zetten zodat ie t programma weer soort van opnieuw opstart, dan moet er wel een cls na komen maar dat spreekt voor zich.

batch is trouwens wel heel makkelijk, Java heb ik meer moeite mee


offtopic

Ik ben 13 en vind die oude pc's die een groen zwart scherm hebben geweldig of zo n oude ibm bak, vet spul!

Mijn school loopt ook achter met systeem beheer de ene pc draait op xp en de ander weer op 7 :facepalm:
je kan trouwens ook gewoon in de bios komen bij mij op school zijn al genoeg pc's door gesneufeld.
Dat novel wat ze daar gebruiken is trouwens ook niet alles.