Форум

OlegALL · #1 (**permalink**) 30.11.2014, 22:36

Привет. Код: http://jsfiddle.net/nz2c2cva/1/

надо переписать строку проверки условия,
комментарий специалиста:
"В конструкторе проверять this, если this не твой инстанс,
то вызывать заново конструктор внутри себя уже с new",
причём это решение должно быть лучше

if (this.$) {
	return new $(selector);
}

Написал так: if (this instanceof window.constructor) , но специалист не принял, сказав, что ничего не улучшилось

Так же, скажите, почему такой вариант лучше

Vlasenko Fedor · #2 (**permalink**) 30.11.2014, 22:50

if (!(this instanceof $)) {
    return new $(selector);
}

зачем учиться так, чтоб за тебя решали?

krutoy · #3 (**permalink**) 30.11.2014, 23:49

не уверен, что правильно понял, но, похоже, ты хотел вот так сделать

<html>
<head>
<meta charset="windows-1251" />
</head>
<body>
 
<div class="class0"> <a href="">link</a> </div>
<div class="class0"></div>
<div class="class0"></div>
<script>

var $ = function (selector){
        var o=Object.create($.prototype)
	o.tags = document.querySelectorAll(selector);
        return o
}

$.prototype.addClass = function(className){
	for (var i = 0; i < this.tags.length; i++){
		this.tags[i].classList.add(className);
	}
	return this;
}

$.prototype.copyHtml = function(){
       var html=this.tags[0].innerHTML
       for(var i = 1; i < this.tags.length; i++){
           this.tags[i].innerHTML=html
       }
}

$('div').addClass('class2').copyHtml();


</script>
</body>
</html>

krutoy · #4 (**permalink**) 01.12.2014, 00:00

И вот так,

var $ = function (selector){
			
			this.tags = document.querySelectorAll(selector);

никогда не делай.
Если уж используешь эту клоунаду, сначала рекурсивный вызов через new, а уже потом присваивание. Например,

$=function(selector){
   if (!(this instanceof $)) return new $(selector);
   this.tags = document.querySelectorAll(selector);
   ...

а в твоем случае, this===window, соответственно, ты туда срешь.

krutoy · #5 (**permalink**) 01.12.2014, 00:18

А вообще, эту хрень проще безо всякого prototype написать, он тут ни к чему, по-моему

<html>
<head>
<meta charset="windows-1251" />
</head>
<body>
 
<div class="class0"> <a href="">link</a> </div>
<div class="class0"></div>
<div class="class0"></div>
<script>




var $ = function (selector){
        var o=Object.create($)
	o.tags = document.querySelectorAll(selector);
        return o
}

$.addClass = function(className){
	for (var i = 0; i < this.tags.length; i++){
		this.tags[i].classList.add(className);
	}
	return this;
}

$.copyHtml = function(){
       var html=this.tags[0].innerHTML
       for(var i = 1; i < this.tags.length; i++){
           this.tags[i].innerHTML=html
       }
}

$('div').addClass('class2').copyHtml();


</script>
</body>
</html>

но все зависит от юзкейса, конечно.

OlegALL · #6 (**permalink**) 01.12.2014, 07:08

Сообщение от Poznakomlus

if (!(this instanceof $)) {
    return new $(selector);
}

зачем учиться так, чтоб за тебя решали?

Я абсолютно не знаю как это делать, поэтому и спрашиваю

Спасибо, а почему так лучше?

OlegALL · #7 (**permalink**) 01.12.2014, 07:10

Сообщение от krutoy

А вообще, эту хрень проще безо всякого prototype написать, он тут ни к чему, по-моему

<html>
<head>
<meta charset="windows-1251" />
</head>
<body>
 
<div class="class0"> <a href="">link</a> </div>
<div class="class0"></div>
<div class="class0"></div>
<script>




var $ = function (selector){
        var o=Object.create($)
	o.tags = document.querySelectorAll(selector);
        return o
}

$.addClass = function(className){
	for (var i = 0; i < this.tags.length; i++){
		this.tags[i].classList.add(className);
	}
	return this;
}

$.copyHtml = function(){
       var html=this.tags[0].innerHTML
       for(var i = 1; i < this.tags.length; i++){
           this.tags[i].innerHTML=html
       }
}

$('div').addClass('class2').copyHtml();


</script>
</body>
</html>

но все зависит от юзкейса, конечно.

Код не надо изменять, это задание, которое почти выполнено

danik.js · #8 (**permalink**) 01.12.2014, 07:35

Сообщение от OlegALL

Я абсолютно не знаю как это делать, поэтому и спрашиваю

Ды должен был спросить не как это сделать, а почему твой вариант неправильный. А перед этим разобраться и пояснить нам что такое конструктор, и как работает оператор new. Если бы разобрался, то понял сам чего от тебя хотят и реализовал бы.

OlegALL · #9 (**permalink**) 01.12.2014, 09:02

Я понимаю, что такое конструктор и new.

Тем не менее, объясните, почему проверка

if (!(this instanceof $))

лучше, чем

if (this.$)

Кроме того,
даже используя отладчик не пойму как работает этот код:

$.prototype.html = function(){
	                var clone = new $('_');
			clone.tags = this.tags;
			clone.toString = function() {
				console.log("|");
				return this.tags[0].innerHTML;
			};
			console.log("||");
			return clone;
}

Точнее - в каком порядке исполняются инструкции return.
Как я понял сначала отрабатывает return clone, затем функция toString()?
Отладчик не заходит в toString

nerv_ · #10 (**permalink**) 01.12.2014, 10:35

Сообщение от OlegALL

но специалист не принял

безобразие

Сообщение от OlegALL

Я понимаю, что такое конструктор и new

если бы понимал, не писал бы такой ерунды

а еще не понятно, зачем такое извращение

$.prototype.html = function(){
    var clone = new $('_');
    clone.tags = this.tags;
    clone.toString = function() {
        return this.tags[0].innerHTML;
    };
    return clone;
}

если достаточно

// getter
$.prototype.html = function(){
    return this.tags[0].innerHTML;
}

http://api.jquery.com/html/
http://code.jquery.com/jquery-2.1.1.js

Форум

Справочник

Discord чат

Курсы javascript