diff --git a/hw10_program_optimization/README.md b/hw10_program_optimization/README.md index 939cbb4..1bd1c42 100644 --- a/hw10_program_optimization/README.md +++ b/hw10_program_optimization/README.md @@ -30,6 +30,13 @@ GetDomainStat(r, "edu") // {"quinu": 1} - удалять имеющийся лишний код (кроме функции `GetDomainStat`); - добавлять юнит-тесты. +**Обратите внимание на запуск TestGetDomainStat_Time_And_Memory** +```bash +go test -v -count=1 -timeout=30s -tags bench . +``` + +Здесь используется билд-тэг bench, чтобы отделить обычные тесты от тестов производительности. + ### Критерии оценки - Пайплайн зелёный и нет попытки «обмануть» систему - 4 балла - Добавлены юнит-тесты - до 3 баллов diff --git a/hw10_program_optimization/stats.go b/hw10_program_optimization/stats.go index b871102..5cee6f1 100644 --- a/hw10_program_optimization/stats.go +++ b/hw10_program_optimization/stats.go @@ -5,7 +5,6 @@ import ( "fmt" "io" "io/ioutil" - "regexp" "strings" ) @@ -29,38 +28,33 @@ func GetDomainStat(r io.Reader, domain string) (DomainStat, error) { return countDomains(u, domain) } -type users [100000]User +type Users []User -func getUsers(r io.Reader) (result users, err error) { +func getUsers(r io.Reader) (Users, error) { + result := make([]User, 10000) content, err := ioutil.ReadAll(r) if err != nil { - return + return result, err } lines := strings.Split(string(content), "\n") - for i, line := range lines { + for _, line := range lines { var user User if err = json.Unmarshal([]byte(line), &user); err != nil { - return + return result, err } - result[i] = user + result = append(result,user) } - return + return result, err } -func countDomains(u users, domain string) (DomainStat, error) { +func countDomains(u Users, domain string) (DomainStat, error) { result := make(DomainStat) - + i:=0 for _, user := range u { - matched, err := regexp.Match("\\."+domain, []byte(user.Email)) - if err != nil { - return nil, err - } - - if matched { - num := result[strings.ToLower(strings.SplitN(user.Email, "@", 2)[1])] - num++ - result[strings.ToLower(strings.SplitN(user.Email, "@", 2)[1])] = num + if strings.Contains(user.Email,"."+domain) { + result[strings.ToLower(strings.SplitN(user.Email, "@", 2)[1])]++ + i++ } } return result, nil