Ks 2 1 #175
Ks 2 1 #175
Conversation
| @@ -0,0 +1,14 @@ | |||
| package Lesson_2 | |||
|
|
|||
| fun main() { | |||
There was a problem hiding this comment.
Есть нарушения в git флоу, то есть в логике работы с коммитами и ветками.
Обрати внимание, сначала ты сделал одну задачу, запушил и все ок.
Затем ты закоммитил в эту же ветку другую задачу задачу (либо создал следующую ветку из предыдущей рабочей ветки) – это видно в истории коммитов.
Что из этого следует? В пулл реквест с одной задачей попали изменения по задаче, которые должны быть только в другой ветке. Это нарушает суть разделения кода задач по веткам. Задача из одной ветки не должна знать и пересекаться с задачей из другой ветки.
На большом проекте затрагивается много файлов и если в одном ПР делать другие изменения (из другой задачи), то при слиянии может быть много проблем и/или гит конфликтов. Например, над кодом другой задачи уже работает программист, изменения из второй задачи не проверены и не протестированы (тестировщики проверяют функционал по конкретному тех. заданию в рамках данного описания задачи в конкретной ветке).
Новые ветки создаем только из мастера (который предварительно надо обновить, если до этого мержились задачи в гитхабе), чтобы не было пересечений с другими тасками)
Что делать?
в данном пулл реквесте надо:
- переключиться в ветку текущего ПР
- удалить лишний файл с задачей (физически из проекта)
- сделать коммит (с изменением удаления)
- прислать ссылку на этот же пулл реквест снова в бот
И новые ветки всегда создаем из основной ветки master.
src/main/kotlin/Lesson_2/L1.kt
Outdated
| val ball_3 = 3.0 | ||
| val ball_4 = 5.0 | ||
|
|
||
| val totalBall = (ball_1 + ball_2 + ball_3 + ball_4)/4.0 |
There was a problem hiding this comment.
Все таки не совсем корректно с точки зрения логики, количество студентов не может быть вещественным числом (дробным). Баллы у тебя и так являются дробными числами, поэтому достаточно будет сделать количество студентов Int. И давай это число также вынесем в отдельную переменную.
| val ball_4 = 5.0 | ||
| val students = 4 | ||
|
|
||
| val totalBall = (ball_1 + ball_2 + ball_3 + ball_4)/students |
There was a problem hiding this comment.
Напоминаю, все арифметические знаки обосабливаются пробелами.
В IDEA есть автоформатирование кода: Находясь в файле Ctrl+Alt+L (для windows) или ⌥+⌘+L (для osx). Рекомендую взять в привычку прожимать автоформат всегда перед коммитом, чтобы все отступы и пробелы расставить по своим местам.
Также подтяни сюда мастер ветку, чтобы убрать лишние изменения.
No description provided.