Pequeño error en script para editar archivos de texto

rogertm · #1 (**permalink**) 09/10/2008, 09:01

Hola... estaba preparando un script para editar archivos vie url, me funciona bien, pero tiene un pequeño fallo y es que si el archivo pedido no existe me manda un error y ya mi cabecita no me dio para mas

y tambien quiero ver como hacer para editar los archivos via URL, pues ahora solo los edita en el server local. osea, que funciona con rutas del tipo /carpeta/file.html pero no con http://localhost/carpeta/file.html

El error que da es:
Warning: file(eyj) [function.file]: failed to open stream: No such file or directory in D:\WAMP-SERVER\wamp\www\foo\file-edit.php on line 63

Warning: Invalid argument supplied for foreach() in D:\WAMP-SERVER\wamp\www\foo\file-edit.php on line 63

aqui el codigo:

Código php:

Ver originalCopiar<?php
if(!isset($_SESSION)){
    session_start();
}
 
// comprobamos si el formulario
// ha sido enviado correctamente
if(isset($_POST['abrir']) && $_POST['abrir'] == 'Abrir archivo'){
    if (!empty($_POST['archivo'])){
        $archivo = $_POST['archivo'];
        $_SESSION['archivo'] = $archivo;
        // comprobamos que existe el archivo
        if (!file_exists($archivo) && !is_file($archivo)){
            $msg = "El archivo no existe";
        } else {
            //$archivo_abierto = $archivo;
            $msg = "La ruta del archivo es <b>$archivo</b>";
        }
    } else {
        $msg = "Debe introducir la diraccion del archivo a editar";
    }
}
 
// editamos el archivo abierto
if(isset($_POST['editar']) && $_POST['editar'] == 'Editar archivo'){
    $archivo = $_SESSION['archivo'];
    $archivo_editado = $_POST['archivo_editado'];
    if(is_writable($archivo)){
        if(!$gestor = fopen($archivo, 'w')){
            $msg = "No se puede abrir el archivo $archivo";
            exit;
        }
        // editamos el archivo
        if(fwrite($gestor, html_entity_decode($archivo_editado)) === false){
            $msg = "No se puede escribir en el archivo $archivo";
        }
        // Exito...!
        $msg = "El archivo $archivo se edito con exito";
        fclose($gestor);
        //unset($_SESSION['archivo']);
    }else{
        $msg = "No se peude escribir en el archivo $archivo_editado";
    }
}
 
// guardamos el archivo
// y destruimos todas las variables
if(isset($_POST['guardar']) && $_POST['guardar'] == 'Guardar'){
    unset($archivo);
    unset($_SESSION['archivo']);
}
?>
 
<form action="<?php $_SERVER['PHP_SELF']; ?>" method="POST">
    <p>URL: <input type="text" name="archivo" value="" size="50" />
    <input type="submit" name="abrir" value="Abrir archivo" /></p>
<?php echo "<p>".$msg."</p>"; ?>
    <textarea name="archivo_editado" rows="25" cols="100">
<?php
// mostramos el contenido
// del archivo abierto
if($archivo){    
    foreach (file($archivo) as $lineas){
        echo htmlentities($lineas);
    }
}
?>
    </textarea>
    <p>
    <input type="submit" name="editar" value="Editar archivo" />
    <input type="submit" name="guardar" value="Guardar" />
    <input type="reset" value="Restablecer" />
    </p>
</form>

quisiera me ayudaran a descubrir por donde estan los fallos aqui, que le sobre y que le falta

gracias de antemano

GatorV · #2 (**permalink**) 09/10/2008, 10:12

Hola rogertm,

El problema es que en la linea 63, asumes que el archivo esta abierto y existe, en ninguna parte de tu código tienes un control de flujo para ver si existe o no.

Te recomiendo re-escribas tu algoritmo para que tengas un control de flujo valido y puedas hacer lo que pides.

Saludos.

pateketrueke · #3 (**permalink**) 09/10/2008, 10:16

que manera mas limpia de escribir PHP ... :P

pero, tu error... tal ves sea unsetear $archivo...

y no comprobar que existe antes de usarlo en el foreach

suerte!!

rogertm · #4 (**permalink**) 09/10/2008, 11:13

bueno, ya pase por eso (creo) ahora lo que hice fue meter el foreach dentro de la comprobacion, cuando defino $archivo:

Código php:

Ver originalCopiarif(isset($_POST['abrir']) && $_POST['abrir'] == 'Abrir archivo'){
    if (!empty($_POST['archivo'])){
        $archivo = $_POST['archivo'];
        $_SESSION['archivo'] = $archivo;
        // comprobamos que existe el archivo
        if (!file_exists($archivo) && !is_file($archivo)){
            $msg = "El archivo no existe";
        } else {
            //$archivo_abierto = $archivo;
            if($archivo){
                foreach (file($archivo) as $lineas){
                    $codigo = htmlentities($lineas);
                }
            }
            $msg = "La ruta del archivo es <b>$archivo</b>";
        }
    } else {
        $msg = "Debe introducir la diraccion del archivo a editar";
    }
}

y al menos no me da el error... pero como llamo a $codigo en el text area??? porque con echo no pincha

nomeacabodeaprenderlapalabrejaesahighlight

GatorV · #5 (**permalink**) 09/10/2008, 11:20

El problema es tu ciclo, por cada vuelta sobreescribes el valor de $codigo, tienes que agregar usando .= en tu foreach.

Saludos.

rogertm · #6 (**permalink**) 09/10/2008, 11:45

Resuelto...

ademas le meti otro foreach cuando por fin se puede escribir y editar el archivo, y redefino $codigo, porque si no no me volvia a mostrar el contenido del archivo una vez editado y asi lo puedes ir editando cuantas veces quieras y guardarlo una vez terminado todo, al final quedo todo asi:

Código php:

Ver originalCopiar<?php
if(!isset($_SESSION)){
    session_start();
}
 
// comprobamos si el formulario
// ha sido enviado correctamente
if(isset($_POST['abrir']) && $_POST['abrir'] == 'Abrir archivo'){
    if (!empty($_POST['archivo'])){
        $archivo = $_POST['archivo'];
        $_SESSION['archivo'] = $archivo;
        // comprobamos que existe el archivo
        if (!file_exists($archivo) && !is_file($archivo)){
            $msg = "El archivo no existe";
        } else {
            //$archivo_abierto = $archivo;
            if($archivo){
                foreach (file($archivo) as $lineas){
                    $codigo .= htmlentities($lineas);
                }
            }
            $msg = "La ruta del archivo es <b>$archivo</b>";
        }
    } else {
        $msg = "Debe introducir la diraccion del archivo a editar";
    }
}
 
// editamos el archivo abierto
if(isset($_POST['editar']) && $_POST['editar'] == 'Editar archivo'){
    $archivo = $_SESSION['archivo'];
    $archivo_editado = $_POST['archivo_editado'];
    if(is_writable($archivo)){
        if(!$gestor = fopen($archivo, 'w')){
            $msg = "No se puede abrir el archivo $archivo";
            exit;
        }
        // editamos el archivo
        if(fwrite($gestor, html_entity_decode($archivo_editado)) === false){
            $msg = "No se puede escribir en el archivo $archivo";
        }
        // Exito...!
        $msg = "El archivo $archivo se edito con exito";
        if($archivo){
            foreach (file($archivo) as $lineas){
                $codigo .= htmlentities($lineas);
            }
        }
        fclose($gestor);
    }else{
        $msg = "No se peude escribir en el archivo $archivo_editado";
    }
}
 
// guardamos el archivo
// y destruimos todas las variables
if(isset($_POST['guardar']) && $_POST['guardar'] == 'Guardar'){
    unset($archivo);
    unset($_SESSION['archivo']);
}
?>
 
<form action="<?php $_SERVER['PHP_SELF']; ?>" method="POST">
    <p>URL: <input type="text" name="archivo" value="" size="50" />
    <input type="submit" name="abrir" value="Abrir archivo" /></p>
<?php echo "<p>".$msg."</p>"; ?>
    <textarea name="archivo_editado" rows="25" cols="100">
<?php
// mostramos el codigo
// o el texto del archivo a editar
echo $codigo;
?>
    </textarea>
    <p>
    <input type="submit" name="editar" value="Editar archivo" />
    <input type="submit" name="guardar" value="Guardar" />
    <input type="reset" value="Restablecer" />
    </p>
</form>

gracias milllll

okram · #7 (**permalink**) 09/10/2008, 12:22

No entiendo porqué usas file() para partir tu archivo en líneas ycrear un ciclo paracada una de ellas si después vas a juntar todo nuevamente en la variable $codigo. Me refiero a esta parte:

Código php:

Ver originalCopiarif($archivo){
    foreach (file($archivo) as $lineas){
        $codigo .= htmlentities($lineas);
    }
}

Si puedes usar en vez file_get_contents():

Código php:

Ver originalCopiarif($archivo){
    $codigo = htmlentities(file_get_contents($archivo));
}

Igual en la línea 44.

Saludos,

rogertm · #8 (**permalink**) 09/10/2008, 12:37

Gracias Okram... tienes toda la razon...

Cita:

No te acostaras sin aprender algo nuevo...

rogertm

yoni-jaa · #9 (**permalink**) 10/11/2008, 20:19

Muy bien

tu script Funciona pero no guarda por favor areclalo para que huarde el archivo