Skip to content

1task - #1

Open
t3mqa wants to merge 2 commits into
mainfrom
dev
Open

t3mqa wants to merge 2 commits into
mainfrom
dev

Conversation

@t3mqa

@t3mqa t3mqa commented Sep 9, 2023

Copy link
Copy Markdown
Owner

No description provided.

@@ -0,0 +1,73 @@
import java.util.Scanner;
public class Calculator {
boolean check;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Имя должно отображать для чего создана переменная. Это для проверки. Чего?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Тем более ты ее еще и не используешь
image

void finalOut() {
Calculator calculator = new Calculator();
String end;
if (costPerPeople < 2) end = "бль";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Если есть несколько if else, их нужно оборачивать в фигурные скобки
https://kotlinlang.ru/docs/control-flow.html

@AleksandrIlinskii

Copy link
Copy Markdown

Приложение крашится если ввести невалидные данные
image

@AleksandrIlinskii

Copy link
Copy Markdown

Приложение крашится если ввести неправильную цену
image

}


}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Поправь форматирование в файле, меню code - reformat code

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