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
Owner: | changed from Egor Suvorov to Maria Chizhova |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 5 years ago by
Owner: | changed from Maria Chizhova to Egor Suvorov |
---|---|
Version: | 1.0 → 2.0 |
comment:3 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
Пожалуйста, изменяйте Type
на "ожидается проверка"
comment:4 Changed 5 years ago by
Owner: | Egor Suvorov deleted |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Корректность:
- Зависимости от заголовков в
Makefile
не соответствует реально включаемым заголовкам. - В задании запрещено использовать
[]
.
Стиль:
- Следующие порядки должны совпадать: порядок определений
my_*
, порядок объявленийmy_*
, порядок описания этих функций в стандарте, порядок этих функций в задании, порядок объявлений тестовых функций, порядок определений тестовых функций, порядок вызовов тестов. - В тестах лучше не используйте
my_strcmp
/my_strlen
/strcmp
/strlen
для проверки. Проверяйте прямо содержимое массивов напрямую.
comment:5 Changed 5 years ago by
Owner: | set to Egor Suvorov |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 2.0 → 3.0 |
comment:6 Changed 5 years ago by
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
Owner: | changed from Maria Chizhova to Egor Suvorov |
---|---|
Type: | ожидаются исправления → ожидается проверка |
comment:8 Changed 5 years ago by
Owner: | changed from Egor Suvorov to Maria Chizhova |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Корректность полностью есть, поздравляю!
Стиль — 2/3:
- Скобочки в
min
не нужны. - Ставьте пробел перед фигурной скобочкой (в тестах не стоит).
- Не нужен
<malloc.h>
n
иm
путаются. Назовите их так, чтобы их названия были как-то связаны с названиями строк, длину которых они хранят. Например,dest_len
илиlen1
(в зависимости от функции).
comment:9 Changed 5 years ago by
Owner: | changed from Maria Chizhova to Egor Suvorov |
---|---|
Type: | ожидаются исправления → ожидается проверка |
comment:10 Changed 5 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
10/10, успех!
Note: See
TracTickets for help on using
tickets.
Тесты проходит, лайк, остаток по корректности:
lab_02
, а неlab2
.obj/test.o
неверные зависимости вMakefile
.restrict
в сигнатурах — по требованию в задании его надо оставлять.test_my_strlen
проверяет неmy_strlen
.'\0
в строковых литералах никогда писать не надо — он добавляется автоматически и всегда означает конец строки в стиле Си.По стилю:
str.h
иtest_str.h
. А ещё лучше — в том же порядке, в каком они идут в задании и в стандарте.#include "string.h"
— кажется, это стандартный заголовок. Надо включать как<string.h>
test_my_strcat
иtest_my_strcpy
лучше не пользоватьсяmy_strcmp
, а честно проверять все элементы массива по одному. Заодно явно покажете, что вы проверяете нулевой символ.