JavaRush/Помощь по Java задачам/Нужна помощь начинающему программисту)))

23 уровень

26.10.2022
487просмотров
12комментариев

Нужна помощь начинающему программисту)))

Решен

Всем привет. Решил "поиграться" с Java и посмотреть на свои знания)) Задача была такая: Создать базу данных "Индивидуальных предпринимателей" (я из РБ - у нас тут ИП, плюс я бывший ИП-шник, поэтому в голову пришла такая задача))) И вроде бы красиво всё написал (на мой взгляд), конечно мой код может кого-то шокировать))), но как есть) Код такой: Класс Main:

import java.util.ArrayList;

public class Main{
    public static void main(String[] args) {
        ArrayList<IP> BaseDate = new ArrayList<>();
        IP.addNewIP(BaseDate);
    }
}

Класс IP:

import java.util.ArrayList;
import java.util.Scanner;

public class IP {
    private int UNP;
    private String FIO;

    public IP (int UNP, String FIO) {
        this.UNP = UNP;
        this.FIO = FIO;
    }
    public int getUNP() {
        return UNP;
    }
    public String getFIO() {
        return FIO;
    }
    public void setUNP(int UNP) {
        this.UNP = UNP;
    }
    public void setFIO(String FIO) {
        this.FIO = FIO;
    }
    public static void addNewIP(ArrayList<IP> BaseDate) {
        Scanner scanner = new Scanner(System.in);
        while (scanner.nextInt() != 0) {
            BaseDate.add(new IP(scanner.nextInt(), scanner.nextLine()));
        }
    }
}

При запуске программы, при вводе данных (111222333, Иванов Иван Иванович) возникает ошибка:

Exception in thread "main" java.util.InputMismatchException
 at java.base/java.util.Scanner.throwFor(Scanner.java:943)
 at java.base/java.util.Scanner.next(Scanner.java:1598)
 at java.base/java.util.Scanner.nextInt(Scanner.java:2263)
 at java.base/java.util.Scanner.nextInt(Scanner.java:2217)

Почему она возникает? Я разве не правильно ввел данные?

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

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

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

Zelen353535

Уровень 23

26 октября 2022, 12:41

решение

В общем всем спасибо за ответы и что не прошли мимо) Естественно что мой код далёк от идеального и что его можно написать намного лучше, НО я хотел понять, что конкретно не так в МОЁМ коде) И я разобрался)) Ответ был неожиданным, если честно)) Если вдруг кому интересно)) Всё дело в использовании класса "Scanner" и в его алгоритмах работы, ну и в "while" было условие не корректное (как подсказал один из пользователей в комментариях). Переписав код с использованием класса "BufferedReader" всё заработало)

public static void addNewIP(ArrayList<IP> BaseDate) {
    try {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        while (true) {
            System.out.println("Введите УНП нового предпринимателя");
            String nextIP = reader.readLine();
            if(nextIP.equals("exit")) {
                break;
            }
            int newUNP = Integer.parseInt(nextIP);
            System.out.println("Введите ФИО нового предпринимателя №");
            String newFIO = reader.readLine();
            BaseDate.add(new IP(newUNP, newFIO));
        }
    } catch (Exception e) {
        System.out.println("Error");
    }
}

А если говорить про "Scanner", то я СЛУЧАЙНО, методом тыка, увидел вот что)

import java.util.Scanner;

public class Main {
    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);
        int x = scanner.nextInt();
        String y = scanner.nextLine();
        System.out.println("Вы ввели число: " + x + " и строку: " + y);
    }
}

Если запустить этот код, то если после ввода числа (int x) нажимаешь "Enter" (что на мой взглял логично), то какого-то хрена))) символ перевода строки "\n" записывается в строку "String y". В общем, я был "НЕМНОГО" удивлён)) После чего начал гуглить и узнал, что "Scanner" - он такой)) и поэтому и кидало исключение

Gans Electro

Уровень 51

26 октября 2022, 11:12

while (scanner.nextInt() != 0) дело в том месте. Вы сканируете первое число в условий цикла. Дальше ЕЩЕ ОДНО ЧИСЛО, только потом ФИО. Если бы было запись типа. (5, 111222333, Иванов Иван Иванович) то он бы работал. Ладно не работает, вот:

import java.util.ArrayList;
import java.util.HashMap;
import java.util.Scanner;

public class TEST {


    public static class Main {
        public static void main(String[] args) {
            IP.addNewIP();
            IP.getIP();
        }
    }


    public static class IP {
        private static HashMap<Integer, String> BaseDate = new HashMap<>();
        private static ArrayList<Integer> ID = new  ArrayList<>();
        private static ArrayList<String> Names = new  ArrayList<>();

        public static void addNewIP() {
            Scanner scanner = new Scanner(System.in);
            while (scanner.hasNext()) {
                String line = scanner.nextLine();

                if (line.equals("exit")) {
                    break;
                }
                String IdAndName[] = line.split(", ");

                ID.add(Integer.parseInt(IdAndName[0]));
                Names.add(IdAndName[1]);

                Integer universalID = Integer.parseInt(IdAndName[0]);
                BaseDate.put(universalID, IdAndName[1]);
            }
        }

        public static void getIP() {
            for (Integer id : BaseDate.keySet()) {
                String value = BaseDate.get(id);
                System.out.println(id + " --> " + value);
            }
        }

        public static void getID() {
            for (Integer id: ID){
                System.out.println(id);
            }
        }
        public static void getNames() {
            for (String names: Names){
                System.out.println(names);
            }
        }
    }
}

Gans Electro

Уровень 51

26 октября 2022, 11:15

public static class IP {
        private static HashMap<Integer, String> BaseDate = new HashMap<>();
//        private static ArrayList<Integer> ID = new  ArrayList<>();
//        private static ArrayList<String> Names = new  ArrayList<>();

        public static void addNewIP() {
            Scanner scanner = new Scanner(System.in);
            while (scanner.hasNext()) {
                String line = scanner.nextLine();

                if (line.equals("exit")) {
                    break;
                }
                String IdAndName[] = line.split(", ");

//                ID.add(Integer.parseInt(IdAndName[0]));
//                Names.add(IdAndName[1]);

                Integer universalID = Integer.parseInt(IdAndName[0]);
                BaseDate.put(universalID, IdAndName[1]);
            }
        }

        public static void getIP() {
            for (Integer id : BaseDate.keySet()) {
                String value = BaseDate.get(id);
                System.out.println(id + " --> " + value);
            }
        }

        public static void getID() {
            for (var id: BaseDate.keySet()){
                System.out.println(id);
            }
        }
        public static void getNames() {
            for (var names: BaseDate.keySet()){
                System.out.println(BaseDate.get(names));
            }
        }
    }

Денис Enterprise Java Developer

26 октября 2022, 11:50

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

Денис Enterprise Java Developer

26 октября 2022, 10:10

while (scanner.nextInt() != 0) {
            BaseDate.add(new IP(scanner.nextInt(), scanner.nextLine()));
        }

Вот этот кусок это просто генератор ошибок. Твоя программа будет работать правильно в очень редких случаях (да и то если будет). Посмотри лучше в сторону BufferedReader для консольного ввода, и в идеале подумай как передавать данные в твое приложение вообще без консоли. Например выстави какое ни будь API наружу. Что бы человек посылал сообщение на него и твоя приложенька его обрабатывала. Вообще по коду есть масса замечаний. Начиная с конвенций типа нейминга переменных и заканчивая собственно структурой. Если ты хочешь создать базу данных - то разумно будет использовать как хранилище (внезапно) базу данных :) Все переменные именуются с маленькой буквы в lowerCamelCase кстати. Теперь пройдемся по логике. У тебя есть модель (IP) хотя правильно было бы ее назвать PE - Private Entrepreneur Зачем тебе в этой модели сеттеры? Ну допустим ФИО правда может смениться (хотя я не помню как это законодательно регулируется в таком случае), но код вряд ли поменяется :) Имхо эти поля можно сделать private final и даже не пытаться делать сеттеры. В крайнем случае можно будет запись удалить и создать новую в будущем. Метод по добавлению записи в базу не должен находиться в модели. Почитай про MVC паттерн. Модель в себе несет только данные. Вьюха отвечает за их репрезентацию, контроллер делает магию. Ну и в целом, не рассчитывай что твоей приложенькой будут пользоваться как задумано, всегда считай что пользователь олень и пытайся застраховаться. Критические места обвешивай try-catch структурами, продумывай failsafe сценарии и тому подобное. Твое приложение не должно падать при неправильном вводе например.

Zelen353535

Уровень 23

26 октября 2022, 10:35

Спасибо за ответ и за критику) Люблю критику))) Под**б засчитан про "базу данных"))) Посмеялся от души))) Просто я ещё на 14-м уровне) и поэтому такой код. БД и BufferedReader ещё не проходил) Я просто хотел проверить уровень своих знаний, на текущий момент) Сеттеры и Геттеры добавил, так как в лекциях было сказано, что лучше их добавить, даже если они не нужны в дальнейшем. Если не сложно, то напиши пожалуйста эту часть кода, чтобы не выскакивала ошибка "InputMismatchException"

while (scanner.nextInt() != 0) {
            BaseDate.add(new IP(scanner.nextInt(), scanner.nextLine()));
        }

Т.е. чтобы всё осталось как есть, только с изменённым кодом в "while", ну если это конечно возможно сделать.

Денис Enterprise Java Developer

26 октября 2022, 10:35

В принципе, то что пытался написать ты выглядит вот так:

class Scratch {
    private final static List<PrivateEntrepreneur> peDataBase = new ArrayList<>();

    public static void main(String[] args) {
        createNewPeAndAdd();
        listPEs();
    }

    private static void createNewPeAndAdd() {
        String newPePattern = "Please enter new PE %s: ";
        try (BufferedReader console = new BufferedReader(new InputStreamReader(System.in))) {
            System.out.printf(newPePattern, "first name");
            String firstName = console.readLine();
            System.out.printf(newPePattern, "last name");
            String lastName = console.readLine();
            System.out.printf(newPePattern, "patronymic");
            String patronymic = console.readLine();
            System.out.printf(newPePattern, "unpCode");
            int unpCode = Integer.parseInt(console.readLine());

            peDataBase.add(buildPE(firstName, lastName, patronymic, unpCode));

        } catch (IOException ioException) {
            System.out.println("Wrong input, PE won't be created");
        }
    }

    private static void listPEs() {
        peDataBase.forEach(System.out::println);
    }

    public static PrivateEntrepreneur buildPE(String firstName, String lastName, String patronymic, int unpCode) {
        return new PrivateEntrepreneur(unpCode, new Person(firstName, lastName, patronymic));
    }
}

Денис Enterprise Java Developer

26 октября 2022, 10:35

public record PrivateEntrepreneur(int unpCode, Person person) {

    @Override
    public String toString() {
        return String.format("PE %s %s %s : UNP: %d\nOwner: {\n%s}\n",
                person.firstName,
                person.lastName,
                person.patronymic,
                unpCode,
                person);
    }
}

public record Person(String firstName, String lastName, String patronymic) {

    @Override
    public String toString() {
        return String.format("\tFirst name: %s\n\tLast name: %s\n\tPatronymic: %s\n", firstName, lastName, patronymic);
    }
}

Вывод будет примерно такой:

Please enter new PE first name: ivanov
Please enter new PE last name: ivan
Please enter new PE patronymic: ivanovich
Please enter new PE unpCode: 123456
PE ivanov ivan ivanovich : UNP: 123456
Owner: {
	First name: ivanov
	Last name: ivan
	Patronymic: ivanovich
}

Денис Enterprise Java Developer

26 октября 2022, 10:37

Не заметил твой ответ, посмотри два моих следующих сообщения. Единственное что, я там record использую, так что не на всех JDK это заведется. Если не взлетит переделай просто в классы с полями и конструкторами.

Михаил

Уровень 41

26 октября 2022, 09:05

Например, InputMismatchException переводится как "исключение - несоответствие ввода", а дальше "at java.util.Scanner.nextDouble" означает "при Scanner.nextDouble". Значит речь шла о неправильно введённом дробном числе. в какой то момент похоже вместо int тебе строка вводится, вот и выпадает в ошибку. Попробуй продумать или ограничения или обработку таких ошибок.

Zelen353535

Уровень 23

26 октября 2022, 09:13

Да, я почитал в интернете, что это за ошибка, типо я не правильео ввёл "int", но так я же ввожу: 111222333 Иванов Иван Иванович Что не так в моём вводе? Я конечно ещё исключения не изучал, но покопавшись в интернете, в классе "Main" сделал следующее:

import java.util.ArrayList;

public class Main {
    public static void main(String[] args) throws Exception {
        try {
            ArrayList<IP> BaseDate = new ArrayList<>();
            IP.addNewIP(BaseDate);
        }
        catch(Exception e) {
            System.out.println("Error");
        }
    }
}

Вывод в консоль "Error". Почему?

Zelen353535

Уровень 23

26 октября 2022, 09:19

Я так понимаю, что в методе "addNewIP", нужно сделать проверку через "if - else", на ввод данных? Или нет?