Си диез / Говнокод #2501 Ссылка на оригинал

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
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
public class PlayerWindow
{
    private readonly ScheduleManager _scheduleManager = new ScheduleManager();

...

    private void OpenSchedule(string timesheetFilename)
    {
        if (!_scheduleManager.OpenSchedule(timesheetFilename))
        {
            _isOpen = false;
            return;
        }

        _isOpen = false;
    }
}

Собственное творчество. ^_^

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

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

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

    Та не говнокод это.. Ну, спутал true с false, и че? Не интересно..
    Ответить

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

      Ну тогда можно было ограничиться без ветвления, чем-то подобным: _isOpen = _scheduleManager...
      Да и я зря не уточнил - это единственное место, где "изменяется" _isOpen. И при этом, программа работает нормально.
      Ответить

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

        Господи! Что я делаю... Пытаюсь доказать, что пишу тру-говнокод. о_О
        Ответить

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

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

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

              Это на меня семестр проектирования uml херово действует) Я имел ввиду член класса "_isOpen"
              Ответить

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

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

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

    ↑
    8
    ↓