такой код — знак плохого алгоритма\программиста или это норма?

а в чём проблема?

ТС боится, что эту халабуду можно было сделать лаконичнее

ТС, я тебе даже так скажу. когда ты под действием веществ или когда ты старишься, ты уже не сможешь так легко всё оптимизировать, просто потому что твой мозг будет слабее. тогда может случиться даже что будешь делать вложенные ifы в то время, когда можно было использовать andы.

однако, такое может случиться и, банально, когда условие будет иначе нечитабельно.

так что, до тех пор, пока код не плодит лишних итераций и не жрёт лишней памяти, и делает именно то что нужно, может и не очень лаконичными выражениями, то всё ок..

ну и да,: Readability counts.

Не знаю где ты живешь+работаешь, но внимание к тестированию это действительно тенденция, там где этого еще не было и нет. Видимо начинает доходить, что просто увидеть что оно работает как-то это совершенно не гарантия провала/позора.

Да, но автор в данном случае именно и делает, что читает этот код.

Ничего очень критичного нет как по мне.

Можно разбить на функции и убрать дублирование. И не использовать названия переменных вроде counter_2. И вообще не писать функции длиннее половины экрана. И использовать пустые строки. И комментировать непонятные выражения.

В остальном всё нормально.