[php] problemi nell'uso di mkdir()

[Dobrobuz]

Ciao a tutti ho pure io lo stesso problema con il comando mkdir

I valori delle variabili che utilizzo sono i seguenti:

$path = "./img";
$pathThumbs = "Thumbs"; 

il comando seguente viene utilizzato in un blocco IF

if(is_dir($path . "/" . $this->pathThumbs) ||
  (!is_dir($path . "/" . $this->pathThumbs) && mkdir($path . "/" . $this->pathThumbs, 0755))
.... 

Qualcuno ha qualche idea del perchè non funziona? Se creo manualmente la cartella va tutto bene, ma ho la necessitò che sia automatizzato.

Grazie

[foreach]

if(!is_dir($path . "/" . $this->pathThumbs)) {
mkdir($path . "/" . $this->pathThumbs, 0755);
} 

[Dobrobuz]

Ciao l'idea era quella di utilizzare uno stesso blocco IF
per comunicare in caso di errore un unico messaggio.

if(is_dir($path . "/" . $this->pathThumbs) ||
(!is_dir($path . "/" . $this->pathThumbs) && mkdir($path . "/" . $this->pathThumbs, 0755)) 

Credo che il malfunzionamento sia dovuto all'inserimento del comando mkdir all'interno di una condizione if?

Eventualmente potrei modificare l'intero script e gestire in modo diverso la condizione.
Consigli?

[dreadnaut]

Manca una parentesi in quella riga... e se avessi postato il messaggio di errore sarebbe stato decisamente più facile capire cosa non andava

Il blocco è però eccessivamente complicato a mio parere, tanto che si fatica a capire cosa tu voglia fare. Direi che attendi una condizione positiva, cioé se la directory esiste, o se sei riuscito a crearla, allora lo script va avanti; altrimenti, messaggio di errore. Quindi:

$dir = $path . "/" . $this->pathThumbs;

if ( is_dir($dir) || mkdir($dir, 0755) ) {
  // la directory c'è, lavoriamoci
} else {
  // non c'è, o non siamo riusciti a crearla: messaggio di errore
} 

Tutto qua. Non serve ripetere is_dir() negata, perché se la prima parte di un or ( || ) è vera, la seconda non viene neanche valutata.

nessun problema in generale ad usare funzione nelle condizioni dell'if, anzi, è un'ottima pratica per rendere il codice più leggibile (senza esagerare però )

[Dobrobuz]

Grazie mille,
direi che come dici tu è molto più leggibile e soprattutto funziona!!

Alla prossima e buona settimana