Opened 3 years ago

Closed 3 years ago

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

WW_strings sysoev.sergej

Reported by: Sergey Sysoev Owned by: Дмитрий Лапшин (lapshin)
Component: WW_strings Version:
Keywords: Cc:

Description


Change History (6)

comment:1 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Sergey Sysoev
Type: ожидается проверкаожидаются исправления

У тебя свои тесты падают, я точно могу проверять?) Но ничего не выводят. Подумай, почему.

Отступы перемешанные (табы и пробелы).

Вввах как тесты написаны. Слишком сильно, столько не просили, но раз сделал то круто.

8/10.

comment:2 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Ай, я проглядел вывод одного своего теста, у тебя strcmp лажает с пустыми строчками. 7/10.

comment:3 Changed 3 years ago by Sergey Sysoev

Owner: changed from Sergey Sysoev to Дмитрий Лапшин (lapshin)
Type: ожидаются исправленияожидается проверка
Version: 2.0

Кажется, падали у меня не тесты, а я не указал return 0 в конце main и вообще main у меня оказался void. В C++ можно было не писать в конце return 0, gcc делал это за меня...

Отступы поправил, это я решил попробовать написать всё в nano вместо CLion, ещё и по ssh через windows terminal. При копировании табы превращались в пробелы.

Дописал тесты на пустые строки в strcmp и поправил ошибку, доигрался с однострочниками.

Заодно поправил всё, на что указала ide(какой-то тест я прогонял несколько раз подряд).

comment:4 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Sergey Sysoev
Type: ожидается проверкаожидаются исправления
Version: 2.0

Текущая версия strcmp прямо UB! Очень страшно. 9.

comment:5 Changed 3 years ago by Sergey Sysoev

Owner: changed from Sergey Sysoev to Дмитрий Лапшин (lapshin)
Type: ожидаются исправленияожидается проверка

Изменил

return *(int *) s1 - *(int *) s2;

на

return (int) *s1 - (int) *s2;

comment:6 Changed 3 years ago by Дмитрий Лапшин (lapshin)

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

Четыре звёздочки, а какой эффект! Но и каст к инту не нужен, char при вычислениях расширяется до int. 10.

Note: See TracTickets for help on using tickets.