Setup fixes #216
Setup fixes #216
Conversation
| else | ||
| # auth failed | ||
| zenity --warning --title="$_YD_auth_title" --text "$auth_output" | ||
| showAuthDialog |
There was a problem hiding this comment.
А зачем повторый вызов авторизации через рекурсию делать? В цикле что-ли нельзя?
There was a problem hiding this comment.
В цикле - while $notAuthorized? Используя функцию не нужно заводить переменную, повторный вызов явно указывает на точку входа, не нужно проверять возможную вложнность циклов - функциональный вариант более читабельный.
There was a problem hiding this comment.
Завести функцию и заваливать стек рекурсивными ее вызовами это конечно экономнее чем завести одну переменную для одного (не вложенного) простого цикла. :)
Извините, но я в душе максималист-оптимизатор и рекурсия вместо цикла - для мня - стрельба из пушки по воробьям.
Рекурсия бывает нужна - не спорю. Но не тут.
Где там нужна проверка вложенности каких циклов, и чем это вредит читабельности - мне тоже не понять...
|
Через Google translate |
No description provided.