1. PHP / Говнокод #7242

    +153

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    $connect = $db->sql_query("SELECT `come_sid`, `come_time`, `come_ip`, `secret_type` FROM `".PREFIX."_gen[user]` WHERE `uid` = '" . $this->USER_UID . "';");
$rowset = $db->sql_fetchrow();

if( $connect == false ) {
	$return = false;
} else if( $rowset == false ) {
	unset( $return );
	$return = false;
} else if( ( intval($rowset['come_time']) + intval($conf_safety["time_sess_adm"]) ) < time()) {
	unset( $return );
	$return = false;
} else if( $rowset['come_ip'] != IPClient() ) {
	unset( $return );
	$return = false;
} else if( $rowset['secret_type'] != secret_hash() ) {
	unset( $return );
	$return = false;
} else if( $rowset['come_sid'] != SID ) {
	unset( $return );
	$return = false;
} else {
	unset( $return );
	$return = true;
}

    Бесполезные условия...

    Запостил: nethak, 13 Июля 2011

    Комментарии (12) RSS

    • ava nobody 13.07.2011 23:56 # 0

      чем они бесполезные ? Кто-то просто не перегружает базу условиями в запросе; Тут еще по хорошему в запросе в конце прописать limit 1 если запись одна.
      Ответить

      • ava nethak 14.07.2011 00:45 # −3

        Оно было бы полезно вот так вот:
        $connect = $db->sql_query("...");
        $rowset = $db->sql_fetchrow($connect);

        А в выше указанном примере ключ не был передан, и он вернёт всегда TRUE если $connect не вернёт FALSE а FALSE он вернёт только в том случаи, если уч. записи с UID указанным не будет.

        Вот я и говорю бесполезное.
        Ответить

        • ava bugmenot 14.07.2011 00:56 # +1

          в том случаи
          Ответить

          • ava nethak 14.07.2011 01:07 # −3

            Что-то не так?
            Ответить

            • ava guest 14.07.2011 01:14 # 0

              >Что-то не так?
              В согласовании и склонении слов в предложениях.
              Он тебя понял, просто троллит, тк сказать по делу нечего.
              Ответить

              • ava nethak 14.07.2011 01:17 # −8

                показать все, что скрытоА ну извиняюсь, просто за частую печатаю быстро, и быстрей чем успеваю подумать =)
                Ответить

                • ava bugmenot 14.07.2011 01:44 # 0

                  в этом и проблема

                  см. тж. "за частую"
                  Ответить

                • ava TarasB 14.07.2011 10:03 # +3

                  А ты тогда печатай не за частую, а за редкую.
                  http://pix.timeout.ru/25589.jpeg
                  Ответить

                • ava Lure Of Chaos 14.07.2011 12:09 # 0

                  функция "думать" опциональна и лениво аттачена?
                  Ответить

                  • ava guest 14.07.2011 13:38 # −1

                    Просто опционально. "Лениво думают", с точки зрения ФП - это, когда думают, только когда потребовалось подумать. А тут периодически просто не думают.
                    Ответить

        • ava nobody 14.07.2011 14:02 # 0

          Во первых класс по работе с бд нам сюда никто не выкладывал, во вторых можно предположить что он отлавливает последний запрос и разбивает его на массив; Так что мы не телепаты чтобы знать как именно работает ваш велосипед который работает с бд
          Ответить

    • ava guest8 09.04.2019 11:10 # −999

      показать все, что скрытоvanished
      Ответить

    Добавить комментарий