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

    +157

    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
    function ajaxGetProcesses()
{
	$db = $this->site->DB;

	$res = array();

	$_POST['title'] = iconv('UTF-8', 'WINDOWS-1251', $_POST['title']);

	if (!empty($_POST['title']))
	{
		$query = 'SELECT * FROM '.$this->site->prefix_db.'_processes_alters WHERE `title` LIKE \''.addcslashes($_POST['title'], '\'').'%\' LIMIT 10';
		$db->query($query);
		while ($i = $db->fetch())
		{
			$res[] = '"'.addcslashes($i['title'], '"').'"';
		}
	}
	// кто ж виноват, что в середине 2010 года все еще приходится писать на пхп4 :-(
	// json отсутствует, библиотеку подключать - жопа - не та сложность вывода
	echo '['.implode(', ', $res).']';
	exit();
}

    Запостил: netrain, 11 Ноября 2010

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

    • ava istem 11.11.2010 14:59 # +2

      А современный пхп-код построен исключительно из оупенсорсовских кирпичей, прикрученных друг к дружке или "абы как" или "как-нибудь эдак"
      Ответить

    • ava Анонимус 11.11.2010 15:00 # −1

      школолошный автор реальне думает, что на пятом пыхе он будет писать лучше)))))
      Ответить

      • ava andremacareno 11.11.2010 15:23 # −4

        Главное отличие 4-го пиэйчпи от 5.3 - отсутствие goto.
        Ответить

      • ava Arigato 11.11.2010 18:36 # −8

        показать все, что скрытоСовершенно верно, хуже может быть только перл.
        Ответить

        • ava Анонимус 11.11.2010 18:44 # +2

          в этом коде смешаны знания об http, sql и кодировках. от переписки на пятый он не изменится)
          потому-то он и дерьмо) впрочем, в мире пыхомакак так и принято
          Ответить

          • ava Arigato 11.11.2010 18:49 # −8

            показать все, что скрытоОт переписки на перл он тоже лучше не станет. Но, боюсь, что стать хуже может...
            Ответить

            • ava Анонимус 11.11.2010 18:53 # +3

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

              • ava Arigato 11.11.2010 18:57 # −6

                показать все, что скрытоА толку шило на мыло менять? Перловская макака - это конечно звучит гордо.
                Вот говнокод сайт на php, тебе не тошно на нем сидеть, не воняет?
                Ответить

                • ava Анонимус 11.11.2010 19:04 # 0

                  перловых макак почти не бывает: макаке не просто осилить мутный перл.

                  мне не тошно, я же тут юзер
                  сырцы не правлю
                  Ответить

              • ava Lure Of Chaos 11.11.2010 19:04 # +1

                1. программист на перле напишет такой перл(пардон за тавтологию), что, даже не будучи говнокодом, его будет трудно читать - это особенность перла со всеми его $_ и s///
                2. может быть, сейчас времена изменились, но когда-то выставлялось в заслуги пыху то, что он работает как модуль апача, тогда как перл на каждый запрос порождается новый процесс

                если уж задуматься, на чем рисовать веб-аппликацию, то есть еще всякие java и его производные, в т.ч. фреймворки, groovy c его grails, и, конечно, asp.net
                Ответить

                • ava Arigato 11.11.2010 19:10 # −5

                  показать все, что скрытоВерно подмечено. Более того, какой двин на перле ни возьми, говнокод хуже php'шного.
                  Ответить

                  • ava Lure Of Chaos 11.11.2010 19:36 # +3

                    товарищ, не ешьте на ночь = )

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

                    • ava dwinner 15.11.2010 12:46 # −2

                      В том-то и сложность, что большой есть соблазн писать на ПХП Гавнокод. Писать на ПХП скрипты устойчивые к ошибкам, оптимизированные и т.д. и т.п. довольно непростое дело. Самодисциплина нужна определённая, тогда и будет хороший код! Facebook и vkontakte - наглядный пример профессиональных ПХП-шников!
                      Ответить

                      • ava Arigato 15.11.2010 13:04 # 0

                        > Facebook и vkontakte - наглядный пример профессиональных ПХП-шников!

                        Что-то я сомневаюсь. Вы видели исходники?
                        Ответить

                        • ava dwinner 15.11.2010 13:38 # 0

                          Исходники - нет, но работает всё превосходно и относительно быстро!
                          Ответить

                          • ava Arigato 15.11.2010 13:42 # 0

                            Говнокод тоже часто работает как надо.
                            Ответить

                          • ava Lure Of Chaos 16.11.2010 01:13 # 0

                            > но работает всё превосходно и относительно быстро!
                            потому что серваки отличные + всякие кешеры, ускорители, к тому же у фейсбука пхп-код скомпилен
                            Ответить

                            • ava bugmenot 16.11.2010 01:53 # +1

                              да ладно, у них ферма с десятками тысяч серверов, вот и шевелится оно
                              Ответить

                      • ava Lure Of Chaos 16.11.2010 01:11 # +1

                        хороший код на пхп -- это как лямбды на бейсике
                        Ответить

                      • ava bugmenot 16.11.2010 01:41 # 0

                        fakebook и vcuntakte - профессиоанальные, ага, i lold
                        Ответить

                        • ava Arigato 16.11.2010 11:09 # 0

                          Профессиональный говнокод - это когда говнокодство превращается в профессию.
                          Ответить

    • ava Lure Of Chaos 11.11.2010 17:42 # +2

      за WINDOWS-1251 отдавал бы святой инквизиции
      Ответить

      • ava Анонимус 11.11.2010 18:21 # 0

        можно подумать что русские комментарии в кои лучше))

        хотя хуже всего utf 8, ибо 2 байта будет для русских букв
        Ответить

        • ava Lure Of Chaos 11.11.2010 18:47 # −4

          utf8 хотя бы частично решает проблему русских крякозябл. мало? есть и utf16, и unicode.

          вообще я не только про однобайтовость, но еще и про то, почему WINDOWS-1251,а не ISO-8859-5 какой-нибудь
          Ответить

          • ava Oleg_quadro 12.11.2010 00:07 # +2

            Почему частично?

            Конец фразы "и unicode" дискредитировал.
            Я минуснул за это, прошу не обижаться.
            (unicode - стандарт кодирования, utf8 - одна из его реализаций).
            Ответить

      • ava punishment 14.11.2010 14:18 # −1

        зря вы так, потом вас отдадут инквизиции за утф-8
        Ответить

        • ava bugmenot 16.11.2010 01:43 # 0

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

    • ava j0kz 12.11.2010 00:39 # 0

      вот строка
      $db = $this->site->DB;

      прям глаз режет. бля, ну почему не ссылка, зачем копировать объект....
      Ответить

      • ava netrain 15.11.2010 09:07 # 0

        Потому что там так круто построен класс. Если просто передавать ссылку - все грохнется нафиг. Ибо в объекте же и хранится результат выполнения запросов.
        Т. е. код
        $query = 'SELECT * FROM `table`';
$db->query($query);
while ($i = $db->fetch())
{
$query2 = 'SELECT * FROM `table2` WHERE `somefield` = '.$i['somefield'];
$db->query($query2);
while ($i2 = $db->fetch())
{
// something
}
}

        работать не будет, ибо результат выборки перепишется на первой же итерации внешнего цикла. А таких мест в CMS достаточно, чтобы не браться за ее переделку.
        Ответить

    • ava Мистер Хэнки 12.11.2010 13:49 # +2 

      $_POST['title'] = ...

      а разве подобные массивы не должны быть неизменяемыми?
      Ответить

      • ava bugmenot 12.11.2010 14:02 # 0

        по логике - должны, но это же пхп
        Ответить

        • ava Lure Of Chaos 12.11.2010 22:15 # 0

          это придумано для говна типа:
          <?php
$_POST['title'] = 't1';
include('page.php');
$_POST['title'] = 't2';
include('page.php');

          page.php:
          <?php
echo $_POST['title']
          Ответить

          • ava punishment 14.11.2010 14:26 # 0

            каждый думает в меру своей испорченности.
            Как будто от использования
            $title = $_POST['title'];
include 'page.php';
$title = 't2';
include 'page.php';


            page.php

            echo $title;


            что-то изменится.
            Ответить

        • ava punishment 14.11.2010 14:20 # −2

          Глобальные массивы "по логике" должны быть неизменными? новые умы, новые идеи:)
          Ответить

          • ava xXx_totalwar 14.11.2010 14:39 # +1

            глобальные массивы должны быть?
            Ответить

          • ava bugmenot 14.11.2010 14:49 # +1

            ну как бы, если подумать над этим вопросом головой, а не жопой =)
            Ответить

          • ava Мистер Хэнки 15.11.2010 19:56 # 0

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

        • ava dwinner 15.11.2010 12:51 # +1

          Ахереть!.. Вы сами должны за этим следить!
          Ответить

          • ava bugmenot 15.11.2010 18:19 # 0

            ...чтобы никакой гад не написал $_SERVER['REQUEST_METHOD'] = 'HEAD'
            Ответить

            • ava istem 16.11.2010 05:38 # 0

              Да. Глобальные массивы они такие...
              Хорошо хоть нет глобальных ссылок...
              Ответить

            • ava Arigato 16.11.2010 11:11 # 0

              В этом что-то есть. Кстати, иногда нечто подобное используется.
              Ответить

              • ava bugmenot 16.11.2010 11:58 # 0

                если это прописать в auto_prepend_file то можно собрать много лулзов
                Ответить

                • ava Arigato 16.11.2010 15:40 # 0

                  Например, при переходе на php3, можно один раз прописать что-то такое:
                  $_SERVER = $HTTP_SERVER_VARS;
                  И не мучиться :)
                  Ответить

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