Как правильно создавать папки?

Question

[Дмитрий Онацкий]

Ребят, какой пример кода лучше всего?

function picture_dir_create( $dir , $id ) {

        $f_level = ceil( $id / 1000000 );
        $s_level = ceil( ( $id - (($f_level-1)*1000000) ) / 1000 );

        $levelF_dir = $dir . $f_level . '/';
        $levelS_dir = $levelF_dir . $s_level . '/';
        $levelE_dir = $levelS_dir . $id . '/';

        // От так?
        
        if( ! file_exists($levelF_dir) ) mkdir($levelF_dir, 0777);
        if( ! file_exists($levelS_dir) ) mkdir($levelS_dir, 0777);
        if( ! file_exists($levelE_dir) ) mkdir($levelE_dir, 0777);
        
        // Или можно и так? 

        @mkdir($levelF_dir, 0777);
        @mkdir($levelS_dir, 0777);
        @mkdir($levelE_dir, 0777);

    }

Заранее спасибо!

Comments

[Дмитрий Онацкий]

Хотя правильнее будет тот вариант который выше. Но обязательно ли проверку делать? Критично ли для функции mkdir наличие существующего каталога.

Answer 1

[Muhammad]

Первое. Второй вариант - говнокод

UPD: php.net/manual/ru/function.mkdir.php

Можно передать третий аргумент true и сделать только одну проверку.

Comments

[Дмитрий Онацкий]

Спасибо за отзыв.

[Mikhail Osher]

говнокод - допускать неотлавливаемую ошибку в операциях с io/tcp итд (fsockopen, mkdir, ....)

[Евгений Обыкновенный]

Вопрос в том, а не будет ли лишней работы? Ведь внутри mkdir происходит проверка существования. Кстати, нечто подобное делается в Си при использовании fopen() с флагом w. Если существует откроет, если нет - создаст.

[Mikhail Osher]

Евгений Обыкновенный: да, и бросается E_WARNING. Я детально объяснил в своем ответе ниже.

[Евгений Обыкновенный]

Mikhail Osher: знаю, что возникает ошибка. А чем она вредна? Это замедляет работу или что? Не из вредности спрашиваю, мне действительно интересно.

[Muhammad]

Евгений Обыкновенный: а что если нет прав на создание папки? Ее как не было, так и не будет. И ты об этом не узнаешь

[Евгений Обыкновенный]

Muhammad: какое значение возвращает mkdir, если папка существует?

Answer 2

[xmoonlight]

Тогда уж так правильнее:

if( ! (file_exists($levelF_dir) || file_exists($levelS_dir) || file_exists($levelE_dir))) {
   mkdir($levelF_dir, 0777);
   mkdir($levelS_dir, 0777);
   mkdir($levelE_dir, 0777);
} else {
  /* АХТУНГ! ОДНА ИЗ ПАПОК УЖЕ БЫЛА СОЗДАНА! */
}

Comments

[Дмитрий Онацкий]

Это не три отдельные папки. Это ветка. Это не то. Да и это. К примеру у нас существует одна из папок. И что выходит тогда? Условие дает добро и пропускает. Дальше создается существующая папка и все, привет ошибка.

[xmoonlight]

Дмитрий Онацкий: почему пропускает то?! там же !()

Answer 3

[Mikhail Osher]

Emits an E_WARNING level error if the directory already exists.
Emits an E_WARNING level error if the relevant permissions prevent creating the directory.

Обновил ответ. Спасибо месье DevMan

/**
 * Create a directory
 *
 * @param string $path Directory path
 * @param int $mode Permissions
 * @return bool TRUE if directory created or exists, FALSE on failure
 */
function makedir($path, $mode)
{
  if (is_dir($path)) {
    return true;
  }

  if (is_writable($path)) {
    return mkdir($path, $mode, true);
  }

  return false;
}

Comments

[DevMan]

не учите плохому.
за собаку, там где без нее можно обойтись, нужно бить по рукам. больно. и долго.

[Mikhail Osher]

DevMan: воу, ребята, доки читаем.) тут без нее нельзя обойтись. Для первого E_WARNING делается проверка на is_dir, а второй E_WARNING, касательно невозможности создания ввиду недостаточных привелегий пользователя, под которым запущен PHP - нельзя отловить человеческим способом.

[DevMan]

Mikhail Osher: > а второй E_WARNING, касательно невозможности создания ввиду недостаточных привелегий пользователя, под которым запущен PHP - нельзя отловить человеческим способом
ога. php.net/manual/en/function.is-writable.php

[Mikhail Osher]

DevMan: Вы абсолютно правы. Я и забыл про эту функцию.

[DevMan]

Mikhail Osher: по правкам: is_writeable нужно вызывать не с создаваемым путем, а с родительским.

[Mikhail Osher]

DevMan: я подразумеваю абсолютный путь

[DevMan]

Mikhail Osher: не в этом дело. если $path - директория, которую мы хотим создать, то должно быть не is_writeable($path), a is_writeable(dirname($path)).