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

    +102

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    private static string GetRootedCurrentConfigurationFile(string configurationFile)
{
if (string.IsNullOrEmpty(configurationFile))
{
throw new ArgumentException(Resources.ExceptionStringNullOrEmpty, "configurationFile");
}
if (!File.Exists(configurationFile))
{
throw new FileNotFoundException(string.Format(CultureInfo.CurrentCulture, Resources.ExceptionConfigurationLoadFileNotFound, new object[] { configurationFile }));
}
if (!Path.IsPathRooted(configurationFile))
{
return Path.Combine(AppDomain.CurrentDomain.BaseDirectory, configurationFile);
}
return configurationFile;
}

    Собснно Говнокод by Microsoft (Ent Lib 5.0.414.0)

    Запостил: TrueLauncher, 30 Июля 2010

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

    • ava 3.14159265 30.07.2010 15:28 # 0

      где здесь говнокод, %username%?
      Ответить

      • ava Nagg 30.07.2010 16:34 # 0

        согласен, абсолютно адекватный код. разве что неотмформатированный слегка.
        Ответить

    • ava Lure Of Chaos 30.07.2010 19:22 # 0

      но что-то настораживает. Может быть, можно короче переписать?
      Ответить

    • ava TrueLauncher 30.07.2010 20:36 # 0

      Заминусовали не разобравшись!!! А вот присмотреться, мы передаем в метод путь к файлу обрезанный (типа folder\file) на выходе метод должен вернуть AppDomain.CurrentDomain.BaseDirectory + наш обрезанный путь. А вот это что тогда:
      if (!File.Exists(configurationFile))
      {
      throw new FileNotFoundException(string.Format(Cult ureInfo.CurrentCulture, Resources.ExceptionConfigurationLoadFile NotFound, new object[] { configurationFile }));
      }
      Правильно Гавнакод!!!! Как у нас может найтись файл по куску пути?
      И в комент даешь "минус" %username%?
      Ответить

      • ava bober_maniac 30.07.2010 21:38 # 0

        Он будет искать файл в текущей директории, то есть в том самом AppDomain.CurrentDomain.BaseDirectory
        Ответить

        • ava Nagg 30.07.2010 21:44 # 0

          не факт. текущая директория и директория сборки могут быть разными
          Ответить

          • ava bober_maniac 30.07.2010 21:52 # 0

            Рефлектором посмотрите File.Exists
            Ответить

            • ava Nagg 30.07.2010 22:25 # 0

              Посмотрел. AppDomain.CurrentDomain.BaseDirectory там нигде не увидел.
              Ответить

            • ava Nagg 30.07.2010 22:30 # 0

              Вот вам пища для размышления:

              string file = "some file near exe.txt";
bool exists = File.Exists(file); //true
Directory.SetCurrentDirectory(@"C:\");
exists = File.Exists(file); //false
exists = File.Exists(Path.Combine(AppDomain.CurrentDomain.BaseDirectory, file)); //true
              Ответить

        • ava TrueLauncher 30.07.2010 22:57 # 0

          не буду с Вами спорить. Проверено на практике - не работает!!! Sources were modified.

          Вот вам доказательства и ответ их сапорта:
          http://entlib.codeplex.com/Thread/View.aspx?ThreadId=221852
          Ответить

    • ava wvxvw 31.07.2010 00:51 # +1

      Вообще, в любом случае надо было проверять IsPathRooted сначала, а то так вообще не понятно, что File.Exists проверяет. Похоже на fizzbuzz. Т.е. не могли определиться, проверить ли сначала, что файл существует, или что файл существует тм, где надо.
      Ответить

    • ava Nigma143 01.08.2010 13:34 # 0

      http://entlib.codeplex.com/Thread/View.aspx?ThreadId=204281
      Ответить

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