Añadir input

[Maganius]

Buenas,,,, tengo un pequeño problema con este script que arme...

Se supone que agregar un input si X es 1, pues si lo hace ahora el problema:

Cuando entra al if la primera vez tiene que crear el input ya que este no existe..
pero si yo clckeo por 2da vez el mismo boton este deberia de salir por el else ya que imgfile_1 si existe y no deberia de repetir el procedimiento pero no encuentro el error y si repite el mismo procedimiento tantas veces que clickee en el boton

Código Javascript:

function image(x){
    switch(x){
                case 1:
                    if(document.getElementsByName("imgfile_"+x).name==undefined){
                        input = document.createElement("input");
                        input.name = "imgfile_"+x;
                        input.type = "file";
                        document.body.appendChild(input);
                    }else{
                        alert("else");
                    }
                break;
            }
}
 

[zerokilled]

buenas,
tienes mal la condicion de if. getElementsByTagName siempre devuelve una coleccion (objeto parecido a un array) aunque no encuentre ningun elemento. de modo que la funcion en si no te devuelve un elemento en particular. dicha coleccion no define la propiedad name, por tanto la condicion siempre sera cierta. ¿que puedes hacer? siguiendo el codigo que ya tienes, determina la longitud de la coleccion en la condicion. si es cero significa que no existe ningun elemento con dicho nombre, y viceversa, uno o mas quiere decir que existen elementos. simplemente lees la longitud e inviertes su valor boolean con el operador not (!).

[Maganius]

No entiendo muy bien tu propuesta, me podrias explicar con algun ejemplo, aunque sea inventado una vez que lo entienda lo implemento a mi codigo,

Gracias

[zerokilled]

como te decia, getElementsByTagName devuelve una coleccion. ese objeto tiene la propiedad length, el cual es un numero indicando cuantos elementos existen en esa coleccion. en javascript, cuando usas el operador not con un numero, dicha operacion resulta en un boolean donde !0 es equivalente a true porque cero convertido a boolean es falso. para cualquier otro numero, la operacion resulta false. por tanto, lo que tenias que hacer era !getElementsByTagName("nombre").length.

[Maganius]

Nose porque pero no me funciono,

Igual lo pude solucionar de otra manera que no me afecta "por ahora" creo :P
Lo único que hice fue darle un ID unico a cada input y luego hago la comprobacion dejo el script por si alguien mas le sucede lo mismo

Código Javascript:

function image(x){
    switch(x){
                case 1:
                    if(!document.getElementById("file_"+x)){
                        input = document.createElement("input");
                        input.name = "imgfile_"+x;
                        input.type = "file";
                        input.id   = "file_"+x; //le indique un ID unico
                        document.body.appendChild(input);
                    }
            }
}
 

Gracias por ayudarme zerokilled