Foros del web

rogertm · 09-oct-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 · 09-oct-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 · 09-oct-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 · 09-oct-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 · 09-oct-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 · 09-oct-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 · 09-oct-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 · 09-oct-2008, 12:37

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

Cita:

No te acostaras sin aprender algo nuevo...

rogertm

yoni-jaa · 10-nov-2008, 20:19

Muy bien

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

09-oct-2008, 10:12	#2 (permalink)
GatorV $this->role('moderador'); 26.480 Mensajes Ubicación: /home/ags/	Respuesta: Pequeño error en script para editar archivos de texto 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. __________________ Blog Web

09-oct-2008, 10:16	#3 (permalink)
pateketrueke Colaborador 7.880 Mensajes Ubicación: Mexihco-Tenochtitlan	Respuesta: Pequeño error en script para editar archivos de texto 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!! __________________ ¿Buscas un framework Amigable para PHP? → Como contactar conmigo

09-oct-2008, 11:20	#5 (permalink)
GatorV $this->role('moderador'); 26.480 Mensajes Ubicación: /home/ags/	Respuesta: Pequeño error en script para editar archivos de texto El problema es tu ciclo, por cada vuelta sobreescribes el valor de $codigo, tienes que agregar usando .= en tu foreach. Saludos. __________________ Blog Web

09-oct-2008, 12:22	#7 (permalink)
okram Colaborador 3.398 Mensajes Ubicación: *	Respuesta: Pequeño error en script para editar archivos de texto 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 original Copiar if($archivo){ foreach (file($archivo) as $lineas){ $codigo .= htmlentities($lineas); } } Si puedes usar en vez file_get_contents(): Código php: Ver original Copiar if($archivo){ $codigo = htmlentities(file_get_contents($archivo)); } Igual en la línea 44. Saludos, __________________ - Asesoría en programación PHP [Paginación PHP]

09-oct-2008, 12:37	#8 (permalink)
rogertm Colaborador 1.528 Mensajes Ubicación: 23.133355, -82.353773	Respuesta: Pequeño error en script para editar archivos de texto Gracias Okram... tienes toda la razon... Cita: No te acostaras sin aprender algo nuevo... rogertm __________________ mi twitter @roger213tm Kilómetro Cero Bloggers Cuba

10-nov-2008, 20:19	#9 (permalink)
yoni-jaa 8 Mensajes	Respuesta: Pequeño error en script para editar archivos de texto Muy bien tu script Funciona pero no guarda por favor areclalo para que huarde el archivo

Herramientas
Mostrar Versión Imprimible Enviar por Correo
Desplegado
Modo lineal Cambiar a Modo Hibrido Cambiar a Modo Hilado