Лучший рефакторинг для ужасного цикла While (True)

Нам действительно нужно рефакторинг циклов while (true)? Иногда это стандарт кодирования, и большинство разработчиков привыкли к такой структуре. Если вам нужно хорошо подумать о том, как реорганизовать этот код, уверены ли вы, что это хорошая идея?

Goto раньше был паршивой овцой в стандартах кодирования. Я встречал алгоритмы, в которых goto делал код более читаемым и короче. Иногда не стоит проводить рефакторинг (или лучше использовать goto).

С другой стороны, в большинстве случаев вы можете избегать while (true).

Я бы предпочел

start:

   // code goes here

goto start;

Это наиболее четко выражает намерение. Удачи в преодолении ваших стандартов кодирования. (Интересно, сколько кармы мне это будет стоить).

Чего в этом так страшно? Попробуйте найти общее условие прерывания и преобразовать его в начало цикла. Если это невозможно - хорошо.

Когда я сталкиваюсь с циклом while (true), это говорит мне либо

1. the break condition is not easily tested at the top (or bottom) of the loop,
   • there are multiple break conditions,
   • или предыдущий программист был слишком ленив, чтобы правильно разложить цикл.

1 и 2 означают, что вы также можете придерживаться while (true). (Я использую for(;;), но, на мой взгляд, это стиль.) Я с другим плакатом, чего бояться этого? Я боюсь зацикленных петель, которые прыгают через обручи, чтобы петля скатывалась "должным образом".

Зачем рефакторинг? И что же такого "ужасного" в этой конструкции? Он широко используется и хорошо изучен.

Если он не сломан, не чините его.

Замените True условием, которое вы собирались использовать для выхода из цикла.

В случае службы или фонового потока вы можете использовать:

volatile bool m_shutdown = false;
void Run()
{
    while (!m_shutdown)
    { ... }
}

Ситуация «работать вечно» иногда является частью более крупной машины состояний. Многие встроенные устройства (с бесконечными циклами) на самом деле не работают вечно. У них часто есть несколько рабочих режимов, и они будут последовательно переключаться между этими режимами.

Когда мы создавали контроллеры тепловых насосов, некоторое время работал режим самотестирования при включении питания (POST). Затем был предварительный режим сбора окружающей среды, который длился до тех пор, пока мы не выяснили все зоны, термостаты и все такое.

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

Я предпочитаю рассматривать «вечный» цикл как действительно один рабочий режим - в какой-то момент в будущем могут быть другие.

someMode= True
while someMode:
    try:
        ... do stuff ...
    except SomeException, e:
        log.exception( e )
        # will keep running
    except OtherException, e:
        log.info( "stopping now" )
        someMode= False

При некоторых обстоятельствах ничто из того, что мы до сих пор не видели, устанавливает someMode в False. Но мне нравится делать вид, что в какой-то будущей версии будет изменение режима.

#define ever 1
for (;ever;)

?

Мех, просто оставьте это как есть, хотя (правда), вероятно, настолько разборчиво, насколько вы собираетесь получить ...

эээ, рефакторинг .....

• Замените бесконечный цикл бесконечной рекурсией :-)

хорошо, если у вас есть язык, поддерживающий вызовы Tail ....

Если вы хотите, чтобы это продолжалось бесконечно до полного прерывания выполнения программы, я не вижу ничего плохого в while (true). Я столкнулся с этим недавно в службе сбора данных .NET, которая объединила while (true) с thread.sleep, чтобы просыпаться каждую минуту и ​​опрашивать стороннюю службу данных на предмет новых отчетов. Я подумал о его рефакторинге с помощью таймера и делегата, но в конечном итоге решил, что это самый простой и легкий для чтения метод. В 9 случаях из 10 это явный запах кода, но когда нет условия выхода, зачем усложнять задачу?

Я не против, когда бесконечный цикл содержится в окне и умирает вместе с ним.

Подумайте о рекурсии Хассельхоффа.

void whiletrue_sim(void)
  {
    //some code
    whiletrue_sim();
  }

Предупреждение: ваш стек может переполниться.