Opened 5 years ago

Last modified 4 years ago

#218 reopened ожидаются исправления

WW #2

Reported by: saldyrkina.alisa Owned by: Vasily Alferov
Component: WW_strings Version: 2.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 5 years ago by Vasily Alferov

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

Здорово. Пока 6/10.

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

4/7: минус балл за strcat и минус два за strcmp.

  • Твои же тесты не проходят.
    $ ./lab2 
    lab2: src/test_str.c:24: test_strcmp: Assertion `b1 == b2' failed.
    Aborted (core dumped)
    
  • size_t не работает, потому что он объявлен в stddef.h, нужно его подключить.
  • strlen, strcpy ОК.
  • strcat: забываешь поставить '\0' в конце, поэтому наши автотесты не проходят.
  • strcmp: не работает, но ты и сама в курсе. Причина: continue работает не так. Он просто сразу перескакивает на следующую операцию цикла. Однако если continue не случится, цикл всё равно продолжит выполняться. А то, что ты хотела сделать (продолжить *только* если условие правда), делается так:
    if (!(условие)) {
        break;
    }
    

То есть "выйти из цикла, если условие не выполнено".

Стиль.

2/3. По пробелам: они не нужны между названием функции и открывающей скобкой, но нужны после слов for, while, а также перед всеми фигурными скобками.

comment:2 Changed 5 years ago by saldyrkina.alisa

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

comment:3 Changed 5 years ago by Vasily Alferov

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

=(

comment:4 Changed 4 years ago by Vasily Alferov

Resolution: задача НЕ сдана
Status: closedreopened

comment:5 Changed 4 years ago by Vasily Alferov

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

Круто, гораздо лучше. Сейчас 9/10. Можешь закрыть, если не будешь исправлять.

Единственное замечание по корректности: strcmp. Кажется, например, строки "ab" и "ac" у тебя равны. Чуть-чуть запуталась.

Note: See TracTickets for help on using tickets.