私は完全に動作する以下のコードを持っています。しかし、それを見るだけで、より短くてエレガントな方法が必要であると私は信じています。 Switch()は明らかに答えではないので、私は入れ子になっています。ネストされたIFを短くしてください
if (mode == 1)
{
if (distance <= 4000)
{
modeValue = "1F";
}
else if (distance > 4000 && distance <= 8000)
{
modeValue = "2F";
}
else if (distance > 8000 && distance <= 12000)
{
modeValue = "3F";
}
else if (distance > 12000)
{
modeValue = "F 0-5";
}
}
else if (mode == 2)
{
if (distance <= 500)
{
modeValue = "";
}
else if (distance > 500 && distance <= 4000)
{
modeValue = "2F";
}
else if (distance > 4000 && distance <= 8000)
{
modeValue = "3F";
}
else if (distance > 8000 && distance <= 12000)
{
modeValue = "4F";
}
else if (distance > 12000)
{
modeValue = "F 0-5";
}
}
[CodeReview](http://codereview.stackexchange.com/)でこの質問を試してください。 – Jonesopolis
クリーンアップする方法の1つは、すべてを方法化することです。物事をするもう一つの方法は、上記のロジックに基づいてmodeValueを導出するメソッドを含むオブジェクトのプロパティを距離にすることです。それを完全に取り除くのではなく、もっと組織化しています。 –
短い方が良いとは限りません。コードが正常に機能している場合は、理解して維持しやすいようにしてください。 – Nasreddine