Есть ли лучший способ реорганизовать этот код?

Question

Ниже мой код

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

Прежде всего, мне нужно проверить, если prop.plan [plan_id] является существовать затем использовать свои данные. Можно ли сделать этот код более красивым, потому что я дважды написал инструкцию if.

Answer 1

Кстати, я думаю, может быть, вместо того, чтобы использовать ==, === лучше в этом случае:

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, или пустую строку.