Opened 5 years ago

Closed 5 years ago

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

WW #2 golovin.valeriy

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

Description


Change History (8)

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

Owner: changed from Дмитрий Лапшин (lapshin) to golovin.valeriy
Summary: WW #2 → WW #2 golovin.valeriy

Не собирается. 0.

Подсказка: он не может найти include/str.h.

Кстати: svn status крайне полезен!

comment:2 Changed 5 years ago by Egor Suvorov

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

comment:3 Changed 5 years ago by golovin.valeriy

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

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

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

Ничего не изменилось!

comment:5 Changed 5 years ago by golovin.valeriy

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

Если до сих пор нет, то я не знаю.

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

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

А теперь норм.

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

  1. Неверное имя исполняемого файла программы.
  2. Заголовок strlen не соотвествует стандарту языка Си.
  3. strcpy возвращает неверное значение.
  4. strcat некорректна с замашками на неопределённое поведение.

Стиль:

  1. strcat: дублирование кода.
  2. str.c: подключить str.h?
  3. Конструкции вида 
    if () {
    return
} else if () {
    return
} else {
    return
}
    вообще говоря содержат избыточные else.
  4. А если не избыточные — перебор лучше по отступам писать как выше.
  5. Пробелы в объявлениях функций (int foo(int a, int b) {).
  6. Неконсистентные пробелы в #include.
  7. Зачем в main.c stdio.h?

Много-много недочётов. 4/10.

comment:7 Changed 5 years ago by golovin.valeriy

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

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

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

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

  1. Nice!
  2. main содержит UB.

Стиль:

  1. У вызовов функций между именем функции и аргументами обычно нет пробела.
  2. Неконсистентные пробелы в #include.

8/10.

Note: See TracTickets for help on using tickets.