1. Python / Говнокод #9935

    −99

    1. 1
    2. 2
    3. 3
    referer = ''
if 'HTTP_REFERER' in request.META :
    referer = request.META['HTTP_REFERER']

    И подобного кода в проекте тонны...

    Запостил: Scipetr, 12 Апреля 2012

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

    • ava loststy 12.04.2012 17:57 # −2

      Лол, джанго?

      Я обычно делаю
      referer = request.META.get('HTTP_REFERER', None)
      Ответить

    • ava Lure Of Chaos 12.04.2012 18:29 # +1

      неоптимально, но не особо и страшно.
      Ответить

      • ava Scipetr 12.04.2012 18:49 # +1

        Давай к нам у нас этого не особо и страшно дохера. Буквально вчера закончил рефакторить один модуль, выкинув вот такие костыли сократил код раза в четыре. И теперь смотришь и хоть понимаешь что этот модуль делает )
        Ответить

        • ava Lure Of Chaos 12.04.2012 19:03 # 0

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

        • ava lucidfoxGovno 12.04.2012 20:21 # 0

          лол, люра job offer получил
          Ответить

    • ava WGH 12.04.2012 18:43 # +1

      Иллюстрация типичной ситуации, которая возникает, если человек не хочет читать маны.
      Ответить

      • ava Lure Of Chaos 12.04.2012 19:02 # +1

        а мне кажется, это иллюстрация другой типичной ситуации, когда прогер пытается обьяснять свои мысли машине, переводя с человеческого на ЯП. Очень ясно представляю процесс рождения кода:

        "если реферер существует..." (пишет: if 'HTTP_REFERER' in request.META :)
        "тогда... мне он понадобится" (пишет: referer = request.META['HTTP_REFERER'])
        *спотыкается на ошибку неинициализированной переменной
        "бля, забыл, что мне нужно, если реферера нет... а его, оказывается, может не быть!"
        *быренько дописывает referer = ''
        "хм, ну тогда... пустой. о, работает!"
        дальше руки копипастят, голова на покое.
        Ответить

        • ava govnozmey 14.04.2012 21:33 # +1

          Отнюдь.
          Просто пыхер, привыкший к чему-то вроде
          $value = 'default';
if (isset($array['key'])) { $value = $array['key']; }
          Ответить

    • ava guest8 09.04.2019 13:05 # −999

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

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