Opened 5 years ago

Closed 5 years ago

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

WW # 2

Reported by: luchinin.aleksey Owned by: Дмитрий Свиридкин
Component: WW_strings Version: 3.0
Keywords: Cc:

Description


Change History (6)

comment:1 Changed 5 years ago by luchinin.aleksey

Component: WW_makeWW_strings

comment:2 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to luchinin.aleksey
Type: ожидается проверкаожидаются исправления

Контрольный набор тестов не пройден: strcmp возвращает неверный результат.

Проверте еще раз с пустой строкой; когда одна строка является началом другой; проверьте с разным порядком аргументов.

  1. У вас отсутствуют тесты для strlen.
  1. Передавать в тестирующие функции вспомогаетельные буфферы -- не очень хорошая практика. Лучше, чтоб функции сами внутри себя выделяли все, что им нужно. И очищали.
  1. В заголовках лучше указывать имена аргументов.
  1. Имена переменным следует давать осмысленные, а не однобуквенные.

comment:3 Changed 5 years ago by luchinin.aleksey

Owner: changed from luchinin.aleksey to Дмитрий Свиридкин
Type: ожидаются исправленияожидается проверка
Version: 1.02.0

Попытался исправить недочёты

comment:4 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to luchinin.aleksey
Type: ожидается проверкаожидаются исправления

Тесты пройдены

  1. Как думаете, сможет ли кто-либо сразу угадать, откуда и куда копируются строки, если в аргументах strcpy будут ваши имена аргументов, а const -- отсутствовать? Подумайте над этим и сделайте хорошо. Во всех функциях.
  1. strcmp позволяет определить упорядоченность строк. В тестах это совершенно не как не отражено.
  1. Точно ли нужен библиотечных strcpy для проверки вашего?

comment:5 Changed 5 years ago by luchinin.aleksey

Owner: changed from luchinin.aleksey to Дмитрий Свиридкин
Type: ожидаются исправленияожидается проверка
Version: 2.03.0

Попытался исправить недочёты

comment:6 Changed 5 years ago by Дмитрий Свиридкин

Resolution: задача сдана
Status: assignedclosed
Note: See TracTickets for help on using tickets.