Класс, который хранит в себе данные о заказе

[NovichokJS]

Помогите плз. Как сделать корректно чтобы при создании заказа генерировался id заказа и чтобы логика метода confirmOrder и метода isValidType была корректной?
Метод confirmOrder должен проставлять ордер в статус confirmed = true (если заказ не был подтвержден раньше) и дату конфирма ставить в текущую.
Метод isValidType должен принимать только два значения Buy, Sell - метод возвращает true, если это требование выполняется, и false если нет.
Мой код:

class Order {
  confirmed = false;
  id = Math.floor(Math.random() * 100);
  dateCreated = new Date();
  dateConfirmed;
  constructor(price, city, type) {
    this.price = price;
    this.city = city;
    this.type = type;
  }
  checkPrice() {
    if (this.price > 1000) {
      return true;
    }
    return false;
  }
  confirmOrder() {
    if (this.confirmed == false) {
      this.confirmed = true;
    }

    this.dateConfirmed = new Date();
  }

  isValidType() {
    if (this.type == false) {
      return 'Buy';
    }
    return 'Sell';
  }
}

[рони]

NovichokJS,

confirmOrder() {
    if (!this.confirmed) {
    this.confirmed = true;
    this.dateConfirmed = new Date();
      }
  }

  isValidType() {
     return ['Sell', 'Buy'].includes(this.type);
  }

[NovichokJS]

спасибо, а как корректно id заказа прикрутить в код? то, что я написал id = Math.floor(Math.random() * 100); - это неверно. Это должен быть наверно рандомное число от 1 до бесконечности и чтобы при новом заказе id не повторился

[voraa]

спасибо, а как корректно id заказа прикрутить в код? то, что я написал id = Math.floor(Math.random() * 100); - это неверно. Это должен быть наверно рандомное число от 1 до бесконечности и чтобы при новом заказе id не повторился

Можно использовать Date.now(); - У всех заказов, поступивших не в одну и ту же милисекунду будет разный id. Если есть вероятность, что в одну ms поступит несколько заказов, тогда
Date.now()*100 + Math.floor(Math.random() * 100)

[рони]

чтобы при новом заказе id не повторился

сделайте счётчик или так ...

id = self.crypto.randomUUID();

[NovichokJS]

id = self.crypto.randomUUID();

ого, такого я не проходил еще. А вот так корректно? или как лучше?

class Order {
  confirmed = false;
  dateCreated = new Date();
  dateConfirmed;
  id;
  constructor(price, city, type) {
    this.price = price;
    this.city = city;
    this.type = type;
  }
  checkPrice() {
    if (this.price > 1000) {
      return true;
    }
    return false;
  }
  confirmOrder() {
    if (!this.confirmed) {
      this.confirmed = true;
      this.dateConfirmed = new Date();
      this.id = Math.floor(Math.random() * 100);
    }
  }

  isValidType() {
    return ['Sell', 'Buy'].includes(this.type);
  }
}

[рони]

NovichokJS,
вам виднее когда нужен id

[рони]

NovichokJS,

let count = 0;
class Order {
  confirmed = false;
  dateCreated = new Date();
  dateConfirmed;
  id = count++;

[NovichokJS]

let count = 0;
class Order {
  confirmed = false;
  dateCreated = new Date();
  dateConfirmed;
  id = count++;

Нужно в рамках класса. Как правильно вообще сгенерировать рандомно id при создании заказа? и чтобы у него был тип данных String?

[рони]

чтобы у него был тип данных String?

alert(typeof self.crypto.randomUUID() );