Конкретно по коду отписал ниже. Как хелпер - ну может быть, почитайте на досуге
php.net/manual/ru/spl.files.php// Я надеюсь вы знаете про PSR4 и символ подчеркивания тут случайно
class MV_Files
{
// PSR2: методы пишутся в lowerCamelCase
// Нет проверки аргументов. что произойдет, если $name будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
public function CreateFile($path, $name, $right = 0777)
{
if (!file_exists($path . '/' . $name)) {
// Посмотрите http://php.net/manual/ru/function.touch.php
fclose(fopen($path . '/' . $name, 'w'));
chmod($path . '/' . $name, $right);
// PSR2: Перед return перевод строки
return true;
// Не ясно, зачем вам тут else?
} else {
return false;
}
}
// PSR2: методы пишутся в lowerCamelCase
// Нет проверки аргументов. что произойдет, если $name будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
public function CreateFileContent($path, $name, $text, $right = 0777)
{
if (!file_exists($path . '/' . $name)) {
// file_put_contents, не?
$file = fopen($path . '/' . $name, 'w');
fwrite($file, $text);
fclose($file);
chmod($path . '/' . $name, $right);
// PSR2: Перед return перевод строки
return true;
// Не ясно, зачем вам тут else?
} else {
return false;
}
}
// PSR2: методы пишутся в lowerCamelCase
// Нет проверки аргументов. что произойдет, если $path будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
public function AddFileContent($path, $text)
{
if (file_exists($path)) {
// file_put_contents($path, $content, FILE_APPEND)
$file = fopen($path, 'a');
fwrite($file, $text);
fclose($file);
// PSR2: Перед return перевод строки
return true;
// Не ясно, зачем вам тут else?
} else {
return false;
}
}
// PSR2: методы пишутся в lowerCamelCase
// Нет проверки аргументов. что произойдет, если $path будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
public function OverwriteFileContent($path, $text)
{
if (file_exists($path)) {
// file_put_contents($path, $content)
$file = fopen($path, 'w');
fwrite($file, $text);
fclose($file);
// PSR2: Перед return перевод строки
return true;
// Не ясно, зачем вам тут else?
} else {
return false;
}
}
// PSR2: методы пишутся в lowerCamelCase
// Нет проверки аргументов. что произойдет, если $oldname будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
// Чего вдруг метод статический?
public static function RenameFile($oldname, $newname) {
if (file_exists($oldname)) {
return rename($oldname, $newname);
} else {
return false;
}
}
// PSR2: методы пишутся в lowerCamelCase
// Нет проверки аргументов. что произойдет, если $path будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
public function DeleteFile($path)
{
if (file_exists($path)) {
return unlink($path);
} else {
return false;
}
}
// Нет проверки аргументов. что произойдет, если $path будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
public function getFileContent($path)
{
if (file_exists($path)) {
return file_get_contents($path);
} else {
return false;
}
}
// Нет проверки аргументов. что произойдет, если $path будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
// "size" обычно трактуется как число, а не форматированная строка.
public function getFileSize($path)
{
if (file_exists($path)) {
// Конструкция - гавно)). Если хотите интовый $size - то сохраняйте в него инт
(int)$size = filesize($path);
switch (true) {
case $size >= 1073741824:
$size = round($size / 1073741824 * 100) / 100 . ' GB';
break;
case $size >= 1048576:
$size = round($size / 1048576 * 100) / 100 . ' MB';
break;
case $size >= 1024:
$size = round($size / 1024 * 100) / 100 . ' KB';
break;
default:
$size .= ' Bytes';
break;
}
return $size;
} else {
return false;
}
}
// Нет проверки аргументов. что произойдет, если $path будет массивом?
// Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
// Прочитайте http://php.net/manual/ru/class.splfileinfo.php
public function getFileInfo($path)
{
if (file_exists($path)) {
return pathinfo($path);
} else {
return false;
}
}
// Нет проверки аргументов. что произойдет, если $dirname будет массивом?
// Прочитайте http://php.net/manual/ru/class.directoryiterator.php
public function getFileList($dirname)
{
$dir = array_diff(scandir($dirname), ['.', '..']);
$files = [];
// Рекомендую перед управляющими конструкциями делать перевод строки, так читать проще
foreach ($dir as $value) {
if (is_file($value)) {
$files[] = $value;
}
}
// PSR2: перед return перевод строки
return $files;
}
// Лишний перевод строки
}
