Conversation
nikolay-977
left a comment
nikolay-977
left a comment
There was a problem hiding this comment.
Можно улучшить: хорошей практикой является добавление README.md с описанием проекта, которое включает название, технологии в проекте (например, Java 11, JUnit 4.13.2, maven 3.9.0 и т.д.), настройка (если необходима) и как запускать (mvn clean test - кавычки позволяют выполнять запуск из файла README.md). Как оформлять README.md можно ознакомиться в статье https://blog.jetbrains.com/idea/2022/08/markdown-support-improvements/
.idea/compiler.xml
Outdated
| <project version="4"> | ||
| <component name="CompilerConfiguration"> | ||
| <annotationProcessing> | ||
| <profile name="Maven default annotation processors profile" enabled="true"> |
There was a problem hiding this comment.
Необходимо исправить: в ПР-е не должно быть лишних файлов (.DS_Store, .idea, .build, .target за исключением файлов отчета). Это локальные файлы. Загружать локальные файлы на удаленный репозиторий не принято: это считается плохой практикой. Они не несут полезной информации, но занимают место.
| final Feline feline; | ||
|
|
||
|
|
||
| boolean hasMane; |
There was a problem hiding this comment.
Можно улучшить: Для всех полей следует указывать модификаторы доступа.
| public void checkAnimalFamilyWithException(){ | ||
| String expectedResult = "Неизвестный вид животного, используйте значение Травоядное или Хищник"; | ||
| Exception actual = Assert.assertThrows(Exception.class,() -> animal.getFood("Вегетарианец")); | ||
| assertEquals("Не тот ответ", expectedResult, actual.getMessage()); |
| this.expectedFood = expectedFood; | ||
| } | ||
|
|
||
| @Parameterized.Parameters |
There was a problem hiding this comment.
Можно улучшить: В параметризованных тестах для аннотации @Parameterized.Parameters лучше использовать аргумент name: @Parameterized.Parameters(name = "Стоимость булочки. Тестовые данные: {0} {1}"), где {0}, {1} - индексы параметров. С аргументом name наименование теста отобразится в отчете. А использование индексов параметров повысит информативность проверки.
| public void checkKittensAmount(){ | ||
| assertEquals("Количество котят не совпадает", 1, feline.getKittens()); | ||
| } | ||
| @Test |
There was a problem hiding this comment.
Можно улучшить: для расстановки пробелов и табуляции в соответствии с настройками Code Style в Idea используй сочетание клавиш Ctrl+Alt+L. Также можно установить курсор на корневую директорию и нажать клавиш Ctrl+Alt+L, отметить чек-боксы: cleanup code, а также optimize imports(для оптимизации импортов) и rearrange entries(для изменения порядка переменных и методов в соответствии с best practics)
src/test/java/LionTest.java
Outdated
| } | ||
| @Test | ||
| public void checkKittens(){ | ||
| assertEquals(5, feline.getKittens(5)); |
There was a problem hiding this comment.
Необходимо исправить: не следует использовать числа, значение которых не является очевидным (антипатерн магические числа). Лучше сохранить в переменную с наименованием, отражающим суть содержимого.
| Lion lion = new Lion("Самец", feline); | ||
| assertEquals("Лев ест другую еду", List.of("Животные", "Птицы", "Рыба"), lion.getFood()); | ||
| } | ||
|
|
There was a problem hiding this comment.
Можно улучшить: лишних пропусков строк следует избегать. Для зрительного разделения кода достаточно использовать один пропуск строки. Пропуски строк должны быть однообразными.
2 participants
No description provided.