2017-04-10 9 views
0

私はformPropsオブジェクトを反復して、キーをチェックし、エラーオブジェクトに同じキーの値を設定します。このコードをどのようにDRYできますか?

if (!formProps.email) { 
    errors.email = "Please enter an email" 
} 
if (!formProps.password) { 
    errors.password = "Please enter a password" 
} 
if (!formProps.passwordConfirm) { 
    formProps.passwordConfirm = "Please enter a password confirmation" 
} 
if (formProps.password != formProps.passwordConfirm) { 
    errors.password = "Passwords must match" 
} 
+5

を表示することができますので、配列を送信するなど、あなたのコードを改善する方法についての提案を探しているのだろうか?これは、スタックエクスチェンジのコードレビューサイトで指示されます。 http://codereview.stackexchange.com/ – amallard

答えて

0

これは、より良いあなたは、同時にすべてのエラー

var errors = []; 
!formProps.email ? errors.push("Please enter an email") : null; 
!formProps.password ? errors.push("Please enter a password") : null; 
!formProps.passwordConfirm ? errors.push("Please enter a password confirmation") : null; 
formProps.password != formProps.passwordConfirm ? errors.push("Passwords must match") : null; 
if(errors.length > 0){ 
    return errors; 
} 
関連する問題