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

    +151

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    $friends = mysql_query("(SELECT * FROM  `friends` WHERE `from` LIKE  '{$act}' AND  `to` LIKE  '{$act}' AND  `isaccepted` =  '1') UNION (SELECT  `id` ,  `name` ,  `surname` FROM  `users_info`);");
		$friend = array();$loop=0;
		while($fetch = mysql_fetch_assoc($friends)) {
			if ($fetch['from'] == $act) $search=$fetch['to']; else $search=$fetch['from'];
			$friend[$loop]=mysql_fetch_assoc(mysql_query("SELECT `id`,`name`,`surname` FROM `users_info` where `id`='{$search}'"));
			$loop++;
		}

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

    Запостил: yanislavb, 05 Января 2015

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

    • ava yanislavb 05.01.2015 23:54 # 0

      fixed
      $friends = mysql_query("SELECT * from `friends` where `from` LIKE '{$act}' AND `to` LIKE '{$act}' AND `isaccepted` = '1'");
      Ответить

    • ava artembegood 06.01.2015 01:02 # +1

      тут же не stackoverflow вроде
      Ответить

      • ava guest 06.01.2015 01:06 # −8

        показать все, что скрытоБлядь, человек вопрос задал: потрудись что-нибудь по делу ответить! Ему не важно, стековерфлоу тут или ещё какая нибудь хуйня. Вам должно быть лестно, что Вашему авторитету и опыту доверяют, а Вы чморите этих людей... Разве так бывает?..
        Ответить

        • ava guest 06.01.2015 01:15 # −4

          Может, денег хотите за советы? Эта позиция тоже понятна, какой-нибудь выхухоль будет пользоваться Вашими советами и клепать коммерческие проги, нехорошо, не в духе советских граждан. Но я лично сам за себя сейчас говорю. Шареваре я не пишу, а уж если пишу - долбаю бетонные плиты своей бошкой, не Борманда. Всё таки, совесть есть - жаль, не у всех.
          Ответить

        • ava Stallman 06.01.2015 09:34 # +1

          Да мне как-то похуй, доверяет ли мне очередной пыхошколотрон.
          Ответить

        • ava bormand 06.01.2015 09:39 # +6

          > Блядь, человек вопрос задал
          "Почему не работает" - настолько расплывчатый вопрос, что на него влом отвечать, а телепатия у меня плохо прокачана. Вопросы в такой форме может задавать только секретарша, а не программист.
          Ответить

          • ava bormand 06.01.2015 09:52 # +5

            Так что дам только вот такие советы:
            1) Локализуй место, где возникает проблема. Тебе помогут лог сервера, var_dump() и mysql_error().
            2) Если там ошибка мускуля - загугли ее, должно найти пару тредов с аналогичной ситуацией.
            3) Если ошибок не выводится, но код работает неправильно - изучай чего выводят var_dump'ы и сравнивай со своими мыслями.
            4) Если совсем ничего не помогает - можешь задать вопрос, НО, не в форме "почему не работает", которая подразумевает, что ты нихуя не пытался разобраться, а в духе "в строке 115 в цикл ни разу не входит, но никаких ошибок и исключений не возникает".

            P.S. Юзай PDO, заебали уже со своим mysql_* и недоэкранировками.
            Ответить

            • ava guest 06.01.2015 14:47 # +3

              Про дебагер не говори @ vardump советуй @ пыхоламеров плоди
              Ответить

              • ava bormand 06.01.2015 14:50 # +2

                Так у пыхи дебаггер есть? Не знал...
                Ответить

                • ava inkanus-gray 06.01.2015 18:27 # +2

                  Разумеется! Запускаем бинарник php из-под gdb.
                  Ответить

                • ava Fike 06.01.2015 22:09 # +2

                  xdebug, и он на удивление жорош
                  Ответить

                  • ava inkanus-gray 06.01.2015 22:11 # 0

                    Но ведь с ним инклуды тормозят: http://govnokod.ru/17413#comment260759
                    Ответить

                    • ava Fike 06.01.2015 22:24 # +2

                      вам дебаггер или продакшен?
                      Ответить

                      • ava Stallman 06.01.2015 22:51 # +2

                        У 95% макак это одно и то же.
                        Ответить

                        • ava inkanus-gray 07.01.2015 01:04 # 0

                          А как дела у оставшихся 5% макак?
                          Ответить

                          • ava bot 08.01.2015 20:34 # 0

                            Другие 5% отлаживают продакшен средствами других языков.
                            PS. Хуйню сморозил, но да всё равно
                            Ответить

                            • ava bormand 08.01.2015 21:16 # 0

                              > средствами других языков
                              Больше безумия для бога безумия:
                              <script type="text/javascript">(function() { console.log("<?php var_dump($var); ?>"); })();</script>
                              Жаль, что переносы строк всю идею обосрут ;(
                              Ответить

                              • ava 1024-- 08.01.2015 21:23 # 0

                                Вот интересно: в соседнем треде ЛишпГовно всем показывает лямбду, словно экзотическую обезьянку, а у нас в жс это обыденность.

                                Комментарии про скобки (как-то сразу они в глаза бросились) прочитал, да. И порадовался.
                                Ответить

                              • ava Stallman 08.01.2015 21:24 # 0

                                >console.log
                                Вот пользователи осла обрадуются-то...
                                Ответить

                                • ava bormand 08.01.2015 21:30 # 0

                                  > Вот пользователи осла обрадуются-то...
                                  Для них можно предусмотреть alert():
                                  if (!console) {
    console = { log: function (s) { alert(s); } }
}
                                  P.S. А в восьмом ie вроде же есть соснолька?
                                  Ответить

                                  • ava Stallman 08.01.2015 21:31 # 0

                                    Но, емнип, если соснолька не включена, со typeof console == 'undefined'
                                    Ответить

                                    • ava bormand 08.01.2015 21:33 # 0

                                      > если соснолька не включена, со typeof console == 'undefined'
                                      Ну значит мой код с if (!console) окажется как нельзя кстати. Больше боли и унижений для ослов.
                                      Ответить

                                      • ava Stallman 08.01.2015 21:36 # +1 

                                        if (!console) {
    console = {
        log: function (s) {
            alert(s);
            window.location.href = 'http://firefox.com/';
        }
    }
}
                                        Ответить

                                        • ava bormand 08.01.2015 21:41 # 0

                                          Упростим:
                                          if (!console) {
    window.location.href = 'http://firefox.com/';
}
                                          Ответить

                                          • ava bot 08.01.2015 21:58 # 0

                                            Мне кажется, или IE на фразу !console таки может ругнуться фразой "Javascript Error"? (я знаю, он и не такое может)
                                            Ответить

                                            • ava bormand 08.01.2015 22:19 # 0

                                              Хрен знает, в линухе нет IE :(
                                              Ответить

                                              • ava Stallman 08.01.2015 22:22 # 0

                                                Зато есть виртуалочка и желание пострадать хуйнёй вместо работы. -_-
                                                Ответить

                                            • ava Stallman 08.01.2015 22:21 # 0

                                              Проверил, таки да, валится, при чем предательски молча.
                                              Ответить

                                              • ava bormand 08.01.2015 22:24 # 0

                                                Т.е. надо через typeof?
                                                Ответить

                                                • ava Stallman 08.01.2015 22:29 # 0

                                                  Получается так, да. По идее можно еще через !window.console, но тут я хз, как поведут себя другие браузеры.
                                                  Кстати, а IE8 для кого-нибудь еще актуально вообще?
                                                  Ответить

                                                  • ava bot 09.01.2015 01:22 # 0

                                                    Таки да, IE8 (и др.) ещё актуально.
                                                    И таки да, лучше !window.console (другие браузеры переваривают на ура), а ещё лучше - через православное typeof.
                                                    Ответить

                              • ava bot 08.01.2015 21:24 # 0

                                Акела промахнулся.
                                Это javascript необратимо extends PHP, а не наоборот.
                                Ответить

                              • ava bormand 08.01.2015 21:25 # 0

                                Вроде всё пофиксил:
                                function var_dump_to_browser($var) {
    ob_start();
    var_dump($var);
    $data = str_replace(
        array("\\", "\"", "\n"),
        array("\\\\", "\\\"", "\\n"),
        ob_get_contents()
    );
    ob_end_clean();
    ?><script type="text/javascript">(function() { console.log("<?php print $data; ?>"); })();</script><?php
}
                                Ответить

                                • ava Stallman 08.01.2015 21:28 # 0

                                  >var_dump_to_browser
                                  Какое-то не похапешное название совсем. Вот, например, vdtb_new2() - другое дело.
                                  Ответить

        • ava guest 06.01.2015 14:46 # +2

          Зачем тупорылой пыхобляди помогать? Она даже про pdo не вкурсе!! Зачем дегенератов плодить?
          Ответить

        • ava guest 09.01.2015 00:06 # 0

          Молодцы, опять подлизали. 

          Лижет ногу,
       лижет руку,
лижет в пояс,
         лижет ниже,-
как кутенок
          лижет
              суку,
как котенок
       кошку лижет.
А язык?!
    На метров тридцать
догонять
    начальство
            вылез -
мыльный весь,
         аж может бриться,
даже
 кисточкой не мылясь.


          ~Новогодный тост~
...Говорят, когда пожилым рабочим, строившим многоэтажки, попадаются в руки спичечные коробки, они машинально начинают укладывать их друг на друга - "строят этажку". Пусть же наш с Вами жизненный путь прервётся раньше, чем мы станем мыслить столь же шаблонно.
          Ответить

    • ava Dummy00001 06.01.2015 02:09 # 0

      очевидно! луп в ноль поставлен - `$loop=0;` - поэтому и не лупится!
      Ответить

    • ava inkanus-gray 06.01.2015 13:55 # +1

      1. Перед строкой №4 ставим var_dump($fetch) и смотрим, что пришло.
      2. А нам точно нужен цикл? А что, если попробовать JOIN или подзапрос?

      Вообще если возникает желание сделать SQL-запрос в цикле, то надо задуматься, правильно ли мы спроектировали алгоритм.
      Ответить

      • ava bormand 06.01.2015 14:47 # +1

        > А что, если попробовать JOIN
        Судя по первой строке автор хотел попробовать JOIN. Вот только JOIN и UNION спутал :)
        Ответить

      • ava guest 06.01.2015 14:48 # 0

        Про дебагер не говори @ vardump советуй @ пыхоламеров плоди
        Ответить

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