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

    +46

    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
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    ...
    
    //----------------------------------@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
    	    //обработка картинки для анонса 
    	    $model->preview_picture = CUploadedFile::getInstance($model, 'preview_picture');
    
    	    //убедимся что каталоги созданы
    	    if (@mkdir($curr_month_folder, 0777))
    		chmod($curr_month_folder, 0777);
    	    if (@mkdir($img_dir, 0777))
    		chmod($img_dir, 0777);
    	    if (@mkdir($img_dir_thumbs, 0777))
    		chmod($img_dir_thumbs, 0777);
    	    if (@mkdir($temp_img_dir_base, 0777))
    		chmod($temp_img_dir_base, 0777);
    	    if (@mkdir($temp_img_dir, 0777))
    		chmod($temp_img_dir, 0777);
    	    if (@mkdir($temp_img_dir_snap, 0777))
    		chmod($temp_img_dir_snap, 0777);
    
    	    //если  картинка для анонса не передана
    	    if (!$model->preview_picture) {
    		//проверяем не была ли она загружена ранее во временую папку
    		if ($form->model->image) {
    		    $model->preview_picture = $form->model->image;
    		}
    		//а нет так нет
    	    } else
    
    ...

    форматирование сохранено.

    Вообще весь кусок около 500 строк, а и эти 500 откопипизжены еще
    в 7 методов этого класса.
    ...за что мне все это

    Запостил: Razban_Guestov, 26 Ноября 2012

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

    • показать все, что скрыто
      /me is crying by tears of empathy
      Ответить
    • Этот копипаст не раскрывает сути говнокода
      Ответить
      • вполне раскрывает - одно это чего стОит
        if (@mkdir($curr_month_folder, 0777))
        chmod($curr_month_folder, 0777);
        Ответить
        • Я так и не понял, то ли там просто копипаст вместо цикла, толи отборная разнообразная хрень...
          Ответить
    • А почему чмоды без собачек?

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

      Если же вы хотите отучить PHP-программиста ставить собачки, чтобы заткнуть интерпретатор в случае ошибок - заставьте его ставить собачки в каждом доступном для этого месте и пояснять в комментарии все возможные ошибки, которые каждая собачка затыкает. Тогда, восприняв эти слова всерьез (пообещайте сократить его дневной рацион бананов вдвое в случае невыполнения требования), он наконец-то изучит документацию и способы обработки ошибок в PHP. В следующем проекте снимите требование на обязательные собачки - качество кода заметно улучшится.
      Ответить
      • От ногтей не помогает. Проверено на себе. Я всегда выгрызаю все ровно. Без сосредотачивания.
        Ответить
        • Это способ действует только на детей дошкольного возраста. Взрослого такой лажей уже не проведешь.
          Ответить
          • Так и тогда не действовало.
            Ответить
            • В особо тяжелых случаях рекомендуют грызть ногти на ногах.
              Ответить
              • Рекомендую попробовать ножные ногти ближайшего бомжа.

                Если не понравится - перестанете грызть.
                Понравится - и вам и бомжу удовольствие.
                Ответить
                • - Маша, не грызи ногти! На ногах! У дедушки! И вообще отойди от гроба!
                  Ответить
                • Месье знает толк в извращениях
                  Ответить
              • Грыз, сейчас не достаю.
                Ответить
    • :D
      Ответить

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