Можно переписать так:
var findFiles = Directory.EnumerateFiles(path, "*.*", SearchOption.AllDirectories)
.Where(s => s.EndsWith(".opus", comp) || s.EndsWith(".ogg", comp) || s.EndsWith(".tak", comp));
return findFiles.Any(File.Exists);
Строго говоря, это не будет оптимальнее. Под оптимальнее скорее всего, вы имеете в виду быстрее. Эта задача решается всегда за линейное время, так что если у вас перебор в цикле по массиву до первого подходящего элемента, это уже оптимально.
Метод Any
делает то же самое, что и ваш код, но его вызов чуть короче.
С другой стороны, как тут подсказали в комментарии, вызывать File.Exists
не нужно, поскольку метод EnumerateFiles
возвращает существующие файлы.
А предикат проверки расширений можно вынести в Any
, так что код можно упростить так:
public bool IsExistAnyAudioFile(string path)
{
var files = Directory.EnumerateFiles(path, "*.*", SearchOption.AllDirectories);
return files.Any(s => s.EndsWith(".opus", StringComparison.OrdinalIgnoreCase)
|| s.EndsWith(".ogg", StringComparison.OrdinalIgnoreCase)
|| s.EndsWith(".tak", StringComparison.OrdinalIgnoreCase));
}
UPDATE
Обновил код по комментарию @АндрейNOP. Расширения вынес как приватное статическое поле класса, чтобы инициализировались только одни раз.
private readonly ICollection<string> extentions = new HashSet<string>(StringComparer.InvariantCultureIgnoreCase)
{
".opus",
".ogg",
".tak"
};
public bool IsExistAnyAudioFile(string path)
{
return Directory.EnumerateFiles(path, "*.*", SearchOption.AllDirectories);
.Select(Path.GetExtension)
.Any(extentions.Contains);
}
Directory.EnumerateFiles
? Просто напишитеreturn findFiles.Any();
File.Exists
, до того как результатом воспользуется внешний код :)