it-swarm.com.ru

% x был неожиданным в это время. пакетный скрипт

@echo off
for /f "tokens=1,2 delims=," %%x in (my.csv) do (
if %M% LSS %%x set M=%%x
)
echo Max X Value= %M%

Иногда это работает нормально, иногда это терпит неудачу со следующей ошибкой:

%x was unexpected at this time.
11
stack

Проблема в том, что вы используете %m% внутри цикла for. Это оценивается, когда цикл читается (перед любыми итерациями вообще). Другими словами, весь цикл, вплоть до закрывающей круглой скобки, читается и оценивается перед выполнением. Таким образом, %m% всегда будет его начальным значением, независимо от того, что вы фактически установили в цикле.

Надеюсь, пример должен проиллюстрировать это:

set val=7
for %%i in (1) do (
    set val=99
    echo %val%
)
echo %val%

что приводит к неожиданному (для некоторых):

7
99

просто потому, что %val% в первом операторе echo интерпретируется (т. е. интерпретируется весь цикл for) перед выполнением любого из них.

Вам нужно отложенное расширение вместе с чем-то, что приведет к тому, что значение m будет установлено равным первому %%x независимо. Использование команды setlocal и !m! вместо %m% задержит оценку m до тех пор, пока строка не будет выполнена.

Кроме того, если изначально установить для m пустое значение и задать для него значение %%x, когда оно is nothing, то первое значение %%x будет загружено в m.

@echo off
setlocal enableextensions enabledelayedexpansion
set m=
for /f "tokens=1,2 delims=," %%x in (my.csv) do (
    if "!m!" == "" set m=%%x
    if !m! lss %%x set m=%%x
)
echo Max X Value = !m!
endlocal

Используя приведенный выше код с этим файлом my.csv:

1,a
2,b
10,c
3,d

в результате получается:

Max X Value = 10

как ожидается, или, для ваших образцов данных в другом комментарии:

422,34
464,55
455,65
421,88

ты получаешь:

Max X Value = 464
19
paxdiablo

Не задана переменная среды с именем M, поэтому при интерпретации этой строки:

if %M% LSS %%x set M=%%x

После 1-го раунда замен это выглядит для переводчика как

if LSS %x set M=%x

Чтобы избежать проблем, о которых упоминает paxdiablo about %M% расширяется только при первом цикле, вы можете использовать обсуждаемую им функцию отложенного расширения или перенести тестирование и настройку M в подпрограмму, которая вызывается из цикла, но существует вне цикл, поэтому он интерпретируется (и расширяется) при каждом вызове:

@echo off
set M=
for /f "tokens=1,2 delims=," %%x in (my.csv) do (
    call :setmax %%x
)
echo Max X Value= %M%
goto :eof

:setmax
if "%M%" == "" set M=%1
if %M% LSS %1 set M=%1
goto :eof
2
Michael Burr

Проблема в вашем if %M% утверждении. Где% M%? объявите это сначала, например

@echo off
set M=""
for /f "tokens=1,2 delims=," %%x in (file) do (
if %M% LSS %%x set M=%%x
)
echo Max X Value= %M%

Альтернативно, вы можете использовать vbscript

Set objFS = CreateObject("Scripting.FileSystemObject")
Set objArgs = WScript.Arguments
strFile = objArgs(0)
Set objFile = objFS.OpenTextFile(strFile)
t=0
Do Until objFile.AtEndOfStream
    linenum = objFile.Line
    strLine = objFile.ReadLine  
    s = Split(strLine,",")
    For Each num In s       
        WScript.Echo "num "&num
        If Int(num) >= t Then
            t=Int(num)
        End If  
    Next
    WScript.Echo "Max for line:" & linenum & " is " & t
Loop

пример вывода

C:\test>type file
422,34464,55455,65421,88

C:\test>cscript //nologo test.vbs file
Max for line:1 is 65421

ОБНОВЛЕНИЕ: чтобы найти максимальное значение столбца мудрый

Set objFS = CreateObject("Scripting.FileSystemObject")
Set objArgs = WScript.Arguments
strFile = objArgs(0)
Set objFile = objFS.OpenTextFile(strFile)
t=0
Do Until objFile.AtEndOfStream
    linenum = objFile.Line
    strLine = objFile.ReadLine  
    s = Split(strLine,",")
    If Int(s(0)) >= t then      
            t=Int(s(0))
    End If      
Loop
WScript.Echo "Max is " & t & " (line: " & linenum & ")"

Результат

C:\test>type file
422,34
464,55
455,65
421,88

C:\test>cscript //nologo test.vbs file
Max is 464 (line: 2)
0
ghostdog74