Opened 5 years ago

Closed 5 years ago

#287 closed ожидается проверка (задача сдана)

lab_04 kreslavskiy.kirill

Reported by: Kreslavski Kirill Owned by: Дмитрий Лапшин (lapshin)
Component: WW_mergesort Version:
Keywords: Cc:

Description

i haven't done the actual mergesort, but it compiles and it has the structure you asked for. that's it for now.

Change History (3)

comment:1 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Kreslavski Kirill
Summary: lab_04 → lab_04 kreslavskiy.kirill
Type: ожидается проверка → ожидаются исправления

Нуууууу ок.

comment:2 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Kreslavski Kirill to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка

Late

Стиль:

  1. Include guards.
  2. Шта? 
    #include "mergesort.h"

#ifndef memcpy
#include <string.h>
#endif

#ifndef memcpy
#include <stdlib.h>
#endif
    #ifdef не читает объявления, да и вообще шта? <br/> Если это твоя попытка делать include guards, она:
    • Провальна (некоторые библиотеки что-то такое делают, но на дефайнах (условно, _NEED_SIZE_T), а не на символах.
    • Тем более не в файлах реализации.
    • Тем более на один и тот же символ.
    • Инклуд должен защищать своё содержимое, а не чтобы им пользовались аккуратно.
  3. Пробелов местами не хватает.
  4. Цикл копирования памяти просится в функцию.

Корректность: работает!

Это в целом 8/10, но после дедлайна. Зачту с штрафом, получится 6.

comment:3 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.