Discussion :

[Olivier Regnier]

Bonjour,

Je souhaiterai rajouter au script un test afin que chaque argument ne soient pas utilisés seuls (tout en sachant qu'ils ne peuvent pas être combinés entre eux) sinon cela ne passe pas.

ex:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
KO
script.cmd -c
usage: script.cmd [-c | -g | -d | -v | -r] file

OK
script.cmd -c c:\

Voici le code:

Code bat :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
@echo off
 
REM.-- Prepare the command processor --
SETLOCAL ENABLEEXTENSIONS
SETLOCAL ENABLEDELAYEDEXPANSION
 
REM.-- Set the title --
SET title=%~nx0 %version%
TITLE %title%
 
REM.-- Do something useful --
 
:: Define the list of parameters
SET args=-c -g -d -v -r
 
:: Check parameters
SET "arg=%~1"
SHIFT
IF /I "%arg%"=="-c" (
    GOTO c
) ELSE IF /I "%arg%"=="-g" (
    GOTO g
) ELSE IF /I "%arg%"=="-d" (
    GOTO d
) ELSE IF /I "%arg%"=="-v" (
    GOTO v
) ELSE IF /I "%arg%"=="-r" (
    GOTO r
) ELSE (
    GOTO u
)
 
:c
ECHO C
GOTO:EOF
 
:g
ECHO G
GOTO:EOF
 
:d
ECHO D
GOTO:EOF
 
:v
ECHO V
GOTO:EOF
 
:r
ECHO R
GOTO:EOF
 
:u
ECHO usage: %0 [-c ^| -g ^| -d ^| -v ^| -r] file
GOTO:EOF

Pourriez-vous me donner un petit coup de main s'il vous plaît ?
Merci

[sachadee]

Salut Olivier,

Une solution :

Code bat :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
@echo off
 
REM.-- Set the title --
SET title=%~nx0 %version%
TITLE %title%
 
REM.-- Do something useful --
 
:: Define the list of parameters
SET "args=-c -g -d -v -r"
 
:: Test the validity of the parameters
for %%a in (%args%) do (
 if [%1]==[%%a] (
  if not [%2]==[] goto:%%a))
echo Argument manquant ou Invalide :
ECHO usage: %0 [-c ^| -g ^| -d ^| -v ^| -r] file
exit/b
 
:-c
ECHO C
GOTO:EOF
 
:-g
ECHO G
GOTO:EOF
 
:-d
ECHO D
GOTO:EOF
 
:-v
ECHO V
GOTO:EOF
 
:-r
ECHO R
GOTO:EOF
 
:-u
 
GOTO:EOF

[Olivier Regnier]

Coucou

Tout d'abord merci pour ton aide.

Voici le code complet:

Code bat :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
@echo off
 
REM.-- Prepare the command processor --
SETLOCAL ENABLEEXTENSIONS
SETLOCAL ENABLEDELAYEDEXPANSION
 
REM.-- Version history --
REM         X.X YYYYMMDD   Author Description
SET version=1.0 &:20141203 leon   initial version
SET version=%version: =%
 
REM.-- Set the default color of the command prompt --
COLOR 07
 
REM.-- Set the title --
SET title=%~nx0 %version%
TITLE %title%
 
REM.-- Do something useful --
 
:: Define the list of parameters
SET "args=-c -g -d -v -r"
 
:: Check the parameters
FOR %%a in (%args%) DO (
 IF [%1]==[%%a] (
  IF [%2]==[%%a] GOTO:u
  IF [%2]==[-] GOTO:u
  IF NOT [%2]==[] GOTO %%a
 )
)
GOTO:u
EXIT /b
 
:-c
ECHO C
GOTO:EOF
 
:-g
ECHO G
GOTO:EOF
 
:-d
ECHO D
GOTO:EOF
 
:-v
ECHO V
GOTO:EOF
 
:-r
ECHO R
GOTO:EOF
 
:u
ECHO usage: %0 [-c ^| -g ^| -d ^| -v ^| -r] file
GOTO:EOF

Voici la partie où j'ai rajouté deux tests supplémentaires:

Code bat :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
:: Check the parameters
FOR %%a in (%args%) DO (
 IF [%1]==[%%a] (
  IF [%2]==[%%a] GOTO:u
  IF [%2]==[-] GOTO:u
  IF NOT [%2]==[] GOTO %%a
 )
)
GOTO:u
EXIT /b

L'objectif c'est qu'il ne doit pas être possible de saisir 2 paramètres en même temps.

Le script a été testé et fonctionne correctement mais j'aimerai savoir si le code est propre ou pas ?

Encore merci

[sachadee]

Je te conseil également de tester si on a un 3éme argument.

Ensuite le test du 2éme argument tu peux le faire avant (call:test)

Donc je te conseille qqe chose comme ça :


(Tu n'a pas besoin de l'expansion retardée)

Code bat :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
@echo off
 
REM.-- Version history --
REM         X.X YYYYMMDD   Author Description
SET version=1.0 &:20141203 leon   initial version
SET version=%version: =%
 
REM.-- Set the default color of the command prompt --
COLOR 07
 
REM.-- Set the title --
SET title=%~nx0 %version%
TITLE %title%
 
REM.-- Do something useful --
 
:: Define the list of parameters
SET "args=-c -g -d -v -r"
 
:: Check the parameters
 
IF not [%3]==[] goto:x
call:test %2
 
for %%a in (%args%) do (
 if [%1]==[%%a] (
  if not [%2]==[] goto:%%a))
goto:u
exit/b
 
GOTO:u
EXIT /b
 
:-c
ECHO C
GOTO:EOF
 
:-g
ECHO G
GOTO:EOF
 
:-d
ECHO D
GOTO:EOF
 
:-v
ECHO V
GOTO:EOF
 
:-r
ECHO R
GOTO:EOF
 
:u
ECHO usage: %0 [-c ^| -g ^| -d ^| -v ^| -r] file
GOTO:EOF
 
:x
ECHO Error [%3] Too much parameters
GOTO:EOF
 
:test
 
set "$valeur=%1"
if "%$valeur:~0,1%"=="-" (
  ECHO Error [%1] Only one parmeter allowed
  exit/b
)

[Olivier Regnier]

Merci beaucoup, ton aide m'a été précieuse.

Avant de clôturer cette discussion, je pèche pour afficher le nombre total de fichiers traités dans un répertoire toujours pour le même script.

Voici le partie concernée:

Code bat :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
:: Generate the index file 
:-g
SET /a done=1
IF EXIST %2\%def_f% (
  ECHO %def_f% ^(generating^): file already present
 ) ELSE (
  ECHO %def_f% ^(generating^): complete
  ECHO.
  FOR /f %%b IN ('"dir /b /a:-d "%2\*.*" 2>nul"') DO (
   ECHO !done! of !done! ^= %%b: complete
   DIR /b /a:-d "%2\*.*" > %2\%tmp_f%
   SET /a done=!done!+1
  )
  TYPE %2\%tmp_f% | %t_fstr% -v "index.tmp" >> %2\%def_f%
  DEL %2\%tmp_f%
)
GOTO:EOF

à l'exécution, j'obtiens l'affichage suivant:

makeindex -g e:
index.txt (generating): complete

1 of 1 file: complete
2 of 2 file: complete
3 of 3 file: complete
4 of 4 file: complete
5 of 5 file: complete

au lieu de:
1 of 5 file: complete
2 of 5 file: complete
3 of 5 file: complete
4 of 5 file: complete
5 of 5 file: complete

Merci d'avance de ta et/ou vos réponses

[devilsnake88]

Bonjour,
Je serais d'avis de dire qu'il faudrait deux FOR, le premier pour compter le nombre total de fichiers dans le dossier et le second est celui que tu fais déjà.

Exemple:

Code bat :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
REM Dans ton FOR qui compte le nombre de fichiers du dossier
for /f %%A in ('dir /a-d-s-h /b ^| find /v /c ""') do set compteur=%%A
 
REM Puis dans ton FOR actuel
ECHO !done! of !compteur! ^= %%b: complete

Bon c'est expliqué à l'arrache mais je pense que c'est compréhensible (j'espère).