Почему не работает оператор if?

Question

[SUS_ITS_LOVE]

Вот код:

$tb = new Table;
 
$type = $_POST['type'];

If($type == 'surname'){
  echo "1";
  $question = $_POST['surname_3'];
}
If($type == 'id_publick'){
  $question = $_POST['id_3'];
}
If($type == 'phone_num'){
  $question = $_POST['phone_num_3'];
}

$res = $tb->query($type, $question);
$tb->printBD();

Когда я пытаюсь вывести переменную $question то там просто пустота...
Если я сменю условие оператора на "!=" то переменная изменяется.
Думаю просто оператор условия неправильный.
Вот кстати сам ПОСТ:

Array ( [type] => id_publick [surname_3] => Поліщук [phone_num_3] => 122333123321 [id_3] => 85476 )

Comments

[Alvi Isakn]

Ты в переменную $TYPE что записываешь?
Ты записываешь туда , то что передается тебе постом так.
И дальше что делаешь?
$res = $tb->query($type, $question);
Получается что метод QUERY который должен принимать SELECT или DELETE или UPDATE у тебя принимает surname, id_publick, phone_num т.е
вместо query("SELECT surname FROM имя таблицы WHERE surname = surname_3 или $question) он принимает query("surname, surname_3 )
Конечно у тебя ничего не будет работать

[SUS_ITS_LOVE]

Alvi Isakn, $tb ето обект моего класа, для которого єта функция имеет другое назначение!

[Alvi Isakn]

А может просто попробовать так $type = $_POST без добавления значений;

Answer 1

[Rsa97]

Потому что оператор if, а не If

Comments

[SUS_ITS_LOVE]

Не в этом проблема( На сколько я помню, в Джава так ,а в ПШП с большой

[SUS_ITS_LOVE]

Konata Izumi, Но оно все равно помогло, может проблема в чем то другом ещё?

[Lander]

SUS_ITS_LOVE, А чё бы вам не включить ошибки и не посмотреть самому?

[Rsa97]

SUS_ITS_LOVE, PHP, как и большинство других языков программирования, имеет регистрозависимый синтаксис. Строчная и прописная буквы в нём - это разные буквы.

[SUS_ITS_LOVE]

Lander, Так в том и проблема что никах ошибок нет! Как будто просто не исполнилось условие

[green_goo]

в php регистр роли не играет, хоть iF можно
SUS_ITS_LOVE,

<?php
var_dump($_POST);
exit();

Что выведет?

[Rsa97]

SUS_ITS_LOVE, Так а что мешает вывести результаты условий? Или, что лучше, переписать на switch/case

switch ($_POST['type']) {
  case 'surname':
    echo "1";
    $question = $_POST['surname_3'];
    break;
  case 'id_publick':
    $question = $_POST['id_3'];
    break;
  case 'phone_num':
    $question = $_POST['phone_num_3'];
    break;
  default:
    die 'Всё очень плохо';
}

[Alvi Isakn]

Языку PHP пох с какой буквы ты пишешь if или IF или If iF
$a = 11;
iF($a > 10){
echo 'Ok';
}

$a = 11;
If($a > 10){
echo 'Ok';
}

$a = 11;
IF($a > 10){
echo 'Ok';
}

[Alvi Isakn]

Просто перед $questions поставь ECHO или echo
ECHO $question;
EcHo $question;
eChO $question;

[Vitaliy Orlov]

Konata Izumi, хехе, двойка тебе по матчасти, сам сходи и почитай доку

Регистрозависимые
- переменные
- константы
- ключи массивов
- свойства классов
- константы в классах

РегистроНЕзависимые
- функции
- конструкторы
- имена методов
- ключевые слова и конструкции (if, else, null, foreach, echo etc.)

это все будет работать одинаково, ошибок не будет

<?php

IF (2==2) echo 123;
iF (2==2) Echo 123;
If (2==2) eCHo 123;

[SUS_ITS_LOVE]

green_goo, array(4) { ["type"]=> string(25) " surname " ["surname_3"]=> string(14) "Поліщук" ["phone_num_3"]=> string(10) "0965783456" ["id_3"]=> string(5) "85476" }

[Rsa97]

green_goo, SUS_ITS_LOVE, Сейчас перепроверил себя, действительно, в ключевых словах PHP регистр неважен.
Там вообще бардак, строки, имена переменных, констант и методов класса регистронезависимы, а ключевые слова, имена функций, классов и свойств классов - регистрозависимы.

[SUS_ITS_LOVE]

Alvi Isakn, Пробовал

[Konata Izumi]

Vitaliy Orlov, уже проверил кодом

[Rsa97]

SUS_ITS_LOVE, " surname " != "surname"
Используйте trim()

[SUS_ITS_LOVE]

Rsa97, Уже, не помогло!

[Konata Izumi]

SUS_ITS_LOVE,

>>> ' word ' == 'word'
=> false
>>> 'word' == 'word'
=> true

[SUS_ITS_LOVE]

Konata Izumi, Я имел виду, что после трима условие все-равно не исполнилось!

[green_goo]

SUS_ITS_LOVE,

Я имел виду, что после трима условие все-равно не исполнилось!

А вар дампом проверил, что то что стоит слева оператора сравнения и справа идентично?

Используйте trim()

А еще лучше отправлять данные без лишних символов сразу.

Answer 2

[Vitaliy Orlov]

Я думаю, у тебя в условиях ошибка

Попробуй вывести переменную

$type = $_POST['type'];
var_dump($type);

и посмотреть что там оказывается в итоге

так же назначь $question значение по-умолчанию, чтобы понимать что ни одно из условий не отработало

$question = 'n/a';
...
If($type == 'surname'){
  echo "1";
  $question = $_POST['surname_3'];

кроме того, обрати внимание на оператор switch с ним было бы удобнее

$type = $_POST['type'];

switch($type) {
    default: // если тип не опознан сгенерировать ошибку
        throw new Exception('Undefined type: '.$type);
    break;
    case 'surname': 
        $question = $_POST['surname_3'];
    break;
    case 'id_publick': 
        $question = $_POST['id_3'];
    break;
    case 'phone_num': 
        $question = $_POST['phone_num_3'];
    break;
}

Comments

[SUS_ITS_LOVE]

Все это я пробовал, переменная $type коректно записывается, но условие не исолняется

[SUS_ITS_LOVE]

SUS_ITS_LOVE, SUS_ITS_LOVE, вот вардамп string(25) " surname "

[Konata Izumi]

SUS_ITS_LOVE, " surname " с пробелами по краям?
Попробуйте через trim очищать type

[SUS_ITS_LOVE]

Konata Izumi, Уже) но нет!

[Konata Izumi]

SUS_ITS_LOVE, Что нет то?
Вы сравниваете строку с пробелами со строкой без пробелов. Так вот они не равны друг другу.
Выше уже написали.

[SUS_ITS_LOVE]

Konata Izumi, Я имел виду, что после трима условие все-равно не исполнилось!

[Konata Izumi]

SUS_ITS_LOVE, как вы применили трим, покажите новый код

[SUS_ITS_LOVE]

$tb = new Table;

$type = $_POST['type'];
trim($type);
If($type == 'surname'){
  $question = $_POST['surname_3'];
}
echo $type;
echo $question;

[aglgl]

попробуйте условия на цифры поменять за место строки, чтобы исключить вариант с пробелами

[Konata Izumi]

SUS_ITS_LOVE,

trim($type);

Это функция, а не процедура. Она вернула обрезанную строку. Вы её никуда не записали.

Так надо было:
$type = trim($type);

[SUS_ITS_LOVE]

Konata Izumi, ДААААА, роботает! Очень спасибо всем!

[Vitaliy Orlov]

SUS_ITS_LOVE, попробуй вот так и напиши нам результат

...
$type = $_POST['type'] ?? '';
switch(trim($type)) {
    default: // если тип не опознан сгенерировать ошибку
        throw new Exception('Undefined type: '.$type);
    break;
    case 'surname': 
        $question = $_POST['surname_3'] ?? '-empty-';
    break;
    case 'id_publick': 
        $question = $_POST['id_3'] ?? '-empty-';
    break;
    case 'phone_num': 
        $question = $_POST['phone_num_3'] ?? '-empty-';
    break;
}

var_dump([$type, $question]);
..

[SUS_ITS_LOVE]

Vitaliy Orlov, Ми уже нашли ошибку, просто перед всем нужен бил трим! Спасибо вам очень за вашу помощь!

[Konata Izumi]

SUS_ITS_LOVE, а пользовались бы phpstom + xdebug или банальными вардампами, сами бы нашли эту ошибку минут за 5.

[SUS_ITS_LOVE]

Konata Izumi, ну я только учусь, но возьму метод на заметку!