2

Появилась надобность написать метод, который по указанному пути будет удалять из директории один файл. Ниже приведены варианты. Не могу решить, какой из них наиболее подходящий с точки зрения читаемости и производительности.

Method #1

public static void ClearDirectory(string path)
{
    var di = new DirectoryInfo(path);
    foreach (var file in di.GetFiles())
        file.Delete();
}

Method #2

public static void ClearDirectory(string path)
{
    var di = new DirectoryInfo(path);
    di.GetFiles()[0].Delete();
}

Method #3

public static void ClearDirectory(string path)
{
    var di = new DirectoryInfo(path);
    var _file = di.GetFiles()[0];
    _file.Delete();
}

Method #4

public static void ClearDirectory(string path)
{
    new DirectoryInfo(path).GetFiles()[0].Delete();
}

На скриншоте ниже показано среднее время выполнения каждого метода. Как можно заметить - последние два имеют наименьший результат. введите сюда описание изображения

Нужен совет. Какой из данных методов предпочтительнее использовать?

6
  • 1
    Или я туплю, но только 1 метод удаляет все файлы, остальные удаляют только первый файл из всех. 18 июл 2018 в 13:58
  • 1
    в 1м методе удаляются все файлы, а в остальных только первый
    – Mikhail M
    18 июл 2018 в 13:58
  • 3
    Первый метод - удаление всего в директории, все остальные делают все тоже самое, разница только в том, что где то вы сгруппировали различные методы в одну "портянку", а где то наоборот, разбили все по своим переменным. Тут вы экономите совершенно не на том... Что касается самого задания, то на кой черт вам перебор не пойму, если файл 1? Вызывайте просто File.Delete("C:\\MyFile.txt"); и будет вам счастье.
    – EvgeniyZ
    18 июл 2018 в 14:04
  • 1
    @EvgeniyZ имхо это можно ответом постить, я бы плюсанул
    – tym32167
    18 июл 2018 в 14:19
  • @EvgeniyZ И я плюсану, пишите. Ответ полностью на все поставленные вопросы дан: экономия на спичках, разницы толком нет, удаляется разное, непонятно зачем цикл.
    – A K
    18 июл 2018 в 14:34

1 ответ 1

6

Давайте разберем ваши методы:

  • Первый метод - Здесь вы при вызове задаете ему некий путь к директории, потом получаете всю информацию о ней (и о каждом файле) и затем уже в цикле пробегаете по всем файлам в данной директории, удаляя каждый. По сути вам не нужно знать всю информацию о директории, а значит можно от DirectoryInfo отказаться и использовать просто пути до файлов, метод в итоге будет такой:

    public static void ClearDirectory(string path)
    {
        foreach (var file in Directory.GetFiles(path))
            File.Delete(file);
    }
    
  • Другие методы - Вы также при вызове задаете некий путь, получаете всю информацию о данной директории и после этого удаляете первый файл в данной директории. Все методы у вас одинаковы, разница только в том, что во втором методе вы вынесли отдельно в переменную di информацию о текущей директории; в третьем методе вы еще дополнительно вынесли информацию о файле в переменную _file; ну а в четвертом вы вовсе все написали сплошной "портянкой". Я вам больше даже скажу, все эти методы у вас имеют серьезную ошибку (попробуйте указать путь к пустой директории).


  • Производительность - В данном случае вы экономите на спичках, большого прироста в написание кода тем или иным способом вы не получите, так что пишите так, как вам нравиться, как подходит под стиль текущего кода и как правильно. Оптимизируйте другие участки своего кода, где к примеру идет работа с большим объемом данных, а не на выносе отдельных участков кода в отдельные переменные.
  • Читаемость кода - Тут тоже, у каждого свое представление, свой стиль, я к примеру люблю минимализм, если это хорошо читается и возможно разместить в одну строку, то почему бы и нет. Пример переписанного первого метода в одну строку, (как с читаемостью?):

    public static void ClearDirectory(string path) => new DirectoryInfo(path).GetFiles().ToList().ForEach(x=>x.Decrypt());
    
  • Правильное решение - Вы пишите:

    написать метод, который по указанному пути будет удалять из директории один файл.

    Так может вам проще сразу удалить файл по пути? Вызовите всего один метод File.Delete(file);, в который передаете путь до нужного файла, не нужно циклов, не нужно получать полную информацию. Если не известно имя файла, то получайте просто путь первого файла в директории (пример var file = Directory.GetFiles(path).FirstOrDefault();), этого будет вполне достаточно.

В общем, удачи и не экономьте на спичках!

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.