Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

PHP (5703)
C++ (2739)
Куча (2337)
JavaScript (2030)
C# (1929)
Java (1503)
Си (1102)
Pascal (649)
Python (585)
1C (536)
SQL (434)
ActionScript (294)
Objective C (204)
bash (200)
Perl (196)
Assembler (148)
Ruby (144)
VisualBasic (140)
Lua (51)
Go (31)
Swift (29)
Haskell (28)
Kotlin (15)

PHP / Говнокод #11797

+38
1. 1
```
if (!file_exists($this->filePath)) fopen($this->filePath, 'r');
```
норм чё

Запостил: dizballanze, 19 Сентября 2012

Tweet
Комментарии (10) RSS
- GoodTalkBot 19.09.2012 04:51 # +2
  
  если нет файла то создать. или нет?
  Ответить
  - roman-kashitsyn 19.09.2012 09:10 # +2
    
    Handle потеряли один, некрасиво. Есть touch.
    Ответить
  - Vindicar 19.09.2012 09:28 # +4
    
    'r' не создаёт вроде. Так что это скорее "если нет файла то вылететь с ошибкой". Ну и да, handle потеряли.
    Ответить
  - Lowezar 19.09.2012 10:21 # +2
    
    Чтоб таким макаром его создать - надо было любую опцию кроме r/r+ тогда. А так получится просто E_WARNING, что нет такого файла... Единственный смысл, который я в этом вижу - это собственный error_handler который перебросит ошибку в Exception, который можно где-то выше обработать.
    Но странно почему его ручками тогда не кинуть, и что-то сомневаюсь что здесь такой случай. :)
    Ответить
    - Steve_Brown 19.09.2012 10:48 # +2
      
      А если есть такая папка, эффект тот же самый будет или исключение? (тогда мог бы быть вариант: проверить, что файл можно создать)
      Ответить
      
      Lowezar 19.09.2012 10:57 # +2
      
      Зависит от системы. В винде получишь warning "Permission Denied", в *nix если в режимах только r - таки откроет. Правда, никогда не пытался читать что там. :)
      Ответить
      
      Vindicar 19.09.2012 14:36 # +4
      
      В никсах вроде разрешение r на папке позволяет сделать листинг (открытие известного пути внутри каталога задаёт право x).
      Ответить
- Lure Of Chaos 19.09.2012 15:13 # +2
  
  вывод - бессмысленное гуано
  Ответить
- cyperh 20.09.2012 18:26 # −2
  
  Если файл не существует, пытаемся его открыть с флагом readonly => бредятина, может описка, а может это из новых технологий :P
  p.s. http://php.net/manual/en/function.fopen.php
  Ответить
- qlqRjbp 25.08.2021 09:40 # 0
  
  vanished
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только во вторник, пятницу или субботу
Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

PHP / Говнокод #11797

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

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