function indexOf() mejorada

[derkenuke]

Estoy creando una function indexOf mejorada, pero no me sale. Se trata de este esquema:

getIndexComa(str,ind);

str es una cadena llena de comas, y ind indica el numero de la coma. Por ejemplo getIndexComa("hola,que,tal,estas",2); deberia devolver el indice de la segunda coma, es decir, la que hay entre 'que' y 'tal'.

Lo he intentado pero no me sale. Ayuda por favor.

[caricatos]

Hola derkenuke:

Es la segunda vez que me ocurre hoy, que mientras edito el mensaje, se me envía solo...

...Te decía que para cosas así uso el split... fíjate si esto te sirve:

Código PHP:

  <script>
    function getIndexComa(cadena, lugar)    {
        var dividir = cadena.split(",");
        if (dividir.length <= lugar) return -1;
        else    {
            var total = lugar;
            for (i = 0; i < lugar; i ++)
                total += dividir[i].length;
            return total;
        }
    }
</script> 
  

Saludos

[derkenuke]

Esta muy bien, funciona perfecto. Gracias caricatos!

[caricatos]

:

Estupendo...

Saludos

[JavierB]

Hola, derkenuke y caricatos.

Me pareció interesante la idea y estuve enredando un rato con el código, hasta que salió esto que dejo aquí por si resulta de interés:

Código PHP:

  <script>
cad="Esto,es,una,frase,para,probar,una,nueva,función";
String.prototype.indexOfNum = function (sep,num) {
    var pos=0; var cad=this;
    for (i=0;i<num;i++) 
        if ( (p=cad.indexOf(sep)) !=-1 ) {
        pos+=p;
        cad=cad.substr(p+1,250);
        }
        else return null;
    return pos+num-1;
}
cadena=cad.indexOfNum(",",2);
alert(cadena);
</script> 
  

Saludos,

[derkenuke]

Me parece muy bueno tambien, otra manera valida de hacerlo.

[JavierB]

Hola de nuevo.

Me alegro de que te haya gustado, pero remirando el código de caricatos, creo que donde he puesto return null debería poner return -1 que está más acorde con la función original (indexOf)

Saludos,

[caricatos]

Hola derkenuke y JavierB:

He contrastado el ejemplo de JavierB con el mío, y hay una diferencia de 1, cuando se indexa se empieza por el 0, y por eso cuando en cuestiones numéricas hay fallos se suele devolver -1...

Por eso en mi ejemplo para que devuelva el mismo resultado hay que restar uno...

var total = lugar - 1; // cambiando esta línea.

Y para que la propuesta de JavierB devuelva -1 en los fallos, habría que cambiarlo en la linea que devuelve null

Por otra lado hay una chapucilla (creo ) en donde pone

cad=cad.substr(p+1,250);

supongo que 250 se refiere a la longitud de la cadena, si es así creo que sería conveniente

cad=cad.substr(p+1, this.length);

Usar los prototipos me parece una idea muy buena y elegante (ya lo dije en anteriores ocasiones)

Otra variante con prototipos sería:

Código PHP:

  String.prototype.subcadena = function(cadena, lugar)     {
    var dividir = this.split(cadena);
    if (dividir.length <= lugar) return -1;
    else    {
        var total = (lugar - 1) * cadena.length;;
        for (i = 0; i < lugar; i ++)
            total += dividir[i].length;
        return total;
    }
} 
   

Saludos

[caricatos]

Javi: A pesar del choque, pensabamos lo mismo...

Saludetes

[JavierB]

Buen choque caricatos

Lo de cad=cad.substr(p+1,250); no es una chapucilla, es una chapuza completa Quise poner 255 y además estaba pensando más en otro lenguaje en el que la longitud máxima de una cadena es 255, pero si no vuelvo a estar equivocado no es el caso de JavaScript.

En fín Pilarín, que se agradece la corrección y espero acordarme la próxima

Saludos,

[caricatos]

Hola otra vez:

JavierB: Fíjate en el inspector que estoy haciendo (los eventos en explorer me tienen bloqueado)...

En el último div hay un formulario y puedes ver el atributo maxLength = 2147483647 (si lo ves con explorer)

Por si le interesa a alguien, el enlace es éste:
http://www.pepemolina.com/DOM/index.html

Saludos