История изменений

По теме уже указали (из-за точки с запятой у тебя if отдельно, блок отдельно, да и код неполный, непонятно, где строка инициализируется).

Но я бы ещё добавил:

1. Если ты обрабатываешь строку, выводить её после каждой итерации цикла — плохая идея. Лучше после цикла выведи результат.

2. Если я правильно понял твой замысел, весь этот цикл элементарно заменяется одним QString::replace().

3. Объявлять счётчик и границы цикла (если они больше нигде не нужны) обычно лучше всего внутри самого for. Да, из этого правила бывают исключения, но тут явно не тот случай.

P.S. И кстати, чем у тебя с QuaZip дело закончилось? Ты из каждой темы исчезаешь, ни решённой не помечаешь, ни наоборот.

По теме уже указали (из-за точки с запятой у тебя if отдельно, блок отдельно, да и код неполный, непонятно, где строка инициализируется).

Но я бы ещё добавил:

1. Если ты обрабатываешь строку, выводить её после каждой итерации цикла — плохая идея. Лучше после цикла выведи результат.

2. Если я правильно понял твой замысел, весь этот цикл элементарно заменяется одним QString::replace().

3. Объявлять счётчик и границы цикла (если они больше нигде не нужны) обычно лучше всего внутри самого for. Да, из этого правила бывают исключения, но тут явно не тот случай.

P.S. И кстати, чем у тебя с QuaZip дело закончилось?

По теме уже указали (из-за точки с запятой у тебя if отдельно, блок отдельно, да и код неполный, непонятно, где строка инициализируется).

Но я бы ещё добавил:

1. Если ты обрабатываешь строку, выводить её после каждой итерации цикла — плохая идея. Лучше после цикла выведи результат.

2. Если я правильно понял твой замысел, весь этот цикл элементарно заменяется одним QString::replace().

3. Объявлять счётчик и границы цикла (если они больше нигде не нужны) обычно лучше всего внутри самого for. Да, из этого правила бывают исключения, но тут явно не тот случай.

По теме уже указали (из-за точки с запятой у тебя if отдельно, блок отдельно).

Но я бы ещё добавил:

1. Если ты обрабатываешь строку, выводить её после каждой итерации цикла — плохая идея. Лучше после цикла выведи результат.

2. Если я правильно понял твой замысел, весь этот цикл элементарно заменяется одним QString::replace().

3. Объявлять счётчик и границы цикла (если они больше нигде не нужны) обычно лучше всего внутри самого for. Да, из этого правила бывают исключения, но тут явно не тот случай.