JavaRush/Помощь по Java задачам/Подскажите пожалуйста , где ошибаюсь

41 уровень

23.11.2019
488просмотров
13комментариев

Подскажите пожалуйста , где ошибаюсь

Вопрос по задаче

Решен

Вроде все возможные варианты описаны. При таком коде не проходит проверку, если равенство котов. Если убрать все else, то наоборот: равенство проходит, а остальное нет. Как объединить, чтобы все выполнялось?

Реализуй метод boolean fight(Cat anotherCat):
продумай механизм драки котов в зависимости от их веса, возраста и силы.
Сравни каждый критерий по отдельности, чтобы победителем был тот, у кого были лучшие показатели по большинству критериев.
Метод должен определять, выиграли ли мы (this) бой или нет, т.е. возвращать true, если выиграли и false - если проиграли.
Если ничья и никто не выиграл, возвращаем false.

Требования:

Класс Cat должен содержать конструктор без параметров.
Класс Cat должен содержать всего три поля: age, weight и strength. Поля должны быть публичные.
В классе Cat должен быть метод fight.
В методе fight реализовать механизм драки котов в зависимости от их веса, возраста и силы согласно условию.
Метод должен возвращать одно и тоже значение, если мы деремся с одним и тем же котом.
Если некий кот1 выигрывает у кота кот2, то кот2 должен проигрывать коту кот1.

package com.javarush.task.task05.task0502; /* Реализовать метод fight */ public class Cat { public int age; public int weight; public int strength; public Cat() { } public boolean fight(Cat anotherCat) { if ((this.age==anotherCat.age)&&(this.weight==anotherCat.weight)&&(this.strength==anotherCat.strength)) return false; else if ((this.age>anotherCat.age)&&(this.weight>anotherCat.weight)&&(this.strength>anotherCat.strength)) return true; else if ((this.age>anotherCat.age)&&(this.weight>anotherCat.weight)&&(this.strength<=anotherCat.strength)) return true; else if ((this.weight>anotherCat.weight)&&(this.strength>anotherCat.strength)&&(this.age<=anotherCat.age)) return true; else if ((this.age>anotherCat.age)&&(this.strength>anotherCat.strength)&&(this.weight<=anotherCat.weight)) return true; else if ((this.age>anotherCat.age)&&(this.weight<=anotherCat.weight)&&(this.strength<=anotherCat.strength)) return true; else if ((this.weight>anotherCat.weight)&&(this.age<=anotherCat.age)&&(this.strength<=anotherCat.strength)) return true; else if ((this.strength>anotherCat.strength)&&(this.age<=anotherCat.age)&&(this.weight<=anotherCat.weight)) return true; else return false;//напишите тут ваш код } public static void main(String[] args) { } }

Комментарии (13)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Тимур Исхаков Java Developer в GridNine

23 ноября 2019, 07:54

Код перегруженный и плохо читается, друг. Попробуй реализовать счетчики для каждого кота. Не хочу все объяснять, чтобы ты сам понял, но если совсем непонятно, объясню подробнее, только скажи

Алексей

Уровень 41

23 ноября 2019, 07:56

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

Тимур Исхаков Java Developer в GridNine

23 ноября 2019, 08:01

Ошибка в логике с 27 строки по 32. Программа возвращает true - это неправильно

Тимур Исхаков Java Developer в GridNine

23 ноября 2019, 08:10

В общем так. Если ты хочешь оставить код в таком виде, исправь эти строки на FALSE, и все будет работать. НО!!! Код твой не рациональный. Я рекомендую переписать заново используя счетчики. Это очень просто, изменяй счетчик кота для каждого параметра, который ты сравниваешь, в пользу того кота, у которого показатель выше, и в конце сравни уже их. Это выдумка, а не теория, программирование все из этого состоит.

Ksenia Volkova Java Developer в DXC

Master

23 ноября 2019, 08:13

Считаешь, что тут, например, должно быть false??

else if ((this.age>anotherCat.age)&&(this.weight>anotherCat.weight)&&(this.strength<=anotherCat.strength))
return true;

Не, не будет так работать.

Тимур Исхаков Java Developer в GridNine

23 ноября 2019, 08:15

Я извиняюсь, строки перепутал)) Потому что открыл в Intellij. Строки 27 - 32. 33 строка вообще мусор

Ksenia Volkova Java Developer в DXC

Master

23 ноября 2019, 08:20

🙂 Как же мусор? Без нее не скомпилируется

Тимур Исхаков Java Developer в GridNine

23 ноября 2019, 08:28

Действительно. ~~Не совсем~~ Совсем не понимаю почему

Ksenia Volkova Java Developer в DXC

Master

23 ноября 2019, 08:30

Потому что не могут все return быть зависимы от выполнения условий. Хотя бы один должен выполняться независимо от них (то есть быть либо вне блока If вообще, либо - как тут - в else).

Тимур Исхаков Java Developer в GridNine

23 ноября 2019, 08:36

Ок, спасибо, буду знать.

Ksenia Volkova Java Developer в DXC

Master

23 ноября 2019, 07:45

А если бы было не 3 параметра, а больше? как бы выглядел твой код?

Алексей

Уровень 41

23 ноября 2019, 07:49

Это уже другой вопрос, решал задачу, исходя из знаний в моей голове и по условию параметров 3. Если можете, укажите все же на ошибку в моем коде, чтобы он считал и равных и различающихся котов корректно. Спасибо

Ksenia Volkova Java Developer в DXC

Master

23 ноября 2019, 08:02

решение

Это, на самом деле, не другой вопрос. Тебе кажется, что можно решить перебором. Ну в принципе, можно, да. Только вариантов комбинаций будет больше. У тебя три параметра, у каждого сравнения три варианта исхода (больше, меньше, равно) и все это может по-разному комбинироваться. Например, равны будут не только коты с одинаковыми параметрами, но и кот (1, 1, 2) с котом(2, 1, 1) (+ все остальные комбинации равенства).