Как сделать эту функцию не такой страшной?

Question

[v- smerti]

func dataValid(result []string) bool {
	if v.IsLogin(result[0]) == true {
		if isLoginAvailable(result[0]) == true {
			if v.IsEmail(result[1]) == true {
				if isEmailAvailable(result[1]) == true {
					return v.IsPass(result[2])
				} else {
					return false
				}
			} else {
				return false
			}
		} else {
			return false
		}
	} else {
		return false
	}
}

Какие будут советы по исправлению этого говнокода?
Зарание спасибо

Comments

[Oleg Shevelev]

Сам этот говнокод написал сам его и разгребай:)

[v- smerti]

Oleg Shevelev: хех)) А зачем если тут толпы людей за отметку могут вместо тебя провести рефакторинг ?))

[Oleg Shevelev]

Владимир Грабко: отметки такого рода не конвертируются. Конвертируются только полезные ответы на полезные и сложные вопросы, где можно чему-то научиться.

[v- smerti]

Oleg Shevelev: я не те отметки имел введу. Я про "отметить решением"

[Oleg Shevelev]

Владимир Грабко: я про них же. Они бесполезны.

[Oleg Shevelev]

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

Answer 1

[alexxandr]

переписать на С :)
if (v.IsLogin(result[0]) && isLoginAvailable(result[0]) && v.IsEmail(result[1]) && isEmailAvailable(result[1])) {
return return v.IsPass(result[2]) }

Comments

[Oleksiy Chechel]

И чем это отличается от аналогичного кода в go?

[LeEnot]

Олексiй Чечель: я полагаю, это сарказам был )

Answer 2

[Oleksiy Chechel]

func dataValid(result []string) bool {
	return v.IsLogin(result[0]) && isLoginAvailable(result[0]) &&
		v.IsEmail(result[1]) && isEmailAvailable(result[1]) &&
		v.IsPass(result[2])
}

Comments

[v- smerti]

бред. Оно вернёт не 1 значение а значение отработк каждой из функций

[Oleksiy Chechel]

Владимир Грабко: Я не обижаюсь на такие выпады) Перечитай свой комментарий через годик-другой, посмеёшься)))
Каждая из функций возвращает булево значение. Логический оператор && вернет true только при условии, что каждая из функций вернет true. Если хоть одна из них вернет false, то и общий результат будет false. Вот такая простая физика.

[v- smerti]

сори. не знал.

Answer 3

[Олег Цилюрик]

func dataValid( result []string ) bool {
   if v.IsLogin(result[0]) && isLoginAvailable(result[0]) && v.IsEmail(result[1]) && isEmailAvailable(result[1]) {
      return v.IsPass(result[2])
   } else {
      return false
   }
}

Comments

[Oleksiy Chechel]

Можно else убрать)

Answer 4

[uvelichitel]

func (data Data) IsValid() bool{
    return v.IsLogin(data[0]) && isLoginAvailable(data[0]) && v.IsEmail(data[1]) && isEmailAvailable(data[1]) && v.IsPass(data[2]) 
}
// fmt.Println(Data(result).IsValid())

Answer 5

[Никита]

Зачем вообще else? И булевы функции можно подставлять в if без операторов сравнения.

func dataValid(result []string) bool {
  if v.IsLogin(result[0])  {
    if isLoginAvailable(result[0])  {
      if v.IsEmail(result[1])  {
        if isEmailAvailable(result[1])  {
          return v.IsPass(result[2])
        }
      }
    }
  }
  return false
}

Comments

[Oleksiy Chechel]

Зачем вообще if?

Answer 6

[Oleg Shevelev]

func dataValid(result []string) bool {
  if v.IsLogin(result[0]) == true {

Вы уверены что там всегда будут прилетать значения в функцию?:) Прилетит что-нибудь пустое и вуаля...

Comments

[v- smerti]

да. Пустое не может прилететь.

[Oleg Shevelev]

Владимир Грабко: ну да ну да...

[v- smerti]

Oleg Shevelev: там перед передачей данных сюда идёт проверка данных. :D так что я уверен на 99%

[Oleg Shevelev]

Владимир Грабко: ну это плохое решение, однозначно не годится для пакетного кода. А в Golang почти весь код бьётся на пакеты.

Answer 7

[Виталий Яковенко]

Или же, если нет желания делать всё в одну строку - простейший вариант с ранними return:

if !v.IsLogin(result[0]) {
  return false;
}
if !isLoginAvailable(result[0]) {
  return false;
}
if !isEmailAvailable(result[1]) {
  return false;
}
return v.IsPass(result[2])