Ok, I know the loop part is invalid, but hopefully helps give an idea of what I want to do. Ultimately, I want to get prompted for the 3 values (commented out the prompts for now and hard coding for ease of testing), then I want it to run through the loop once per CD (so 8 times in this example). I want it to name the .mp3 file with an incrementing CD#... CD1, CD2, CD3, etc. If anyone has any improvements, those are welcome too! I'm posting my full code here to share with the community.
@ECHO OFF REM set /p BookName="Book Name: " REM set /p AuthorName="Author Name: " REM set /p CDNumber="Number of CDs: " set BookName=Victory and Honor set AuthorName=W.E.B. Griffin set CDNumTot=8 set PathName=C:\Rip\%AuthorName%\%BookName% mkdir "C:\Rip\%AuthorName%\%BookName%\" REM cd C:\Program Files (x86)\freac ECHO CD Count: %CDNumTot% FOR /L %%N IN (1, 1, %CDNumTot%) DO ( Set FileName=%PathName%\%BookName% CD%%N.mp3 ECHO Filename: %FileName% ECHO\ ECHO Ripping CD# %%N... ECHO freaccmd -track all -o %FileName% ECHO CLS ECHO\ ECHO\ set /A NextCD=%%N ECHO Next: %NextCD ECHO Please Eject CD and insert next CD# %NextCD%... pause ) Final version of this code (WORKING thanks to @JoeNahmias):
@ECHO OFF SETLOCAL ENABLEEXTENSIONS ENABLEDELAYEDEXPANSION set /p BookName="Book Name: " set /p AuthorName="Author Name: " set /p CDNumTot="Total number of CDs: " set PathName=C:\Rip\%AuthorName%\%BookName% FOR /L %%N IN (1, 1, %CDNumTot%) DO ( cls ECHO\ ECHO Please insert CD# %%N... ECHO\ pause SET FileName=%PathName%\%BookName% CD%%N.mp3 cls ECHO\ ECHO Ripping CD# %%N... ECHO\ "C:\Program Files (x86)\freac\freaccmd" -track all -o "!FileName!" REM You need the id3 executable downloaded/installed for the next line: REM id3 -l "!BookName!" -a "!AuthorName!" -n %%N "!FileName!" rundll32 user32.dll,MessageBeep -1 ECHO\ ECHO Please eject CD# %%N... pause ) cls ECHO\ ECHO DONE! pause 1 Answer
You want the FOR /L command. For example:
C:\>SET num=7 C:\>FOR /L %I IN (1, 1, %num%) DO @echo %I 1 2 3 4 5 6 7 Note that you have to double the lead percent sign in a batch file. So, for your CD ripping loop:
FOR /L %%n IN (1, 1, %CDNumTot%) DO ( SET FileName=%PathName%\%BookName% CD %%n.mp3 ... ) If you're having difficulty, try putting the following in a batch file and seeing if you get it working:
@ECHO OFF SET num=7 FOR /L %%I IN (1, 1, %num%) DO @echo %%I Try this for a complete solution:
@ECHO OFF SETLOCAL ENABLEEXTENSIONS ENABLEDELAYEDEXPANSION REM set /p BookName="Book Name: " REM set /p AuthorName="Author Name: " REM set /p CDNumTot="Total number of CDs: " set BookName=Victory and Honor set AuthorName=W.E.B. Griffin set CDNumTot=8 set PathName=C:\Rip\%AuthorName%\%BookName% REM mkdir "C:\Rip\%AuthorName%\%BookName%\" REM cd C:\Program Files (x86)\freac ECHO CD Count: %CDNumTot% FOR /L %%N IN (1, 1, %CDNumTot%) DO ( ECHO Please Eject CD and insert next CD# %%N... pause SET FileName=%PathName%\%BookName% CD%%N.mp3 ECHO Filename: !FileName! ECHO. ECHO Ripping CD# %%N... ECHO freaccmd -track all -o "!FileName!" ECHO CLS ECHO. ECHO. ) 
