Нормальный ли codestyle?

Question

[vetsmen]

Доброго времени суток, серверный обработчик сокет запроса:

socket.on('errorData', async (id) => {
			id = parseInt(id, 10);
			if(!socket.payload || !socket.payload.userid) {
				socket.emit('onError', 'Вы не авторизованы');
				return;
			}
			if(!id) {
				socket.emit('onError', 'Некорректные данные');
				return;
			}

			let item = await connection.query('SELECT * FROM UserData WHERE id = ?', [id]);

			if(!item || !item[0]) {
				socket.emit('onError', 'Некорректные данные');
				return;
			}

			if(!item[0].userid || (item[0].userid !== socket.payload.userid + '')) {
				socket.emit('onError', 'Некорректные данные');
				return;
			}

			let updateUserItems = await connection.query('UPDATE UserData SET ? WHERE id = ?', [{status: 1}, id]);

			socket.emit('errorData', {
				success: true,
				id: id
			});
		});

Нормальный ли codestyle? От callback-hell'a вроде бы получилось уйти, однако мне такой принцип все равно кажется неправильным.
Проблема в том, когда обработчик большой, когда очень много проверок всяких данных - получается около 70 строк такого кода, а то и больше. Есть какие-нибудь советы?

Comments

[sim3x]

Обработка авторизации, валидация входящих данных, рав скуль запрос, сообщения об ошибках с информативностью 0

А коде-стайл - ок
Правда зачем парится с кодестайлом, если там 2-3 общепринятых стандарта и выбор по сути между гуглстайлом и airbnb?

Answer 1

[kulaeff]

Не знаю, что там с кодом на 70 строк, но этот кусок вполне нормальный. Разве что пустые строки до и после if'ов поставить и у стрелочной функции убрать скобки, если параметр один. И вообще юзать eslint.