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));

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

5
  • Работает, но не уверен, что пройдёт ревью)
    – леха
    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
    @леха, стандартный вид цикла это while(){} ну не как не do{}while(), когда-нибудь, схватите с ним жопа-болей.
    – And
    29 окт 2019 в 13:10
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 делали, чтобы его применять :)
    – Mikhailo
    29 окт 2019 в 12:52
0

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

if() {
}

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

while (true) {
Тело
}

Либо

for(;;) {
}

Ваш ответ

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

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