Opened 3 years ago

Closed 3 years ago

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

WW_mergesort (lab_03) Korkin

Reported by: korkin.igor Owned by: Антон Филатов
Component: WW_mergesort Version: 2.0
Keywords: Cc:

Description

Оно не работает и не должно.

Change History (6)

comment:1 Changed 3 years ago by Антон Филатов

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

Нет, это пустая посылка. Когда я говорил, что вы должны заслать хоть что-то, я имел ввиду что-то, что хоть немного близко к решению. Смотрите, прошла неделя, а вы, по сути, не сделали ничего. В субботу вы быстренько запушите решение, которое у меня почему-то не скомпилируется из-за забытой случайно точки с запятой, я поставлю вам 0, а вы будете ругаться, что какой препод глупый, не смог один символ поправить, чтобы не ставить 0.
Я не хочу возникновения подобных ситуаций, поэтому требую, чтобы к первой посылке было послано хотя бы не компилируемое нечто, но которое делает приблизительно то, что нужно.

Эта посылка считается пустой. Первое предупреждение. В будущем такие вещи будут приравниваться к той ситауции, когда вы не запушили ничего и не создали тикет

comment:2 Changed 3 years ago by korkin.igor

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

Мое абсолютно непрошенное мнение относительно этой ситуации: ваши условия на сдачу работ все-таки стоит озвучивать, когда вы даете работу, а не когда кто-то эти условия не выполнил.

И да, я хотел попросить, чтобы вы не сразу закрывали тикет, если так вообще можно, потому что по прошлому проекту у меня 6 из 10, и я даже не знаю почему и не могу в будущем не допускать таких же ошибок. Возможно, вы это где-то в другом месте пишете, но я как-то не нашел.

comment:3 Changed 3 years ago by Антон Филатов

Мое абсолютно непрошенное мнение относительно этой ситуации: ваши условия на сдачу работ все-таки стоит озвучивать, когда вы даете работу, а не когда кто-то эти условия не выполнил.

Я говорил на лекции, что первая посылка должна быть спустя неделю. Я действительно не сказал, что в этой посылке должно быть что-то более осознанное, чем пустые файлы, о чём я и сказал в первом комментарии, после этого попросил вас больше так не делать, а затем на занятии сказал это ещё раз.
Я прошу прощения, если мой комментарий обидел вас, я не преследовал этой цели.

И да, я хотел попросить, чтобы вы не сразу закрывали тикет, если так вообще можно, потому что по прошлому проекту у меня 6 из 10, и я даже не знаю почему и не могу в будущем не допускать таких же ошибок. Возможно, вы это где-то в другом месте пишете, но я как-то не нашел.

Вот ваш прошлый тикет
https://trac.compscicenter.ru/spbsu/ticket/17
Он хоть и закрыт, но всё равно доступен, там же написано, почему 6. Скорее всего вам на почту должно было прийти уведомление об изменении статуса тикета вместе с моим комментарием по нему.
Я закрываю все тикеты, по которым прошёл дэдлайн, поскольку в противном случае тикеты будут копиться и мешать, отделять старые тикеты от новых будет сложно.
Если вы хотите найти закрытый тикет, чтобы прочитать мои комментарии, можете воспользоваться строкой поиска в правом верхнем углу сайта или перейти в раздел "view tikets -> All Tickets By Milestone (Including closed)"

Так, теперь по сути посылки

Make работает как-то косячно. Если папка obj уже создана, он не собирает проект (даже если там что-то изменилось), а выпадает с ошибкой. Это считается ошибкой стиля

По корректности замечаний нет, все тесты пройдены 7/7
Стиль:

  • кажется, что в mergesort.h не требуются все, указанные вами хедеры
  • кажется, что от копипасты в main можно избавиться

2/3 за стиль

Итого 9/10. Оценка окончательная
Пожалуйста, закройте тикет самостоятельно, как только прочитаете его, а также убедитесь, что даже после закрытия вы можете найти его по поиску. Будущие тикеты я буду сразу по наступлению дэдлайна после проверки

Last edited 3 years ago by Антон Филатов (previous) (diff)

comment:4 Changed 3 years ago by Антон Филатов

Бонусное задание не оценивается, поскольку по правилам вы должны набрать 10/10, чтобы бонус был оценён. Хотя по сути всё выполнено правильно

comment:5 Changed 3 years ago by Антон Филатов

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

comment:6 Changed 3 years ago by korkin.igor

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

Хорошо, все понятно понятно. На почту, кстати, уведомления об изменении тикетов не приходят. Как искать тикеты я понял.

Note: See TracTickets for help on using tickets.