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

    +153.9

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    <?php
        /* ... */
	$sbj->hasNewMsgs = false;
	if (User::isLogged()) {
if (0) {
		if (/* Condition */) {
			$sbj->hasNewMsgs = true;
		}
}
	}
        /* ... */
?>

    Пишу форум. Выкладываю в публичный доступ более менее стабильную версию. Обнаруживается баг: «когда в теме есть новые сообщения — она не помечается, хотя категория, которая её содержит — помечается, как "с новыми сообщениями"». Неделю ленюсь залезть в исходники, найти, где же этот баг закрался (представляя, сколько кода надо перелопатить). Таки зашёл и сразу к модели, а там...
    Видно, дебажил и забыл убрать «экранизацию»

    Запостил: Shock, 12 Августа 2009

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

    • ava guest 13.08.2009 11:28 # 0

      Зачем if (Condition) в if-е? Ведь можно if (User::isLogged() and Condition) {...
      Ответить

      • ava guest 14.08.2009 03:31 # 0

        я часть, которая не несет смысловой нагрузки, — вырезал.
        Ответить

    • ava guest 20.08.2009 11:58 # 0

      "дебажил и забыл убрать" - это не говнокод. вот если б специально сделал, тогда да )
      Ответить

    • ava guest6 29.08.2023 01:35 # 0

      >Пишу форум.
      зачем?
      Ответить

      • ava OCETuHCKuu_nemyx 29.08.2023 23:07 # 0

        Что за глупый вопрос? Каждый в жизни должен посадить дерево, построить дом, родить ребёнка и написать сайт на пхп.
        Ответить

    • ava OCETuHCKuu_nemyx 29.08.2023 23:07 # 0

      Ну как, написал?
      Ответить

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