2011-10-14 69 views
2

所有,问题与create_row()函数

这段JavaScript代码动态地添加基于用户的下拉框选择输入框。我已经将它放在了需要的地方,但现在,当我测试时,“msds_copy”和“cofa_copy”两个案例都显示“html2”的相同数据。 “msds_copy”选项应使第二个表单框中显示“区域/语言”,而不是“批次编号”。

任何想法,为什么会发生这种情况?

inquiry_type_onchange: function(e) { 
     var place_order = 1, 
      order_status = 2, 
      telalert_signup = 3, 
      invoice_questions = 4, 
      msds_copy = 5, 
      cofa_copy = 6, 
      html = null, 
      html2 = null, 
      inquiry = e.target, 
      id = "inquiry_type_addendum", 
      form_row = dojo.query("."+id); 

     //Clear any possible previous additions. 
     if (form_row != null) 
      form_row.forEach(dojo.destroy); 

     //Add the correct new field to the form. 
     switch (inquiry.selectedIndex) { 
      case place_order: 
       html = this.create_form_row(id, "Account Number:"); 
       break; 
      case order_status: 
       html = this.create_form_row(id, "Order Number:"); 
       break; 
      case telalert_signup: 
       html = this.create_form_row(id, "Account Number:"); 
       break; 
      case invoice_questions: 
       html = this.create_form_row(id, "Invoice Number"); 
       break; 
      case msds_copy: 
       html = this.create_form_row(id, "Product Name:"); 
       html2 = this.create_form_row(id + "_2", "Region/Language:"); 
      case cofa_copy: 
       html = this.create_form_row(id, "Product Name:"); 
       html2 = this.create_form_row(id + "_2", "Batch Number:"); 
      default: 
     } 

     if (html == null) return; 
     //Place the new element below the inquiry_type field. 
     var placeat = dojo.byId('buttons'); 
     dojo.place(html, placeat, "before"); 
     if(html2!=null) 
      dojo.place(html2, placeat, "before"); 
    }, 

    create_form_row: function(id, label) { 
     //Container 
     var a = dojo.create("div", { id: id, className: "question inquiry_type_addendum", style: "padding-top:4px;" }); 
     //Label 
     var b = dojo.create("div", { className: "label", innerHTML: label, style: "margin-top:8px;" }, a); 
     //Field 
     var c = dojo.create("div", { className: "field" }); 
     var d = dojo.create("span", { className: "full_number_span span" }); 
     var e = dojo.create("input", { type: "text", className: "textbox full_number", name: label }, d); 
     dojo.place(d, c); 
     dojo.place(c, a); 
     return a; 
    } 
}); 

回答

0

你缺少你break S和开关的情况下msds_copy正在执行,然后到下一个案件cofa_copy下降。

 case msds_copy: 
      html = this.create_form_row(id, "Product Name:"); 
      html2 = this.create_form_row(id + "_2", "Region/Language:"); 
      break; // <---- 
     case cofa_copy: 
      html = this.create_form_row(id, "Product Name:"); 
      html2 = this.create_form_row(id + "_2", "Batch Number:"); 
      break; // <---- 
+0

哦,哇。在这里盯着屏幕太久了!非常感谢! – ndisdabest