Skip to content

Pull Request - #1

Open
KorolkovN wants to merge 3 commits into
mainfrom
dev
Open

KorolkovN wants to merge 3 commits into
mainfrom
dev

Conversation

@KorolkovN

Copy link
Copy Markdown
Owner

Надеюсь я понял правильно про Pull Request

Comment thread src/main/java/org/example/Main.java Outdated
@@ -0,0 +1,87 @@
package org.example;

import java.util.Scanner;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🍏Скачет форматирование кода, старайся прожимать ctrl+alt+l перед тем, как делаешь коммит. https://devcolibri.com/%D0%B3%D0%BE%D1%80%D1%8F%D1%87%D0%B8%D0%B5-%D0%BA%D0%BB%D0%B0%D0%B2%D0%B8%D1%88%D0%B8-intellij-idea/

Comment thread src/main/java/org/example/Main.java Outdated
public class Main {
public static void main(String[] args) {
Scanner scanner = new Scanner(System.in);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🍏🍏Лишние пустые строки

Comment thread src/main/java/org/example/Main.java Outdated
scanner.close();
}

private static int getNumberOfPersons(Scanner scanner) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🍏Можно было тоже вынести в отдельный класс, так как у каждого класса должны быть единственная ответственность (принцип Single responsibility из SOLID https://habr.com/ru/articles/454290/)

Comment thread src/main/java/org/example/Main.java Outdated
return totalCost / persons;
}

public String getRussianEnding(int persons) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Почему окончание слова рубль зависит от количества человек? Окончание должно зависеть от числа рублей, которое получается в результате работы калькулятора

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants