Lesson 19 and 20 by johnmango · Pull Request #86 · KFalcon2022/practical-tasks

johnmango · 2024-05-17T09:42:42Z

Lesson 19 is done.
Lesson 20 all tasks done.

KFalcon2022 · 2024-05-22T17:24:58Z

src/com/walking/lesson19_object_methods/Main.java

+                      new Car(2020, "e555ee55", "red", "skoda")
+                     };
+
+        CarService dataBase = new CarService(cars);


database - это одно слово:) но название некорректно - carService и очевиднее и лучше. Так-то, он не обязан хранить данные - вполне может обращаться внутри себя к другому классу, ответственному за хранение

KFalcon2022 · 2024-05-22T17:26:13Z

src/com/walking/lesson19_object_methods/model/Car.java

+            hashcode += color == null ? 0 : color.hashCode();
+            hashcode += model == null ? 0 : model.hashCode();
+        }
+    return hashcode;


нарушен контракт equals-hashcode. Сейчас у одинаковых по икуалз объектов могут быть разные хэшкоды, что недопустимо

KFalcon2022 · 2024-05-22T17:26:51Z

src/com/walking/lesson19_object_methods/model/Car.java

+            hashcode += color == null ? 0 : color.hashCode();
+            hashcode += model == null ? 0 : model.hashCode();
+        }
+    return hashcode;


не хватает пустой строки перед return. И поехала табуляция. ctrl+alt+L - автоформатирование кода

KFalcon2022 · 2024-05-22T17:27:25Z

src/com/walking/lesson19_object_methods/model/CarService.java

@@ -0,0 +1,18 @@
+package com.walking.lesson19_object_methods.model;
+
+public class CarService {


сервис - не модель. Обычно сервисы выносят в отдельный пакет

KFalcon2022 · 2024-05-22T17:28:06Z

src/com/walking/lesson19_object_methods/model/CarService.java

+public class CarService {
+    private final Car[] cars;
+
+    public CarService(Car[] allcars) {


переменные, прааметры методов и поля именуются в camelCase. Т.е. здесь - allCars. Впрочем, просто cars было бы достаточно

KFalcon2022 · 2024-05-22T17:28:54Z

src/com/walking/lesson19_object_methods/model/CarService.java

+        this.cars = allcars;
+    }
+
+    public Car searchCar(Car userCar) {


нет смысла в постфиксе Car. Вполне логично, что метод поиска в CarService ищет объекты типа Car. Т.е. можно назвать метод search. Или, что более распространено - find

KFalcon2022 · 2024-05-22T17:29:58Z

src/com/walking/lesson19_object_methods/model/CarService.java

+            }
+        }
+        return null;
+    }


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

KFalcon2022 · 2024-05-22T17:34:39Z

src/com/walking/lesson20_exceptions/task1_catchException/model/File.java

+
+public class File {
+    private String fileName;
+    private final FileType fileType;


почему я могу менять все, кроме типа файла? Вполне логично заменить, например, txt на sql/properties/другой текстовый формат. В целом, мелкое замечание

KFalcon2022 · 2024-05-22T17:34:47Z

src/com/walking/lesson20_exceptions/task1_catchException/model/File.java

+    private final FileType fileType;
+    private int fileSize;
+
+    public File(String fileName, FileType fileType, int fileSize){


пробел перед {

KFalcon2022 · 2024-05-22T17:34:54Z

src/com/walking/lesson20_exceptions/task1_catchException/model/File.java

+    }
+
+    @Override
+    public String toString(){


KFalcon2022 · 2024-05-22T17:35:41Z

src/com/walking/lesson20_exceptions/task1_catchException/service/FileService.java

+
+    public File search (String userFile) throws FileNotFoundException {
+        for (File file : files) {
+            if (userFile.equals(file.getFileName())) {


в File не переопределен икуалз

KFalcon2022 · 2024-05-22T17:36:48Z

src/com/walking/lesson20_exceptions/task1_throwsException/service/FileNotFoundException.java

@@ -0,0 +1,7 @@
+package com.walking.lesson20_exceptions.task1_throwsException.service;
+
+public class FileNotFoundException extends Exception{


не хватает пробела перед {. Логичнее было бы наследоваться от RuntimeException - ты ведь файл ищешь не в файловой системе, а в ин-меморе хранилке - массиве

KFalcon2022 · 2024-05-22T17:37:49Z

src/com/walking/lesson20_exceptions/task2/Main.java

+        if (userFigure == 1) {
+            figure = new Square();
+        } else {
+            figure = new Triangle();


хватило бы тернарки. Или switch-case, который выглядел бы лучше при потениальном увеличении числа наследников фигуры

KFalcon2022 · 2024-05-22T17:38:01Z

src/com/walking/lesson20_exceptions/task2/Main.java

+        } else {
+            figure = new Triangle();
+        }
+        System.out.println(figure.buildFigure(userWidth));


пустая строка перед этой строчкой

KFalcon2022 · 2024-05-22T17:38:21Z

src/com/walking/lesson20_exceptions/task2/model/Square.java

@@ -0,0 +1,22 @@
+package com.walking.lesson20_exceptions.task2.model;
+
+public class Square implements Figure{


пробел

KFalcon2022 · 2024-05-22T17:38:35Z

src/com/walking/lesson20_exceptions/task2/service/InputValidationException.java

@@ -0,0 +1,7 @@
+package com.walking.lesson20_exceptions.task2.service;
+
+public class InputValidationException extends RuntimeException{


пробел

KFalcon2022 · 2024-05-22T17:39:33Z

src/com/walking/lesson20_exceptions/task3/Main.java

 public class Main {
    public static void main(String[] args) {
+        Animal[] animals = {new Cat(), new Dog(), new Cow()};
+            soundAll(animals);


лишний таб.

Kot added 18 commits November 16, 2023 12:06

My version

334cb54

My version using INTERFACE added.

88e5e20

My version using INTERFACE added.

0657fa4

Task 2 done

b480990

Task 3 done

1c23a1d

Lesson 17, task 1 is done.

f0a718c

Lesson 17, task 2 is done.

4f528c5

Lesson 17, task 2 and task 3 are done.

07ef60a

Merge remote-tracking branch 'origin/for-pr' into for-pr

32ea4f1

Lesson 18 task 1 done

1b8f0db

Lesson 18 all tasks done

82556f9

Lesson 18 all tasks done

d9b545e

lesson 19 done

fba1cf4

lesson 19 done

05d4449

lesson 20 task 1

4ac7d5b

lesson 20 task 2

1edaacb

lesson 20 task 2

88685a7

lesson 20 all tasks

2d9a103

johnmango changed the title ~~Lesson 19 PR~~ Lesson 19 and 20 May 21, 2024

KFalcon2022 reviewed May 22, 2024

View reviewed changes

src/com/walking/lesson20_exceptions/task1_catchException/model/File.java

}

@Override

public String toString(){

Copy link
Copy Markdown

Owner

KFalcon2022 May 22, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

KFalcon2022 reviewed May 22, 2024

View reviewed changes

		@@ -0,0 +1,18 @@
		package com.walking.lesson19_object_methods.model;

		public class CarService {

		@@ -0,0 +1,7 @@
		package com.walking.lesson20_exceptions.task1_throwsException.service;

		public class FileNotFoundException extends Exception{

		@@ -0,0 +1,22 @@
		package com.walking.lesson20_exceptions.task2.model;

		public class Square implements Figure{

		@@ -0,0 +1,7 @@
		package com.walking.lesson20_exceptions.task2.service;

		public class InputValidationException extends RuntimeException{

Conversation

johnmango commented May 17, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

johnmango commented May 17, 2024 •

edited

Loading