Opened 3 years ago

Closed 3 years ago

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

WW_intrusive_list, Cherepanova Alyona, lab_04

Reported by: cherepanova.alyona Owned by: Антон Филатов
Component: WW_intrusive_list Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 3 years ago by Антон Филатов

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

Во всех тестах на экран выводится всё почти правильно.
Уберите, пожалуйста, лишний пробел в конце строки. И исправьте опечатку в слове unknown
Но на каждый тест ругается валгринд. Вот пример его вывода на тесте, где дёргаются все описанные вами функции:

==2988== error calling PR_SET_PTRACER, vgdb might block
==2988== Conditional jump or move depends on uninitialised value(s)
==2988==    at 0x4EAC071: _IO_vfscanf (vfscanf.c:1021)
==2988==    by 0x4EB7FD7: __isoc99_scanf (isoc99_scanf.c:37)
==2988==    by 0x108A59: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4EA8FDD: _IO_vfscanf (vfscanf.c:1103)
==2988==    by 0x4EB7FD7: __isoc99_scanf (isoc99_scanf.c:37)
==2988==    by 0x108A59: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4EAA942: _IO_vfscanf (vfscanf.c:1188)
==2988==    by 0x4EB7FD7: __isoc99_scanf (isoc99_scanf.c:37)
==2988==    by 0x108A59: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4FA5D60: __strcmp_ssse3 (strcmp.S:144)
==2988==    by 0x108A6C: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4FA5D60: __strcmp_ssse3 (strcmp.S:144)
==2988==    by 0x108A87: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4C33DC4: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2988==    by 0x108A87: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4FA5D60: __strcmp_ssse3 (strcmp.S:144)
==2988==    by 0x108AD0: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4FA5D60: __strcmp_ssse3 (strcmp.S:144)
==2988==    by 0x108B1C: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4FA5D60: __strcmp_ssse3 (strcmp.S:144)
==2988==    by 0x108B44: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4FA5D60: __strcmp_ssse3 (strcmp.S:144)
==2988==    by 0x108B6C: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4C33DC4: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2988==    by 0x108B6C: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4C33DC4: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2988==    by 0x108B1C: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4C33DC4: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2988==    by 0x108AD0: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4C33DC4: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2988==    by 0x108B44: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== Use of uninitialised value of size 8
==2988==    at 0x4C33DC4: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2988==    by 0x108A6C: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988==
==2988== HEAP SUMMARY:
==2988==     in use at exit: 40 bytes in 2 blocks
==2988==   total heap usage: 8 allocs, 6 frees, 1,160 bytes allocated
==2988==
==2988== 40 (16 direct, 24 indirect) bytes in 1 blocks are definitely lost in loss record 2 of 2
==2988==    at 0x4C2FB0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2988==    by 0x108C89: init_list (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==    by 0x108A41: main (in /home/wwu/cpp20/cpp20/cherepanova.alyona/lab_04/lab_04)
==2988==
==2988== LEAK SUMMARY:
==2988==    definitely lost: 16 bytes in 1 blocks
==2988==    indirectly lost: 24 bytes in 1 blocks
==2988==      possibly lost: 0 bytes in 0 blocks
==2988==    still reachable: 0 bytes in 0 blocks
==2988==         suppressed: 0 bytes in 0 blocks

Помимо утечки памяти, он скорее всего ругается на вашу переменную char* s из main, под которую вы не выделили память.
Эти ошибки фиксятся быстро, но по формальным требованиям я, к сожалению, вынужден поставить вам 0 за корректность

стиль:

  • make пересобирает что-то даже если ничего не изменилось
  • кажется, что слишком много пустых строчек в main.c
  • в clist.h два раз затесались include guard-ы
  • лишний #include "assert.h" в clist.c

Стиль: 2,5/4
Итого: 2,5/12

comment:2 Changed 3 years ago by cherepanova.alyona

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

comment:3 Changed 3 years ago by Антон Филатов

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

Тесты пройдены! 8/8 за корректность

Стиль:

  • make по прежнему делает что-то даже если ничего не изменилось в проекте
  • рекомендую вместо head -> data писать head->data. Вы же вокруг точки (если бы там была точка, а не стрелочка) не ставите пробелы

3,5/4

Итого: 11,5/12

Note: See TracTickets for help on using tickets.