- 1
- 2
- 3
- 4
- 5
if (!($conf_content = file_get_contents($data_path."voc.conf"))) {
die("Cannot open voc.conf file, please check your DATA PATH ($data_path) parameter");
}
eval($conf_content);
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+24
if (!($conf_content = file_get_contents($data_path."voc.conf"))) {
die("Cannot open voc.conf file, please check your DATA PATH ($data_path) parameter");
}
eval($conf_content);
при условии что voc.conf является пхпфайлом, глупо делать такую мутотень когда можно просто обойдись file_exists и include
неправда. file_get_contents использован правильно - другое дело, что его надо сравнивать с === false, иначе условие будет выполнятся также если файл пустой.
А если файл есть, а прав на него нет?
прав нет | файла нет | кармы нет | мозгов нет == cannot open == (file_get_contents === false)