Почему не срабатывает запись в файл?

Question

[KTG]

@echo off
SetLocal EnableDelayedExpansion
for /F "delims=" %%i in (pack.id) do set pid=%%i
echo Last pack number: %pid%

	set /P pid_flag=Обновить номер пакета? [Y/N]:
	if /i %pid_flag% EQU y (
	
		set /A pid=%pid%+1 > pack.id	
		goto :end_pid_question
	) 
	if /i %pid_flag% EQU n (
		set pid=%pid%
	) ELSE (
		echo Try again...
		goto :pid_question
	)

Пустой вывод в файл. Да же при отсутствии файла pack.id они создаются, но ничего не записывается.
ПО факту нужно брать оттуда цифру, делать +1 и записывать обратно в файл.
если делаю: echo %pid% > pack.id
пишет что вывод команд echo отключен.
если делаю:echo+%pid%>pack.id то почему-то записывает 3 при первом запуске, и больше ничего.

2. Есть ещё вторая проблема
set /P pid_flag=Обновить номер пакета? [Y/N]:
Если не вводить символы и просто нажать Enter, то вываливается "ошибка синтаксиса".
А обработать на отсутствие данных не получается.
Пробовал:
if not defined
if not exist
if not
Не взлетели (

Answer 1

[res2001]

если делаю: echo %pid% > pack.id
пишет что вывод команд echo отключен.

Это происходит когда переменная %pid% не определена (отсутствует).

Если не вводить символы и просто нажать Enter, то вываливается "ошибка синтаксиса".

Ошибка возникает на этой строке:
if /i %pid_flag% EQU y
когда переменная %pid_flag% не определена. Используйте такой вариант:
if /i "%pid_flag%" EQU "y"
И не ясно какого вывода в файл вы ждете?
Если от этой команды:
set /A pid=%pid%+1 > pack.id
то не дождетесь - set ничего не выводит на консоль, если она выполнилась успешно.

Comments

[KTG]

Тут на самом деле 2 разные ошибки, по проверке вводимого значения, и ошибка при обновлении номера.

1. Ошибка при запросе данных от пользователя:
Спасибо. Помогло, а то брал в кавычки только "Y" и получал другую ошибку
"Непредвиденное появление: "y".". Надо было ещё в кавычки взять "%pid_flag%

2. Ошибка при записи файла.
А как тогда выводить? Есть файл pack.id. там всего 1 цифра: 0
По идее я устанавливаю переменную
for /F "delims=" %%i in (./pack.id) do set pid=%%i
И в зависимости от выбора пользователя, хочу либо обновлять её в файле делая +1, либо оставлять какая есть.
Здесь прибавляю +1 к значению:
set /a pid=%pid%+1
И потом хочу записать в файл
echo %pid%>pack.id
Выдает ошибку на экран: Режим вывода команд на экран (ECHO) отключен.
Если брать имя файла в кавычки, то эту же ошибку пишет в файл.
Так то же не работает.
echo+%pid%>pack.id

При всём при этом, в конце файла я могу добавить строку echo Current pack:
%pid%
И увижу что значение он посчитал.

[KTG]

Если я в консоли вручную определяю переменную
set pid=1
а затем делаю
set pid=%pid%+1>pack.id
То переменная в файле обновляется. Т.е. ожидаю что и set может писать в файл.

[KTG]

по всей видимости echo+%pid%>pack.id все таки записывает в файл, но
почему-то только старое значение.

[res2001]

KTG, Устанавливать переменную из файла можно так:
set /p "pid="<pid.id
В переменной окажется содержимое последней строки файла.

Предусмотрите вариант, когда либо файла pid.id нет, либо он пустой (т.е. переменная pid из файла не проинициализировалась). После чтения файла:
if not defined pid set "pid=1"
В этом случае у вас pid всегда будет что-то содержать и echo %pid% не даст ошибки.

Кроме того, есть вариант без ошибочного echo:
echo.%pid%>pid.id
В таком варианте команды ошибки не будет, но если pid не определена, то в файл запишется только "перевод строки".

То переменная в файле обновляется. Т.е. ожидаю что и set может писать в файл.

Вы ошибаетесь. В файл пишется только то что команда выводит на консоль. Оператор > это оператор перенаправления вывода. Если вывода нет, то в файл ничего не запишется.

[res2001]

KTG,

echo+%pid%>pack.id

echo+ оказался аналогом echo. - не знал :-)
Чтоб set выполняла арифметические операции надо использовать set /a (см. set /?)

[KTG]

Вот полностью код:

@echo off
SetLocal EnableDelayedExpansion

set /p "pid="<pack.id
if not defined pis set "pid=1"
echo Last pack number: %pid%

:pid_question
	set /P pid_flag=Обновить номер пакета? [Y/N]:
	if /i "%pid_flag%" EQU "y" (
	
		set /a "pid="%pid%+1
		echo.%pid%>pack.id
		goto :end_pid_question
	) 
	if /i "%pid_flag%" EQU "n" (
		set pid=%pid%
	) ELSE (
		echo Try again...
		goto :pid_question
	)
	
:end_pid_question
echo Current pack number: %pid%

Всё равное не пишет новое значение, хотя в консоли выводит что прибавилось.
Ну и на самом деле если файла нет то он при set /a берет значение pid как 0

[res2001]

KTG, Используйте такой вариант:
echo.!pid!>pack.id
Это "отложенное расширение переменной среды", смотри setlocal /? и set /?
Отложенное расширение актуально когда вы в отдельном блоке (в скобках) присваиваете переменной новое значение и здесь же ее используете.
По умолчанию отложенное расширение выключено, а весь блок рассматриваются cmd как одна команда и переменные подставляются один раз при входе в блок. Соответственно при выключенном отложенном расширении переменные внутри блока имеют то же значение, что и при входе в блок (за исключением счетчиков цикла).

[KTG]

res2001, да. сработало.
*facepalm* Столько времени убил.... я уж и не помню зачем мне понадобилось отложенное расширение )))

Спасибо. всё заработало.