"स्ट्रिंग गणित" क्या है और यह बुरा क्यों है?

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: String Concatenation slow performance? Array.join('')?

पसंदीदा तरीका एक सरणी का उपयोग करें और शामिल होने के लिए है:

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

जोड़ा अधिक:

मुझे लगता है कि आप अपने कोड में एक गुप्त बग हो सकता है: आप से बचने जा करने के लिए प्रकट नहीं होते हैं आपके डेटा मूल्य। यदि उनमें से कोई भी एम्परसैंड शामिल करता है, तो आप परेशानी में होंगे। अपने सभी डेटा मानों के लिए बचें() का उपयोग करें।

ps। और यह एक असली बग है कि अन्य डेवलपर चूक गया। स्ट्रिंग गणित मुद्दा एक प्रदर्शन/रखरखाव मुद्दा है।

जोड़ा गया:

मैं अपने ईमेल लिखने की धारा (जल्दी) दुबारा लिखा। मुझे लगता है कि एक टुकड़ा सरणी का उपयोग करते समय यह क्लीनर (और थोड़ा तेज होगा)।

.... 
} 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:: नीचे संपादित ठीक है, इसे जावास्क्रिप्ट, वापस + है: पी

---

मुझे लगता है कि वह शायद की तरह कुछ करने के लिए बात कर:

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

यानी संयोजन के लिए . का उपयोग कर ।

Answer 3

क्या आप शायद तारों का उपयोग करके संख्यात्मक डेटा संग्रहित/कुशल बना रहे हैं? यह शायद ही कभी एक अच्छा विचार है।

Answer 4

Skilldrick के जवाब का विस्तार करने के लिए:

वहाँ का उपयोग कर "+" आपके चर में से एक तक तार (अपनी भाषा के आधार पर) concat को कुछ भी गलत नहीं एक स्ट्रिंग नहीं है है:

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

पहली पंक्ति आउटपुट "0" हो सकता है (क्योंकि यह स्ट्रिंग को किसी संख्या में कनवर्ट करने का प्रयास करता है)। दूसरी पंक्ति अपेक्षित लेखन के रूप में काम करती है "0: हाय!
"।

Answer 5

चूंकि आप विकास के लिए नए हैं, इसलिए इस डेवलपर के साथ चर्चा करना होगा कि "स्ट्रिंग गणित" क्या है, आप इसे फिर से कब कर रहे हैं, और इससे कैसे बचें, इसकी पहचान कैसे कर सकते हैं। फिर, यहां वापस आएं और अपने प्रश्न का उत्तर दें ताकि हम देख सकें कि यह "स्ट्रिंग गणित" वास्तव में क्या है - आपके साथी देव के परिप्रेक्ष्य से।

Answer 6

जब से तुम जावास्क्रिप्ट के साथ अपने प्रश्न retagged, तो आपके सहयोगी अपने कोड में कीड़े कि, Strange javascript addition problem

मूल रूप से "1" + 1 जावास्क्रिप्ट में 11 का आकलन करती तरह से पूछे जाने वाले हैं, जबकि 1 + 12 का मूल्यांकन मतलब हो सकता है। अब एक चर के साथ + के पहले तर्क को प्रतिस्थापित करें और आप कुछ अप्रत्याशित व्यवहार प्राप्त कर सकते हैं।

Answer 7

शायद यह आपके सहकर्मी के साथ इस तरह की रेखाएं हैं। सिद्धांत में यह बिल्कुल सही कोड है, लेकिन इसे पढ़ने के लिए काफी असंभव है।

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

समारोह और चर्चा यहाँ पर एक नजर है: http://frogsbrain.wordpress.com/2007/04/28/javascript-stringformat-method/

आप आसानी से अपने जे एस को यह समारोह जोड़ सकते हैं और फिर आप की तरह कुछ करने के लिए कोड के इस भयानक लाइन बदल सकते हैं:

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.

करता कौन वास्तव में अन्य उत्तर फिट नहीं हम यहाँ है। मैं सबसे ऊपर के जवाब के साथ उत्तर देने के लिए जा रहा हूं, क्योंकि यह शायद सबसे उपयोगी है, लेकिन सिर्फ सूचित करना चाहता था।