Хорош ли такой шаблон работы с AJAX?

Question

[tundramani]

В работе использую такой простой шаблонный код:

let отправить_данные_на_сервер = function(данные)
{
	let объект_запроса = new XMLHttpRequest()
	объект_запроса.open('POST', 'сохранить_данные.php')
	объект_запроса.send(данные)

	объект_запроса.onreadystatechange = function()
	{
		if ( объект_запроса.readyState === 4 )   // 4 означает: запрос завершен
		{
			if ( объект_запроса.status < 200 || объект_запроса.status > 299) // успешный ответ сервера — https://developer.mozilla.org/en-US/docs/Web/HTTP/Status
			{
				alert('сервер не найден')
				return 
			}

			let ответ = объект_запроса.responseText
			alert(ответ) //используем ответ
		}
	}
}

Вопрос - в этом коде есть всё что нужно?
Или есть ошибка?
Или можно еще что-то добавить?

//пс:
мне не нужны более новые методы запросов к серверу
мне тупо хватает этой классики

Comments

[Дмитрий Беляев]

а если статус ответа будет не 200? 201 или 204 вполне себе тоже ответы об успехе, как и многие другие

[tundramani]

спс
исправил

https://developer.mozilla.org/en-US/docs/Web/HTTP/...

[Андрей]

Такое ощущение что вы вчера ещё на 1с кодили

[tundramani]

в профессиональной индустрии ты раб и должен работать по стандартному шаблону

а я - свободный программист - я именую переменные так как мне удобно

а ты - страдай

Answer 1

[hzzzzl]

проще так
https://learn.javascript.ru/fetch

let response = await fetch(url);

if (response.ok) { // если HTTP-статус в диапазоне 200-299
  // получаем тело ответа (см. про этот метод ниже)
  let json = await response.json();
} else {
  alert("Ошибка HTTP: " + response.status);
}

Comments

[tundramani]

забыл добавить:

мне не нужны более новые методы запросов к серверу
мне тупо хватает этого пока

[hzzzzl]

js-css-html-php, ну фиг знает, зачем возиться со всеми этими onreadystatechange , когда уже 5+ лет как есть fetch, который одной строчкой это всё делает

[Леонид Князев]

Если вам хватает, значит в нём есть всё, чего вам хватает.

[tundramani]

fetch можно использовать без await?

если да то покажи такой аналог моего кода

[Виктор П.]

await по сути синтаксический сахар, всё можно написать без него, почитайте тему Promise

Answer 2

[Alex]

в этом коде есть всё что нужно?

А вам чего-то не хватает? Или вам тупо хватает этого пока?

Или есть ошибка?

А что-то не работает?

Или можно еще что-то добавить?

Добавить всегда можно. Но можно ли что-то отнять?

В целом, если вам нужен ревью кода, то я бы такое не пропустил. Просто за использование устаревших техник. Это ухудшает читаемость и работать другим людям сложнее, особенно новичкам.
Но ведь

мне не нужны более новые методы запросов к серверу

, ногда не ясно что вам нужно?

Answer 3

[Dmitriy Mozgovoy]

Это уже архаизм, к тому дикий велосипед. Зачем 100500 раз опять пытаться писать сырую обертку над XMLHttpRequest? Берите fetch или какую то стороннюю библиотеку, если там надо что то, чего у fetch нету (типа прогресса upload) и вперед.