Opened 5 years ago

Closed 5 years ago

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

WW_2

Reported by: Maria Chizhova Owned by: Egor Suvorov
Component: WW_strings Version: 3.0
Keywords: Cc:

Description


Change History (10)

comment:1 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to Maria Chizhova
Type: ожидается проверкаожидаются исправления

Тесты проходит, лайк, остаток по корректности:

  1. Название исполняемого файла — lab_02, а не lab2.
  2. У obj/test.o неверные зависимости в Makefile.
  3. Не хватает ключевых слов restrict в сигнатурах — по требованию в задании его надо оставлять.
  4. test_my_strlen проверяет не my_strlen.
  5. Нулевой символ '\0 в строковых литералах никогда писать не надо — он добавляется автоматически и всегда означает конец строки в стиле Си.

По стилю:

  1. Объявляете и определяйте везде функции в одном и том же порядке. У вас сейчас, как минимум, отличается в str.h и test_str.h. А ещё лучше — в том же порядке, в каком они идут в задании и в стандарте.
  2. #include "string.h" — кажется, это стандартный заголовок. Надо включать как <string.h>
  3. В test_my_strcat и test_my_strcpy лучше не пользоваться my_strcmp, а честно проверять все элементы массива по одному. Заодно явно покажете, что вы проверяете нулевой символ.

comment:2 Changed 5 years ago by Maria Chizhova

Owner: changed from Maria Chizhova to Egor Suvorov
Version: 1.02.0

comment:3 Changed 5 years ago by Egor Suvorov

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

Пожалуйста, изменяйте Type на "ожидается проверка"

comment:4 Changed 5 years ago by Egor Suvorov

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

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

  1. Зависимости от заголовков в Makefile не соответствует реально включаемым заголовкам.
  2. В задании запрещено использовать [].

Стиль:

  1. Следующие порядки должны совпадать: порядок определений my_*, порядок объявлений my_*, порядок описания этих функций в стандарте, порядок этих функций в задании, порядок объявлений тестовых функций, порядок определений тестовых функций, порядок вызовов тестов.
  2. В тестах лучше не используйте my_strcmp/my_strlen/strcmp/strlen для проверки. Проверяйте прямо содержимое массивов напрямую.

comment:5 Changed 5 years ago by Maria Chizhova

Owner: set to Egor Suvorov
Type: ожидаются исправленияожидается проверка
Version: 2.03.0

comment:6 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to Maria Chizhova
Type: ожидается проверкаожидаются исправления

У меня падают ваши тесты (может, падали и раньше, но я не запускал):

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.
Assertion failed!

File: src/test_str.c, Line 35

Expression: *(s1 + 5) == '\0'

Подсказка: UB.

comment:7 Changed 5 years ago by Maria Chizhova

Owner: changed from Maria Chizhova to Egor Suvorov
Type: ожидаются исправленияожидается проверка

comment:8 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to Maria Chizhova
Type: ожидается проверкаожидаются исправления

Корректность полностью есть, поздравляю!

Стиль — 2/3:

  1. Скобочки в min не нужны.
  2. Ставьте пробел перед фигурной скобочкой (в тестах не стоит).
  3. Не нужен <malloc.h>
  4. n и m путаются. Назовите их так, чтобы их названия были как-то связаны с названиями строк, длину которых они хранят. Например, dest_len или len1 (в зависимости от функции).

comment:9 Changed 5 years ago by Maria Chizhova

Owner: changed from Maria Chizhova to Egor Suvorov
Type: ожидаются исправленияожидается проверка

comment:10 Changed 5 years ago by Egor Suvorov

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

10/10, успех!

Note: See TracTickets for help on using tickets.