Как можно улучшить этот код?

Question

[OlyaSoz]

class Pizza {
    constructor(crust, toppings, howMany) {
        this.crust = prompt('Choose your crust: ');
        this.howMany = +prompt('How many toppings do you want?');
        this.array = [];
    }

    makeTopping() {
        for(let i = 1; i <= this.howMany; i++) {
            this.toppings = prompt('Choose your topping: ');
            this.array.push(this.toppings);
        }
    }

    makePizza() {
        console.log(`Your order is done! You choose ${this.crust} crust with these toppings: ${this.array.join(', ')}`);
    }
}

let personalPizza = new Pizza;
personalPizza.makeTopping();
console.log(personalPizza.makePizza());

Comments

[Urukhayy]

Параметры конструктора не используются?
Зачем в цикле присваивать результат от prompt в this.toppings? Можно сделать локальную переменную (let, var)

[OlyaSoz]

Urukhayy, Спасибо) да, параметры не используются, забыла убрать из конструктора)

Answer 1

[Urukhayy]

class Pizza {
    constructor() {
        this.crust = prompt('Choose your crust: ');
        this.array = new Array(+prompt('How many toppings do you want?')).fill(0);
    }

    makeTopping() {
        this.array = this.array.map(function (t) { return prompt('Choose your topping: ') })
    }

    makePizza() {
        return `Your order is done! You choose ${this.crust} crust with these toppings: ${this.array.join(', ')}`
    }
}

let personalPizza = new Pizza();
personalPizza.makeTopping();
alert(personalPizza.makePizza());

Comments

[OlyaSoz]

Спасибо большое)

Answer 2

[Дмитрий Беляев]

class Pizza {
    constructor() {
        this.crust = prompt('Choose your crust: ');
        this.toppingsCount = +prompt('How many toppings do you want?') || 0;
        this.toppings = [];
    }

    makeToppings() {
        const {toppings, toppingsCount} = this;
        for(let i = toppingsCount; i--;) {
            toppings.push(prompt('Choose your topping: '));
        }
    }

    makePizza() {
        const {crust, toppings} = this;
        return `Your order is done! You choose ${crust} crust with these toppings: ${toppings.length ? toppings.join(', ') : 'nothing'}`;
    }
}

let personalPizza = new Pizza();
personalPizza.makeTopping();
console.log(personalPizza.makePizza());

Comments

[OlyaSoz]

Спасибо)

Answer 3

[Robur]

не использовать prompt (если это реальный проект для реальных людей)
не использовать '+' для конвертации (если нет цели выпендриться знанием корявостей JS)
убрать взаимодействие с пользователем в конструкторе и сделать отдельный метод. (если это реальный проект для реальных людей)
заменить this.toppings на const toppings
дать array осмысленное имя.
Убрать класс вообще (опционально, если этот код именно в таком виде и используется)
console.log в последней строке бессмысленнен. Либо возвращайте значение из функции, либо вызывайте ее просто так.

Остальное - вкусовщина.

Comments

[Robur]

Ах да, и самое главное - перед тем как это все делать, определить понятие "лучше".

[OlyaSoz]

Это, конечно, не реальный проект, но спасибо, буду знать)

Answer 4

[Александр Косицын]

Нет. Он идеален))

Answer 5

[Сергей delphinpro]

параметры не используются, забыла убрать из конструктора)

а зря. Первоначальный путь был верным.

Пицца не может сама спросить покупателя, как ей приготовиться =))
Кто-то другой получит данные от пользователя, передаст их объекту Pizza

const crust = prompt('Choose your crust: ');
const howMany = +prompt('How many toppings do you want?');
const personalPizza = new Pizza(crust, howMany);

Это, пожалуй, главная ошибка в коде - архитектурная. Все остальное, о чем писали - стилистические мелочи.