javascript function if, else условия

Question

Я новичок в программировании, я создал следующую функцию Javascript с нуля. Но, он не выполняет correctly.It есть некоторые ошибки, но я не могу очистить errors.please помочь мне,

<script type = "text/javascript"> 
     function GetSelectedRow(lnk) { 
      var row = lnk.parentNode.parentNode; 
      if (row.cells[0].innerHTML == "checkbox") 
      { 
       var ItemCode = row.cells[1].innerHTML; 
       var ItemName = row.cells[2].innerHTML; 
       var Unit = row.cells[3].innerHTML; 
       var Price = row.cells[4].innerHTML; 
       var StockQty = row.cells[5].innerHTML; 
       document.getElementById('<%=TxtICode.ClientID %>').value = ItemCode; 
       document.getElementById('<%=TxtIName.ClientID %>').value = ItemName; 
       document.getElementById('<%=DropDownUnit.ClientID %>').value = Unit; 
       document.getElementById('<%=TxtPrice.ClientID %>').value = Price; 
       document.getElementById('<%=TxtStockQty.ClientID %>').value = StockQty; 
       return false; 

      } 
      else (row.cells[0].innerHTML != checkbox) 
      { 
       var ItemCode = row.cells[0].innerHTML; 
       var ItemName = row.cells[1].innerHTML; 
       var Unit = row.cells[2].innerHTML; 
       var Price = row.cells[3].innerHTML; 
       var StockQty = row.cells[4].innerHTML; 
       document.getElementById('<%=TxtICode.ClientID %>').value = ItemCode; 
       document.getElementById('<%=TxtIName.ClientID %>').value = ItemName; 
       document.getElementById('<%=DropDownUnit.ClientID %>').value = Unit; 
       document.getElementById('<%=TxtPrice.ClientID %>').value = Price; 
       document.getElementById('<%=TxtStockQty.ClientID %>').value = StockQty; 
       return false; 
      } 

     } 
    </script> 

Answer 1

Если вы, в самом деле, хотите проверить второе условие, если первое является ложным затем,

else (row.cells[0].innerHTML != checkbox)

должно быть:

else if(row.cells[0].innerHTML != checkbox)

но ваше else состояние такое же, как вы r if условие (кроме того, что вы забыли котировки около checkbox в else), поэтому, если row.cells[0].innerHTML == "checkbox", то ваш код никогда не попадет в ветвь else. Вы, скорее всего, вообще не хотите никакого условия в другом.

Кроме того, если ваша функция головы вниз один из этих двух путей (истина/ложь), и они оба возвращают false то кажется, что либо ваш используют, что для выхода из функции, которые были бы не нужны, так как ваша функция заканчивается после того, как заявление if в любом случае или вы хотите, чтобы false всегда было возвращаемым значением, что не имеет никакого смысла.

Наконец, примерно половина кода в вашей истинной ветви идентична коду в вашей ложной ветви. Этот код должен работать независимо от того, что, так что просто напишите его один раз, за ​​пределами обоих ветвей, и пусть ваши ветви if просто беспокоятся о том, что отличается при каждом обстоятельстве (DRY: «Do not Repeat Yourself») является лучшей практикой в ​​любом программировании язык.)

function GetSelectedRow(lnk) { 

    var ItemCode, ItemName, Unit,Price, StockQty; 
    var row = lnk.parentNode.parentNode; 

    if (row.cells[0].innerHTML == "checkbox") { 
     ItemCode = row.cells[1].innerHTML; 
     ItemName = row.cells[2].innerHTML; 
     Unit = row.cells[3].innerHTML; 
     Price = row.cells[4].innerHTML; 
     StockQty = row.cells[5].innerHTML; 

    } else { 
     ItemCode = row.cells[0].innerHTML; 
     ItemName = row.cells[1].innerHTML; 
     Unit = row.cells[2].innerHTML; 
     Price = row.cells[3].innerHTML; 
     StockQty = row.cells[4].innerHTML; 
    } 

    document.getElementById('<%=TxtICode.ClientID %>').value = ItemCode; 
    document.getElementById('<%=TxtIName.ClientID %>').value = ItemName; 
    document.getElementById('<%=DropDownUnit.ClientID %>').value = Unit; 
    document.getElementById('<%=TxtPrice.ClientID %>').value = Price; 
    document.getElementById('<%=TxtStockQty.ClientID %>').value = StockQty; 

}