Opened 5 years ago

Closed 5 years ago

#276 closed ожидаются исправления (задача сдана)

WW #4

Reported by: stotskiy.nikita Owned by: stotskiy.nikita
Component: WW_mergesort Version: 2.0
Keywords: Cc:

Description


Change History (6)

comment:1 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to stotskiy.nikita
Type: ожидается проверкаожидаются исправления

Не компилируется:

osboxes@osboxes:~/cpp2019/cpp19/stotskiy.nikita/lab_04$ make
mkdir -p obj
gcc -c -g -Iinclude -Wall -Wextra -Werror -fsanitize=address src/mergesort.c -o obj/mergesort.o 
In file included from src/mergesort.c:1:0:
include/mergesort.h:1:29: error: unknown type name ‘size_t’
 void mergesort(void *array, size_t elements, size_t element_size, int (*comparator)(const void*, const void*));
                             ^
include/mergesort.h:1:46: error: unknown type name ‘size_t’
 void mergesort(void *array, size_t elements, size_t element_size, int (*comparator)(const void*, const void*));
                                              ^
Makefile:13: recipe for target 'obj/mergesort.o' failed
make: *** [obj/mergesort.o] Error 1

comment:2 Changed 5 years ago by stotskiy.nikita

Owner: changed from stotskiy.nikita to Egor Suvorov
Type: ожидаются исправленияожидается проверка

comment:3 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to stotskiy.nikita
Type: ожидается проверкаожидаются исправления

Отправлено в 21:00:49, но пока допустим, сделаю объявление в чат на будущее.

По корректности есть 1/10, попытка засчитана.

comment:4 Changed 5 years ago by stotskiy.nikita

Owner: changed from stotskiy.nikita to Egor Suvorov
Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:5 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to stotskiy.nikita
Type: ожидается проверкаожидаются исправления

Корректность 5.5/7:

  1. Не хватает include guard.
  2. В mergesort.h слишком много #include, надо только один.
  3. Выводятся лишние пробелы в конце строки.

Стиль 1/3:

  1. *_gt_comparator у вас не gt.
  2. string_gt_comparator — воспользуйтесь strcmp.
  3. Проверяйте в main не первые символы строк, а строки целиком (strcmp).
  4. Пробелы вокруг бинарных операторов.
  5. Не объявляйте переменные в начале функции.
  6. В my_mergesort внутренние форы бесполезны.
  7. elementsleft --> left_elements.
  8. В my_merge можно сделать один цикл и один if вместо трёх циклов.

comment:6 Changed 5 years ago by Egor Suvorov

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

Кажется, так и осталось.

Захотите дорешать — переоткрывайте тикет.

Note: See TracTickets for help on using tickets.