¿Qué es "matemática de cuerdas" y por qué es malo?

Question

Recientemente, un compañero desarrollador me reprendió por usar "matemáticas de cuerda" en una aplicación que escribí. Soy bastante nuevo en todo el desarrollo, sin formación formal, y no he oído hablar de este tema. ¿Qué es?

código en cuestión:

$('.submit-input').click(function() { 
    var valid = true; 
    $('input, select, radio').removeClass('error'); 
    $('.error-message').hide(); 

    $('.validate').each(function() { 
     if($(this).val() == $(this).attr('default')){ 
      valid = false; 
      $(this).addClass('error'); 
     } 
    }); 

    if(!$('select[name="contact"] option:selected').val() != ''){ 
     $('select[name="contact"]').addClass('error'); 
     valid = false; 
    } 

    if(!$('input[name="ampm"]:checked').length){ 
     $('input[name="ampm"]').addClass('error');   
     valid = false; 
    } 

    if(!valid){ 
     $('.error-message').css('display','block'); 
     return false; 
    } else { 

     var services_selected = 'Services Selected: '; 
     services_selected += $('.l3').text() + ', ' + $('.l4').text() + ', ' + $('.l5').text() + '; ' + $('.l6').text(); 
     var prices = 'Prices: '; 
     prices += $('.l7').text() + ', ' + $('.l8').text() + ', ' + $('.l9').text() + ', ' + $('.l10').text(); 
     var name = 'Name: '; 
     name += $('input[name="name"]').val(); 
     var phone = 'Phone: ' 
     phone += $('input[name="phone"]').val(); 
     var time = 'Preferred contact time: '; 
     time += $('select[name="contact"] option:selected').val() + $('input[name="ampm"]:checked').val(); 

     $.ajax({ 
      url: 'php/mailer.php', 
      data: 'services_selected=' + services_selected +'&prices=' + prices + '&name=' + name + '&phone=' + phone + '&time=' + time, 
      type: "POST", 
      success: function() { 
       $('#email_form_box .container').children().fadeOut(500, function() { 
        $('#email_form_box .container').html('<div style="margin:20px auto;text-align:center;width:200px;">yada yada yada<br /><span class="close">Close</span></div>'); 
       }); 
      } 
     }); 
    } 

}); 

Editar: La esencia me estoy aquí es que esto no es una expresión coloquial de desarrollo estándar, y probablemente debería hablar con el chico que me dio impertinencias en el primer lugar. Entonces haré eso. Gracias chicos. Volveré con una respuesta, o marcaré a quien ya sabía.

Answer 1

En la mayoría de las implementaciones de navegadores Javascript, la concatenación de cadenas es lenta debido a una copia excesiva.Ver JavaScript: String Concatenation slow performance? Array.join('')?

método preferido es usar una matriz y unirse a:

var pieces = ["You purchased "]; 
pieces.push(num, " widgets."); 
el.innerHTML = pieces.join(''); 

añadido más:

que cree que puede tener un error al acecho en su código: no parece estar escapando sus valores de datos Si alguno de ellos incluye un ampersand, estarías en problemas. Use escape() para todos sus valores de datos.

ps. Y este es un error real que el otro desarrollador se perdió. El problema matemático de cuerdas es un problema de rendimiento/mantenimiento.

Agregado:

me volvió a escribir su sección de composición de correo electrónico (rápidamente). Creo que es más limpio (y será un poco más rápido) cuando se utiliza una matriz de piezas.

.... 
} else { 

var d = []; // the post_data pieces table 

d.push ('services_selected='); // Start the services_selected value 
d.push ('Services Selected: '); 
d.push ($('.l3').text(), ', ', $('.l4').text(), ', ', $('.l5').text(), 
     '; ', $('.l6').text()); 

d.push ('&prices='); // Start the prices value 
d.push ('Prices: '); 
d.push ($('.l7').text(), ', ', $('.l8').text(), ', ', $('.l9').text(), 
     ', ', $('.l10').text()); 

d.push ('&name='); // Start the name value 
d.push ('Name: ', $('input[name="name"]').val()); 

d.push ('&phone='); // Start the phone value 
d.push ('Phone: ', $('input[name="phone"]').val()); 

d.push ('&time='); // Start the timevalue 
d.push ('Preferred contact time: ', 
     $('select[name="contact"] option:selected').val(), 
     $('input[name="ampm"]:checked').val()); 

    $.ajax({ 
     url: 'php/mailer.php', 
     data: d.join(''), 
     type: "POST", 
     success: function() { 
      $('#email_form_box .container').children().fadeOut(500, function() { 
       $('#email_form_box .container').html('<div style="margin:20px auto;text-align:center;width:200px;">yada yada yada<br /><span class="close">Close</span></div>'); 
      }); 
     } 
    }); 
} 

Answer 2

Edit: Bien, mi mal, usted no usa + para la concatenación. Editado a continuación:

Edit2: Ok, es JavaScript, de vuelta a +: P

---

Creo que probablemente refiriéndose a algo como:

$my_html = "<p>" + someVar + "<em>" + somethingImportant + "</em></p>"; 

es decir, utilizando . para la concatenación .

Answer 3

¿Está almacenando/manipulando datos numéricos usando cadenas? Rara vez es una buena idea.

Answer 4

Para extender la respuesta de Skilldrick:

No hay nada mal uso de "+" a concat cuerdas (dependiendo de su idioma) hasta que uno de sus variables no es una cadena:

echo 0 + ": hi!<br />"; 
echo 0 .. ": hi!<br />"; 

La primera línea podría dar salida a "0" (mientras intenta convertir la cadena a un número). La segunda línea funciona como se esperaba escribiendo "0: hola!
".

Answer 5

Dado que eres nuevo en el desarrollo, lo mejor que puedes hacer es debatir con este desarrollador qué es "String math", cómo puedes identificar cuándo lo estás haciendo nuevamente y cómo evitarlo. Luego, regrese aquí y responda su propia pregunta para que podamos ver lo que realmente es esta "matemática de cadenas", desde la perspectiva de su compañero desarrollador.

Answer 6

Ya que reetiquetado su pregunta con javascript, entonces su colega podría significar errores en su código que conducen a preguntas como Strange javascript addition problem

Básicamente "1" + 1 evalúa a 11 en javascript, mientras 1 + 1 evalúa a 2. Ahora reemplace el primer argumento de + con una variable y puede obtener un comportamiento inesperado.

Answer 7

es probable que su compañero de trabajo tenga problemas con líneas de este tipo. en teoría, este es el código perfectamente correcto, pero es prácticamente imposible de leer.

services_selected += $('.l3').text() + ', ' + $('.l4').text() + ', ' + $('.l5').text() + '; ' + $('.l6').text(); 

echar un vistazo a la función y la discusión aquí: http://frogsbrain.wordpress.com/2007/04/28/javascript-stringformat-method/

puede añadir fácilmente esta función a sus JS y entonces usted puede cambiar esta línea horrible código para algo como:

services_selected = '{0} , {1}, {2}, {3}; {4}'.format($('.l3').text(), $('.l4').text(), $('.l5').text(), $('.l6').text()); 

Answer 8

Esto es lo que pienso cuando escucho "String math". Yo también le gritaba a él.

public String StringAdd (String str1, String str2){ 
    int int1, int2; 
    switch (str1){ 
     case "Zero": 
     int1 = 0; 
     break; 
     case "One": 
     int1 = 1; 
     break; 
     //...etc... 
     default: 
     throw new BadNumberSpellingException("You spelled a number wrong."); 
    } 
    switch (str2){ 
     case "Zero": 
     int2 = 0; 
     break; 
     //...etc... 
    } 

    int result = int1 + int2; 
    switch (result){ 
     case 0: 
     return "Zero"; 
     case 1: 
     return "One"; 
     case 2: 
     return "Two"; 
     //etc.... 
    } 
} 

Answer 9

bien, así que aquí está la respuesta que me dijo:

I should have said inline string concatenation/parsing, which is a potential injection vulnerability and a sign of sloppy code or bypassing the framework.

que no encaja exactamente con las otras respuestas que tenemos aquí. Voy a dar el cheque a la respuesta con la mayoría de los votos a favor, ya que es probablemente el más útil, pero solo quería informar.