僕がWebエンジニア1年目に受けたコードレビュー①
※実際の開発現場のコードを元にして作成
【タスク】
、
、
のどれか一つが表示されるコードを書きましょう
Beforeのコード
ーーーーーーーーーーーーーーーーー
return (
<>
{isDog && }
{!isDog && !isCat && }
{isCat && }
</>
);
ーーーーーーーーーーーーーーーーー
【レビュアーからのコメント】
視覚的に「どれか一つしか当てはまらない（排他制御）」ということがわかりにくいので書き直してください
After（「どれか一つしか表示されない」という意図が一目で分かる構造）
ーーーーーーーーーーーーーーーーーー
const getAnimal = () => {
if (isDog) {
return "";
} else if (isCat) {
return "";
} else {
return "";
}
};
return getAnimal();
ーーーーーーーーーーーーーーーーーーー
改善ポイント
・「どれか1つだけ」という意図が一目で分かる排他構造に改善
#Webエンジニア1年目のコードレビュー
、、のどれか一つが表示されるコードを書きましょう
Beforeのコード
ーーーーーーーーーーーーーーーーー
return (
<>
{isDog && }
{!isDog && !isCat && }
{isCat && }
</>
);
ーーーーーーーーーーーーーーーーー
【レビュアーからのコメント】
視覚的に「どれか一つしか当てはまらない（排他制御）」ということがわかりにくいので書き直してください
After（「どれか一つしか表示されない」という意図が一目で分かる構造）
ーーーーーーーーーーーーーーーーーー
const getAnimal = () => {
if (isDog) {
return "";
} else if (isCat) {
return "";
} else {
return "";
}
};
return getAnimal();
ーーーーーーーーーーーーーーーーーーー
改善ポイント
・「どれか1つだけ」という意図が一目で分かる排他構造に改善
#Webエンジニア1年目のコードレビュー