1. Pascal / Говнокод #4321

    +90

    1. 1
    2. 2
    if DirectoryExists(ExtractFilePath(Application.ExeName)+'Folder')=false then
CreateDir(ExtractFilePath(Application.ExeName)+'Folder');

    Запостил: Мартин, 09 Октября 2010

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

    • ava maniac 09.10.2010 17:23 # +1

      Ну и?..
      Ответить

    • ava Lure Of Chaos 09.10.2010 19:46 # −3

      ага, создатьпапку типа c:\Program Files\AppFolder (вместо c:\Program Files\App\Folder)
      Ответить

      • ava Stalker 09.10.2010 21:02 # +3

        ExtractFilePath() возвращает путь с разделителем. Говно не в этом, а в дублировании кода.
        Ответить

    • ava ZLOvar 10.10.2010 09:01 # +3

      Говно здесь в If условие = false, вместо If not условие.
      Ответить

    • ava Arigato 11.10.2010 19:34 # +3

      Много говна:
      1. = false вместо not
      2. Использование 2 раза ExtractFilePath(Application.ExeName)+'Fo lder' вместо того, что бы завести под это дело переменную.
      3. Да и вообще, CreateDir просто вернет false, если директория уже существует, так что проверка тут вообще не нужна.
      Ответить

      • ava ctm 12.10.2010 09:37 # 0

        + нет проверки на успешность выполнения createdir - может прав создания папки нету...
        Ответить

        • ava Arigato 12.10.2010 21:56 # 0

          Тогда все равно надо будет делать проверку DirectoryExists, т.к. может папка не создалась из-за того, что уже существует, а может из-за отсутствия прав или возможностей.
          Ответить

        • ava bugmenot 17.10.2010 22:27 # 0

          + на современных виндах потенциально Writeln('Hello, UAC!'); { за писание в %ProgramFiles% гнать из профессии ссаными тряпками }
          и вообще плясать от абсолютного пути вместо workingDir - неправильно
          Ответить

          • ava ctm 22.10.2010 11:11 # 0

            не факт, что ExtractFilePath(Application.ExeName) = workingDir
            может тут нужна именно папка с ехешником - откуда нам знать?
            Ответить

    • ava rety 16.11.2010 17:33 # 0

      ну не знает человек forcedirectory ну и ладно...
      Ответить

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