no encuenro mi error en procesando mis archivos php

pithon · #1 (**permalink**) 16/05/2011, 06:10

Estoy intentando procesar mis archivos para si hay un error aparte de mostrar error me envie otra vez al formulario pero no me funciona alguien me dice el porque:

codigo:

Código PHP:

Ver original<html>
    <head>
      <script language='javascript' type='text/javascript'>
       function volver(){
        location.href='formulario.html';
       }
       </script>
    </head>
    <body>
 
<?php
$error=false;
if($_FILES['archivo']['type']!='image/pjpeg'){
    echo 'Este archivo no se puede subir solo se acepta archivos tipo jpeg';
    $error=true;
}
if($_FILES['archivo']['size'] >50000){
    echo 'Este archivo no se puede subir es demasiado grande solo se aceptan hasta 50000 bytes';
    $error=true;
}
if($_FILES['archivo']['error']!=0){
    echo 'Hay un error en el archivo';
}
if ($error){
    echo '<input type="button" name="retorno" id="retorno" value="volver" onClick="javascript:volver();";';
}else{
      $numFiles = count($_FILES['archivo']['tmp_name']);
    for($i=0; $i<$numFiles; $i++){
       if($_FILES['archivo']['name'][$i]){
if (move_uploaded_file($_FILES['archivo']['tmp_name'][$i],'misarchivos/'.$_FILES["archivo"]["name"][$i])) {
  }else{
    echo 'Error, no se pudo mover la imagen ';
    }
       }
    }
?>

alexg88 · #2 (**permalink**) 16/05/2011, 06:21

No entiendo porque si hay error pones un botón que al hacer click te lleve al formulario.

Si lo que quieres es volver al formulario, utiliza la función header de php:

Código PHP:

Ver originalheader('Location: formulario.html');

IEKK · #3 (**permalink**) 16/05/2011, 06:24

Cita:

Iniciado por alexg88

No entiendo porque si hay error pones un botón que al hacer click te lleve al formulario.

Si lo que quieres es volver al formulario, utiliza la función header de php:

Código PHP:

Ver originalheader('Location: formulario.html');

¿Y si quiere usar un botón qué?

No he mirado mucho de tu código pero o lo has copiado mal aquí o eso no te puede funcionar porte te falta cerrar una llave }
Concretamente la del else que está antes del for.
Supongo que ahí estaría tu fallo.

pithon · #4 (**permalink**) 16/05/2011, 06:29

bueno eso lo puedo cambiar pero esque igual no me funciona

IEKK · #5 (**permalink**) 16/05/2011, 06:37

Hombre, si tienes las llaves mal ya te digo yo que no te iba a funcionar de ninguna de las formas ...

¿Donde se para tu código? ¿Te entra en los ifs? ¿Que valor tiene $error?
¿Cuando no hay errores en tu imagen hace la otra parte de forma correcta?
También puedes hacer un echo y decirnos donde se para tu código.

Nosotros no sabemos que estás subiendo ni como tienes el formulario ni los valores que te devuelven.

pithon · #6 (**permalink**) 16/05/2011, 06:39

gracias por contestar IEKK pero ha sido un fallo de escritura en el codigo si que tengo esta llave cerrada el problema que me da es que cuando envio los archivos me salen en pantallas los echo de error cuando realmente no hay ningun error

IEKK · #7 (**permalink**) 16/05/2011, 06:45

Amm ok, bueno dinos que errores te dan. Así es más rápido, sabemos el tipo y la línea.
Postea el warning que te dé

pithon · #8 (**permalink**) 16/05/2011, 06:47

no me da ningun error solo que el codigo no me funciona como tendria que hacerlo asea cuando mando los archivos me sale en pantalla todos los echo de error

IEKK · #9 (**permalink**) 16/05/2011, 06:49

Disculpa ya te entendí.
Te está mostrando los echo aún cuando supuestamente no hay errores.
Vayamos por partes ¿Cuales te muestra? Es posible que se deba a la variable $error.

... O quizás esté en el $_FILES['archivo']

Por cierto es diferente:

Código PHP:

  if($_FILES['fichero']['type']=="image/jpeg"){...}
if($_FILES['fichero']['type']=="image/pjpeg"){...}

pithon · #10 (**permalink**) 16/05/2011, 06:53

por si lo quereis saber despues del ultimo echo antes de cerrar el else y el bucle creo un codigo para redimensionar las imagenes y despues cierro con las tres llaves del else y bucle for

IEKK · #11 (**permalink**) 16/05/2011, 06:55

Por favor ¿Me haces un echo de estas variables?

- $_FILES['fichero']['type']
- $error

Para ir descartando cosas

pithon · #12 (**permalink**) 16/05/2011, 06:57

IEKK yo lo tengo asin:

Código PHP:

Ver originalif($_FILES['fichero']['type']!="image/jpeg"){...}

si es diferente tendria que mostrar el echo si es igual no y en este caso es igual no tendria que mostrar nada y me muestra todos los echo

IEKK · #13 (**permalink**) 16/05/2011, 07:00

Cita:

Iniciado por pithon

IEKK yo lo tengo asin:

Código PHP:

Ver originalif($_FILES['fichero']['type']!="image/jpeg"){...}

Pues si te fijas en el primer código pusiste claramente pjpeg.
Yo te recomiendo que pongas las dos, pero como quieras.
En cualquier caso dinos que valor te está devolviendo $_FILES['fichero']['type']
$_FILES['archivo']['size']...

Si te entra en los if es que es obvio que los tienes mal. PHP no inventa nada.

Sólo hazles un echo...

... y dinos que te devuelve.

pithon · #14 (**permalink**) 16/05/2011, 07:02

perdona pero no te entiendo donde como que te haga un echo de esas variables

IEKK · #15 (**permalink**) 16/05/2011, 07:13

Dios mio...

Te pido que hagas un echo para comprobar que los valores del $_FILE sean correctos. ¿Por qué no lo haces y ya está? -.-'

Te repito si te entra en los if y te hace los echo (Y la imagen está acorde a lo que tu estableces) Es que hay un fallo en la condición.

Llevo no sé ni cuantos post pidiendo que hagas un simple echo, que tardas 5 segundos en hacerlo y podrías descartar errores...

Código PHP:

  //Antes de los if....
echo 'Tipo: '.$_FILES['fichero']['type'];
echo 'Tama&ntilde;o: '.$_FILES['archivo']['size'];

Es muy problable que tu error venga de ahí porque cambias el valor de la variable $error en los siguientes if

Código PHP:

  if($_FILES['archivo']['type']!='image/pjpeg'){
    echo 'Este archivo no se puede subir solo se acepta archivos tipo jpeg';
    $error=true;
}
if($_FILES['archivo']['size'] >50000){
    echo 'Este archivo no se puede subir es demasiado grande solo se aceptan hasta 50000 bytes';
    $error=true;
}

Sube una iamgen y dinos que tienes en los $_FILE....

pithon · #16 (**permalink**) 16/05/2011, 07:18

IEKK ya lo se que php no engaña esque hay biene lo que estoy haciendo mal que no consigo resolver todos los echos se muestran en pantallas por su puesto que no se tendrian que mostrar por eso es ahi donde ago algo mal y no lo localizo

IEKK · #17 (**permalink**) 16/05/2011, 07:21

Lo siento me rindo contigo. Te digo que hagas un echo y tú R que R.
Sólo tenías que decirnos que valores te daba la variable para ir descartando posibilidades. Como comrpenderás no estamos ejecutando tú código y no somos adivinos.

Me parece que pedirte que hagas un echo y nos muestres los valores para saber porque te entraban en los if() no era mucho pedir.

Nexus10 · #18 (**permalink**) 16/05/2011, 07:22

pithon, pon lo que te ha dicho IEKK.

Antes de los if, después de $error=false pon esto:

Código PHP:

  echo 'Tipo: '.$_FILES['archivo']['type'];
echo 'Tama&ntilde;o: '.$_FILES['archivo']['size'];

Ahí te va a salir el valor que toma la condición en los if, para saber por qué te entra.

Luego pon aquí:

Me sale esto:

Tipo: (lo que te salga)
Tamaño: (lo que te salga)

pithon · #19 (**permalink**) 16/05/2011, 07:27

vale vale me salio esto Tipo: Tamaño: Array

Nexus10 · #20 (**permalink**) 16/05/2011, 07:29

Perdona, cuando te escribí el código puse:

$_FILES['fichero']['type']

pero tenías que poner:

$_FILES['archivo']['type']

luego lo edité. Comprueba que lo has puesto bien y vuelve a poner lo que te sale.

pithon · #21 (**permalink**) 16/05/2011, 07:38

Nexus10 ya lo puse bien me salio lo que te dije
Tipo: ArrayTamaño: Array

con este error:

( ! ) Notice: Undefined variable: i in C:\wamp\www\misarchivos\archivos.php on line 19

perdona no me da ningun error de php era una letra mal puesta solo que los tipos y tamaño me sale eso

no funciona bien le puse un bucle for
codigo

Código PHP:

Ver original<html>
    <head>
      <script language='javascript' type='text/javascript'>
       function volver(){
        location.href='formulario.html';
       }
       </script>
    </head>
    <body>
 
<?php
$numFiles = count($_FILES['archivo']['tmp_name']);
    for($i=0; $i<$numFiles; $i++){
        if($_FILES['archivos']['name'][$i]){
$error=false;
if($_FILES['archivo']['type']!='image/pjpeg'){
    echo 'Este archivo no se puede subir solo se acepta archivos tipo jpeg';
    $error=true;
}
if($_FILES['archivo']['size'] >50000){
    echo 'Este archivo no se puede subir es demasiado grande solo se aceptan hasta 50000 bytes';
    $error=true;
}
if($_FILES['archivo']['error']!=0){
    echo 'Hay un error en el archivo';
}
if ($error){
    echo '<input type="button" name="retorno" id="retorno" value="volver" onClick="javascript:volver();";';
}else{
      $numFiles = count($_FILES['archivo']['tmp_name']);
    for($i=0; $i<$numFiles; $i++){
       if($_FILES['archivo']['name'][$i]){
if (move_uploaded_file($_FILES['archivo']['tmp_name'][$i],'misarchivos/'.$_FILES["archivo"]["name"][$i])) {
  }else{
    echo 'Error, no se pudo mover la imagen ';
    }
    //aqui redimensiono las imagenes
 
       }
    }
}
?>

lo que me muestra este error

Notice: Undefined index: archivos in C:\wamp\www\misarchivos|archivos.php on line 14
aunque funciona bien

Nexus10 · #22 (**permalink**) 16/05/2011, 07:58

¿Te funciona bien?, ¿¿en serio??.
¿Has visto lo que te salían las variables que te dijimos antes?, ¿entiendes lo que significaba eso?

pithon · #23 (**permalink**) 16/05/2011, 08:01

nexus10 NO me funciona bien
entiendo que tendria que hacer un bucle ya que son multiples archivos y corregir los if verdad?

Nexus10 · #24 (**permalink**) 16/05/2011, 08:10

Desde luego los if los tienes que cambiar. El bucle ya depende de lo que quieras hacer y como lo quieras hacer.

Tienes que aprender a depurar el código. Para saber si una variable está tomando el valor que corresponde pones echo de la forma que te dijimos antes. Para saber que valores tiene un array lo puedes mostrar con print_r($array), así sabrás si es unidimensional o multidimensional para saber como acceder a los elementos.
Este tipo de cosas son las que tienes que aprender primero antes de meterte a programar porque si no vas a tener muchísimos problemas.

pithon · #25 (**permalink**) 16/05/2011, 08:12

ahora funciona bien solo que que salen tantos botones como errores aigan y solo quiero un boton como puedo corregir el bucle

muestro codigo :

Código PHP:

Ver original<?php
$numFiles = count($_FILES['archivo']['tmp_name']);
    for($i=0; $i<$numFiles; $i++){
        if($_FILES['archivos']['name'][$i]){
$error=false;
if($_FILES['archivo']['type'][$i]!='image/pjpeg'){
    echo 'Este archivo no se puede subir solo se acepta archivos tipo jpeg';
    $error=true;
}
if($_FILES['archivo']['size'][$i] >50000){
    echo 'Este archivo no se puede subir es demasiado grande solo se aceptan hasta 50000 bytes';
    $error=true;
}
if($_FILES['archivo']['error'][$i]!=0){
    echo 'Hay un error en el archivo';
}
if ($error){
    echo '<input type="button" name="retorno" id="retorno" value="volver" onClick="javascript:volver();";';
}else{
    
if (move_uploaded_file($_FILES['archivo']['tmp_name'][$i],'misarchivos/'.$_FILES["archivo"]["name"][$i])) {
  }else{
    echo 'Error, no se pudo mover la imagen ';
    }
    //aqui redimensiono las imagenes
 
       }
    }
    }
 
?>

alexg88 · #26 (**permalink**) 16/05/2011, 08:35

Saca el trozo donde pones el botón fuera del bucle para que sólo te aparezca una vez.

Nexus10 · #27 (**permalink**) 16/05/2011, 08:35

Pues si quieres que el bucle se corte en cuanto haya un error tendrás que poner un break dentro del bucle:

if ($error){
break;
}

y luego el botón fuera del bucle:

if ($error){
<input......>
}

pithon · #28 (**permalink**) 16/05/2011, 09:02

perfecto era eso lo que queria un ultimo detalle quiero mostrar el archivo que esta mal lo hice algo asin:

Código PHP:

Ver originalif($_FILES['archivo']['size'][$i] >50000){
   echo 'El archivo'.$i.' no se puede subir es demasiado grande solo se aceptan hasta 50000 bytes';
    $error=true;

lo que pasa que siempre me avisa un archivo antes como el array empieza a contar de 0 como lo podria arreglar?

Nexus10 · #29 (**permalink**) 16/05/2011, 09:51

Súmale uno al índice.

echo 'El archivo'.($i+1).' no se puede subir es demasiado grande solo se aceptan hasta 50000 bytes';

pithon · #30 (**permalink**) 16/05/2011, 10:15

a vale es verdad gracias.

JOER aunque aun no funciona como quiero todo va bien pero al final despues de cerrar llaves abro conexion con db y grabas los datos pero quiero mostrar un echo diciendo que todo fue bien osea desde la subida de archivos hasta la insercion en db pero no logro consegirlo tengo algo asin:

Código PHP:

Ver original<?php
     $numFiles = count($_FILES['archivo']['tmp_name']);
    for($i=0; $i<$numFiles; $i++){
        if($_FILES['archivo']['name'][$i]){
$error=false;
if($_FILES['archivo']['error'][$i]!=0){
   echo 'Hay un error en el archivo<br>';
}
if($_FILES['archivo']['type'][$i]!='image/jpeg'){
    echo 'Este archivo no se puede subir solo se acepta archivos tipo jpeg';
    $error=true;
}
if($_FILES['archivo']['size'][$i] >50000){
   echo 'El archivo'.($i+1).' no se puede subir es demasiado grande solo se aceptan hasta 50000 bytes';
    $error=true;
}
 
if ($error){
    break;
}
else{
    
if (move_uploaded_file($_FILES['archivo']['tmp_name'][$i],'misarchivos/'.$_FILES["archivo"]["name"][$i])) {
   }else{
 echo 'no se pudo subir imagen';
   }
//codigo para redimensionar imagenes y guardarlas en carpeta servidora 
if (imagejpeg($imagen_nueva,'misarchivos/'.$_FILES['archivo']['name'][$i])){
    }else{
    echo 'No se realizo el thumbnail';
    }
}
        }
    }
   if( $error){  echo '<input type="button" name="retorno" id="retorno" value="volver" onClick="javascript:volver();";';
}
 
    
 // codigo para guardar datos en db    
?>