HW2 is completed #2
Conversation
timersha
force-pushed
the
hw02_unpack_string
branch
from
e03b2d4 to
324200a
Compare
| func Unpack(str string) (string, error) { | ||
| inputIndex := 0 | ||
| resultIndex := 0 | ||
| backslash := '\\' |
There was a problem hiding this comment.
Рекомендую для неизменяемых значений использовать константы
| resultIndex := 0 | ||
| backslash := '\\' | ||
| input := []rune(str) | ||
| result := make([]rune, 0, 200) |
There was a problem hiding this comment.
Для итоговой строки рекомендую использовать strings.Builder
| if utf8.RuneCountInString(string(input)) == 0 { | ||
| return "", nil | ||
| } |
There was a problem hiding this comment.
Лучше проверить на исходной строке и до всех инициализаций
| for inputIndex <= inputLen { | ||
| r := input[inputIndex] |
There was a problem hiding this comment.
Гораздо проще и надежнее анализировать текущий и запоминать предыдущий символы, без обращения по индексу
| func nextIndex(sl *[]rune, index int) int { | ||
| slLen := math.Max(0, float64(len(*sl)-1)) | ||
| nextIndex := index + 1 | ||
| return int(math.Min(float64(slLen), float64(nextIndex))) | ||
| } | ||
|
|
||
| func previousIndex(sl *[]rune, index int) int { | ||
| slLen := math.Max(0, float64(len(*sl)-1)) | ||
| previousIndex := index - 1 | ||
| return int(math.Min(float64(slLen), math.Max(0, float64(previousIndex)))) |
There was a problem hiding this comment.
Кажется, что в качестве аргумента можно использовать вычисленную длину и не передавать в функцию оригинальный слайс, ещё и по указателю.
Для поиска минимального и максимального значений, рекомендую использовать встроенные функции min и max
| {input: `qwe\4\5`, expected: `qwe45`}, | ||
| {input: `qwe\45`, expected: `qwe44444`}, | ||
| {input: `qwe\\5`, expected: `qwe\\\\\`}, | ||
| {input: `qwe\\\3`, expected: `qwe\3`}, |
There was a problem hiding this comment.
Будет здорово добавить собственных кейсов, расширяющих существующие сценарии тестирования
Домашнее задание №2 «Распаковка строки»
Чек-лист студента (Что это?)
go mod tidy..syncфайл. Зачем его удалять?Критерии оценки
Зачёт от 7 баллов