Как то можно красиво написать if-ы?

Question

[nurdus]

Добрый день.
Глупый вопрос: можно ли красиво написать (убрав вложенность)?!

if (socket.request.user !== undefined) {
  if (socket.request.user.id !== undefined) { 
    // do something...

    } else {
    console.log('error 1');
  }
} else {
  console.log('error 2');
}

Comments

[Ярослав Иванов]

(socket.request.user !== undefined) ? (socket.request.user.id !== undefined) ? do something... :  console.log('error 1') : console.log('error 2');

Answer 1

[Anton Mashletov]

if (socket.request.user === undefined) {
    console.error('error'); // или throw
    return;
}

if (socket.request.user.id === undefined) {
    console.error('error'); // или throw
    return;
}

// todo

Comments

[M-ka]

НЕ ПРАВИЛЬНО
ошибка сценария при отсутствии user на поиск id

[Anton Mashletov]

M-ka, с чего это вдруг? Если user не найден вызовется return.

[M-ka]

Anton Mashletov, с того вдруг, если он не найдет, закрешется все...
было бы if else if, тогда бы вопрос не возникал, но отсутствует else...

[Anton Mashletov]

M-ka, ты совсем что ли. Если socket.request.user не установлен, то выполнится console.error('error'); и, ВНИМАНИЕ НА ЭКРАН, return. После return скритп дальше выполяться не будет и НЕ будет проверки на id.

[M-ka]

Anton Mashletov, а где в начальном условии вообще идется о return? это то, что все начали выдавать за желаемое... потому наличие ретурна для себя обвел в коммент ибо он в изначальном правиле отсутствовал, что подразумевает вообще развитие до много чего и написали просто как пример, не углубляясь во то, каковой там вообще обработчик респонсов...

[Anton Mashletov]

M-ka, автор просил убрать вложенность. Я убрал.
Ты же в своем ответе, по сути, повторил тоже самое, что и автор. С той же вложенностью. В чем польза его?

Если у тебя таких проверок будет 50? 50-уровней if-else друг в друга? Посмотри как делают это во фреймах популярных - return something или exception ибо 99% дальше программа выполяться не сможет.

PS: А вообще, есть такой прекрасный паттерн NullObject.

[M-ka]

Если у тебя таких проверок будет 50? 50-уровней if-else друг в друга?

у меня их более 50 и реализация совсем другая....
могу подкинуть тему гугла, абстрактный класс, модульность, mv*(etc)... вообщем много можно разного сделать, вместо 50ти if для удобства реализации...
(ой, толь дочитал до конца твое сообщение, еще и про паттерн улыбнуло... но пока остаемся на прежнем месте, без знания того, что необходимо достичь, структура сохраняется...)

[Anton Mashletov]

M-ka, в чем смысл твоих сообщений мне? Отстань. Автор просил убрать вложенность.
Вложенность я убрал. Ошибки сценария тут быть не может, это бред. return/throw в if'ах обычная практика простой валидации. Автор не указал, что их нельзя использовать. Автора мой ответ устроил, в отличие от твоего. Не надо растекаться мыслью по древу.

Answer 2

[Rsa97]

if (socket.request.user === undefined) {
  console.log('error 2');
} else if (socket.request.user.id === undefined) { 
  console.log('error 1');
} else {
  // do something...
}

Answer 3

[Илья Герасимов]

Уже много хороших ответов, вставлю и свой кусочек, просто что бы было ;)

var result = socket.request.user && socket.request.user.id && (() => {
	// its okey!
	console.log('hello:', socket.request.user.name);

	return {
		// result
	}
})();

if (!result)
	return console.log('something wrong.')

Comments

[Stepanya]

Вам самому-то нравится?

[M-ka]

да и работать не везде будет подобное...

[Илья Герасимов]

эй эй полегче, это один из, а для решение проблем с "работать везде" есть babel или пишем по старинке. ну и это стандартная практика для react-темплейтов (jsx), которая даже в их документации приводится, что-то в этом духе:
{this.props.user && <SelfProfileComponent/>}
или так:

{this.props.user && this.props.user.rating > 50 && <MegaFunnyBlock/>}

Answer 4

[M-ka]

еще вариант

if ('user' in socket.request) {
   if ('id' in socket.request.user) {
      //ok
   } else {
      //err
   }
} else {
   //er
}

or
//вопрос в том, чего нужно добиться.... 
//без рабочего кода невозможно понять, что должно произойти, что бы оптимизировать
var response = 'err'
if ('user' in socket.request) {
   if ('id' in socket.request.user) {
   //response = ok
   } else {
   //response = errX
   }
}

or
 'user' in socket.request && 'id' in socket.request.user //ok?

//etc...