проблема с readyState

[kilohertz_.]

Ошибка: Uncaught TypeError: Cannot read property 'readyState' of undefined в строчке

if(this.xhr.readyState == 4) {

function ajax(callback) { //json object
	this.xhr;
	this.result = {"result":"empty"};
	
	this.send = function(addr, string, method, async){
		this.xhr = new XMLHttpRequest();
		this.xhr.open(method,addr,async);
		if(method == 'POST') this.xhr.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');
		this.xhr.send(string);
		this.xhr.onreadystatechange = this.get;
	}
	
	this.get = function(){
		if(this.xhr.readyState == 4) {
			this.result = JSON.parse(this.xhr.responseText);
			if(this.result.status == 'complete') {
				if(!(callback === null)) {callback}
			}
			if(this.result.status == 'error') {
				return showMsg(1,this.result.error);
			}
		}
	}
	
}

[Safort]

Это из-за того, что инициализация объекта происходит только при вызове метода send. Перенеси содержимое 6 строки на 2 строку.

[kilohertz_.]

Не помогло
И данные при этом с сервера загружаются ведь!
XHR finished loading: POST "http://***/index.php".

[danik.js]

Привыкай к необычному поведению this в JavaScript. Кратко говоря - куда будет указывать this зависит от того, кто и как вызывает функцию, а не от того где и как она объявлена.


this.get.call('хрень') - и внутри метода this указывает не на экземпляр ajax, а на 'хрень'.

Примерно тоже самое у тебя сейчас и происходит:

this.xhr.onreadystatechange = this.get;

Тут ты дал объекту xhr ссылочку на некую функцию. Теперь объект, когда получит ответ от сервера, вызовет этот метод, причем установит в качестве this - себя любимого!

if(this.xhr.readyState == 4) {

Тут this указывает на xhr, у которого нет свойства xhr. Исправь на if (this.readyState ...)


Еще раз: метод не знает, чьим "методом" он является. И вообще, в js нет методов. Есть просто функции.

Можешь жестко задать this таким образом:

this.xhr.onreadystatechange = this.get.bind(this);

И метод заработает без изменений.

Функция bind возвращает функцию с жестко прописанным this.

[danik.js]

var ajax = new Ajax();
ajax.get() - так get будет вызван в контексте this, равном ajax.

var get  = ajax.get;

get() - а так this уже "потеряется" и будет тупо указывать на глобальный объект.

xhr.get = ajax.get;

xhr.get() - так this будет указывать на xhr. Что у тебя и происходит.

[danik.js]

var ajax = new Ajax();

Вот кстати вкурил дружок почему констурторы надо с большой буквы называть? Бегом переименовывай )

[kilohertz_.]

Спасибо! Буду переделывать/доделывать)