Вложенные вспомогательные функции внутри функции для удобства чтения

Question

Так что я сейчас читаю Clean Code, и мне очень нравится идея супер маленьких функций, каждая из которых рассказывает свою собственную «историю». Мне также очень нравится, как он пишет, как писать код, чтобы читать его в терминах «ТО абзацев», которые я решил переименовать в «0» для

В любом случае, я был рефакторинг много кода, чтобы включить более значимые имена и сделать поток, в котором он будет читаться немного лучше, и я наткнулся на то, о чем я не уверен, и, возможно, некоторые гуру здесь могут дать мне солидный совет!

Я знаю, что код-стили является весьма спорной и субъективной темой, но, надеюсь, я не буду получать развернутый из этого поста.

Спасибо всем!

PSA: Я ноб, свежий из колледжа, создающего веб-приложение, использующее стек MEAN для внутреннего проекта в рамках стажировки на данный момент.

Чистый код рефакторинг

//Modal Controller stuff above. vm.task is an instance variable 
vm.task = vm.data.task; 
castTaskDataTypesForForm(); 

    function castTaskDataTypesForForm() { 
    castPriorityToInt(); 
    castReminderInHoursToInt(); 
    castDueDateToDate(); 
    getAssigneObjFromAssigneeString(); 
    } 

    function castPriorityToInt() { 
    vm.task.priority = vm.task.priority === undefined ? 
     0 : parseInt(vm.task.priority); 
    } 

    function castReminderInHoursToInt() { 
    vm.task.reminderInHours = vm.task.reminderInHours === undefined ? 
     0 : parseInt(vm.task.reminderInHours); 
    } 

    function castDueDateToDate() { 
    vm.task.dueDate = new Date(vm.task.dueDate); 
    } 

    function getAssigneObjFromAssigneeString() { 
    vm.task.assignee = getUserFromId(vm.task.assignee); 
    } 

Возможно лучше рефакторинг?/Мой вопрос ----------------------------

//Modal Controller stuff above. vm.task is an instance variable 
vm.task = vm.data.task; 
castTaskDataTypesForForm(); 

    function castTaskDataTypesForForm() { 
    castPriorityToInt(); 
    castReminderInHoursToInt(); 
    castDueDateToDate(); 
    getAssigneObjFromAssigneeString(); 

    function castPriorityToInt() { 
     vm.task.priority = vm.task.priority === undefined ? 
     0 : parseInt(vm.task.priority); 
    } 

    function castReminderInHoursToInt() { 
     vm.task.reminderInHours = vm.task.reminderInHours === undefined ? 
     0 : parseInt(vm.task.reminderInHours); 
    } 

    function castDueDateToDate() { 
     vm.task.dueDate = new Date(vm.task.dueDate); 
    } 

    function getAssigneObjFromAssigneeString() { 
     vm.task.assignee = getUserFromId(vm.task.assignee); 
    } 
    } 

Answer 1

Публикация примера IIFE здесь, поэтому у меня больше возможностей для работы. Я не говорю, что это лучший вариант, это тот, который я использовал бы с информацией, которую дал нам ОП.

var castTaskDataTypesForForm = (function() { 
    var castPriorityToInt = function castPriorityToInt() { ... }, 
     castReminderInHoursToInt = function castReminderInHoursToInt() { .. }, 
     castDueDateToDate = function castDueDateToDate() { ... }, 
     getAssigneObjFromAssigneeString = function getAssigneObjFromAssigneeString() { ... }; 
    return function castTaskDataTypesForForm() { 
     castPriorityToInt(); 
     castReminderInHoursToInt(); 
     castDueDateToDate(); 
     getAssigneObjFromAssigneeString(); 
    }; 
}()); 
vm.task = vm.data.task; 
castTaskDataTypesForForm(); 

Таким образом, вспомогательные функции определяются только один раз и сохраняются в закрытом помещении. Вы можете исключить синтаксис var x = function x, если вы предпочитаете стиль функции x().

edit: Если функция вызвана только один раз, ваши собственные примеры, вероятно, являются более чистым кодом. Причина, по которой вы будете использовать синтаксис IIFE, будет заключаться в том, чтобы вспомогательные функции были доступны только основной функции, например, в вашем собственном втором примере.

Answer 2

MEANjs иногда использует второй метод (например, с обратными вызовами) , Я лично считаю, что это хорошо, если вы не собираетесь использовать эти помощники вне основной функции.