Conversation
WoWaster
left a comment
WoWaster
left a comment
There was a problem hiding this comment.
Забавно, в стайлгайде прямо не написано, хотя предполагается, что файлы называются в CamelCase.
There was a problem hiding this comment.
Решение корректное. Хотя по асимптотике (либо уже было на дискретке, либо вот-вот будет на Python) не идеальное. При желании можно найти более эффективное решение. Будет зачтено при инициализации переменных и использовании \n
| int x; | ||
| int t; |
There was a problem hiding this comment.
Неинициализированные переменные! В остальный файлах тоже!
| int x; | ||
| int t; | ||
| scanf("%d", &x); | ||
| t = x * x; |
There was a problem hiding this comment.
t можно было бы объявить прямо здесь
| scanf("%d", &x); | ||
| t = x * x; | ||
| int result = (t + x) * (t + 1) + 1; | ||
| printf("%d", result); |
There was a problem hiding this comment.
А вот изменения в этом файле я не вижу :( Поэтому комментарии будут прямо здесь.
- Куча неиницилизированных переменных!
- Вспомините математическое определение неполного частного,
rобычно означает остаток и сбивает с толку. Если переименовать, можно будет обойтись без комментариев
int r = 0; // сколько раз вмещается b в a- Переменные у нас в курсе называются в camelCase, а не в snake_case.
- Для логических значений лучше использовать тип
boolизstdbool.h. Более того,|-- это побитовое ИЛИ. Чем оно отличается от логического расскажем чуть позже, пока просто не путайте|и||. - В случае если отдельно разбираются случаи когда числа одного знака и разных знаков, можно проверять знак произведения.
remember_aможно было объявить и присвоить позже.- Можно было написать так:
if (b == 0) {
printf("division by zero\n");
return 1;
}и уменьшить вложенность кода.
|
Откатила коммит к предыдущему состоянию, когда еще нет ветки с третьим уроком. Исправила ошибки в файлах. |
WoWaster
left a comment
WoWaster
left a comment
There was a problem hiding this comment.
2 домашку я Вам зачёл целиком, но вливать пока нельзя, потому что здесь оказались коммиты с третьей домашкой. Вы можете попробовать погуглить, как теперь это разнести в разные ветки. Мой вариант решения проблемы (возможно слишком сложный):
- Сделать ветку для 3 домашки от main
- Сделать туда cherry-pick коммитов, в которых живёт третья домашка
- Запушить эту ветку.
- Перейти на ветку для 2 домашки
- Сделать интерактивный ребейз и снести коммиты с 3 домашкой
- Сделать force push этой ветки
На консультации можно сделать это вместе с преподавателем, если страшно и/или гугл не знает более лёгкого пути.
P.S. действия в коммитах обычно пишутся в настоящем времени (add, revert)
| int rememberA = abs(a); | ||
|
|
||
| if (b == 0) { | ||
| printf("devision by zero\n"); |
There was a problem hiding this comment.
Оно, конечно, division, но это уже мелочи
| int howManyTimesBInA = 0; | ||
| scanf("%d %d", &a, &b); | ||
|
|
||
| int underZero = false; // флаг, чтобы понять какие значения введены |
There was a problem hiding this comment.
Здесь тоже не заметил. Логические переменные принято называть в стиле isUnderZero (а ещё корректнее isNegative)
yaprogrammer18-yanchi
force-pushed
the
Lesson_2_CodeStyle
branch
from
0f2a727 to
d22be0f
Compare
|
Делала все строго по вашей инструкции, надеюсь, получилось и я нигде не напутала с коммитами... Получается, если все окей будет с этой веткой, я смогу запулреквестить ветку с третьим уроком и именно ту, куда переносила коммиты? |
2 participants
Added hw:
Это второй пул реквест, первый я закрыла, так как по ошибке в эту ветку добавила решения задач из третьего урока... В данный момент тут только задачи из Code style урока.