Glagolev ruslan request to check homework2 by RuslanGlagolev · Pull Request #17 · MykolaSuperman/BaseHomeWork

RuslanGlagolev · 2019-12-10T16:59:58Z

No description provided.

MykolaSuperman

У вас не все дз выполненые, у вас не решенно последняя задача из дз Лучший делитель

MykolaSuperman · 2019-12-14T09:58:13Z

HomeWork2/Glagolev_Ruslan_task1.cpp

+int main() {
+  int n;
+  int sum = 0;
+  int i = 0;


давайте будем приучаться давать переменным более читабельные имена)
например ну сложно понять что означает n? я думаю что лучше будет написать number
или например i думаю что лучше будет написать count_digit_in_number

MykolaSuperman · 2019-12-14T10:01:14Z

HomeWork2/Glagolev_Ruslan_task1.cpp

+    n /= 10;
+    i++;
+  }
+  double avarage = (double)sum / (double)i;


смотрите не нужно явно преобразовывать оба выражения к double, достаточно хотябы чтобы один из аргументов делителя был вещественного типа.
И потом не используйте С стиль приведения типов, используйте только С++ стиль std::static_cast
Т.е. лучше было бы так

double avarage = sum / std::static_cast<double>(i);

MykolaSuperman · 2019-12-14T10:02:23Z

HomeWork2/Glagolev_Ruslan_task1.cpp

+  int sum = 0;
+  int i = 0;
+
+  std::cout << "please, enter n = ";


потом пишите в cout что будет делать ваша программа, т.е. сделайте вывод на экран

MykolaSuperman · 2019-12-14T10:02:48Z

HomeWork2/Glagolev_Ruslan_task2.cpp

+  int n;
+  int first = 0;
+  int second = 0;
+  int i = 0;


как и выше назовите более читабельней имена переменных

MykolaSuperman · 2019-12-14T10:04:01Z

HomeWork2/Glagolev_Ruslan_task2.cpp

+  int first = 0;
+  int second = 0;
+  int i = 0;
+  std::cout << "Please enter number type ******" << std::endl;


опишите пользователю что делает ваша программа, а то просто сложно понять зачем вводить какое то число

MykolaSuperman · 2019-12-14T10:11:20Z

HomeWork2/Glagolev_Ruslan_task3.cpp

+  int s = 1;
+  int f = 0;
+  std::cout << "enter ***" << std::endl;
+  std::cin >> n;


тоже замечание как и выше

HomeWork2/Glagolev_Ruslan_task3.cpp

MykolaSuperman · 2019-12-14T10:14:08Z

HomeWork2/Glagolev_Ruslan_task4.cpp

+
+int main() {
+  int s = 0;
+  int a;


тип переменной s должен быть больше чем тип переменной a, т.к. у вас может быть переполнение

MykolaSuperman · 2019-12-14T10:15:14Z

HomeWork2/Glagolev_Ruslan_task4.cpp

+
+int main() {
+  int s = 0;
+  int a;


потом зачем тут обьявлять переменную a ?
обьявите ее прям в цикле for, ограничивайте область видимости переменным

HomeWork2/Glagolev_Ruslan_task4.cpp

MykolaSuperman

Пожалуйста примените к части кода
https://zed0.co.uk/clang-format-configurator/

HomeWork2/Glagolev_Ruslan_task4.cpp

HomeWork2/Glagolev_Ruslan_task5.cpp

HomeWork2/Glagolev_Ruslan_task6.cpp

HomeWork2/Glagolev_Ruslan_task2.cpp

MykolaSuperman · 2019-12-16T07:16:25Z

HomeWork2/Glagolev_Ruslan_task3.cpp

+  int normal_number;
+  int check_number;
+  int multiplicity_of_number = 1;
+  int reverse_number = 0;


тип reverse_number должен быть больше типа int, так как вы переворачиваете число, соотсветвенно оно может больше чем тип int, точнее быть в него не может поместиться

MykolaSuperman · 2019-12-16T07:17:41Z

HomeWork2/Glagolev_Ruslan_task4.cpp

+      if (number % 2 != 0) {
+        sum += number;
+      } else {
+        number = 0;


зачем тут обнулять переменную? она все равно дальше по коду не будет использоваться

HomeWork2/Glagolev_Ruslan_task4.cpp

MykolaSuperman · 2019-12-16T07:19:33Z

HomeWork2/Glagolev_Ruslan_task4.cpp

+      --max_iteration;
+    } else {
+      std::cout << "Enter number " << number_of_digits << ":" << std::endl;
+      std::cin >> number;


вообще бесмысленный else {
у вас все равно будет начало цикла где вы будете просить ввод заново числа
я думаю что вам этот else нужно просто удалить

HomeWork2/Glagolev_Ruslan_task5.cpp

MykolaSuperman · 2019-12-16T07:27:22Z

HomeWork2/Glagolev_Ruslan_task5.cpp

+  std::cout << "Enter tour number- ";
+  std::cin >> number;
+  for (long long i = 1; i <= number; i++) {
+    if (number % i == 0 && i != number) {


удалите i != number число должно само на себя делиться

MykolaSuperman · 2019-12-16T07:27:32Z

HomeWork2/Glagolev_Ruslan_task5.cpp

+  std::cout << "This program calculates the best divisor of your number.\n";
+  std::cout << "Enter tour number- ";
+  std::cin >> number;
+  for (long long i = 1; i <= number; i++) {


HomeWork2/Glagolev_Ruslan_task6.cpp

MykolaSuperman · 2019-12-16T07:29:58Z

HomeWork2/Glagolev_Ruslan_task6.cpp

+
+int main() {
+  int number;
+  int b = 0;


что такое b ?

MykolaSuperman

прочитайте внимательно замечания по обьявлению переменных. И когда вы исправляете замечания, или прочитали рекомендации нажимайте кнопку Resolve conversation

MykolaSuperman · 2019-12-28T13:25:58Z

HomeWork2/Glagolev_Ruslan_task2.cpp

+int main() {
+  int number;
+  int first_half_of_the_number = 0;
+  int second_half_of_the_number = 0;


обьявляйте переменные по мере их использования, вот зачем тут обьявлять переменные first_half_of_the_number и second_half_of_the_number если реально они будут применяться только после того как вы проверите коректность введенного числа, т.е. проверка что число содержит 6 цифр

MykolaSuperman · 2019-12-28T13:27:14Z

HomeWork2/Glagolev_Ruslan_task3.cpp

+  std::cout << "This program reverses the digits of your number in places.\n";
+  std::cout << "Enter your number-";
+  std::cin >> normal_number;
+  check_number = abs(normal_number);


вот например тут, зачем вверху обьявлять переменную check_number а тут ей присваивать значение почему нельзя сделать сразу
int check_number = abs(normal_number)

MykolaSuperman · 2019-12-28T13:31:01Z

HomeWork2/Glagolev_Ruslan_task5.cpp

+  long long best_sum_number_divisor = 0;
+  long long sum_number_divisor = 0;
+  long long divisor_1;
+  long long best_divisor;


вот зачем обьявлять все тут переменные??? как думаете удобно смотреть весь список переменных в начале, а потом искать по всему коду где же они применяються. Приучайтесь создавать переменные(обьявлять), перед первым их использованием. Не надо все переменные создавать в начале, а потом ниже в коде их инициализировать, так никто не пишет

MykolaSuperman · 2019-12-28T13:33:56Z

HomeWork2/Glagolev_Ruslan_task5.cpp

+      if (divisor < i) {
+        divisor = i;
+      }
+      divisor_1 = divisor;


например зачем выносить в начале main обьявление переменной divisor_1, если она реально используеться только внутри цикла. Т.е. это плохо когда вы увеличиваете скоп переменной, когда ее скоп реально есть только тело цикла. Это плохо для читабельности кода

MykolaSuperman · 2019-12-28T13:35:19Z

HomeWork2/Glagolev_Ruslan_task5.cpp

+  std::cout << "based on the sum of the divisor.\n";
+  std::cout << "Enter tour number- ";
+  std::cin >> number;
+  for (long long i = 1; i <= number; ++i) {


вместо типа long long в цикле лучше писать size_t

MykolaSuperman · 2019-12-28T13:36:18Z

HomeWork2/Glagolev_Ruslan_task6.cpp

+    std::cout << "Enter the correct number- ";
+    std::cin >> number;
+  }
+  for (int a = 0; a != (number + 1) / 2; a++) {


вместо int привыкайте использовать size_t, и только в редких случаях пишите int

MykolaSuperman · 2019-12-28T13:37:02Z

HomeWork2/Glagolev_Ruslan_task6.cpp

+
+int main() {
+  int number;
+  int skimmer = 0;


название переменной огонь)

RuslanGlagolev added 6 commits December 10, 2019 18:50

Resolve task #1

a0f6ce1

Resolve homevork2 task2

c47e95c

Resolve Homework 2 task 2

4cc4859

Resolve homework 2 task 2

f82749a

Resolve homework 2 task3

b44a874

Resolve homework 2 task 4

d3f4e97

MykolaSuperman reviewed Dec 14, 2019

View reviewed changes

RuslanGlagolev added 6 commits December 15, 2019 17:06

Resolve_Glagolev_Ruslan_homework2_task1

9d454f9

Resolve_Glagolev_Ruslan_homework2_task2

b2c1f81

Resolve_Glagolev_Ruslan_homework2_task3

f663015

Resolve_Glagolev_Ruslan_homework2_task4

a472a0a

Glagolev_Ruslan_homework2_task5

896028a

Create_Glagolev_Ruslan_homework2_task6

585db47

MykolaSuperman reviewed Dec 15, 2019

View reviewed changes

HomeWork2/Glagolev_Ruslan_task4.cpp Show resolved Hide resolved

HomeWork2/Glagolev_Ruslan_task5.cpp Show resolved Hide resolved

HomeWork2/Glagolev_Ruslan_task6.cpp Show resolved Hide resolved

RuslanGlagolev added 3 commits December 15, 2019 21:11

Resolve_Glagolev_Ruslan_homework2_task6

f316094

Resolve_Glagolev_Ruslan_homework2_task5

f672cbc

Resolve_Glagolev_Ruslan_homework2_task4

dee8aef

MykolaSuperman reviewed Dec 16, 2019

View reviewed changes

RuslanGlagolev added 5 commits December 25, 2019 22:11

Resolve_Glagolev_Ruslan_homework2_task6

af07e36

Resolve_Glagolev_Ruslan_homework2_task5

977aa85

Resolve_Glagolev_Ruslan_homework2_task4

52a9ebd

Resolve_Glagolev_Ruslan_homework2_task3

4c48332

Resolve_Glagolev_Ruslan_homework2_task2

a02fac0

MykolaSuperman reviewed Dec 28, 2019

View reviewed changes

Conversation

RuslanGlagolev commented Dec 10, 2019

Uh oh!

MykolaSuperman left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

MykolaSuperman left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MykolaSuperman left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants