1. C# / Говнокод #8845

    +122

    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
    while (!requestedTermination)
{
    // ...
    // тут  130 строк кода...
    // ...

    if (requestedTermination)
    {
        break;
    }
    else
    {
        // to prevent excess CPU usage
        Thread.Sleep(100);
    }
}

    requestedTermination - Property, изменяемое другим потоком

    Мораль: не пишите длинные циклы - к концу цикла забудете, какое у него было условие завершения.

    Запостил: burdakovd, 16 Декабря 2011

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

    • ava Lure Of Chaos 16.12.2011 01:36 # 0

      досрочный выход из цикла
      Ответить

      • ava burdakovd 16.12.2011 01:40 # +1

        Если убрать break, то, спустя 100 миллисекунд, мы всё равно вышли бы из цикла.
        Ответить

        • ava guest 16.12.2011 01:42 # −1

          его комментарии как правило не несут вообще никакой смысловой нагрузки можно было и не отвечать
          Ответить

        • ava guest 16.12.2011 17:22 # 0

          А если за эти 100 мс requestedTermination изменится?
          Ответить

          • ava burdakovd 16.12.2011 17:31 # 0

            Во-первых в данном случае оно не изменилось, извне оно может меняться только из false в true, но не обратно.

            Во-вторых, если бы оно могло измениться за 100 мс, то оно могло бы измениться и за несколько тактов между if(requestedTermination) и break.

            Не питайте иллюзий, что "мы успеем сделать %THING% прежде чем значение %VARIABLENAME% изменится".
            Ответить

    • ava guest 16.12.2011 01:40 # −2

      в дисклеймере написано про сишной код, а тут ничего смишного уже давно нет
      Ответить

    • ava guest 16.12.2011 09:23 # +3

      >Thread.Sleep(100);

      напоминает осла из шрека:
      -а мы приехли?
      -нет
      -а сейчас?
      -нет
      ....
      Ответить

      • ava Steve_Brown 16.12.2011 09:34 # 0

        Бывает, что это необходимо... скажем, если в цикле стоит recv. Есть данные? А сейчас? Да, буду спрашивать каждые 100 мс, а теперь?
        Ответить

      • ava guest 16.12.2011 10:46 # +3

        Было вчера на хабре.
        Ответить

    • ava guest8 09.04.2019 12:23 # −999

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

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