Func KeywordsChecker(word string, amount int, positives, negatives []string) (bool, error)

Question

Func KeywordsChecker(word string, amount int, positives, negatives []string) (bool, error)

{
hmap := make(map[string]int8, len(positives)+len(negatives))
for _, word := range positives {
hmap[word] = 1
}
for _, word := range negatives {
hmap[word] = -1
}
word = strings.ToLower(word)
words := strings.Split(word, " ")
var matched int
for _, word := range words {
if n, ok := hmap[word]; ok {
matched += int(n)
}
}
return matched >= amount, nil
} Можно как-то улучшить эту функцию?

#backend #go #programming #russian

0

15.11.2021

7 ответов

4 просмотра

Igor Shekalev

Ну и особый котёл в аду за использование локальных переменных с тем же именем, что и параметр функции.

0

15.11.2021

Roman Sharkov

что значит KeywordsChecker? во-первых название функции не должно быть существительным, оно должно быть императивным (команда) например append (добавь), make (сделай), delete, recover, print из названия не понятно что эта функция делает и зачем она нужна

0

15.11.2021

Igor Shekalev

Roman Sharkov
что значит KeywordsChecker? во-первых название фу...

Не всегда команда, тут больше похоже на IsWordPositive. Результат boolean.

0

15.11.2021

Roman Sharkov

Igor Shekalev
Не всегда команда, тут больше похоже на IsWordPosi...

не всегда конечно, примеры тому new, real, len, cap

0

15.11.2021

Стопосто

Igor Shekalev
Не всегда команда, тут больше похоже на IsWordPosi...

там вообще sentence а не word

0

15.11.2021

Roman Sharkov

Igor Shekalev
Не всегда команда, тут больше похоже на IsWordPosi...

но KeywordsChecker или даже CheckKeywords это вообще не о чём, совершенно не понятно зачем оно нужно

0

15.11.2021

Igor Shekalev · Accepted Answer

Igor Shekalev

Если эта функция вызывается много раз для разных слов, но с одинаковыми negatives и positives, то решение кривовато, map каждый раз заполняется. Ну и single responsibility страдает. Оно не только считает, но и данные для подсчёта формирует.

0

15.11.2021

170 похожих чатов

Func KeywordsChecker(word string, amount int, positives, negatives []string) (bool, error)

7 ответов

Похожие вопросы