Es correcta esta sintaxy en jQuery

[dannelblogger]

Buenos dias compañeros..
Ayer escribí en la seccion de javascript un problema que me surgia exclusivamente con ie7 en un evento onchange, dentro de un tag select box..
link

Como me he quedado un poco mosca, porque la consola de errores no me dice nada claro y no soy capaz de resolver por mas que modifico el código, he decidido hacer la función en jquery..

Pero vamos.. no soy malo.. soy lo siguiente.. jaja, perdon por las risas pero es lo que toca despues de sufrir..

Código Javascript:

// Esto sería lo que tengo ahora mismo.. 
<select name="puntuacion" onchange="ValorarArticulo(this.value,22,3)">    
 
<option value="0">0</option>   
 <option value="1">1</option>
 <option value="2">2</option>
 <option value="3">3</option>
 <option value="4">4</option>
 <option value="5">5</option>  
</select>
<div id='etiqueta'></div> // donde recibiré los datos..
 

La estructura de la misma función en Jquery seria correcta así?
NOTA(debería de pasarle 3 valores a la función, donde var1 sería el (this.value))

Código Javascript:

function ValorarArticulo(var1,var2,var3)
 {
  $(document).ready(function ()
  {
   $('#puntuacion').change(function() // #puntuacion = nombre del tag selectbox..
    {
     $.get("mi_php.php", // archivo que procesará los datos
      {
        valor1: $('#var1').val(), // declaro nombre de variable1 y valor1
        valor2: $('#var2').val(), // declaro nombre de variable2 y valor2
        valor3: $('#var3').val() // declaro nombre de variable3 y valor3
      },
      function(data)
       {
     $('#etiqueta').html(data); // donde recibo respuesta
       });
     });
  });
 }
 

La idea es reemplazar el evento onchange de javascript por el símil en jquery, me he estado documentando un poco desde esta mañana que he llegado de trabajar, pero no me aclaro.. son ya 6 horas que llevo picando y leiendo.. pero ya me puede..

Se aceptan tomatazos..

[Dradi7]

Bueno bueno bueno revisemos tu código.

Primero no creo que sea correcto que pongas el $(document).ready ya que este es un evento que se ejecuta cuando se carga el DOM completamente

Segundo si quieres pasar argumentos a la Funcion Change de JQuery puedes usar los atributos data para poder manejarlos.

Ejemplo

Código HTML:

<select id="puntuacion" data-value1="22" data-value2="3">
...
...
...
</select>
 

Código Javascript:

$(document).ready(function(){
    $('#puntuacion').change(function(){
        var value1 = $(this).value;
        var value2 = $(this).attr("data-value1");
        var value3 = $(this).attr("data-value2");
        $.get("myphp.php",{
            valor1: value1,
            valor2: value2,
            valor3: value3
        },function(){
            $('#etiqueta').html(data);
        });
    });
});
 

[dannelblogger]

muchas gracias por responder.. lo pruebo en cuanto llegue a casa y comento!!