什么是“字符串数学”，为什么它不好？

Question

我最近被一位开发人员在我写的一个应用程序中使用“字符串数学”时苦苦挣扎。我对整个开发项目都很陌生，没有接受过正式培训，我也没有听说过这个问题。它是什么？

代码中的问题：

$('.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>'); 
       }); 
      } 
     }); 
    } 

}); 

编辑：我发现了这里的要点是，这是不是一个标准的开发口语和我也许应该要向谁倾诉给我废话第一的家伙地点。所以我会这样做。多谢你们。我会回复一个答案，或者检查谁已经知道谁。

Answer 1

在大多数Javascript浏览器实现中，串联字符串由于过度复制而速度很慢。见JavaScript: String Concatenation slow performance? Array.join('')?

首选的方法是使用一个数组，并加入：

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

增加了更多：

我认为你可能有一个潜伏的bug在你的代码：你不似乎逃脱你的数据值。如果其中任何一个包含“＆”符号，就会遇到麻烦。对所有数据值使用escape（）。

ps。这是其他开发者错过的一个真正的bug。字符串数学问题是一个性能/可维护性问题。

加：

我重写了您的电子邮件作文部分（快速）。我认为使用一个数组时，它会更干净（并且速度会更快）。

.... 
} 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

编辑：好吧，我的坏，你不使用+串联。编辑如下：

EDIT2：好吧，这是JavaScript的，回+：P

---

我想他很可能指的是这样的：

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

即使用.进行连结。

Answer 3

你可能使用字符串存储/操作数值数据吗？这很少是一个好主意。

Answer 4

为了延长Skilldrick的回答是：

没有什么错用“+”来Concat的字符串（根据您的语言），直到你的变量之一是不是一个字符串：

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

第一行可能会输出“0”（因为它试图将字符串转换为数字）。 第二行如期编写“0：hi！
”。

Answer 5

由于您刚开始接触开发，最好的做法是与开发人员讨论“String math”是什么，您如何识别何时再次使用以及如何避免使用它。 然后，回到这里回答你自己的问题，这样我们就可以从你的开发者的角度看到这个“字符串数学”究竟是什么。

Answer 6

既然你重新标记使用JavaScript你的问题，那么你的同事可能意味着在你的代码中的错误导致的问题，如Strange javascript addition problem

基本上"1" + 1评估在JavaScript 11，而1 + 1计算结果为2。现在用一个变量替换+的第一个参数，你可以得到一些意想不到的行为。

Answer 7

这可能是您的同事遇到的问题。理论上这是完全正确的代码，但几乎不可能读取。

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

看看功能和这里的讨论： http://frogsbrain.wordpress.com/2007/04/28/javascript-stringformat-method/

您可以将此功能轻松地添加到您的JS，然后你可以的代码，这个可怕的行更改为类似：

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

Answer 8

下面是我听到“弦乐数学”时的想法。我也会对他大吼一声。

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

好了，所以这里是他告诉我的答案是：

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

这并不完全适合其他的答案，我们在这里。我打算用最多的赞扬来给支票给出答案，因为它可能是最有用的，但只是想告知。