Crear array usando bucle for

[Astroscuro]

Hola!

llevo un rato mirando en la web a ver si aprendo/entiendo que hay mal en mi sintaxis.

Se trata de validar un formulario, en el cual hay unos campos donde se especifican los datos de unos autores. Además cada autor lleva un campo asociado con su número de orden.

Entonces estoy tratando de coger todos los números de orden en un array, para verificar que son números y después tengo que hacer una comprobación de que todos son distintos, que de eso ya no tengo ni idea.

Hasta la fecha sólo había tratado con arrays en php...

Código Javascript:

// JavaScript Document
function valid() {
    String.prototype.trim = function() { return this.replace(/^\s+|\s+$/g, ""); };
    var patronNaN = /^[0-9]+$/;
    if (pos == 0) {
        alert("No ha seleccionado ning\u00fan autor");
        return false;
    }
    var orden=new Array();
    for (i=1;i<=pos;i++) {
        ape = document.getElementById("ape"+i).value.trim();
        nomb = document.getElementById("nomb"+i).value.trim();
        orden[i] = document.getElementById("orden"+i).value;
        if (!patronNaN.text(orden[i])) {
            alert("el campo de orden debe ser num\u00e9rico");
            return false;
        }
        if (ape.length == 0 || nomb.length == 0) {
            alert("No ha proporcionado informaci\u00f3n suficiente en el campo de Autores");
            return false;
        }
    }
    
    titulo = document.getElementById("titulo").value.trim();
    anio = document.getElementById("anio").value.trim();
    editor = document.getElementById("editor").value.trim();
    tp = document.getElementById("tp").selectedIndex;
    if (titulo.length == 0 || anio.length == 0 || editor.length == 0 || tp == 0) {
        alert("No ha proporcionado informaci\u00f3n suficiente en el campo de Informaci\u00f3n");
        return false;
    }
    return true;
}
 

En concreto la parte del array sería:

Código Javascript:

// JavaScript Document
function valid() {
    var patronNaN = /^[0-9]+$/;
    if (pos == 0) {
        alert("No ha seleccionado ning\u00fan autor");
        return false;
    }
    var orden=new Array();
    for (i=1;i<=pos;i++) {
        orden[i] = document.getElementById("orden"+i).value;
        if (!patronNaN.text(orden[i])) {
            alert("el campo de orden debe ser num\u00e9rico");
            return false;
        }
    }
    return true;
}
 

hasta donde llego se que el error debe estar en la linea:

Código Javascript:

orden[i] = document.getElementById("orden"+i).value;
 

Gracias de antemano. A ver si otros ojos son capaces de ver en dónde me estoy equivocando. Y si me pueden dar una pista de como comprobar que son diferentes los elementos del array, sería estupendo.

Saludos

[David]

Ve si te sirve una función como esta:

Código Javascript:

function repetido(varray) {
    for (var i = 0; i < varray.length; i++) {
        for (var j = i + 1; j < varray.length; j++) {
            if (varray[i] == varray[j]) {
                return true;
            }
        }
    }
    return false;
}
 

[Astroscuro]

Gracias, David, por tu función. Nada más verla recordé haber escrito algo parecido hace tiempo para otra cosa. ¡Que malas son las vacaciones!

El error con el array lo tengo solucionado, y no era tal error. Simplemente se me había colado una 'x' en lugar de una 's' en la línea:

if (!patronNaN.text(orden[i])) {

Desde luego que a veces hace falta levantarse y mirar las cosas al día siguiente con otros ojos.