JavaRush/Помощь по задачам/В чем подвох?

6 уровень

12.11.2019
542просмотров
11комментариев

В чем подвох?

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

Решен

Господа, в чем мой косяк? Вроде бы все делаю правильно (скорее всего нет). Помогите пожалуйста..

Реализуй метод 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 */ import java.util.concurrent.ExecutionException; public class Cat { public int age; public int weight; public int strength; public Cat() { } public boolean fight(Cat anotherCat) { //напишите тут ваш код int my = 0; int another = 0; if (this.age>anotherCat.age) my+=1; else another+=1; if (this.weight>anotherCat.weight) my+=1; else another+=1; if (this.strength>anotherCat.strength) my+=1; else another+=1; if (my>another) return true; else if (another>my&&another==my); return false; } public static void main(String[] args) throws Exception { } }

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

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

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

Даниил Salesforce Developer в Customertimes

Master

12 ноября 2019, 16:01

Ошибка в логике в строке 27

Даниил Salesforce Developer в Customertimes

Master

12 ноября 2019, 16:04

совет: всегда стоит проверить работоспособность и правильность работы того что написал)

Андрей Балабаш

Уровень 6

12 ноября 2019, 16:22

Спасибо. && - логическое И, изменил на ||

if (my>another) return true;
    else
    if (another>my||another==my) return false;

Но вторая часть всегда true. Изменил условие :

if (my>another) return true;
    else return false;

Все равно не проходит :(

Даниил Salesforce Developer в Customertimes

Master

12 ноября 2019, 16:39

Так, для информации: - another > my || another == my это тоже самое что и "аnother >= my";

Даниил Salesforce Developer в Customertimes

Master

12 ноября 2019, 16:42

Может это проделки валидатора... У себя посмотрел - было так: сравниваешь два объекта по такой формуле strength * weight + age * 0.1

Pavel Kurashov

Уровень 17

Expert

12 ноября 2019, 17:01

я свой код посмотрел, у меня вообще: return this.hashCode() + 2 * this.age + 4 * this.weight + 8 * this.strength > anotherCat.hashCode() + 2* anotherCat.age + 4 * anotherCat.weight + 8 * anotherCat.strength; Во раньше времена были! Только я теперь не знаю, причём тут хешкод ))

Андрей Балабаш

Уровень 6

12 ноября 2019, 17:08

Я на самом деле не могу понять, что у меня не так. Я уже реализовал мою же логику без классов, с переменными.

public class MyClass {
    public static void main(String args[]) {
int age1 = 4, age2 = 3;
int weight1 = 3, weight2 = 2;
int strength1 = 3, strength2 = 2;
int my = 0, another = 0;
        if (age1>age2) my=my+1; else another=another+1;
        if (weight1>weight2) my = my+1; else another= another +1;
        if (strength2>strength2) my= my+1; else another=another+1;

        System.out.println("My Cat power: "+ my +", Another Cat power: " + another);
boolean win;
    if (my> another) win = true; else win = false;

    System.out.println("Is My Cat win? :" + win);

    }
}

Результаты удовлетворяют условия задачи логически. Но нужно решить с классами. По всей логике мой новый код должен выдавать корректный результат. Может еще что то не так делаю?

package com.javarush.task.task05.task0502;

/*
Реализовать метод fight
*/

import java.util.concurrent.ExecutionException;

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

    public Cat() {
    }

    public boolean fight(Cat anotherCat) {
        //напишите тут ваш код
        int my = 0;
        int another = 0;
        if (this.age>anotherCat.age) my=my +1; else another=another + 1;
        if (this.weight>anotherCat.weight) my = my+1; else another= another +1;
        if (this.strength>anotherCat.strength) my= my+1; else another=another+1;

        if (my>another) return true;
            else return false;

    }

    public static void main(String[] args) throws Exception {


    }
}

Что значит реализовать "механизм" драки? Возможно я не правильно интерпритирую это понятие? Как мне кажется мой код вполне тянет на механизм :(

Даниил Salesforce Developer в Customertimes

Master

12 ноября 2019, 17:16

На счёт механизма - затея правильная. Ещё один совет: эти 3 строки эквивалентны

my = my + 1;
my += 1;
my++; // этот я считаю самым простым в понимании логики и чтения строки

Как я понял ты можешь сам придумать по какой формуле будет расчитываться победитель, но в твоём коде есть нюанс... Возмём любую строку. К примеру эту:

if (this.strength>anotherCat.strength) my= my+1; else another=another+1;

Что будет если у объектов поля strength будут равны...?

Даниил Salesforce Developer в Customertimes

Master

12 ноября 2019, 17:18

Лучше сделать по образцу как я и "Pavel" предложили - придумай сам любую формулу где будут учитываться все параметры объекта и просто сравнивай "суммы которые будут у каждого объекта по заданой формуле".

Андрей Балабаш

Уровень 6

12 ноября 2019, 17:29

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

    return this.age + this.weight + this.strength > anotherCat.age + anotherCat.weight + anotherCat.strength;

}

Вот сейчас кроме пункта 4 ошибку выдает в последнем. т.е. ругается, что если ничья - то не выводит false Я реально не могу понять, что оно хочет :( P.S. в алгоритме выше, вопрос по поводу strength. Если они равны - бал начисляется another и финальный результат будет false. То, что нам нужно. Опять так. От греха подальше убрал. И все же опять не то.

Андрей Балабаш

Уровень 6

12 ноября 2019, 17:44

решение

public boolean fight(Cat anotherCat) {
    //напишите тут ваш код
    int my = 0,another = 0;

    if (this.age > anotherCat.age) my++; else if (this.age < anotherCat.age) another++;
    if (this.weight > anotherCat.weight) my++; else if (this.weight < anotherCat.weight) another++;
    if (this.strength > anotherCat.strength) my++; else if (this.strength < anotherCat.strength) another++;

        return my>another;
}

Вот так сработало. Не пойму логику проверки. Но настрадался я знатно. Видимо пора привыкать :D