Change History (6)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления

Корректность -- 7/7

Стиль -- 2/3

  1. Длинные строки (>80 символов) нужно разбивать на несколько. Примеры того, как строки должны разбиваться для условий, вызовов функций, объявлений функций и т.д. можешь посмотреть вот тут -- https://google.github.io/styleguide/cppguide.html#Formatting
  2. Не нужно писать свою strcmp, лучше воспользоваться библиотечной.
  3. Строку вывода нужно завершать переводом строки
  4. Не нужно кастовать функцию my_strcmp к другому типу, лучше перепиши функцию так, чтобы она внутри себя кастовала свои аргументы к нужному типу.

comment:2 Changed 3 years ago by Станислав Михайлов

Version: 1.02.0

Исправил ошибки по стилю и сделал бонус-задание.

comment:3 Changed 3 years ago by Станислав Михайлов

Type: ожидаются исправленияожидается проверка

comment:4 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления

Вот так переносить строки не надо:

                if (merge(array, element_size, j, j + i, j + 2 * i, comparator)
                != 0)
                    return -1;

Вместо этого:

                if (merge(array, element_size, j, j + i, j + 2 * i, 
                          comparator) != 0)
                    return -1;

В заголовочном файле длинную строку тоже нужно разбить.

10.8/11

comment:5 Changed 3 years ago by Станислав Михайлов

Type: ожидаются исправленияожидается проверка
Version: 2.03.0

Все поправил.

comment:6 Changed 3 years ago by Святослав Власов

Resolution: задача сдана
Status: assignedclosed

11/11

Note: See TracTickets for help on using tickets.