1. C# / Говнокод #14278

    +129

    1. 1
    2. 2
    3. 3
    4. 4
    string fileName = Path.GetFileName(f);
string fileExt = Path.GetExtension(f);
string i=fileName.IndexOf(fileExt);
if (i > -1) fileName = fileName.Remove(i, fileExt.Length);

    Получение имени файла

    Запостил: sbs, 25 Декабря 2013

    Комментарии (12) RSS

    • ava sbs 25.12.2013 19:39 # −2

      Извиняюсь, я имел ввиду в 3 строке
      int i = fileName.IndexOf(fileExt);
      Ответить

    • ava kegdan 25.12.2013 20:16 # +1

      Я так подозреваю что человек хотел

      var fileName = Path.GetFileNameWithoutExtension(f);

      У меня только один вопрос - как, блеать, можно было пропустить этот метод, если после набора GetF... он в интелисенсе на втором месте стоит?
      Ответить

      • ava 1024-- 25.12.2013 20:19 # +2

        Название метода слишком длинное, чтобы его читать.
        Ответить

        • ava bormand 25.12.2013 20:48 # +1

          Да, если метод назван слишком длинно, следует написать его заново.
          Ответить

          • ava inkanus-gray 25.12.2013 20:51 # +2

            Это что же получается, теперь SDK известных компаний надо переписывать?
            Ответить

          • ava kegdan 25.12.2013 21:11 # +1

            GFNWE лучше?)
            Ответить

            • ava inkanus-gray 25.12.2013 21:28 # +1

              1. Get/Set в названии метода не нужно, потому что можно использовать свойства с геттером/сеттером.
              2. Совмещение двух действий в одном методе не нужно.

              С учётом вышеизложенного получаем: 
              var fileName = new Path(f).fileName.removeExtension();
              Ответить

              • ava bormand 25.12.2013 21:38 # +1

                Хм, removeExtension() - extension method? Или fileName не string?
                Ответить

              • ava defecate-plusplus 25.12.2013 22:06 # +5

                тряхну стариной
                namespace fs = boost::filesystem;
auto filename = fs::path(f).stem();
                Ответить

              • ava kegdan 25.12.2013 22:10 # +1

                Тогда уж Path.removeExtension(Path.fileName(f)) дабы все в одном модуле было
                Чувствуете этот запах?
                Ответить

      • ava sbs 25.12.2013 21:28 # 0

        Спасибо, действительно невнимательность сильно мешает
        Ответить

    Добавить комментарий