画像サイズの長辺を1000pxに変更したい

Question

実現したいこと

複数の画像をアスペクト比はそのままで
長辺が1000pxとなるよう一括変換したいです。

前提

同一フォルダ内にある複数画像の長辺が1000pxとなるように
バッチファイルを作成したいです。

発生している問題・エラーメッセージ

一度バッチファイルを実行し、長辺が1000pxとなった画像を
再度バッチにかけると短辺が1000pxになってしまいます。
運用上、同じ画像を複数回変換する想定はありませんが、
原因が分かればご教示いただきたくお願いいたします。

例：
元画像：600×800
↓バッチ1回目実行
750×1000
↓バッチ2回目実行
1000×1333　（本来は750×1000から変わらない想定）

該当のソースコード

setlocal enabledelayedexpansion

REM # EXP=拡張子を入れてください
set EXP=jpg
REM # SIZE=長辺のサイズを入れてください
set SIZE=1000

for %%f in (.%EXP%) do (
REM 画像の縦幅を取得
for /f "usebackq tokens=" %%i in (magick identify -format '%%h' %%f) do @set HEIGHT=%%i

REM 画像の横幅を取得
for /f "usebackq tokens=*" %%i in (magick identify -format '%%w' %%f) do @set WIDTH=%%i

if !HEIGHT! GEQ !WIDTH! (
REM 縦幅 >= 横幅
magick convert -resize x%SIZE% %%f %%~nf.%EXP%
) else (
REM 縦幅 < 横幅
magick convert -resize %SIZE%x %%f %%~nf.%EXP%
)
)

Comments

[bsdfan]

(回答に記載)

Answer 1

-format '%%h' みたいにシングルクォートをつけているので、HEIGHT や WIDTH がシングルクォート付きになっていて、GEQ で '750' と '1000' を文字列として比較してしまっているんじゃないかと思います。(文字列だと '750' のほうが大きい)

そうだとすると、もともとのサイズが 4桁 x 3桁 だったりすると同じ問題が発生している可能性があります。

-format %%h とクォートを取るか、-format "%%h" とダブルクォートにすればいいのではないでしょうか。

Comments

[misat]

ご回答いただきありがとうございます。
教えていただいた方法で解決いたしました。