Opened 3 years ago

Closed 3 years ago

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

WW #5

Reported by: antonenko.mixail Owned by: Святослав Власов
Component: WW_c_io Version: 2.0
Keywords: Cc:

Description


Change History (7)

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

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

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

  1. Точки выводятся не в том порядке.

Стиль -- 1.5/3

  1. point_list.h зависит от clist.h но не включает его
  2. Функции-однострочники -- плохой паттерн, давай не будем его придерживаться
  3. Magic Numbers -- это плохо. Если бы я не зная задания читал твой код, я бы не понял что вот тут происходит:
    s[2] & (1 << 7) ? 0xFF : 0
    

Напиши функцию is_negative и перепиши обработку отрицательных чисел так, чтобы читая код можно было понять что в нем проиходит.

  1. В point_list.h/cc должны быть только функции работы со списком точек. Функции сохранения/загрузки точек в файл нужно реализовать в main.c

comment:2 Changed 3 years ago by antonenko.mixail

  1. Не понял, где в задании написано, в каком порядке должны выводиться точки.

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

Replying to antonenko.mixail:

  1. Не понял, где в задании написано, в каком порядке должны выводиться точки.

Пример работы есть в задании. Посмотри на него и сравни с тем как у тебя программа работает.

comment:4 Changed 3 years ago by antonenko.mixail

  1. Ну, не знаю, насколько это честно по отношению к бедному студенту. Но я перепишу, если вы говорите, ведь иначе меня отчислят.
  1. Надеюсь, я правильно понял, что "функциии работы со списком точек" --- это те, что скармливаются функции apply. Просто их довольно мало.

comment:5 Changed 3 years ago by antonenko.mixail

Ну и понятно, почему мало. Видимо, не совсем то, что нужно, я делаю.

comment:6 Changed 3 years ago by antonenko.mixail

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

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

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

12/12

Note: See TracTickets for help on using tickets.