Исходное сообщение
"вариант без goto, повторов и функций" Отправлено Аноним, 27-Дек-23 05:31
1. Читабельность вашего решения насктолько хороша, что вы же сами запутались, где надо возвращать true, а где false. Оно и понятно, лесенка из else-ов в конце функции не слишком читабельна. 2. Основной код функции (в данном случае вызов doSomeJob) находится на четвёртом уровне вложенности. То есть с нормальной настройкой отступов левая половина окна редактирования будет пустовать. То есть всё то, с чем люди боролись десятилетиями, чтобы код не разъезжался в стороны, вы одним махом обнулили. 3. В реальном коде вместо doSomeJob() будет пара десятков строк кода. Это ОСНОВНОЙ код функции, ради которого всё и затевалось, но у вас оно почему-то спрятано за 4 уровнем отступа. Кроме того, дойдя до завершающих else, читающему код придётся поворачивать монитор на 90 градусов и прикладывать к нему линейку, чтоб понять, какой из else какую развилку завершает. Кстати, ваш способ используется в Битриксе (российская система управления сайтами), и могу вас заверить, такой вариант, когда основной код имеет глубокую вложенности, а потом ещё завершается мутными rc = false, не шибко читаем. 4. Вы пишете, что код без повторов, но с завидным упорством повторяете else { rc = false; } .

Ваше сообщение
Имя*:
EMail:	Для отправки ответов на email укажите знак ! перед адресом, например, !user@host.ru (!! - не показывать email). Более тонкая настройка отправки ответов производится в профиле зарегистрированного участника форума.
Заголовок*:
Сообщение*:	> 1. Читабельность вашего решения насктолько хороша, что вы же сами запутались, где > надо возвращать true, а где false. Оно и понятно, лесенка из > else-ов в конце функции не слишком читабельна. > 2. Основной код функции (в данном случае вызов doSomeJob) находится на четвёртом > уровне вложенности. То есть с нормальной настройкой отступов левая половина окна > редактирования будет пустовать. То есть всё то, с чем люди боролись > десятилетиями, чтобы код не разъезжался в стороны, вы одним махом обнулили. > 3. В реальном коде вместо doSomeJob() будет пара десятков строк кода. Это > ОСНОВНОЙ код функции, ради которого всё и затевалось, но у вас > оно почему-то спрятано за 4 уровнем отступа. Кроме того, дойдя до > завершающих else, читающему код придётся поворачивать монитор на 90 градусов и > прикладывать к нему линейку, чтоб понять, какой из else какую развилку > завершает. Кстати, ваш способ используется в Битриксе (российская система управления сайтами), > и могу вас заверить, такой вариант, когда основной код имеет глубокую > вложенности, а потом ещё завершается мутными rc = false, не шибко > читаем. > 4. Вы пишете, что код без повторов, но с завидным упорством повторяете > else { rc = false; } .
	Введите код, изображенный на картинке: