LINUX.ORG.RU

История изменений

Исправление den73, (текущая версия) :

У меня так: 

	var rd RegistrationData
	rd.Nickname = c.PostForm("nickname")
	rd.Registrationemail = c.PostForm("registrationemail")
	rd.Password1 = c.PostForm("password1")
	rd.Password2 = c.PostForm("password2")
	appErr := doRegistrationFormSubmit(c, &rd)
	if appErr == nil {
Ты предлагаешь так: 
	var user RegistrationData
	user.Nickname          = ...
	user.Registrationemail = ...
 	user.Password1         = ...
	user.Password2         = ...

        if appErr := doRegistrationFormSubmit(...), appErr == nil {
Отвечаю. RegistrationData - это не пользвоатель, а данные, которые аноним ввёл в форме регистрации. Пользователь тоже есть, и это другое - это тот, кто уже зарегистрировался. Так что rd - это название правильное в рамках моей раскладки понятий.

Насчёт if appErr := , - я это где-то подсмотрел и это достаточно популярно. Здесь хорошо то, что appErr определена только в рамках данного if , т.е. можно потом опять писать appErr :=

Но да, в определённых случаях громоздко. В случае, если помещается в одну строку, честно сказать, криминала не вижу.

Насчёт доки ты прав. Но я стараюсь, по «чистому коду», кодировать смысл функции в её названии, а линтер требует документацию для всех экспортов. В таких случаях я и пишу «на отженись». Что посоветуешь?

Исправление den73, :

У меня так: 

	var rd RegistrationData
	rd.Nickname = c.PostForm("nickname")
	rd.Registrationemail = c.PostForm("registrationemail")
	rd.Password1 = c.PostForm("password1")
	rd.Password2 = c.PostForm("password2")
	appErr := doRegistrationFormSubmit(c, &rd)
	if appErr == nil {
Ты предлагаешь так: 
	var user RegistrationData
	user.Nickname          = ...
	user.Registrationemail = ...
 	user.Password1         = ...
	user.Password2         = ...

        if appErr := doRegistrationFormSubmit(...), appErr == nil {
Отвечаю. RegistrationData - это не пользвоатель, а данные, которые аноним ввёл в форме регистрации. Пользователь тоже есть, и это другое - это тот, кто уже зарегистрировался. Так что rd - это название правильное в рамках моей раскладки понятий.

Насчёт if appErr := , - я это где-то подсмотрел и это достаточно популярно. Здесь хорошо то, что appErr определена только в рамках данного if , т.е. можно потом опять писать appErr :=

Но да, в определённых случаях громоздко. В случае, если помещается в одну строку, честно сказать, криминала не вижу.

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

Исходная версия den73, :

У меня так: 

	var rd RegistrationData
	rd.Nickname = c.PostForm("nickname")
	rd.Registrationemail = c.PostForm("registrationemail")
	rd.Password1 = c.PostForm("password1")
	rd.Password2 = c.PostForm("password2")
	appErr := doRegistrationFormSubmit(c, &rd)
	if appErr == nil {
Ты предлагаешь так: 
	var user RegistrationData
	user.Nickname          = ...
	user.Registrationemail = ...
 	user.Password1         = ...
	user.Password2         = ...

        if appErr := doRegistrationFormSubmit(...), appErr == nil {
Отвечаю. RegistrationData - это не пользвоатель, а данные, которые аноним ввёл в форме регистрации. Пользователь тоже есть, и это другое - это тот, кто уже зарегистрировался. Так что rd - это название правильное в рамках моей раскладки понятий.

Насчёт if appErr := , - я это где-то подсмотрел и это достаточно популярно. Здесь хорошо то, что appErr определена только в рамках данного if , т.е. можно потом опять писать appErr :=

Но да, в определённых случаях громоздко. В случае, если помещается в одну строку, честно сказать, криминала не вижу.