3

Можно ли заменить следующий бесконечный цикл на обычный цикл? Слышал, что всегда можно, но не понимаю как.

while (true) {
    if (c1(a)) {
        a = f1(a);
    }

    if (c2(a)) {
        break;
    }

    a = f2(a);
}

Здесь f1, f2 - функции, возвращающие какое-то значение, зависящее от a, c1, c2 - условия (функции, возвращающие bool), зависящие от a.

upd:

под обычным я понимаю без break, continue, goto и прочих хаков, а чтобы условие было не постоянным/пустым

1
  • 1
    Можно, но смысла никакого.
    – Qwertiy
    29 окт 2019 в 12:59

4 ответа 4

3

Например можно так. Но я бы не сказал что это улучшит читабельность.

for (;!c2(a=!c1(a)?:f1(a));a=f2(a));

Внезапный код-гольф?

4
  • Работает, но не уверен, что пройдёт ревью)
    – леха
    29 окт 2019 в 12:59
  • @леха, do-while прям пройдет? Я бы лучше делал через for.
    – And
    29 окт 2019 в 13:01
  • @леха ну попробовать можно) мне интересно в глаза коллег посмотреть с таким кодом было бы.
    – pavel
    29 окт 2019 в 13:02
  • @And а в чём проблема do-while? это стандартный вид цикла. Условие, которое всегда в конце, имхо лучше, чем искать break где-то в середине тела цикла
    – леха
    29 окт 2019 в 13:02
1

Заменить можно так

do {
  if (c1(a)) {
    a = f1(a);
  }

  cond = !c2(a);

  if (cond)
    a = f2(a);
} while (cond);

но в данном случае бесконечный цикл нагляднее

2
  • 1
    Для заметки, если кому интересно, сильно ли повляет на производительность: сделал бенчмарк кода из вопроса и этого: на маленьком колве итераций код из вопроса чуть быстрее, на большом -- одинаково
    – леха
    29 окт 2019 в 13:12
  • @леха: Проверять подобные минимальные изменения "на производительность", да еще и без информации о содержимом вызываемых функций - совершенно бессмысленное занятие, результаты которого абсолютно ничего не значат. 29 окт 2019 в 14:11
1

Если вы хотите for (обычный) - то можно просто

for(;;) {
    if (c1(a)) {
        a = f1(a);
    }

    if (c2(a)) {
        break;
    }

    a = f2(a);
}

Или

for(;;a = f2(a)) {
    if (c1(a)) a = f1(a);
    if (c2(a)) break;
}

Мне кажется, что так понятнее всего.

7
  • Под обычным я понимаю не содержащий бесконечное условие
    – леха
    29 окт 2019 в 12:48
  • Короче без break и continue
    – леха
    29 окт 2019 в 12:48
  • @леха у нас goto ещё есть)
    – pavel
    29 окт 2019 в 12:48
  • Да и без него тоже
    – леха
    29 окт 2019 в 12:49
  • 1
    Кто говорит - пусть пишут по-другому. Писать нужно так, чтоб было понятно человеку, и чтоб поменьше ошибок делать. Для того и break делали, чтобы его применять :)
    – Mikhajlo
    29 окт 2019 в 12:52
0

Если захотите, то можно ещё использовать Go to, но это не советуется использовать. Главное не забывайте, что нужно условие выхода, оно делается через

if() {
}

Самым практичным является

while (true) {
Тело
}

Либо

for(;;) {
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.