Не понимаю, что не так

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

Решен

public class Cat {
    public int age;
    public int weight;
    public int strength;

    public Cat() {
    }

    public boolean fight(Cat anotherCat) {
        //напишите тут ваш код


           int a=0;
           int b=0;
if (this.age>anotherCat.age)
a++;
if(this.age<anotherCat.age)
b++;
if (this.strength>anotherCat.strength)
a++;
if (this.strength<anotherCat.strength)
b++;
if(this.weight>anotherCat.weight)
a++;
if(this.weight<anotherCat.weight)
b++;
else if(a>b)  return true;

Реализуй метод 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) { //напишите тут ваш код int a=0; int b=0; if (this.age>anotherCat.age) a++; if(this.age<anotherCat.age) b++; if (this.strength>anotherCat.strength) a++; if (this.strength<anotherCat.strength) b++; if(this.weight>anotherCat.weight) a++; if(this.weight<anotherCat.weight) b++; else if(a>b) return true; else if(b>a) return false; } public static void main(String[] args) { Cat cat1 = new Cat(); Cat cat2 = new Cat(); /*cat1.age=8; cat1.weight=5; cat1.strenght=3; cat2.age=7; cat2.weight=4; cat2.strenght=2; */ cat1.fight(cat2); } }

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

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

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

Санжар Мухитдинов

Уровень 7

4 декабря 2019, 18:07

Здравствуйте, попробуем разобраться с вашим кодом? :)

public class Cat {
	public int age;
	public int weight;
	public int strength;

	public Cat() {
	}

	public boolean fight(Cat anotherCat) {
		// напишите тут ваш код
		int a = 0;
		int b = 0;
		if (this.age > anotherCat.age)
			a++;
		if (this.age < anotherCat.age)
			b++;
		if (this.strength > anotherCat.strength)
			a++;
		if (this.strength < anotherCat.strength)
			b++;
		if (this.weight > anotherCat.weight)
			a++;
		if (this.weight < anotherCat.weight)
			b++;
/*
Вы упустили ещё одно условие, если коты оба круты :) (одинаковы)
то должно возвращаться false
*/
		if (a == b)
			b++;
/*
и как правильно заметила Ксения, компилятор не пропустит выдаст ошибку,
"метод должен возвращать тип boolean" если нет прямого возвращения данных
перед выходом из метода, возвращаемое значение не должно находиться внутри
if или else. Следующие за комментарием четыре строки нужно заменить
(на основании выше сказанного) на одну: return a > b;
здесь если a > b то вернётся true, если нет то false.
*/
		else if (a > b)
			return true;
		else if (b > a)
			return false;
	}

	public static void main(String[] args) {
// в методе main не чего создавать не нужно
// так как в условии об этом ничего не сказано
		Cat cat1 = new Cat();
		Cat cat2 = new Cat();
		/*
		 * cat1.age=8; cat1.weight=5; cat1.strenght=3; cat2.age=7; cat2.weight=4;
		 * cat2.strenght=2;
		 */
		cat1.fight(cat2);
	}
}

Если мне удалось объяснить, то я очень этому рад, если нет, пишите будем разбираться. :)

Ksenia Volkova Java Developer в DXC

Master

4 декабря 2019, 06:24

Во-первых, ты видимо не разобрался, как работает esle if. То, что после else, проверяется только в том случае, если в предыдущем if условие не сработало. То есть если твой кот весит меньше соперника, то

else if(a>b)  return true;
  else if(b>a) return false;

выполнено не будет. Отсюда вытекает еще одна проблема - метод должен обязательно вернуть какое-то значение. Если у тебя все команды return будут стоять внутри if - то есть выполняться только в случае выполнения каких-либо условий - компилятор это не пропустит. Хоть один return дожен выполняться независимо от каких-либо условий вообще - либо вне блока if, либо в else.

Анатолий Анатольевич Мишенин

Уровень 9

3 декабря 2019, 21:44

не понятен вопрос, какой конкретно пункт проверки не проходит ?

Ksenia Volkova Java Developer в DXC

Master

3 декабря 2019, 21:37

Могу предположить, что у тебя return только внутри блока if

Роман

Уровень 24

4 декабря 2019, 03:33

прикрепил код

Amedomaro

Уровень 19

3 декабря 2019, 20:04

А где твоё решение ? Как тебе помочь, если мы не видим код ? Отправь на проверку хотя бы 1 раз и будет возможность показать своё решение прикрепляется тут :

Роман

Уровень 24

4 декабря 2019, 03:33

прикрепил код