Не пойму как правильно связать конструкторы с циклом?

Question

[Ruslan]

Здравствуйте, вот мой тестовый код

spoiler

function User (firstName, lastName) {
    this.firstName = firstName;
    this.lastName = lastName;
    
    let date = new Date();
    this.regDate = date.toLocaleDateString() + ' ' + date.toLocaleTimeString();
}

function UserList (User) {
    this.users = [];

    this.addUser = function(User){
        this.users.push(User);
    }

    this.getAllUsers = function(){
        for (let eachUser of this.users) {
            console.log(eachUser);
        }
    }
}

while(true){
    newUser = prompt('Введите имя и фамилию') 
    if (newUser != null) {
        reg = new User(newUser)

        reg2 = new UserList
        reg2.addUser(reg) 
    }

    if (newUser === null) {
        reg2.getAllUsers()
        break;
    }
}

Мне нужно чтобы введенные данные в промпт заносились в массив, а при отмене промпта выводился список всех пользователей. На данном этапе выводится только последний пользователь, не пойму в чем проблема..

Comments

[Alex]

Хотите совет по производительности? Используйте Класы

Answer 1

[gracer]

У вас в цикле новый список постоянно создаётся. Нужно вынести за цикл

reg2 = new UserList
while(true){
    newUser = prompt('Введите имя и фамилию') 
    if (newUser != null) {
        reg = new User(newUser)

        
        reg2.addUser(reg) 
    }

    if (newUser === null) {
        reg2.getAllUsers()
        break;
    }
}

Comments

[Ruslan]

Спасибо, просмотрел)