cláusula de error para file_put_contents

Question

Estoy utilizando esta línea para obtener y guardar una imagen de una URL.

file_put_contents("./images/".$pk.".jpg", file_get_contents($PIC_URL)) 

No estoy seguro de cuál es la mejor manera de manejar un error. En este momento está fallando porque no hay permiso, que se remediará en breve, pero me gustaría que sea capaz de manejar situaciones en las que PIC_URL está vacío o no es una imagen. ¿Debería haber muerto con el error en este nivel (probablemente es mejor para las cosas relacionadas con permisos) o debería consultar más arriba si PIC_URL está vacío, o ambos?

¿Cuál es el mejor enfoque?

Answer 1

No soy lo suficientemente talentoso como para reclamar este es el mejor método, pero me acaba de probar en el camino:

$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

me gustaría utilizar is_writable(), pasándole el nombre de la carpeta si la imagen doesn 't ya existe, o el nombre del archivo si lo hace, antes de intentar escribir la imagen.

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

Answer 3

Ambos, por lo que yo soy preocupación. Especialmente con esas funciones peligrosas de manejo de archivos, la doble verificación no duele. (¿De dónde viene $pk?)

En general, consulte más arriba para obtener una mejor respuesta al usuario y verifique antes de ejecutar por seguridad. Es difícil dar comentarios decentes al usuario cuando solo se comprueba a un nivel bajo. Por otro lado, es difícil verificar todos los errores posibles (como los permisos del sistema de archivos) en un nivel alto y genérico.

Answer 4

intente hacer una función para esto.

<?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 ¿no podría también acaba de hacer

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