Как исправить http: multiple response.WriteHeader calls?

Question

[v- smerti]

Есть функция

import (
	"api/kernel/httprouter"
	"api/kernel/status"
	"database/sql"
	"fmt"
	_ "github.com/go-sql-driver/mysql"
	"net/http"
)
func SelectMethods(w http.ResponseWriter, r *http.Request, ps httprouter.Params) {

	rows, err := DB.Query("SELECT * FROM dev_method WHERE name=?", ps.ByName("methods"))
	if err != nil {
		status.Code503(w, r)
	}
	defer rows.Close()
	bks := make([]*dev_method, 0)
	for rows.Next() {
		bk := new(dev_method)
		err := rows.Scan(&bk.id, &bk.id_menu, &bk.name, &bk.description)
		if err != nil {
			status.Code503(w, r)
		}
		bks = append(bks, bk)
	}
	if err = rows.Err(); err != nil {
		status.Code503(w, r)
	}
	i := 0
	fmt.Fprintf(w, "{")
	for _, bk := range bks {
		i++
		fmt.Fprintf(w, "'%s':['id_menu':'%s','name':'%s','description':'%s']", bk.id, bk.id_menu, bk.name, bk.description)
	}
	fmt.Fprintf(w, "}")

	if i == 0 {
		status.Code400(w, r)
	}
}

status.go
pastebin.com/zMXdargE
router.go
pastebin.com/QzrvJMCw
Если ps.ByName("methods") нету в бд то в терминал пишет

2015/10/23 15:48:11 http: multiple response.WriteHeader calls

Я подозреваю что надо в конструкции

if err = rows.Err(); err != nil {
		status.Code503(w, r)
	}

как то остановить дальнейшее выполнение функции.
Возникает три вопроса.
1. Как исправить фун-цию?
2. Как остановить выполнение любой фун-ции в любом месте?
3. Как избавиться от конструкции

i := 0
	fmt.Fprintf(w, "{")
	for _, bk := range bks {
		i++
		fmt.Fprintf(w, "'%s':['id_menu':'%s','name':'%s','description':'%s']", bk.id, bk.id_menu, bk.name, bk.description)
	}
	fmt.Fprintf(w, "}")

	if i == 0 {
		status.Code400(w, r)
	}

? зарание спасибо.

Comments

[Oleg Shevelev]

return
Поработайте над опрятностью кода, будет и вам проще и работать лучше. Когда формируете вопрос - предоставляйте все необходимые данные. О том что вы используете стандартный database/sql можно толко догадаться. О том что у вас за объект status глобальный тоже не известно. А так - допишите проверку как хотели.

[v- smerti]

Oleg Shevelev: если в каждый if с ошибкой добавить return то эффект == 0.

[Oleg Shevelev]

Владимир Грабко: Ретурн был по поводу "2. Как остановить выполнение любой фун-ции в любом месте?"

На самом деле ничего не мешает делать каскады ошибок, приведу пример. Есть некая функция которая работает с другими функциями и что-то делает, на выходе она либо выдаёт результат и отсутствие ошибки, либо собственно ошибку.

import "errors"

func MyFunc(param string) (i int, err error) {
a, err := Func1(param)
if err {
return
}
b, err := Func2(param)
if err {
return
}
if a > b {
i = 10
return
}
err = errors.New("a <= b")
return
}

При этом возвращаются уже определённые переменные и в любой части кода видно что будет в ошибке, а что в значении.

[v- smerti]

Oleg Shevelev: дополнил вопрос

Answer 1

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

Ошибка возникает потому что код ответа надо писать максимум один раз и перед записью тела ответа. Делайте return при ошибке, это прервёт функцию.
JSON лучше всего формировать с помощью модуля encoding/json, тогда избавитесь от этой некрасивой конструкции.

Comments

[v- smerti]

эта конструкция вообще проверяет выбралось ли что то из базы. Если в каждую проверку err != nil добавить return то ошибка та же.

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

Ошибка потому что вы пишете код ответа после записи в тело ответа. status.Code400(w, r) должен быть перед всеми fmt.Fprintf. i можно вообще убрать если проверять len(bks)

[v- smerti]

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