Правильно написание кода [закрыт]

101

15 сентября 2021, 21:00

Закрыт. На этот вопрос невозможно дать объективный ответ. Ответы на него в данный момент не принимаются.

Хотите улучшить этот вопрос? Переформулируйте вопрос так, чтобы на него можно было дать ответ, основанный на фактах и цитатах.

Закрыт 1 год назад.

Улучшить вопрос

Изучаю JS, часто наблюдаю, как все более-менее опытные дядьки твердят, что функция должна выполнять только одно действие, ибо в противном случае - говнокод.

То есть, если есть некая функция checkPermission то она должна только проверять разрешение и возвращать true/false, но никак не выводить на экран с сообщением.

Собственно, вот мой код:

let access = true;
function checkPermission(access) {
    if(access) alert("access true");
    else alert("access denied!");
}
checkPermission(access);

Следуя Библии всея кодинга, так делать ниок. Извольте отредактировать мой код как положено и пояснить, почему и как надо делать.

Answer 1

В классическом виде код можно представить так:

let access = true;
function checkPermission(access){
    return !!access;
}
function alertPermission(access) {
    let permission = checkPermission(access);
    alert("access " + permission);
}
alertPermission(access);