0

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

if(!is_numeric($q5proclang) || (!is_numeric($q5proclang2))|| (!is_numeric($q5proclang3))|| (!is_numeric($q5proclang4))|| (!is_numeric($q5proclang5)) { 
        $error_result = "Операнды должны быть числами"; 
    }
    else 
      
            $result = $q5proclang + $q5proclang2+$q5proclang3+$q5proclang4+$q5proclang5; 
            
                break;    
        }
}
if(isset($result>200)) {
    echo "Некорректное значение, сумма не должна быть более 200";
 } 

P.S. Поправил [1][ошибка с поправкой]

<?php


try {
    $conn = new PDO("mysql:host=$servername;dbname=$dbname", $username, $password);
$conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  
  session_start();
$id = $_SESSION['id'];
   $sql = "UPDATE  anketa SET q5lang ='$q5lang', q5proclang ='$q5proclang',q5lang2='$q5lang2',q5proclang2='$q5proclang2',q5lang3='$q5lang4',    q5proclang3 = '$q5proclang5',q5lang4='$q5lang3', q5proclang4='$q5proclang3', q5proclang5 ='$q5proclang4',q6tek='$q6tek',q601='$q601',q602='$q602', q6chmes='$q6chmes',q6chtrimes='$q6chtrimes',q6konec='$q6konec' WHERE id = '$id' ";
  $result = null;
if(
    (!is_numeric($q5proclang))  ||
    (!is_numeric($q5proclang2)) ||
    (!is_numeric($q5proclang3)) ||
    (!is_numeric($q5proclang4)) ||
    (!is_numeric($q5proclang5))
) {
    $error_result = "Операнды должны быть числами";
} else {
    $result = $q5proclang + $q5proclang2 + $q5proclang3 + $q5proclang4 + $q5proclang5;
    // break;
}
if($result > 200) {
    echo "Некорректное значение, сумма не должна быть более 200";
}
    else {
    
    $conn->exec($sql);
    echo "New record created successfully";
    
catch(PDOException $e)
    
    echo $sql . "<br>" . $e->getMessage();
    

$conn = null;
echo "<meta http-equiv=refresh content=0;URL=http://co47377.tmweb.ru/354.html>";
} 
?>

0

1 ответ 1

0

Попробуйте это...

тут навел какой-никакой порядок в скобках и вынес объявление переменной $result в область видимости проверки

$result = null;
if(
    (!is_numeric($q5proclang))  ||
    (!is_numeric($q5proclang2)) ||
    (!is_numeric($q5proclang3)) ||
    (!is_numeric($q5proclang4)) ||
    (!is_numeric($q5proclang5))
) {
    $error_result = "Операнды должны быть числами";
} else {
    $result = $q5proclang + $q5proclang2 + $q5proclang3 + $q5proclang4 + $q5proclang5;
    // break;
}
if($result > 200) {
    echo "Некорректное значение, сумма не должна быть более 200";
}

Вариант 2

Можно еще все это дело перенести в массив, пробежаться по нему и сделать проверку для каждого (и там же сложить заодно)

$result = null;
$nonNumeric = false;
$error_result = null;

$operands = [
    $q5proclang,
    $q5proclang2,
    $q5proclang3,
    $q5proclang4,
    $q5proclang5
];
foreach ($operands as $operand) {
    if(!is_numeric($operand)) {
        $nonNumeric = true;
        break;
    }
    $result += $operand;
}

if($nonNumeric) {
    echo "Операнды должны быть числами";
} else {
    if($result > 200) {
        echo "Некорректное значение, сумма не должна быть более 200";
    } else {
        echo "Результат: $result";
    }
}

После PS

<?php
$sql = "UPDATE  anketa SET q5lang ='$q5lang', q5proclang ='$q5proclang',q5lang2='$q5lang2',q5proclang2='$q5proclang2',q5lang3='$q5lang4',    q5proclang3 = '$q5proclang5',q5lang4='$q5lang3', q5proclang4='$q5proclang3', q5proclang5 ='$q5proclang4',q6tek='$q6tek',q601='$q601',q602='$q602', q6chmes='$q6chmes',q6chtrimes='$q6chtrimes',q6konec='$q6konec' WHERE id = '$id' ";
try {
    $conn = new PDO("mysql:host=$servername;dbname=$dbname", $username, $password);
    $conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

    session_start();
    $id = $_SESSION['id'];
    $result = null;
    if(
        (!is_numeric($q5proclang))  ||
        (!is_numeric($q5proclang2)) ||
        (!is_numeric($q5proclang3)) ||
        (!is_numeric($q5proclang4)) ||
        (!is_numeric($q5proclang5))
    ) {
        $error_result = "Операнды должны быть числами";
    } else {
        $result = $q5proclang + $q5proclang2 + $q5proclang3 + $q5proclang4 + $q5proclang5;
        // break;
    }
    if($result > 200) {
        echo "Некорректное значение, сумма не должна быть более 200";
        echo "<meta http-equiv=refresh content=0;URL=http://co47377.tmweb.ru/354.html>";
    } else {
        $conn->exec($sql);
        echo "New record created successfully";
        $conn = null;
    }
} catch(PDOException $e) {
    echo $sql . "<br>" . $e->getMessage();
}
?>
10
  • Спасибо за помощь, но вот попробовал оба варианта, оба делают php недоступным, как и в изначальной версии, сейчас отредактирую топик и выложу полный пхп код, возможно, я не так вношу 6 авг 2022 в 17:59
  • @Full0fb3ans напишите как приложите 6 авг 2022 в 18:02
  • Приложил после p.s. 6 авг 2022 в 18:06
  • @Full0fb3ans, поправил там у вас баланс скобок.... но если честно то вопросов стало больше чем ответов... попробуйте самый последний вариант и если что пишите 6 авг 2022 в 18:13
  • Есть продвижение, теперь он хотя бы проходит, но пишет SQLSTATE[HY000] [2002] No such file or directory, скриншот приложил 6 авг 2022 в 18:22

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.