是否有更好的方法来重构这段代码？

Question

下面是我的代码

if(props.plan[plan_id]) { 
    if(props.plan[plan_id].number== 0) { 
     return null; 
    } 
    else { 
     return 'some value'; 
    } 
    } 

首先，我需要检查，如果prop.plan [plan_id的数据类型]是存在，那么使用它的数据。 有什么办法可以使这段代码更漂亮，因为我写了两次语句。

Answer 1

BTW，我想也许不是使用==，===在这种情况下更好：

if(props.plan[plan_id]) { 
    return props.plan[plan_id].number === 0 ? null : 'some value'; 
    } 

Answer 2

使用三元运算：

if (props.plan[plan_id]) 
    return (props.plan[plan.id].number == 0) ? null : 'some value'; 

Answer 3

使用&& operator可以节省你从两个if小号因为它会在第一条语句中终止，它可以转换为false（不会尝试检查props.plan[plan_id]的值，除非它存在，并且左侧评估为true）：

return props.plan[plan_id] && props.plan[plan_id].number != 0 ? 'some value' : null; 

Answer 4

为了确保没有运行时错误，我会做到这一点：

return props && props.plan && props.plan[plan_id] ? 'some value' : null 

您将获得some value如果props.plan[plan_id]不存在，或者其值之一：0 ，undefined,null或空字符串。