반복 횟수가 많은 짧은 코드

Question

저는 프로그래밍에있어별로 도움이되지는 않지만 항상 최선을 다합니다. follwong 코드에서 많은 반복을 볼 수 있습니다. 나는 그것을 단축하려고 노력했지만, 나는 그것이 매우 추악하게 남아 있다고 생각한다.

반복 횟수를 줄이려면 어떻게 개선 할 수 있습니까? 당신이 조회를 사용할 수

var values = ["A", "B", "T", "D", "DD"]; 
for(var i = 0; i < values.length; i++) { 
    var len = values[i].length + 1; 
    if(q.substr(0, len) == values[i] + ";"){ 
     $("#imagedocu_art").val(values[i]); 
     $("#imagedocu_text").val(q.substr(len)); 
     $("#new_imagedocu").submit(); 
     break; 
    } 
} 

또는 :

$("#searchterm").keyup(function(e){ 
    var q = $("#searchterm").val(); 

    if(e.keyCode==13){ 
     if(q.substr(0,2) == "A;"){ 
      $("#imagedocu_art").val("A"); 
      $("#imagedocu_text").val(q.substr(2)) 
      $("#new_imagedocu").submit();   
     } 
     if(q.substr(0,2) == "B;"){ 
      $("#imagedocu_art").val("B"); 
      $("#imagedocu_text").val(q.substr(2)) 
      $("#new_imagedocu").submit();   
     } 
     if(q.substr(0,2) == "T;"){ 
      $("#imagedocu_art").val("T"); 
      $("#imagedocu_text").val(q.substr(2)) 
      $("#new_imagedocu").submit();   
     } 
     if(q.substr(0,2) == "D;"){ 
      $("#imagedocu_art").val("D"); 
      $("#imagedocu_text").val(q.substr(2)) 
      $("#new_imagedocu").submit();   
     } 
     if(q.substr(0,3) == "DD;"){ 
      $("#imagedocu_art").val("DD"); 
      $("#imagedocu_text").val(q.substr(3)) 
      $("#new_imagedocu").submit();   
     } 
    } 


    if((q.substr(0,2) == "D;") || (q.substr(0,3) == "DD;")){ 
    if(q.substr(0,2) == "D;"){ 
     var text = q.substr(2); 
    }else{ 
     var text = q.substr(3); 
    } 

Answer 1

당신은 이런 일에 첫 번째 if 블록을 대체 할 수

var values = { A: 1, B: 1, T: 1, D: 1, DD: 1 }; 
var parts = q.split(/;(.*)/, 2); 
if(parts[0] in values){ 
    $("#imagedocu_art").val(parts[0]); 
    $("#imagedocu_text").val(parts[1]); 
    $("#new_imagedocu").submit(); 
} 

실제로 전체 방법은 무엇인가로 대체 될 수 이렇게 :

var q = $("#searchterm").val(); 
var values = { A: false, B: false, T: false, D: true, DD: true }; 

var parts = q.split(/;(.*)/, 2); 

if(e.keyCode==13){ 
    if(parts[0] in values){ 
     $("#imagedocu_art").val(parts[0]); 
     $("#imagedocu_text").val(parts[1]); 
     $("#new_imagedocu").submit(); 
    } 
} 

if(parts[0] in values && values[parts[0]]){ 
    var text = parts[1]; 
} 

Answer 2

정규 표현식은 모든 if 점검을 대체 할 수 있습니다.

if (e.keyCode==13) { 
    var match = q.match(/^([ABTD]|DD);(.*)/); 
    if (match) { 
     $("#imagedocu_art").val(match[1]); 
     $("#imagedocu_text").val(match[2]); 
     $("#new_imagedocu").submit();   
    } 
} 

등록 특급

/  Start of reg exp 
^  Start of string 
(  Start of capture group 
[ABTD] Match any character included inside the [] 
|  OR 
DD  Match the String "DD" 
)  End of capture group 
;  Match a semicolon 
(.*) Match all the remaining 
/  End of Reg Exp 

의 기본 설명이 경기는이 스타일의 모든 문제입니다

null or ["A;FOO", "A", "FOO"] or ["DD;FOO", "D", "FOO"]