План непредвиденных обстоятельств для ошибки fopen в php

Question

Я пишу PHP-приложение с «панелью управления», которое записывает файл prefs с определенными переменными. В каждом POST, если файл не существует, он создается. Если он существует, это unlinked, а новый файл - touched с тем же именем и новыми переменными. Этот файл затем включается на другую страницу с отображением содержимого на основе переменных внутри него.

$file = "phpsettings.php"; 

if (!file_exists($file)) { 
    touch($file); 
    $handle = fopen ($file, 'r+'); 
$str = "<?php \$pref1 = \"$mypref\"; ?>"; 

} else { 

unlink($file); 
    touch($file); 
    $handle = fopen ($file, 'r+'); 
    $str = "<?php \$pref1 = \"$mypref\"; ?>"; 

} 

fwrite ($handle, $str); 
fclose ($handle); 

Безопасный ли способ записи предпочтений при условии, что этот файл будет перезаписан много раз в день? Что является хорошим способом как предупредить пользователя этой панели управления, если файл не был сохранен правильно, и в этом случае, что было бы хорошим планом на случай непредвиденных обстоятельств, чтобы не сломать страницу, этот файл prefs включен не до определения набор значений по умолчанию для заполнения, если !(file_exists)?

Answer 1

Если вы сохраняете свои настройки в массиве, вы можете сериализовать() их и записать в текстовый файл, вместо того, чтобы писать исходный php в файл php и включать его.

Если вы не санировать ввод для тех предпочтений, и, скажем, $ mypref1 представляет чье-то имя, то ничто не мешает их заполнения этого в поле формы:

\"; echo \"PWNED 

и ваш полученный PHP будет

<?php \$pref1 = \"$mypref\"; echo \"PWNED\"; ?> 

Итак, во-первых, хранить ваши предпочтения в массиве и используя Serialize() гораздо безопаснее:

$prefs = array('mypref1' => 'somethingorother'); 
$handle = fopen ($file, 'w'); 
fwrite($handle, serialize($prefs)); 
fclose($h); 

// example code demonstrating unserialization 
$prefs2 = unserialize(file_get_contents($file)); 
var_dump($prefs == $prefs2); // should output "(bool) true" 

В вашем вопросе вы также отмечаете, что если файл существует, он отсоединен. Вы можете просто обрезать его до нулевой длины, передав «w» в качестве второго аргумента fopen - вам не нужно вручную его удалять. Это должно в любом случае установить mtime, отрицая необходимость вызова touch().

Если значения, записываемые в файл, являются предпочтениями, наверняка у каждого предпочтения может быть значение по умолчанию, если не существует сотни? array_merge позволит перезаписать на основе каждого ключа, так что если вы делаете что-то вроде этого:

// array of defaults 
$prefs = array(
    'mypref1' => 'pants', 
    'mypref2' => 'socks', 
); 
if (file_exists($file)) { 
    // if this fails, an E_NOTICE is raised. are you checking your server error 
    // logs regularly? 
    if ($userprefs = unserialize(file_get_contents($file))) { 
     $prefs = array_merge($prefs, $userprefs); 
    } 
} 

Если проблема в том, что есть кучи, и вы не хотите, чтобы инициализировать их всех, вы можете получить метод get_preference, который просто переносит вызов isset в массив prefs.

function get_preference($name, &$prefs) { 
    if (isset($pref[$name])) 
     return $pref[$name]; 
    return null; 
} 
var_dump(get_preference('mypref1', $prefs)); 

За все вопросы этой связи возникает, хотя, реальность такова, что с вашим приложением, в маловероятном случае, что-то делает ошибетесь с FOPEN, следует рассматривать как серьезную неудачу в любом случае, и горстка пользователей, с которыми вы, вероятно, будете пользоваться этой функцией, скоро свяжется с вами довольно быстро, если что-то пойдет не так.

Answer 2

Почему бы не просто использовать функции усечения fopen()? Я верю вместо «r +», вам нужно передать «w +» ... Тогда, если файл существует, он будет усечен, если нет, вы просто создадите новый файл. Таким образом, код становится следующим:

$file = "phpsettings.php"; 
$handle = fopen($file, 'w+'); 
$str = "<?php \$pref1 = \"$mypref\"; ?>"; 
fwrite ($handle, $str); 
fclose ($handle); 

Answer 3

Всегда лучше сохранять состояние пользователя в сеансе и сохранять его только при необходимости.