Есть ли способ упростить этот код PHP?

Question

$a = $_POST['year'];
$b = $_POST['mileage'];
$c = is_numeric($a);
$d = is_numeric($b);
if ($c == False && $d == False) {
    echo = "$c & $d variables are not numeric";
} else {
    echo = "$c & $d variables are numeric";
}

Это код, который я набрал, изучая PHP. Есть ли кто-нибудь, кто может помочь мне их упростить. Мне не нравится, как это выглядело. Я чувствую, что это слишком долго. Новичок здесь (",)

Answer 1

Иногда изменение способа установки значений может быть полезно, особенно при сложных вычислениях, поскольку это помогает отладить код.

Но с простыми присваиваниями лучше (IMHO) попытаться уменьшить количество дополнительные шаги, которые вы делаете в своем коде. Установка переменной для ее последующего использования на другом этапе является накладными расходами для компьютера (хотя и незначительными), но также добавляет дополнительные строки кода. Итак, предполагая, что вы просто хотите показать, являются ли они обеими c или нет, вы можете свернуть все эти переменные в оператор if ...

if (is_numeric($_POST['year']) == False && is_numeric($_POST['mileage']) == False) {
    echo "variables are not numeric";
} else {
    echo "variables are numeric";
}

Это предполагает, что вы уже проверили что $_POST['year'] и $_POST['mileage'] установлены (как и ваш код), вы можете использовать $_POST['mileage'] ?? '', если хотите сделать его более гибким.

Также код говорит, что переменные не являются числовыми c если оба значения не являются числовыми c. Измените && на ||, если значения не являются числовыми c.