Мне одному кажется, что в этом задании некорректно составленное условие...? Что у меня не так? Целый день не могу победить эту задачу.

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

Нужно исправить программу, чтобы компилировалась и работала

Архивный

package com.javarush.task.task07.task0726;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;

/*
Не компилируется задача про котиков
*/

public class Solution {
    public final static ArrayList<Cat> CATS = new ArrayList<>();

    public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        while (true) {

            String name = reader.readLine();
            int age = Integer.parseInt(reader.readLine());
            int weight = Integer.parseInt(reader.readLine());
            int tailLength = Integer.parseInt(reader.readLine());

            Cat cat = new Cat(name, age, weight, tailLength);

            if (name == null || name.isEmpty()) {
                System.out.println(cat.toString());
                break;
            }
            else if (name.length()>0){
                CATS.add(cat);
                if(CATS.size()==1){
                    printList();
                }

                if (CATS.size() == 2) {
                    printList();
                    break;
                }
            }
        }
    }

    public static void printList() {
        for (int i = 0; i < CATS.size(); i++) {
            System.out.println(CATS.get(i));
        }
    }

    public static class Cat {
        private String name;
        private int age;
        private int weight;
        private int tailLength;

        Cat(String name, int age, int weight, int tailLength) {
            this.name = name;
            this.age = age;
            this.weight = weight;
            this.tailLength = tailLength;

        }

        @Override
        public String toString() {
            return "Cat's name: " + name + ", age: " + age + ", weight: " + weight + ", tail: " + tailLength;
        }
    }
}

Задача: Программа вводит с клавиатуры данные про котов и выводит их на экран.

Пример:
Cat's name: Barsik, age: 6, weight: 5, tail: 22 Cat's name: Murka, age: 8, weight: 7, tail: 20

Требования:

Программа должна считывать данные с клавиатуры.
Если пользователь ввел: Barsik, 6, 5 и 22 (каждое значение с новой строки), то программа должна вывести "Cat's name: Barsik, age: 6, weight: 5, tail: 22".
Если пользователь ввел: Murka, 8, 7 и 20 (каждое значение с новой строки), то программа должна вывести "Cat's name: Murka, age: 8, weight: 7, tail: 20".
Если пользователь ввел: Barsik, 6, 5, 22, Murka, 8, 7 и 20 (каждое значение с новой строки), то программа должна вывести две строки: "Cat's name: Barsik, age: 6, weight: 5, tail: 22" и "Cat's name: Murka, age: 8, weight: 7, tail: 20".
Если пользователь ввел пустую строку вместо имени, то программа должна вывести данные на экран и завершиться.

package com.javarush.task.task07.task0726; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; /* Не компилируется задача про котиков */ public class Solution { public final static ArrayList<Cat> CATS = new ArrayList<>(); public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); while (true) { String name = reader.readLine(); int age = Integer.parseInt(reader.readLine()); int weight = Integer.parseInt(reader.readLine()); int tailLength = Integer.parseInt(reader.readLine()); Cat cat = new Cat(name, age, weight, tailLength); if (name == null || name.isEmpty()) { System.out.println(cat.toString()); break; } else if (name.length()>0){ CATS.add(cat); if(CATS.size()==1){ printList(); } if (CATS.size() == 2) { printList(); break; } } } } public static void printList() { for (int i = 0; i < CATS.size(); i++) { System.out.println(CATS.get(i)); } } public static class Cat { private String name; private int age; private int weight; private int tailLength; Cat(String name, int age, int weight, int tailLength) { this.name = name; this.age = age; this.weight = weight; this.tailLength = tailLength; } @Override public String toString() { return "Cat's name: " + name + ", age: " + age + ", weight: " + weight + ", tail: " + tailLength; } } }

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

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

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

Regina Software Developer

Expert

7 июля 2020, 23:11

почему вы пишите так?

if (name == null || name.isEmpty()) {
                System.out.println(cat.toString());
                break;
}

а если у вас несколько котов уже было введено?

Regina Software Developer

Expert

7 июля 2020, 23:13

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

Эрнест

Уровень 19

8 июля 2020, 01:09

Региночка, спасибо вам большое за отклик! Ошибку понял. Почему то условие задачи понял по-другому..

Эрнест

Уровень 19

8 июля 2020, 01:18

package com.javarush.task.task07.task0726;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
/*
Не компилируется задача про котиков
*/
public class Solution {

    public final static ArrayList<Cat> CATS = new ArrayList<>();

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

        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        while (true) {
            String name = reader.readLine();
            int age = Integer.parseInt(reader.readLine());
            int weight = Integer.parseInt(reader.readLine());
            int tailLength = Integer.parseInt(reader.readLine());

            Cat cat = new Cat(name, age, weight, tailLength);

            if (name == null || name.isEmpty()) {
                printList();
                break;
            } else {
                CATS.add(cat);
                printList();
            }
        }
    }


    public static void printList() {
        for (int i = 0; i < CATS.size(); i++) {
            System.out.println(CATS.get(i));
        }
    }

    public static class Cat {
        private String name;
        private int age;
        private int weight;
        private int tailLength;

        Cat(String name, int age, int weight, int tailLength) {
            this.name = name;
            this.age = age;
            this.weight = weight;
            this.tailLength = tailLength;
        }

        @Override
        public String toString() {
            return "Cat's name: " + name + ", age: " + age + ", weight: " + weight + ", tail: " + tailLength;
        }
    }
}

Эрнест

Уровень 19

8 июля 2020, 01:21

Теперь же, по-моему, все условия выполняются, но валидация "ругается" на (4) из (5) условий.

Regina Software Developer

Expert

8 июля 2020, 03:03

решение

1) если имя введено пустое - человек хочет выйти, но вы его сначала еще просите ввести возраст, вес и хвост, я думаю вам стоит сначала проверять что имя не пустое, а уже потом считывать остальное и создавать кота) 2) вам не нужно при добавлении каждого котика выводить список, это нужно сделать один раз за пределами цикла, и соответсвенно убрать этот вызов и при пустом имени, чтобы не дублировался вывод.

Эрнест

Уровень 19

8 июля 2020, 13:08

Спасибо большое😉👍 Все решилось. Перевела с Русского на Русский))