ループ内の場合は0を返すPHP関数[重複]
Aug 18 2020
整数の特定の桁の数を数える単純なPHP関数があります
function countOccurrence($number, $digit){
$result = 0; while ($number > 0){
$lastDigit = $number % 10;
if($lastDigit === $digit){
$result++; } $number = ($number - $lastDigit)/10;
}
return $result;
}
これはそれ自体で完全に正常に機能します[countOccurrence（1111、1）でテストされ、4が返されました]
しかし、数値範囲のループで使用すると、常に0が返されます。
if (isset($_POST['submit'])){
$startNum = $_POST['startNum'];
$endNum = $_POST['endNum'];
$findNum = $_POST['findNum'];
$occur = 0; echo 'from ' . $startNum . ' to ' . $endNum . ' find how many times ' . $findNum . ' occurs';
echo '<br>';
$numRange = makeArray($startNum, $endNum); foreach ($numRange as $number){ $test = countOccurrence($number, $findNum);
echo 'number: ' . $number . ' find: ' . $findNum . ' countOccurrence('.$number.', '.$findNum.'): ' . $test . '<br>'; } echo '<br>'; echo '<p>The digit ' . $findNum . ' occurred ' . $occur . ' times within the range</p>'; echo '<br>'; echo 'countOccurrence(1111, 1) = '; $test = countOccurrence(1111, 1);
echo $test;
}
これから得られる結果は、「number：1 find：1 countOccurrence（1、1）：0」などです。
関数を手動でテストすると問題ないように見えるので、ここでどこがうまくいかないのかよくわかりません
編集：これが私の完全なコードです
<html lang="en">
<head>
<title>Number Counter</title>
<link rel="stylesheet" href="bootstrap/css/bootstrap.min.css">
</head>
<body>
<div class="container">
<form id="form" class="form-group" action="" method="post">
<div class="form-group">
<label for="startId">Start Num</label>
<input type="number" name="startNum" id="startId" class="form-control">
</div>
<div class="form-group">
<label for="endId">End Num</label>
<input type="number" name="endNum" id="endId" class="form-control">
</div>
<div class="form-group">
<label for="findId">Find Num</label>
<input type="number" name="findNum" id="findId" class="form-control">
</div>
<input type="submit" name="submit">
</form>
</div>
<script src="bootstrap/js/jquery-3.5.1.min.js"></script>
<script src="bootstrap/js/jquery.validate.min.js"></script>
<script src="bootstrap/js/bootstrap.min.js"></script>
<script src="validate.js"></script>
</body>
</html>
<?php
function countOccurrence($number, $digit){ $result = 0;
while ($number > 0){ $lastDigit = $number % 10; if($lastDigit === $digit){ $result++;
}
$number = ($number - $lastDigit)/10; } return $result;
}
function makeArray($startNum, $endNum){
$numRange = array(); for($ctr = $startNum; $startNum <= $endNum; $startNum++){
array_push($numRange, $startNum);
}
return $numRange; } if (isset($_POST['submit'])){
$startNum = $_POST['startNum'];
$endNum = $_POST['endNum'];
$findNum = $_POST['findNum'];
$occur = 0; echo 'from ' . $startNum . ' to ' . $endNum . ' find how many times ' . $findNum . ' occurs';
echo '<br>';
$numRange = makeArray($startNum, $endNum); foreach ($numRange as $number){ $test = countOccurrence($number, $findNum);
echo 'number: ' . $number . ' find: ' . $findNum . ' countOccurrence('.$number.', '.$findNum.'): ' . $test . '<br>'; } echo '<br>'; echo '<p>The digit ' . $findNum . ' occurred ' . $occur . ' times within the range</p>'; echo '<br>'; echo 'countOccurrence(1111, 1) = '; $test = countOccurrence(1111, 1);
echo $test;
}
編集：答えてくれてありがとう！期待値を取得するために、このように代わりに$ _POST値をintに解析することになりました。
$startNum = (int) $_POST['startNum'];
$endNum = (int) $_POST['endNum'];
$findNum = (int) $_POST['findNum'];
回答
SirPilan Aug 18 2020 at 10:04
function countOccurrence($number, $digit){
$result = 0; while ($number > 0){
$lastDigit = $number % 10;
if($lastDigit === $digit){ // <-- bug is here
$result++; } $number = ($number - $lastDigit)/10;
}
return $result;
}
厳密な比較を行っているため、これらの値が等しくなることはありません。理由は次のとおりです。
$ _POSTからデータを取得する場合、それは常に文字列です。だからあなたはcountOccurrence(1111, '1')0を返す呼び出しています:)
コメントで述べたように、関数を次のものに置き換えることができます。
- substr_count の代わりに
countOccurrence - range の代わりに
makeArray
あなたがあなたの機能に固執したい場合に備えて。入力をintに変換してから、intvalを使用して関数に渡します。
追加情報
これが将来起こるのを防ぐため。この投稿をご覧ください。このタイプのシナリオで導入されたphpの比較的新しい機能。
TimBrownlaw Aug 18 2020 at 10:10
これはあなたの小さなバグベアがいるところです...
if($lastDigit === $digit){
ここで期待しているのは、値とタイプ（===）による比較です。
gettype（）;を使用してこれを確認できます。
while ($number > 0) { $lastDigit = $number % 10; echo '$lastDigit is a '.gettype($lastDigit); echo ' $digit is a '.gettype($digit); echo '<br>'; if ($lastDigit === $digit) { $result++;
}
$number = ($number - $lastDigit) / 10;
}
POSTからの値は、常にString型です。したがって、$ digitは文字列であり、$ lastDigitは数値（整数）です。===は同じタイプではないため、使用は失敗します。
したがって、PHPにはそれ自体でタイプキャストする機能があるため、===を==に変更すると機能します。IE
if ($lastDigit == $digit) {
PHPは、RHS値をLHSタイプに変換してから、比較を実行しようとします。
これで、比較を実行しているだけで、PHPは$ digitを整数に変換しています。
より安全な方法は、鋳造を強制することです。したがって、$ digitを文字列から整数にキャストして既存のコードを使用できます。
function countOccurrence($number, $digit) { $result = 0;
$digit = intval($digit); // Force $digit to be an integer while ($number > 0) {
$lastDigit = $number % 10;
echo '$lastDigit is a '.gettype($lastDigit);
echo ' $digit is a '.gettype($digit);
echo '<br>';
if ($lastDigit === $digit) {
$result++; } $number = ($number - $lastDigit) / 10;
echo ' $number is a '.gettype($number);
}
return $result;
}
したがって、これからの教訓は、===を使用するときに、タイプも同じであることを確認してください。