"PHP" / Говнокод #1561 Ссылка на оригинал

0

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

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

Запостил: Shock Shock, (Updated )

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

  • ava guest # Ссылка на оригинал 0

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

    • ava guest # Ссылка на оригинал 0 ⇈

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

  • ava guest # Ссылка на оригинал 0

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

    • ava OCETuHCKuu_nemyx # Ссылка на оригинал 0 ⇈

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

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓