可読性のためにファンクション内にヘルパー関数をネストする

Question

現在、私はClean Codeで現在読んでおり、それぞれが独自の「ストーリー」を伝える超小型関数のアイデアが本当に好きです。私はまた、 "TO段落"という言葉でコードをどのように書かなければならないのか、頭の中で一種の名前を "to to"にすることを決めたのが本当に好きです。

とにかくより多くの意味のある名前を含むようにコードをリファクタリングし、それを少し読んでいく流れを作り、私は私が確信している何かに遭遇しました。

私は、コードスタイルは非常に議論の余地があり、主観的な話題ですが、うまくいけば、私はこの投稿でうまくいきません。

ありがとうございました！

PSA：現時点ではインターンシップの内部プロジェクトにMEANスタックを使用しているWebアプリケーションを作成しています。

クリーンコードのリファクタリング

//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の例を掲載すると、私はより多くの作業の余地があります。私はこれが最善の選択肢であると言っているわけではありません。それは、OPが私たちに与えた情報で使用するものです。

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(); 

このようにして、ヘルパー関数は一度しか定義されず、閉じた内部でプライベートに保たれます。関数x（）スタイルが好きな場合は、var x = function x構文をobvで削除できます。

編集：関数が一度しか呼び出されない場合は、独自の例がおそらくよりクリーンなコードになります。 IIFEの構文を使用する理由は、ヘルパー関数を自分の2番目の例のようにmain関数でのみアクセスできるようにするためです。

Answer 2

MEANjsは時には2番目の方法を使用します（コールバックなど） 。私は個人的には、あなたが主な機能の外でそれらのヘルパーを使用するつもりがないなら、それはいいと思う。