sgkochnev / skills Goto Github PK

1. З кейса в свиче - это лишнее, попробуй уйти от этого и сделать проще

2. Пересоздание канала в функции fanIn - тоже лишняя работа, попробуй сделать без этого

1. Как уже говорила, вот такой стиль описания констант в го не используется, скорее в python
   _http

2. Loop также обычно не используется, лучше использовать

for{
    ...
    break
}

Но я поняла, зачем ты его заиспользовал, получилось довольно громоздко, лучше внутренний цикл в отдельную функцию выносить

if err == nil {

Лучше так не делать, иначе код будет вправо разрастаться. Если нужно проверить ошибку, то лучше сделать так:

func f1() {
...
    if err != nil {
        return err
    }
...
}

func f() {
...
    if err := f1(); err != nil {
        return errors.Wrapf(err, "function f1 returned error")
    }
...    
}

4. За количество строк, обрабатываемых параллельно, отвечает количество горутин
   А за количество тредов, в которых будут создаваться горутины отвечает переменная runtime.GOMAXPROCS, в нее не нужно прописывать значение N
   Вообще это значение лучше руками не указывать, а если указывать, то делать его равным ( количеству ядер процессора - 1)

5. Слишком много пересозданий каналов, давай попробуем обойтись одним
   Давай также попробуем использовать пулл воркеров и не создавать горутины при обработке каждой новой строки

6. Давай все же попробуем сделать так, чтобы 1ый тест тоже проходил, т.е. были ссылки по-порядку

7. Попробуй предусмотреть ситуацию, когда сайт, чью ссылку хотим проверить будет не отвечать на запросы