Не работает проверка на число в массиве/GET?

Question

[Человек]

Данные из html формы умножаются друг на друга,
а именно два элемента массива $_GET.
Затем результат умножения выводится отдельной переменной $equals на последней итерации.

Но у меня не работает условие(if), проверка элементов массива на тип число.
Я добавил проверку на число is_integer, чтобы скрипт(умножение) срабатывал,
когда пользователь введет два целочисленных значения в html форму.
Как это пофиксить? Условие не работает =(

<form method ="GET">   
Форма:                                                                                     
<div> 1 число:    <input type="text" name="num1" />    </div>     
<div> 2 число   : <input type= "text" name="num2" /> </div>  
<input type="submit" value="Отправить">
</form>

<br>
<br>

$request = $_GET;

$names = ["num1"=> "1-число:",  "num2"=> "2-число:"];
$default =  ["num1"=> "По умолчанию",  "num2"=> "По умолчанию"];

$equals = 0;
$counter = 0;


foreach($names as $key => $items){


if(isset($request[$key]) && is_integer($request[$key]) ){    // почему не работает это условие?
$equals  = $request["num1"] * $request["num2"];                             
$default[$key] =  htmlentities($request[$key]);                                                                                                                            
}                                                                                    


echo "$names[$key] $default[$key]  <br>";

if(++$counter == count($names)){  // сработает на последней итерации цикла
echo "Равно: ".$equals;
}

}

Comments

[scottparker]

а что да дичь там вообще написана?) что за цикл?

[Человек]

scottparker, ну покажите пример своего кода на моей основе.
мне крайне интересно =)

[scottparker]

Добрый Дельфин, не знаю точной задачи, но
что бы просто перемножить два числа достаточно
$equals = $request["num1"] * $request["num2"];
окей, ты хочешь проверить что значения, являются числами - тогда так

if(is_numeric($request["num1"]) && is_numeric($request["num2"])){ 
   $equals  = $request["num1"] * $request["num2"];
}

[Человек]

scottparker, оказывается, нужно было
просто заменить is_integer на is_numeric и все заработало.

К слову:

if(is_numeric($request["num1"]) && is_numeric($request["num2"])){

тоже самое, что:
if( is_numeric($request[$key]) ){

[scottparker]

Добрый Дельфин, оно то может и показывает правильный результат, но но не значит что это правильно)

и нет, это не тоже самое

[Человек]

scottparker, мне сказали, что я рачонок, так как я сделал двойное умножение.
$equals = $request["num1"] * $request["num2"];

Я пометил комментом строчку.
Я же умножаю один раз, когда условие будет верно.
Я не могу понять, почему идет умножение два раза =(

$request = $_GET;

$names = ["num1"=> "1-число:",  "num2"=> "2-число:"];
$default =  ["num1"=> "По умолчанию",  "num2"=> "По умолчанию"];

$equals = 0;
$counter = 0;

foreach($names as $key => $items){
if(isset($request[$key]) && is_numeric($request["num1"]) && is_numeric($request["num2"]) ){    
$equals  = $request["num1"] * $request["num2"];       // разве здесь идет умножение два раза?           
$default[$key] =  htmlentities($request[$key]);                                                                                                                            
}                                                                                    

echo "$names[$key] $default[$key]  <br>";

if(++$counter == count($names)){  
echo "Равно: ".$equals;
}}

[scottparker]

Добрый Дельфин, все верно сказали, поддерживаю. зачем цикл?

почему ты думаешь что условие выполняется 1 раз?

[Человек]

scottparker, значит, умножение нужно убрать из цикла?
а именно убрать:
$equals = $request["num1"] * $request["num2"];

то есть, чтобы умножение работало снаружи цикла?

[scottparker]

Добрый Дельфин, третий раз спрошу, зачем цикл? зачем эти два массива вначале?

[Человек]

scottparker, цикл чтобы перебирать элементы массива,
а умножение сделал, чтобы посмотреть как это будет работать в цикле.
Как я понял, надо умножение убрать из цикла, так как оно два раза умножается.

Answer 1

[AUser0]

Совсем не то.

$names = ["num1"=> "1-число:",  "num2"=> "2-число:"];
$default =  ["num1"=> "По умолчанию",  "num2"=> "По умолчанию"];
$equals = 0;

foreach($names as $key => $items){
    if(isset($_GET[$key]) && is_number($_GET[$key]) ){
        $default[$key] =  $_GET[$key];
    }
    echo "$items ".htmlentities($default[$key])."<br>\n";
}

$equals  = $default["num1"] * $default["num2"];
echo "Равно: ".$equals."<br>\n";

Comments

[Человек]

В коде ошибка:
Fatal error: Uncaught TypeError: Unsupported operand types: string * string in

$names = ["num1"=> "1-число:",  "num2"=> "2-число:"];
$default =  ["num1"=> "По умолчанию",  "num2"=> "По умолчанию"];
$equals = 0;

foreach($names as $key => $items){
    if(isset($_GET[$key]) && is_integer($_GET[$key]) ){
        $default[$key] =  $_GET[$key];
    }
    echo "$items ".htmlentities($default[$key])."<br>\n";
}

$equals  = $default["num1"] * $default["num2"];
echo "Равно: ".$equals."<br>\n";

[Человек]

Вот финальная моя версия:
Использовал strlen благодаря вам.
При помощи strlen получилось сделать так,
если одна из строк будет пустая, то умножение не сработает.
Например: если в одну строку ввести значение, а в другую нет и нажать кнопку,
то умножение не сработает.

В итоге, умножение сработает,
если пользователь введет цифры в оба поля в html форму.
Как и задумывалось мною.

<form method ="GET">   
Форма:                                                                                     
<div> 1 число:    <input type="text" name="num1" />    </div>     
<div> 2 число   : <input type= "text" name="num2" /> </div>  
<input type="submit" value="Отправить">
</form>

$request = $_GET;
 
$names = ["num1"=> "1-число:",  "num2"=> "2-число:"];
$default =  ["num1"=> "По умолчанию",  "num2"=> "По умолчанию"];
 
$equals = 0;
$counter = 0;
 
 
foreach($names as $key => $items){
 
 
if(isset($request[$key]) && is_numeric($request[$key]) && strlen($request["num1"]) && strlen($request["num2"]) ){    
$equals  = $request["num1"] * $request["num2"];                             
$default[$key] =  htmlentities($request[$key]);                                                                                                                            
}                                                                                    
 
echo "$names[$key] $default[$key]  <br>";
 
if(++$counter == count($names)){  
echo "Равно: ".$equals;
}
}

Почему вы убрали $equals = $default["num1"] * $default["num2"]; из условия isset?

[AUser0]

Добрый Дельфин, потому что вы не понимаете, что делает foreach(), сколько раз будет повторено вычисление $equals и проверен ваш второй if, и почему не надо пихать strlen($request["num1"]) && strlen($request["num2"]) внутрь foreach(). И видимо даже не желаете понимать, иначе использовали бы новый вариант.

[Человек]

AUser0, но в вашем варианте ошибка =(
значит, мне нужно убрать умножение из цикла foreach и выводить умножение и его результат вне цикла?

[AUser0]

Добрый Дельфин, тогда исправьте эту ошибку:

$equals  = intval($default["num1"]) * intval($default["num2"]);

Кстати, вам на заметку, intval("По умолчанию") вернёт число 0.

[Человек]

AUser0, ошибка была в том, что нет такого метода is_number
заменил is_number на is_numeric и ваш код заработал.
Спасибо вам!

$names = ["num1"=> "1-число:",  "num2"=> "2-число:"];
$default =  ["num1"=> "По умолчанию",  "num2"=> "По умолчанию"];
$equals = 0;

foreach($names as $key => $items){
    if(isset($_GET[$key]) && is_numeric($_GET[$key]) ){ // вот здесь ошибка.
        $default[$key] =  $_GET[$key];
    }
    echo "$items ".htmlentities($default[$key])."<br>\n";
}

$equals  = $default["num1"] * $default["num2"];
echo "Равно: ".$equals."<br>\n";

Но когда только заходишь на страницу, когда ничего не нажал выходит это:

Fatal error: Uncaught TypeError: Unsupported operand types: string * string

Ваш код внизу убрал ошибку. Как я понимаю вначале, когда заходишь на страницу и ничего не жмешь,
то происходит умножение пустой строки на пустую строку, что и вызывало ошибку ранее,
поэтому мы преобразуем эти пустые строки в числа при помощи intval?

$equals  = intval($default["num1"]) * intval($default["num2"]);

Немного поигрался и добавил условие, чтобы умножение происходило,
когда массив $GET не пустой и когда пользователь введет любые два значения в виде чисел в html форму

$request = $_GET;                                                                                                              
$names = ["num1"=> "1-число:",  "num2"=> "2-число:"];
$default =  ["num1"=> "По умолчанию",  "num2"=> "По умолчанию"];
$equals = 0;

foreach($names as $key => $items){
if(!empty($request[$key]) ){        
$default[$key] =  htmlentities($request[$key]);                                                                                                              
}                                                                                    
echo "$names[$key] $default[$key]  <br>";
}


if(!empty($request) && is_numeric($request["num1"]) && is_numeric($request["num2"])){
$equals  = $request["num1"] * $request["num2"];  
}
echo "Равно:". $equals;

Выражаю вам огромную благодарность, я много чего понял, благодаря вам =)

Answer 2

[rPman]

$_GET возвращают только строки и массивы, а is_integer проверяет тип переменной, а не ее значение.

Чтобы проверить по значению, есть is_numeric, для строк "1" или " -0.123" оно будет true но "1-ая" уже false

Answer 3

[ThunderCat]

1) Полям желательно дать тип number, тогда есть некоторая вероятность что в нее будут попадать цифровые значения.
2) $request = $_GET; абсолютно лишнее телодвижение.
3) is_integer вернет всегда фалс, так как переменные в $_GET всегда строки. Используйте intval() для приведения к целому, или floatval() для флоат соответственно, после этого можно проверять на empty например.

Comments

[Человек]

Почему массив $_GET всегда строки?
Ведь его значения равны тому, что я введу в поля.
Если я в поля введу целочисленное число,
то и значение GET должно быть целочисленное число

[Человек]

после этого можно проверять на empty например.

нужно два условия в одном if? первое это проверка на empty а второе это intval?
не могли бы вы показать пример с if, с этими двумя условиями

[ThunderCat]

Добрый Дельфин,

Ведь его значения равны тому, что я введу в поля.

Вы путаете понятие представления, типа и интерпретации. Например, 111 !== '111' так как первое является числом, а второе строкой. Из браузера мы можем передать только набор байт, как его принимать и интерпретировать решает сервер. На сервере нет информации какой тип полей использовался для отправки, да и доверять форнтенду сервер не должен, по этому по умолчанию все что пришло интерпретируется как строка.

нужно два условия в одном if? первое это проверка на empty а второе это intval?

Неужели чтение документации такая болезненная процедура? Чаще обращайтесь к докам, там практически все описано, в том числе и варианты применения.

[Человек]

ThunderCat, зачем мне проверять массив на пустоту?
что это даст?
ведь isset же нормально подходит.

[Человек]

ThunderCat, Получилось, что то такое:

if(isset($request[$key]) && is_integer(intval($request[$key])) ) {

здесь получается, что я преобразую строковое значение из GET в целочисленное?
теперь все работает как надо.

хотя так тоже работает:

if(isset($request[$key]) && is_numeric($request[$key]) ) {