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

    +173.4

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    function p_id($p_id)
{
	global $product_id;
	
	foreach($product_id as $k=>$v)if($p_id==$k)
	{
		$sel_r=$k;
		$sel_t=$v;
	}
	return $sel_t;
}

    Реальная функция и даже реально используется.
    Сотрудник, далекий от кодинга, иногда чтобы не отвлекать меня разные мелочи кодит сам... Ну там по образцам типа...
    Функцию эту кстати я трогать не стал исходя из золотого правила - "работает - не трогай!", так и используется и по сей день...

    Запостил: larrygingras2, 09 Апреля 2010

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

    • ava HyperGeek 09.04.2010 19:47 # +0.8

      Интересно, где он такие образцы берет? Наверное на php.net ))
      Кстати, было уже подобное оттуда http://govnokod.ru/2887
      Ответить

      • ava larrygingras2 09.04.2010 19:52 # 0

        Мне скорее интересно, где он такую траву берет...
        А насчет образцов - утверждает что у меня где-то подобный цикл подсмотрел и адаптировал. Ну то есть сам цикл примерно такой структуры имеет право на существование (я даже примерно догадываюсь где именно он у меня подобное увидел), но там всё несколько сложнее разумеется. А он просто взял за основу и упростил для данной задачи :)
        Ответить

        • ava murz 09.04.2010 20:26 # 0

          Какая трава?..... Наоборот, он молодец, раз далек от кодинга и что-то делает сам
          Ответить

          • ava larrygingras2 10.04.2010 12:21 # 0

            Кстати "палка о двух концах"...
            Как правило подобный код делается без всякого понимания, чисто методом "тыка". Не работает - ну наверное вот тут надо поменять - о, заработало, заебись!
            Проблема в том, что зачастую оно может и работает как надо, но создает ТАКИЕ уязвимости, что сердце в пятки уходит при виде подобных шедевров на production сервере... Особенно если речь о компании, работающей в сфере онлайн-коммерции, ЕВПОЧЯ...
            Ответить

      • ava larrygingras2 09.04.2010 20:00 # 0

        "Источником вдохновения", вероятно, послужил вот этот код:
        foreach($bonus_nr as $k=>$v)if($row["total"]>$k&&$max_total<$k)$max_total=$k;
        Ответить

        • ava guest 10.04.2010 13:38 # +1

          И это Вы называете лаконичным кодом?
          Ответить

          • ava larrygingras2 10.04.2010 13:40 # 0

            А как надо?
            Ответить

            • ava HyperGeek 10.04.2010 13:53 # 0

              Отсутствие пробелов != лаконичность ;)
              Ответить

              • ava larrygingras2 10.04.2010 13:57 # 0

                Я в курсе.
                Я имею в виду как надо более лаконично или, скажем так, изящно (с точки зрения грамотного применения языка) оформить данный алгоритм?
                Ответить

                • ava HyperGeek 10.04.2010 14:09 # 0

                  Вот как-то так можно
                  $max_total = array_reduce(array_keys($bonus_nr), 'max', $row['total']);
                  Ответить

                  • ava larrygingras2 10.04.2010 14:19 # 0

                    Результатом выполнения Вашего кода будет всего лишь максимальное значение ключа массива или $row['total'] (если оно больше).
                    Слив защитан
                    Ответить

                    • ava HyperGeek 10.04.2010 14:25 # 0

                      Я лишь модифицировал твой код, там тоже ключи сравниваются )
                      Ответить

                      • ava larrygingras2 10.04.2010 14:32 # 0

                        О какой модификации может идти речь если результат выполнения разный?!
                        Мой код находит максимальный ключ, значение которого меньше $row["total"].
                        Приведенный тобой код чуть более чем безсмысленнен: он всего лишь находит максимальное значение из множества ключей массива и $row['total']
                        Неужели мне нужно привести конкретные примеры входных данных, чтобы было понятно, в чем разница?!
                        Ответить

                        • ava HyperGeek 10.04.2010 14:46 # 0

                          Чорт, не разгледел... каюсь
                          Ответить

                          • ava larrygingras2 10.04.2010 14:51 # +1.2

                            Бывает, ничего страшного :)
                            В конце концов мы сюда не на олимпиаду по информатике пришли, а показать именно особо отмороженные вещи...
                            Ответить

      • ava larrygingras2 09.04.2010 20:33 # 0

        Да, те же яйца
        Адъ :(
        Ответить

    • ava guest 10.04.2010 08:55 # +2

      Скоко похапешники не втычивают долларов в свои программы, а платить им за говнокод больше не станут :Р
      Ответить

      • ava larrygingras2 10.04.2010 12:15 # 0

        К счастью, в нашей компании, в отличии от индусов, объем проделанной работы не измеряется килобайтами кода.
        Лично мой код как правило лаконичен донельзя.
        Ответить

      • ava HyperGeek 10.04.2010 13:56 # +1.4

        Мы берем не в долларах, а в рупиях ) Или что там в Индии ))
        Ответить

    • ava Death 10.04.2010 18:41 # +1.2

      А как дурак использую $product_id [ $p_id ] везде, а оно вот как оказывается надо...
      Эх, придётся везде переписывать "как надо"...
      Вернее так как всё у нас делается (через жопу)
      Ответить

    • ava mrbig66 12.04.2010 09:41 # 0

      Зачем нужен set_r, если возвращается только set_t?
      Ответить

    • ava guest 14.04.2010 23:18 # −5

      показать все, что скрытоА ты ещё не знаешь С++? Тогда мы идём к вам.
      Ответить

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