Opened 5 years ago

Closed 5 years ago

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

WW #2 obryadina.aleksandra

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

Description


Change History (5)

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

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

Стиль и логика:

  1. Как много пустых строк. Они действительно нужны в таком числе?
  2. Файлы с реализацией не тянут свои же заголовки. Это полезно делать.
  3. Непосредственно стиль:
    1. Ничего между именем функции и аргументами (int foo(int n);, int foo(int n) {).

Корректность:
Работает, но сигнатуры функций не соответствуют стандарту Си.

Пока это 8/10. Хорошо!

comment:2 Changed 5 years ago by Obryadina Alexandra

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

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

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

Корректность:
Работает, но сигнатуры функций не соответствуют стандарту Си.

Подсказка: возвращаемый тип strlen.

Стиль:
Сделала лучше, но есть недостаточки:

  1. Дублирование кода в strcat.
  2. Два предложения в одной строчке это нехорошо (s1++, s2++;).

Всё ещё 8.

comment:4 Changed 5 years ago by Obryadina Alexandra

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

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

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

10.

Note: See TracTickets for help on using tickets.