Opened 3 years ago

Closed 3 years ago

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

WW #5

Reported by: Georgiy Angeni Owned by: Святослав Власов
Component: WW_c_io Version: 2.0
Keywords: Cc:

Description


Change History (3)

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

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

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

  1. apply должен работать за один проход

Стиль -- 0.5/3

  1. Длинные строки нужно разбить
  2. Зачем ты дважды объявляешь структуру point?
  3. Функции integer_to_buffer и buffer_to_integer ты сможешь реализовать значительно проще, если немного помедитируешь вот над таким кодом:
    int x = 0x12345678;
    char* bytes = &x;
    assert(bytes[0] == 0x78);
    assert(bytes[1] == 0x56);
    assert(bytes[2] == 0x34);
    assert(bytes[3] == 0x12);
    
  4. Зачем так извращаться с count_with_apply? Никто ж не заставляет печатать длину в той же функции, где ты её считаешь.
  5. output_all_points и output_all_points_binary буду выглядеть лучше, если их тоже реализовать через apply.
  6. Функции загрузки точек из файлов лучше вынести в отдельные функции, а не городить в main.
  7. argc лучше бы проверить assert'ом, и не падать в UB, если вдруг забыл передать аргумент программе.

Бонус 0/2
Отрицательные числа в бинарном формате грузятся неправильно.

comment:2 Changed 3 years ago by Georgiy Angeni

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

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

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

Разыменовывать указатель на функцию не нужно, можно просто писать op(tmp, data);

В остальном всё ок, бонус тоже работает. 12/12

Note: See TracTickets for help on using tickets.