@vetsmen

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

Доброго времени суток, серверный обработчик сокет запроса:
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 строк такого кода, а то и больше. Есть какие-нибудь советы?
  • Вопрос задан
  • 167 просмотров
Решения вопроса 1
@kulaeff
Front-end developer
Не знаю, что там с кодом на 70 строк, но этот кусок вполне нормальный. Разве что пустые строки до и после if'ов поставить и у стрелочной функции убрать скобки, если параметр один. И вообще юзать eslint.
Ответ написан
Комментировать
Пригласить эксперта
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Похожие вопросы