Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ks 1 4 #174

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from
Open

Ks 1 4 #174

wants to merge 7 commits into from

Conversation

StepanovDm
Copy link

No description provided.

.idea/vcs.xml Outdated

Choose a reason for hiding this comment

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

Эти файлы у тебя тянутся с 1 ПРа) Смержи плиз задание 1-1, по которому получен аппрув, а далее подтяни в эту ветку master ветку, чтобы эти изменения отсюда убрались)

val age: Byte = 27
val partOfDay: Float = 0.075F
val seconds: Int = 6480
val partOfYear: Float = 2.0547945205479453E-4F

Choose a reason for hiding this comment

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

Тут ты при помощи F числу 2.0547945205479453E-4 обрезаешь часть знаков после запятой до 2.0547945E-4. Если есть задача где это критично, можно получить ошибку в расчетах.

val lenght: Long = 40_848_600_000
val age: Byte = 27
val partOfDay: Float = 0.075F
val seconds: Int = 6480

Choose a reason for hiding this comment

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

Для этого значения можно применить тип, который занимает еще меньше места в памяти)

val seconds: Int = 6480
val partOfYear: Double = 2.0547945205479453E-4
val apogee: Int = 327000
//в задаче же указано число 327_000, было бы 327_00, то применялся бы тип Short. Это как я понял, возможно я ошибаюсь

Choose a reason for hiding this comment

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

Для числа 6480 можно применить тип short. С числом 327000 все верно.

После получения аппрува от меня ПРы нужно мержить в мастер. Если ты это сделал, то подлей плиз мастер ветку сюда. Благодаря этому могут убраться лишние изменения из этого ПРа

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