ATM

let allCards = [] // краще без "всі"

let sumIsValid = false // булеві змінні прийнято іменувати починаючи з is, has ...

for (let i = 0; i < 10; i++) {
    allCards[i] = 0
}

let cardsNumber = parseInt(prompt("Insert a cards number"))

let sumForDeposit = parseInt(prompt("Insert a sum for deposit"))

for (let i = 0; !sumIsValid; i++) { // те саме for (let i = 0; sumIsValid != true; i++) , поміркуйте як працює умова.
/* що в цьому циклі рахує лічильник і? чи важливо його ініціалізувати нулем? можна так записати:
* for (; !sumIsValid; )  Ви вже відчуваєте наступне питання? А чому не вайл?) Фор круто коли є кількість , підрахунок,
* вайл - коли є умова, хоча ці двоє повністю взаємозамінні */
    if (-1000 > sumForDeposit) { /* а ви читали про логічні оператори? тут би логічний оператор обьєднав би дві умови до
    одної, яка б виносила вердикт валідності даним*/
        sumForDeposit = parseInt(prompt("Sum can't be less then -1000"))
    } else if (sumForDeposit > 1000) {
        sumForDeposit = parseInt(prompt("Sum can't be more then 1000"))
    } else {
        sumIsValid == true // будьте уважні, тут не присвоєння, а порівняння, ну а далі знов стоп-кран..
        break // що до брейку - він показник того, що алгоритм не ідеальний, але існують ситуації,коли він дійсно потрібен.
    }

}

allCards[cardsNumber - 1] = sumForDeposit // а номер картки не перевіряєте на валідність, а ще мінус один, це для чого?

alert('Sum of money on each of your card is ' + allCards)

let totalSumOfDepsit = 0
let index = 0 // декларування та ініціалізацію можна робити у форі (зручно коли змінна після циклу не потрібна)

while (index < 10) { // а ось тут можна було б і форчиком пройтися, бо є ініц, умова, інкремент, все як у форі.)
    totalSumOfDepsit += allCards[index]
    index++
}

alert('A total sum on your account is ' + totalSumOfDepsit)
/* Гарного коду! */