Opened 5 years ago

Closed 5 years ago

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

WW #2 chistyakova.polina

Reported by: chistyakova.polina Owned by: Дмитрий Лапшин (lapshin)
Component: WW_strings Version:
Keywords: Cc:

Description


Change History (9)

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

Resolution: → задача НЕ сдана
Status: assigned → closed

Не в дедлайн.

В целом всё ок, при этом. Надо бы в файлы реализации инклудить свои заголовочные файлы, да и со стилем аккуратнее.

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

Resolution: задача НЕ сдана
Status: closed → reopened

Подожди, это вторая лаба?..

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

Component: WW_make → WW_strings
Status: reopened → assigned

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

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

Стиль:

  1. str.h: зачем stdio.h? main.c: зачем str.h?
  2. strcat: дублирование кода.
  3. Пробел между аргументами и началом функции (void foo() {).
  4. Пробелы вокруг условия управляющей конструкции (while (2 < 3) {).

Корректность:

  1. strcmp сыпется.

Неплохо, 6/10.

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

Version: 1.0

comment:6 Changed 5 years ago by chistyakova.polina

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

Стиль исправила, функцию починила, дублирование кода тоже

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

Owner: changed from Дмитрий Лапшин (lapshin) to chistyakova.polina
Summary: WW #2 → WW #2 chistyakova.polina
Type: ожидается проверка → ожидаются исправления

Корректность:

  1. Заголовок *strlen* не соответствует стандарту!
  2. Кстати, а где restrict?

А в остальном правильно и всё красиво, 9/10.

comment:8 Changed 5 years ago by chistyakova.polina

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

Исправила заголовок)
Но в стандарте нет restrict... (Я даже после этого специально ещё раз посмотрела)

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

Resolution: → задача сдана
Status: assigned → closed

Разглядел в main UB. Но это моя слепота, раньше бы сказал.

10.

Note: See TracTickets for help on using tickets.