Почему повторно не меняется страница на nodejs?

Question

[pavlyk]

Не могу понять один момент, написал обработку параметров url если пустой вывести index.html если нет то error.html

function onRequest (request, response) {
	var parse = urlutils.parse(request.url, true);
	view(parse.pathname, parse.query.text, function (render) {
		response.writeHead(200, {"Content-Type": "text/html"});
		response.write(render);
		response.end();
		// console.log('write ' + parse.pathname);
	});
}

Всё отрабатывает и всё казалось бы хорошо, но я заметил такую фичу , т.к. когда я захожу на сайт мне шлется первым ulr 127.0.0.1/ , а вторым 127.0.0.1/favicon.ico , когда приходит фавикон срабатывает функция что нужно показать error.html и в консоле по команде console.log у меня выводить подряд две строки
/
/favicon.ico
Но отображается правильная страница первая, ошибки нету.... вот не могу понять почему т.к. в коде я не где этот фавикон не проверяют и по идеи должна тогда быть ошибка.

Answer 1

[pavlyk]

function view(page, text, callback) {
	if (page === '/') {
		action = 'render';
		page = 'index';
	} else if (page === '/ajax') {
		translate(text, function (result) {
			callback(result);
		});
		return;
	} else {
		action = 'render';
		page = 'error';
	}
	if (action = 'render') {
		callback(fs.readFileSync('front/' + page + '.html'));
	}
}

Comments

[Alexander Litvinenko]

У вас в коде блокирующие операции.

[pavlyk]

Какие? Код вроде работает, вопрос лишь в том почему ответ приходит 2 раза но второй не отображается

[Alexander Litvinenko]

pavlyk: код работает, почитайте что такое блокирующие операции.

[pavlyk]

Alexander Litvinenko: получается что вызывается блокирующая операция readFileSync , она блокирует выполнение других операций, но после того как она выполнилась, другие операции тоже ведь должны выполниться. Мне сначала приходит первый html и выводиться на экран, а потом приходит html ошибки и он должен тоже вывестись на экран поверх первого, разве нет?

[Alexander Litvinenko]

pavlyk: ощибка ведь приходит на фавикон, который часть другой страницы, следовательно вы ее видите только в дебагере.

А насчет блокирующих операций, лучше всего они понимаютса когда осознаеш что в ноде все операции блокирующие, даже обычные if или for, потому такие долгие операции, как чтения файла надо дробить на более мелкие операции, async функции это просто функции, где кто-то уже сделал это до вас.

Кстати вот тут
if (action = 'render') {
два равно поставьте

[pavlyk]

Спасибо. Все понял!

[pavlyk]

Вынесите в отдельный коммент, отмечу решенным