2

Дайте отзыв по моему решению задачи на Объекты и методы (this). Что нужно исправить или усовершенствовать?

Попробуйте создать новый объект кошки со свойствами и добавьте два несколько методов:

  • метод сна (кошка спит, если сыта)

  • метод кушать (метод принимаем параметр количество каллорий еды)

  • после каждого сна кошка теряет 100 едениц еды (каллорий) Моё решение

let cat = {
  sleep: true,
  eating: 0,

  sleeping: function() {
    if(this.eating>0){
      this.sleep=true;
      alert('Кошка спит');
      this.eating=this.eating-100;
    } else {
      alert('Кошке нужно покушать');
    }
  }
}
sleeping(300);
4
  • интересно, как вы будете заводить вторую кошку. Клонированием?
    – teran
    Commented 18 мар. 2020 в 13:02
  • @Арт'м Калачев Из очевидного - это cat.sleeping(300); и используя this, у Вас всегда будет eating = 0; То-есть кошка всегда будет голодная. Вы изначально хотите передать в функцию сытость кошки?
    – Denis640Kb
    Commented 18 мар. 2020 в 13:08
  • @Denis640Kb да, точно, спасибо за ответ, я забыл поставить название обьекта перед sleeping, а так же убрал this чтоб eating не равнялся 0, а был равен значению которое я введу в функцию.
    – Bart
    Commented 18 мар. 2020 в 13:15
  • @Арт'мКалачев Сейчас до компа доберусь и добавлю своё видение )))
    – Denis640Kb
    Commented 18 мар. 2020 в 13:33

1 ответ 1

-1

У вас кошка никогда не просыпается. и метода кушать я не вижу.

Что это такое

 sleeping(300);

Я не понял. Если это попытка вызвать метод cat.sleeping(), так нужно указывать имя объекта. Да и сам метод у вас не принимает никаких параметров

Я бы делал так

let cat = {
  energy: 0,
  eat: function(energy) {
    this.energy += energy - 0;
  },
  sleep: function() {
    if (this.energy >= 100) {
      this.energy -= 100;
      return true;
    } else
      return false;
  }
};

document.getElementById('eat').addEventListener('click', function() {
  cat.eat(document.getElementById('energy').value);
  console.log(cat.energy);
});

document.getElementById('sleep').addEventListener('click', function() {
  if (cat.sleep())
    console.log('Кошка спит');
  else
    console.log('Кошке нужно покушать');
});
<input id="energy" type="number" value="100"/><input type="button" id="eat" value="Кормить"/><br/>
<input type="button" id="sleep" value="Спать"/><br/>

3
  • Спасибо за ответ, я только недавно начал изучать js и не могу написать код такой сложности как ваш)
    – Bart
    Commented 18 мар. 2020 в 13:27
  • @Арт'мКалачев извините, а что в нем сложного? Commented 18 мар. 2020 в 13:32
  • не знаю некоторых значений, например: document.getElementById, .addEventListener
    – Bart
    Commented 18 мар. 2020 в 13:36

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.