2

Есть класс Animal и сабкласс Cat, хочу, чтоб Cat принимал имя и ограничение на максимальное расстояние для бега.

Но созданный метод выводит всегда результат "бежит" при любом отрицательном параметре расстояния и "не может бежать" при любом положительном параметре. Не пойму, как исправить?

Класс Animal:

public class Animal {
    protected String name;
    protected boolean canRun = true;
    protected boolean canSwim = true;
    protected boolean canJump = true;
    protected int maxRunLength;
    protected double maxJumpHeigth;
    protected int maxSwimLength;
    protected int length;
    protected double jump;



    public Animal(){
    }

    public  Animal(String name) {
        this.name = name;
    }

    public void run(int length) {
        if (length <= this.maxRunLength) {
            System.out.println(this.name + " is running");
        } else {
            System.out.println("can`t run");
        }
    }
}

Класс Cat:

public class Cat extends Animal {
    protected int maxRunLength;
    protected double maxJumpHeigth = 2;
    protected boolean canSwim = false;

    public Cat(){
    }

    public Cat(String name, int maxRunLength) {
        this.name = name;
        this.maxRunLength = maxRunLength;
    }

}
1
  • А если написать this.length в операторе if, то бег будет происходить при любых параметрах...
    – NarX8
    7 сен 2018 в 12:15

1 ответ 1

1

У вас не меняется переменная length у Animal, а если она не меняется, то результат условия if (this.length <= this.maxRunLength) тоже будет постоянным.

public void run(int length) {
    this.lenght += length;
    if (this.length <= this.maxRunLength) {
        System.out.println(this.name + " is running");
    } else {
        System.out.println("can`t run");
    }
}

А если вам и нужен постоянный результат при неизменном length (то есть просто хотите отсекать значения больше maxRunLength, то у вас еще this.maxRunLength не инициализирована, то есть всегда равна 0. Из-за этого и любое положительное отсекает, тк оно больше 0.

4
  • И правда, не подскажите, как его инициализировать, а то я думал, что сделал это...
    – NarX8
    7 сен 2018 в 12:29
  • @NarX8 можно передавать в конструкторе public Animal(String name, int maxRunLength) { this.maxRunLength = maxRunLength; this.name = name; } Советую так же сразу дописать все параметры туда, которые вам нужно инициализировать при создании Animal.
    – Maxgmer
    7 сен 2018 в 12:31
  • Спасибо, все работает для класса Animal, но в классе Cat выдает ту же ошибку (бежит при отрицательных значениях), видимо, я с наследованием или переменными что-то не допонял
    – NarX8
    7 сен 2018 в 12:47
  • 1
    Похоже разобрался, указал в Cat родительский конструктор через super и стало вроде правильно работать.
    – NarX8
    7 сен 2018 в 12:53

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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