Fonction PHP renvoyant 0 si en boucle [dupliquer]
J'ai une fonction PHP simple qui compte le nombre d'un chiffre spécifique dans un entier
function countOccurrence($number, $digit){
$result = 0; while ($number > 0){
$lastDigit = $number % 10;
if($lastDigit === $digit){
$result++; } $number = ($number - $lastDigit)/10;
}
return $result;
}
Ce qui fonctionne parfaitement tout seul [Testé avec countOccurrence (1111, 1) et renvoyé 4]
Mais quand je l'utilise dans une boucle pour une plage de nombres, il renvoie toujours 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;
}
Le résultat que j'obtiens est "nombre: 1 find: 1 countOccurrence (1, 1): 0" et ainsi de suite
Je ne sais pas vraiment où cela va mal ici car cela semble correct lorsque je teste la fonction manuellement
Edit: voici mon code complet
<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;
}
Edit: Merci pour les réponses! J'ai fini par analyser les valeurs $ _POST en int à la place comme ceci pour obtenir les valeurs attendues
$startNum = (int) $_POST['startNum'];
$endNum = (int) $_POST['endNum'];
$findNum = (int) $_POST['findNum'];
Réponses
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;
}
Puisque vous effectuez une comparaison stricte, ces valeurs ne sont jamais égales. Voici pourquoi:
Lorsque vous obtenez des données de $ _POST, il s'agit toujours d'une chaîne. Vous appelez donc countOccurrence(1111, '1')ce qui renverra 0 :)
Comme je l'ai mentionné dans les commentaires, vous pouvez remplacer vos fonctions par celles-ci:
- substr_count au lieu de
countOccurrence - range au lieu de
makeArray
Au cas où vous voudriez vous en tenir à vos fonctions. Convertissez votre entrée en un entier avant de le passer à votre fonction avec intval .
Information additionnelle
Pour éviter que cela ne se produise à l'avenir. Jetez un œil à ce post . C'est une fonctionnalité relativement nouvelle de php, qui a été introduite pour ce type de scénarios.
C'est là que se trouve votre petit bugbear ...
if($lastDigit === $digit){
Ce que vous attendez ici est une comparaison par valeur et par type (===).
Vous pouvez le vérifier en utilisant 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;
}
Les valeurs d'un POST sont invariablement de type String. Donc, comme $ digit est une chaîne et $ lastDigit est un nombre (entier). L'utilisation de === échouera car ils ne sont pas du même type.
Donc, comme PHP a la capacité de transtyper par lui-même ... changer le === en == fonctionnera. C'EST À DIRE
if ($lastDigit == $digit) {
PHP tentera de convertir la valeur RHS en type LHS puis effectuera la comparaison.
Alors maintenant, vous effectuez simplement une comparaison et PHP convertit $ chiffre en entier.
Un moyen plus sûr est de forcer le moulage. Ainsi, vous pouvez convertir $ digit d'une chaîne en un entier et utiliser votre code existant.
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;
}
Donc, la leçon à tirer de ceci, lorsque vous utilisez ===, assurez-vous que vos types sont également les mêmes.