C# と VB.NET の質問掲示板

■No100381 (おさのり さん) に返信

私ならListにつめてAnyでやっちゃいます

var files = new List<string>()
{
  "filePath1",
  "filePath2",
  "filePath3",
  "filePath4",
  "filePath5"
};

if (files.Any(x => File.Exists(x)))
{
  i += 1;
}

■No100381 (おさのり さん) に返信

> そのまま書くと、以下のように非常に長く見た目も良くない為、改善したいです。
> if (File.Exists(filePath1)) || (File.Exists(filePath2)) || (File.Exists(filePath3)) || (File.Exists(filePath4)) || (File.Exists(filePath5))
> {
>     i += 1
> }

そのコードは間違っているのですがそこは本題ではないので置いといて・・・

見やすくする、一目でわかるようにする、後から保守する人のことを考える、組織の
コーディングルールがあればそれに従うというのがまず優先されるべきではないので
しょうか？

好みの問題かもしれませんが、自分的には以下のようなコードが、短くするなどの
ためにこだわっていじくって一目では分からないようなコードにするよりは、良さ
そうだと思うのですが・・・

if (File.Exists(filePath1) || 
    File.Exists(filePath2) || 
    File.Exists(filePath3) || 
    File.Exists(filePath4) || 
    File.Exists(filePath5))
{
    i += 1;
}