Opened 5 years ago

Closed 5 years ago

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

WW #2

Reported by: zhemchuzhina.elizaveta Owned by: zhemchuzhina.elizaveta
Component: WW_strings Version: 2.0
Keywords: Cc:

Description


Change History (10)

comment:1 Changed 5 years ago by Дмитрий Свиридкин

Ок. Тесты пройдены.

  1. Переменным надо дать осмысленые имена вместо однобуквенных. Особенно в заголовках.
  1. Форматирование слегка поехало.
  1. Сохраняйте исходники в кодировке UTF-8
  1. Тесты нормального человека с использованием библиотечных функций для strcat и strcpy у вас проверяют не то что нужно :) Подумайте почему и используйте тесты курильщика :)

comment:2 Changed 5 years ago by Дмитрий Свиридкин

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

comment:3 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to zhemchuzhina.elizaveta

comment:4 Changed 5 years ago by zhemchuzhina.elizaveta

Я могу просто убрать тесты нормального человека, так как тесты курильщика у меня уже есть?

comment:5 Changed 5 years ago by zhemchuzhina.elizaveta

Owner: changed from zhemchuzhina.elizaveta to Дмитрий Свиридкин
Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

Вроде исправила форматирование: выровняла отступы и тд+ дала осмысленные имена переменным.

comment:6 Changed 5 years ago by Дмитрий Свиридкин

https://imgur.com/zUlBEwF

Форматирование вас все-таки пока побеждает :)

comment:7 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to zhemchuzhina.elizaveta
Type: ожидается проверка → ожидаются исправления

comment:8 Changed 5 years ago by zhemchuzhina.elizaveta

А что тут не так?Я правильно поняла, что итоговая оценка 9.8 и можно уже ошибки форматирования не выискивать?(это самое сложное в лабе, правда)

comment:9 Changed 5 years ago by Дмитрий Свиридкин

На скрине лишний уровень вложенности.

Ну как хотите:)

Поставьте clang-format и не мучайтесь. Под Windows тоже есть.

comment:10 Changed 5 years ago by Дмитрий Свиридкин

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.