clause d'erreur pour file_put_contents

Question

J'utilise cette ligne pour obtenir et enregistrer une image à partir d'une URL. Je ne sais pas quelle est la meilleure façon de gérer une erreur. En ce moment, il échoue parce qu'il n'y a pas d'autorisation, ce qui sera corrigé sous peu, mais je voudrais qu'il soit capable de gérer des situations où PIC_URL est vide ou pas une image. Dois-je mourir avec l'erreur à ce niveau (probablement c'est mieux pour les choses liées à la permission) ou devrais-je vérifier plus haut si PIC_URL est vide, ou les deux?

Quelle est la meilleure approche?

Answer 1

Je ne suis pas assez talentueux pour demander c'est la meilleure méthode, mais je voudrais juste tester le long du chemin:

$imageDir = "/path/to/images/dir/"; 
$imagePath = "$imageDir$pk.jpg"; 
if (!is_dir($imageDir) or !is_writable($imageDir)) { 
    // Error if directory doesn't exist or isn't writable. 
} elseif (is_file($imagePath) and !is_writable($imagePath)) { 
    // Error if the file exists and isn't writable. 
} 

$image = file_get_contents(urlencode($PIC_URL)); 
if (empty($image)) { 
    // Error if the image is empty/not accessible. 
    exit; 
} 

file_put_contents($imagePath, $image); 

Answer 2

J'utiliseraient is_writable(), en lui passant le nom du dossier si l'image doesn n'existe pas, ou le nom de fichier si c'est le cas, avant d'essayer d'écrire l'image.

http://uk3.php.net/manual/en/function.is-writable.php

Answer 3

deux, en ce qui me concerne. Surtout avec ces fonctions de gestion de fichiers dangereuses, la double vérification ne fait pas de mal. (D'où vient $pk?)

Généralement, vérifiez plus haut pour un retour plus agréable à l'utilisateur et vérifiez avant d'exécuter pour la sécurité. Il est difficile de donner des commentaires décents à l'utilisateur lors de la vérification à un niveau faible. D'un autre côté, il est difficile de vérifier toutes les erreurs possibles (comme les permissions du système de fichiers) à un niveau élevé et générique.

Answer 4

essayez de créer une fonction pour cela.

<?php 
define('OK', 0); 
deinfe('URL_EMPTY', 1); 
define('WRITING_PROBLEMS',2); 
define('OTHER_PROBLEM', 3); 


function save_pic($pic_url) { 

    $imageDir = '/path/to/images/dir/'; 

    if (!strlen($pic_url)) 
    return URL_EMPTY; 

    if (!is_dir($imageDir) || !is_writable($imageDir)) { 
    return WRITING_PROBLEMS; 
    } 

    $image = file_get_contents(urlencode($pic_url)); 

    $pk = time(); // or whatever you want as key 


    $r = file_put_contents($imagePath.$pk.".jpg", $pic_url); 

    if ($r) 
    return OK; 
    else 
    return OTHER_PROBLEM; 

} 
?> 

Answer 5

Hmm vous ne pourriez pas le faire aussi

file_put_contents($file, $str) or die("Unable to write file!");