Александр Павлюк

Никак не проверить, только цикл.
В го пока нет дженериков, поэтому такие функции отсутствуют в стандартной библиотеке.

По-первых, когда вы создаете буферный ридер и читаете строку bufio.NewReader(c).ReadString('\n'), то ридер на самом деле может прочитать гораздо больше, чем одна строка, на то он и буферный.
Если хотите использовать bufio, то создавайте ридер один раз и дальше передавайте его везде по коду и читайте только из него, а не из net.Conn

Во-вторых, как-то вы очень сложно написали получение файла, ошибка скорее всего именно там.
Это все лучше делать средствами стандартной библиотеки:

limited := io.LimitReader(c, file_size)
err = io.Copy(newfile, limited)

Способ 1: как вы и хотите, переиспользование функции. Пишем функцию, которая возвращает функцию less
https://play.golang.org/p/x_zDfZfwm_Q

func main() {
	arr := []byte{4, 3, 2, 1, 6, 3, 77, 8, 3}
	sort.Slice(arr, bytesAsc(arr))
	fmt.Println(arr)
}

func bytesAsc(arr []byte) func(i, j int) bool {
	return func(i, j int) bool {
		return arr[i] < arr[j]
	}
}

Способ 2: создаем алиас типа, у которого прописаны нужные методы.
https://play.golang.org/p/ZNkOt_jSUbr

func main() {
	arr := []byte{4, 3, 2, 1, 6, 3, 77, 8, 3}
	sort.Sort(BytesAsc(arr))
	fmt.Println(arr)
}

type BytesAsc []byte

func (a BytesAsc) Len() int           { return len(a) }
func (a BytesAsc) Swap(i, j int)      { a[i], a[j] = a[j], a[i] }
func (a BytesAsc) Less(i, j int) bool { return a[i] < a[j] }

Сразу происходит копирование.

unreachable code можно смело убрать
У вас же и так возвращается ошибка в случае отмены контекста:

case <-ctx.Done():
      return nil, ctx.Err()

Но нужно понимать, что контекст будет по таймауту отменяться только если этот таймаут контексту задан.
Например:

ctx, cancel := context.WithTimeout(context.Background(), time.Second)
defer cancel()
resp, err := srv.RPCClient.Call(ctx, topic, payload)
if err != nil { // тут вернет ошибку "context cancelled" при таймауте
  srv.logger.Error("RPC call error. ", err)
  return err
}

Потому что проверка на удовлетворение интерфейсу производится над одним значением, а не над коллекциями.
Если вы возвращаете из функции типа []IA, то и переменная, которую вы возвращаете, должна быть типа []IA. Но никто вам не помешает положить в этот массив элементы типа a.

https://play.golang.org/p/t7VMS0KQDas

func getA() []IA {
	var arr = make([]IA, 10)
	arr[0] = a{}
	return arr
}

Можно например так:

package main

import (
	"errors"
	"fmt"
	"strings"
)

type validator interface {
	Validate() error
}

type FieldEmail string

func (fi *FieldEmail) Validate() error {
	if fi == nil {
		return errors.New("invalid field")
	}

	if !strings.Contains(string(*fi), "@") {
		return errors.New("invalid field")
	}

	return nil
}

func NewFieldEmail(str string) *FieldEmail {
	fieldEmail := FieldEmail(str)
	return &fieldEmail
}

func checkValidators(validators ...validator) error {
	for _, v := range validators {
		if v == nil {
			continue
		}
		if err := v.Validate(); err != nil {
			return err
		}
	}
	return nil
}

func main() {
	fmt.Printf("Error: %v\n", checkValidators(NewFieldEmail("test@test")))

	var email *FieldEmail
	fmt.Printf("Error: %v\n", checkValidators(email))
}

Во-первых, лучше контекст все же брать не Background, а из http-запроса. Таким образом вы будете корректно обрабатывать ситуацию, когда соединение с клиентом разорвалось и не надо больше обрабатывать запрос (контекст отменится).

ctx, cancel := context.WithTimeout(r.Context(), config.Read().HTTP.Timeout.Request*time.Second)

Во-вторых, контекст обычно не используют для ожидания внутренних горутин, он не для этого предназначен. Для этого используют sync.WaitGroup.
Поэтому у вас и возникает проблема, ибо событие отмены контекста еще не значит, что дочерние функции мгновенно завершились. Возникает ситуация, когда чтение канала <-ctx.Done() внутри приведенной вами функции происходит раньше, чем такое же чтение канала внутри функции router.middlewareHandler(h). Поэтому нужно ждать не отмены контекста, а завершения работы горутины.

В-третьих, вам тут вообще не нужна горутина, ибо ожидание ее завершения вы сделали сразу после нее. Вместо этого можно просто написать последовательный код.

В-четвертых, вы, наверное, хотели написать if ctx.Err() != nil {. Но у вас вместо этого ==, что вообще обесценивает код внутри этого ифа.

City = s.Find(".list-jobs__details__info > .icon-location").Text()

чтобы без буфера было, нужно вот этот кусок

body, err := ioutil.ReadAll(resp.Body)
    if err != nil {
        http.Error(w, "error reading response body", http.StatusInternalServerError)
        return
    }

    // write status code and body from proxy request into the answer
    w.WriteHeader(resp.StatusCode)
    w.Write(body)

заменить на этот:

w.WriteHeader(resp.StatusCode)
_, err := io.Copy(w, body)
if err != nil {
    http.Error(w, "error reading response body", http.StatusInternalServerError)
    return
}

Нет, RemoveClass удаляет класс из элементов, а не удаляет элементы с этим классом.
Вам нужна функция Not()

longDescription = s.Find(".vacancy-post").Not(".vacancy-page-title").Text()

Тримы работают вполне нормально https://play.golang.org/p/31OXind1eyt

str := "UserName: Vasya Pupkin"
result := strings.TrimPrefix(str, "UserName: ")
fmt.Println(result)

Почитайте что такое область видимости https://golangs.org/variable-scope
Чтобы переменная city (переменные принято называть с маленькой буквы) была видна внутри Each (76 строчка), вы должны переменную объявить внутри этой же функции, а не во вложенной.
Например, на строке 87 написать
var city string
или же, как у вас закомментировано:
city := ""

Внутри вложенной функции уже будете использовать не :=, а просто =
Потому что равно с двоеточнием создает новую переменную, а вам надо просто присвоить значение в уже существующую переменную.