Problème d'écriture dans mon IF ELSE

Version imprimable

Bonsoir,
Lorsque j'exécute seulement le premier code ci-dessous, je supprime l'utilisateur dont le nom est marqué dans le fichier user.txt, dans mon 2eme code lorsque j'imbrique dans un if else %C% est vide.
Merci de m'éclairairé

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
@echo off
cls

TITLE %0 Script 

set FILE_USER=%CD%\tmp\user.txt

	REM On supprime l'ancien utilisateur
	set /P C= <%FILE_USER%" 
	echo %C%
	net user %C% /del

echo Appuyez sur une touche pour fermer... 
pause > nul

exit

Code:

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
@echo off
cls

TITLE %0 Script 

set FILE_USER=%CD%\tmp\user.txt

REM Creation du nouvel utilisateur
set /p NEW_USER=Saisissez le nouvel utilisateur:

if not exist %FILE_USER% (	
	REM On creer le fichier contenant le nom d'utilisateur
	echo %USERNAME%>%FILE_USER%  
	REM Creation de l'utilisateur
	net user %NEW_USER% /add
	net localgroup Administrateurs %NEW_USER% /add
) else (
	REM On supprime l'ancien utilisateur
	set /P C= <%FILE_USER%" 
	echo %C%
	net user %C% /del
)

echo Appuyez sur une touche pour fermer... 
pause > nul

exit

13/08/2017, 12h56
ericlm128
La faiblesse du batch que veux tu ...

Lorsqu'il y a un bloc d'instruction (boucle for, (), &) l'interpréteur de script batch n'évalue la variable qu'une seule fois avant le bloc.
Dans ton cas, lorsqu’il va dans le else il estime que ta variable C est vide.
Tu modifie bien la valeur de C mais n’empêche qu'il as décidé que durant ce bloc il te dira qu'elle est vide.

Pour le forcer à vérifier de nouveau la valeur réelle de ta variable, dans un bloc, tu doit utiliser SETLOCAL ENABLEDELAYEDEXPANSION et appeler tes variables avec des !

Exemple :
Code:

1 2 3 4 5 6 7 @echo off SETLOCAL ENABLEDELAYEDEXPANSION if "toto" == "toto" ( set /p aa= echo !aa! ) pause
Et oui, il faut être un sacré Geek du batch pour faire ce que tu souhaites.
13/08/2017, 22h21
dudux2

Ah je connaissais pas ce problème. (venant du shell unix)

Merci

Hello,

juste pour completer l'excellente reponse de Eric, tu peux egalement utiliser call

Code:

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
 
@echo off
cls
TITLE %0 Script
set "FILE_USER=%CD%\tmp\user.txt"
 
 
REM Creation du nouvel utilisateur
set /p NEW_USER=Saisissez le nouvel utilisateur:
 
if not exist %FILE_USER% (call:userNexist) else (call:userExist)
 
echo fin
echo Appuyez sur une touche pour fermer...
pause > nul
exit
 
:userExist
	REM On supprime l'ancien utilisateur
	set /P C= <%FILE_USER%"
	echo %C%
exit/b
 
:userNexist
	REM On creer le fichier contenant le nom d'utilisateur
	echo %USERNAME%>%FILE_USER%
	REM Creation de l'utilisateur
exit/b

;)

16/08/2017, 16h57
dudux2

Super!

Maintenant je sais aussi faire une fonction :lol:

Merci