Скрипт очень странно себя ведет. Что делать если в условии всё учтено, но выходит совсем не то?

Question

[Дмитрий Баскаков]

Здраствуй друг. Если ты забрел сюда, то вот код:

case "catalog":
                    $number_of_page = (int)$callback_data[1];
                    $catalog = $parser->catalog();
                    $catalog_count = count($catalog);
                    $catalog = array_slice($catalog,$number_of_page,1);
                    $img = $catalog[0]["img"];
                    $title = $catalog[0]["title"];
                    $link = str_replace(["http://","https://"],"",$catalog[0]["link"]);
                    $next_text = "\xE2\x96\xB6";
                    $prev_text = "\xE2\x97\x80";
                    $next = $number_of_page+1;
                    $prev = $number_of_page-1;
                    $buttons_keys = [
                            ["text" => $prev_text, "callback_data" =>"catalog_".$prev],
                            ["text" => "Подробнее", "url" =>"$link"],
                            ["text" => $next_text, "callback_data" =>"catalog_".$next],
                    ];
                    $i = (int)0;
                    if($number_of_page <= $i){
                        unset($buttons_keys[0]);
                        sort($buttons_keys);
                    }elseif($number_of_page >= ($catalog_count-1)){
                        unset($buttons_keys[2]);
                        sort($buttons_keys);
                    }
                    if($number_of_page != $i){
                        $telegram->deleteMessage([
                            "chat_id" => $callback_chat_id,
                            "message_id" => $callback_message_id,
                        ]);
                    }
                    $buttons = [
                        $buttons_keys,
                    ];
                    $keyboard = json_encode(["inline_keyboard"=>$buttons]);
                    $telegram->sendPhoto([
                        "chat_id" => $callback_chat_id,
                        "photo" => $img,
                        "parse_mode" => "Markdown",
                        "reply_markup" => $keyboard,
                        "caption" => "*$title*",
                    ]);
                    break;

Кстати, $callback_data[1] = 1, а все остальные значения берутся с сайта - всё рабочее.

Проблема в том, что $number_of_page не входит в условия и сообщения не удаляются.
Причем, только при $callback_data[1] = 1 не удаляются. А при 2,3,0 и т.д. - удаляются

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

Если сие данных достаточно, то подскажите где ошибка. Уже n'e(около часа) кол-во минут сижу и не вижу

Comments

[Нурлан]

И это всё в case? Охренеть!
$i = (int)0;
Серьезно?

[Дмитрий Баскаков]

Нурлан, Я чего только не пробовал.

Answer 1

[doublench21]

if($number_of_page <= $i){ ... }
if($number_of_page != $i){ ... }

Если $number_of_page < $i, то оно зайдет в оба условия. Для вас это нормально?

Comments

[Дмитрий Баскаков]

Так код стал короче, ибо не пришлось обрабатывать 3 условия в которых будет один и тот же метод deleteMessage(). - мб костыль. Не спорю. даже без этого условия с конструкцией if-elseif-else не работает

[doublench21]

Дмитрий Баскаков,

if($number_of_page <= $i){
   1 <= 0 Не проходим
} elseif ($number_of_page >= ($catalog_count-1)){
    Суда тоже мы не проходим, Ну не может же быть число $catalog_count-1 меньше  либо равно нулю. 
}
if($number_of_page != $i){
  1 != 0 Не проходим
}

То есть при $number_of_page = 1, единственный возможный случай, когда мы зайдём в одно из условий, а именно 2ое условие, это когда $catalog_count-1 равен 0.

[Дмитрий Баскаков]

doublench21,
То что идет 2 проверки - норма. В общем, если $number_of_page = 1, то получится:

if($number_of_page <= $i){
   1 <= 0 Не проходим, т.к. элемент не является начало массива
} elseif ($number_of_page >= ($catalog_count-1)){
    Не проходим, т.к. элемент и не конец массива
}
if($number_of_page != $i){
  Проходим, ибо 1 не равно 0
}

В общем, в первых 2х условиях проверяется позиция массива.
А в 3м условии проверяется надо ли делать какие-то действия с массивом. И их надо делать, только если ключ не равен нулю

[Дмитрий Баскаков]

doublench21,
Если сделать иначе, то всё так же не будет работать

//
                    if($number_of_page == $i){
                        unset($buttons_keys[0]);
                        sort($buttons_keys);
                    }elseif($number_of_page >= ($catalog_count-1)){
                        unset($buttons_keys[2]);
                        sort($buttons_keys);
                        $telegram->deleteMessage([
                            "chat_id" => $callback_chat_id,
                            "message_id" => $callback_message_id,
                        ]);
                    }else{
                        $telegram->deleteMessage([
                            "chat_id" => $callback_chat_id,
                            "message_id" => $callback_message_id,
                        ]);
                    }

[doublench21]

Дмитрий Баскаков, то есть вы хотите сказать, что при $number_of_page = 0, сюда он не заходит:

if($number_of_page != $i){
                        $telegram->deleteMessage([
                            "chat_id" => $callback_chat_id,
                            "message_id" => $callback_message_id,
                        ]);
                    }

?

[Дмитрий Баскаков]

Он равен нулю. $i так же равна нулю. Должен?

[doublench21]

Дмитрий Баскаков, да.
$callback_data[1] = 1;
$number_of_page = (int)$callback_data[1];
$number_of_page = 1; Почему 0 та?

[Дмитрий Баскаков]

doublench21, Потому что вы написали

Дмитрий Баскаков, то есть вы хотите сказать, что при $number_of_page = 0, сюда он не заходит:

[doublench21]

Дмитрий Баскаков, попутал. Вопрос тот же, ток $number_of_page = 1

[Дмитрий Баскаков]

doublench21,

То что идет 2 проверки - норма. В общем, если $number_of_page = 1, то получится:

if($number_of_page <= $i){
   1 <= 0 Не проходим, т.к. элемент не является начало массива
} elseif ($number_of_page >= ($catalog_count-1)){
    Не проходим, т.к. элемент и не конец массива
}
if($number_of_page != $i){
  Проходим, ибо 1 не равно 0
}

В общем, в первых 2х условиях проверяется позиция массива.
А в 3м условии проверяется надо ли делать какие-то действия с массивом. И их надо делать, только если ключ не равен нулю

[doublench21]

Дмитрий Баскаков, ну так суть такая, что ключ не равен нулю, но туда он не заходит и соответственно и не удаляет? Хотя должен

[Дмитрий Баскаков]

doublench21, Вот именно. Код работает правильно для остальных элементов массива, но не для певрвого

[doublench21]

Дмитрий Баскаков, а вы выводили наверное значение $number_of_page и $i перед проверкой if($number_of_page != $i). Что-то не верится просто.

[Дмитрий Баскаков]

doublench21, Я просто не учтел один момент - Там была обработка как раз для таких случаев, которуя я сам же и написал, но не добавил.

Answer 2

[Нурлан]

Перед if($number_of_page != $i){ var_dump'ом выведите обе переменные и посмотрите на результат.

Comments

[Дмитрий Баскаков]

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

//
                    if($number_of_page == $i){
                        unset($buttons_keys[0]);
                        sort($buttons_keys);
                    }elseif($number_of_page >= ($catalog_count-1)){
                        unset($buttons_keys[2]);
                        sort($buttons_keys);
                        $telegram->deleteMessage([
                            "chat_id" => $callback_chat_id,
                            "message_id" => $callback_message_id,
                        ]);
                    }else{
                        $telegram->deleteMessage([
                            "chat_id" => $callback_chat_id,
                            "message_id" => $callback_message_id,
                        ]);
                    }

Для справки: elseif проверяет не является ли $number_of_page конечным элементом

Answer 3

[Дмитрий Баскаков]

Тема закрыта. Я слепой и невнимательный сами знаете кто. - забыл дописать одно исключение