FizzBuzzのマックス範囲をコマンドラインに入力した値（ターミナルから受け取った値）にしたい。

Question

phpを勉強し始めたのですが、（php　version 7.4）
以下のFizzBuzz問題
•コマンドラインから引数xを受け取る（xは２０以上）
•１〜xまでの自然数を出力
３の倍数の時Fizz
５の倍数の時Buzz
その両方の倍数の時FizzBuzzと出力。

php ファイル名　x（20以上の自然数）

この問題のコマンドラインから引数xを受け取るやり方（解決済み）、またxを20以上に設定するやり方がわかりません。
以下、自分が書いたコードです。（

$argv = $argv[1];
if ($argv > 20); {
	for($i = 1; $i < $argv + 1 ; $i++) {
		if($i % 15 === 0){
			echo "FizzBuzz\n";
		}
		else if($i % 3 == 0){
			echo "Fizz\n";
		}
		else if($i % 5 == 0){
			echo "Buzz\n";
		}
		else{
			echo $i."\n";
		}	
	}
}
echo 'error';

このようなソースコードで

php ファイル名(index.php)　25

を実行すると1~25までのFizzBuzzを表示させることができました！
しかし、
if文で＄argvの値を20以上に設定したつもりですが、現状20以下の数値でも実行されます。
どのように訂正したらよいでしょうか。

Comments

[otn]

コマンドラインから受け取るのと、標準入力から受け取るのとの区別付いてますか？

[tomotomo_1D]

申し訳ございません。
標準入力の意味をあまり理解していないまま、標準入力という言葉を使っていました。
質問内容を訂正致しました。

Answer 1

if ($argv > 20); {

ifのTHEN部分がその直後の;で終わりなので、{以降はifと無関係で、常に実行されます。
つまり、;を取ります。

Comments

[tomotomo_1D]

ありがとうございます！
; を外すことで無事完成させることができました！

Answer 2

otn さんがすでに指摘してくれていますが、tomotomo_1D さんが実装しているやり方は「標準入力から値を読み取る」方法です。
コマンドラインでの引数で値を与える方法は以下となります。

$argv - php.net

質問にある、「20 以上であること」はforの前にifで条件分岐すると良いですよ。

Comments

[tomotomo_1D]

アドバイスありがとうございます。
te2jiさんのアドバイスに沿って改良した結果、少し正解に近づくことができました。ありがとうございます。しかし、まだ
・２０以上の自然数であること
・php ファイル名　25　と打ち込んでも1~２４までのFizzBuzzが表示され２５まで表示されない
という問題が残っています。どのように修正すればよいでしょうか。
質問文も若干変更させていただきました。

[tomotomo_1D]

上記の
・php ファイル名　25　と打ち込んでも1~２４までのFizzBuzzが表示され２５まで表示されない
という問題が残っています。
という問題はたった今、解決できました。

[tomotomo_1D]

無事解決することができました！
ありがとうございます！

[退会済みユーザー]

> ・２０以上の自然数であること

これ、満たしてないです。
以上、以下、未満あたりは境界値を念のためテストすると良いです。

あと、error が想定入力値に対しても出力されるので、else 等を使用して適切に制御してください。