If-Else Issue In Javascript

Question

Я делаю условный оператор в javascript. Когда пользователь вводит значение в поле ввода «Собрать», его значение вместе со значением из поля ввода «CurBal» выполнит добавление и поместит ответ в поле ввода «Баланс».

Если пользователь вводит значение в поле ввода «вклад», его значение будет вычтено из «CurBal», а разница будет также отражена (предположительно) в «балансе»

я это код:

var collect = parseFloat(document.getElementById("Collect").value) ||0; 
var dep  = parseFloat(document.getElementById("Deposit").value) ||0; 
var curr = parseFloat(document.getElementById("CurBal").value) ||0; 
var bal  = document.getElementById("Balance"); 

if (collect.value = true) 
{ 
bal.value = (collect + curr).toFixed(2) 
bal  = document.getElementById("Balance"); 
} 

else if (dep.value = true) 
{ 
bal.value = (curr - dep).toFixed(2) 
bal  = document.getElementById("Balance"); 
} 

HTML-:

<tr> 
<td> &nbsp;Collections: </td> 
<td> &nbsp; <input onchange="addNumbers()" type="text" id="Collect" name="Collect" value="" /> </td> 
</tr> 

<tr> 
<td> &nbsp;Deposits:</td> 
<td> &nbsp;<input onchange="addNumbers()" type="text" id="Deposit" name="Deposit" value="" /> </td> 
</tr> 

<tr> 
<td> &nbsp;Current Balance: </td> 
<td> &nbsp;<input onchange="addNumbers()" type="text" id="CurBal" name= "CurBal" value="<?php echo $all['balance']; ?>" readonly="readOnly" /> </td> 
</tr> 

<tr> 
<td> &nbsp;Balance: </td> 
<td> &nbsp;<input type="text" id="Balance" name="Balance" value="" readonly="readOnly" /> </td> 
</tr> 

Но, его просто выполняя первое условие правильно. Если я ввела значение в поле ввода «Депозит», это не вычитает значение из «CurBal», а просто копирует значение из «CurBal».

Что следует удалить или добавить или изменить? Спасибо за помощь

Дополнительная информация: Пользователь может только ИДЕТ ввести значение «Собрать» ИЛИ «Баланс».

Answer 1

РЕДАКТИРОВАНИЕ НА КОММЕНТАРИИ НИЖЕ

// Collect is now a number 
var collect = parseFloat(document.getElementById("Collect").value) ||0; 

// dep is now a number 
var dep  = parseFloat(document.getElementById("Deposit").value) ||0; 

// Cur is now a number 
var curr = parseFloat(document.getElementById("CurBal").value) ||0; 

// bal is now pointing to an element 
var bal  = document.getElementById("Balance"); 


// What are you trying to compare here? That bal !== 0 perhaps? 
// Collect is a number, so we can do the compare here. 
if (collect !== 0) { 
bal.value = (collect + curr).toFixed(2) 


// Dep, on the other hand, is pointing to an element 
// so we use the property here 
// The leading '+' is a quick way to turn the string value (dep.value is a string) 
// into a number. 
} else if (+dep.value !== 0) { 
bal.value = (curr - dep).toFixed(2) 
} 

** Примечания ** Линия:

if (collect !== 0) { 

можно записать короче

if (collect) { 

, но я всегда рекомендую писать наиболее читаемая версия, которая точно описывает ты имеешь в виду.

Answer 2

У вас есть это:

if (collect.value = true) 
{ 
bal.value = (collect + curr).toFixed(2) 
bal  = document.getElementById("Balance"); 
} 

else if (dep.value == true) 
{ 
bal.value = (curr - dep).toFixed(2) 
bal  = document.getElementById("Balance"); 
} 

Это звучит так, как будто вам нужно это:

if (collect.value = true) 
{ 
bal.value = (collect + curr).toFixed(2) 
bal  = document.getElementById("Balance"); 
} 

if (dep.value == true) 
{ 
bal.value = (curr - dep).toFixed(2) 
bal  = document.getElementById("Balance"); 
} 

У вас есть else if состояние, так что если collect.value верно, ваши другие условия не будут работать.

Обратите внимание: вы также выполняете назначение в своих операциях if, когда на самом деле вы должны проверять равенство. На самом деле вам даже не нужно этого делать; вы можете сделать:

if (collect.value) 
{ 
... 
} 

if (dep.value) 
{ 
... 
} 

Answer 3

Вы назначаете true к collect.value в первом if тесте. Значение выражения присваивания - это назначенное значение; в этом случае true. Таким образом, первый тест всегда будет успешным. Для сравнительных тестов вы должны использовать == или ===. (При необходимости первый оператор преобразует типы, оператор === будет успешным только в том случае, если текущие типы данных уже совпадают).

Для булевых значений лучше использовать if (collect.value) (и т. Д.), А не if (collect.value == true).

Answer 4

вам не хватает двойной знак равенства, если условие. попробуйте изменить состояние.

if (collect.value != 0) 

else if (dep.value != 0) 

Answer 5

Попробуйте

var collect = parseFloat(document.getElementById("Collect").value) ||0; 
var dep  = parseFloat(document.getElementById("Deposit").value) ||0; 
var curr = parseFloat(document.getElementById("CurBal").value) ||0; 
var bal  = document.getElementById("Balance"); 

if (collect != 0) 
{ 
bal.value = (collect + curr).toFixed(2) 
bal  = document.getElementById("Balance"); 
} 

else if (dep != 0) 
{ 
bal.value = (curr - dep).toFixed(2) 
bal  = document.getElementById("Balance"); 
}