On Tuesday, 22 April 2014 17:29:56 UTC+10, stagas wrote:
>
>
>
>
> 2014-04-22 7:44 GMT+03:00 Andrey <andrey....@gmail.com <javascript:>>:
>
>> In addition all previous comments:
>> I don't like name 'self' as alias to 'this' ( I tend to interpret same
>> way as 'this', e.i context of current function). Instead, use same name you
>> would use for your object instance. Example:
>>
>> Client.prototype.doStuff = function() {
>> var client = this;
>> // ... some async code here
>> setTimeout(function() {
>> client.doneTimeout(); // I think it's more explicit than
>> self.doneTimeout()
>> }, 100);
>> };
>>
>>
>>
> Consider the situation where that `client.something()` ends up somewhere
> deeply nested and the function isn't that trivial. Someone reading that
> piece code or jumping to it from search results will have no idea what it
> points to, because it's not obvious; is it an argument, or what? Where did
> it come from?
>
That's exactly the reasoning behind my suggestion: 'self' or 'that' deeply
nested from corresponding 'this' much more difficult to understand than
properly named reference. Also it's more difficult to fine original
reference in a file with multiple 'self' of different type scattered in
different places
>
> These questions waste time and energy. `self` is conventionally the
> instance object for that reason. Conventions are there for that reason.
>
> Then there's refactoring, what if you decide it's not a Client but
> something else. Suddenly `client = this` doesn't make sense. What if you
> move the function over to another prototype because it seems it belongs
> better there? Again, doesn't make sense. Save yourself and everyone else
> some headaches and use `self`. It can never be wrong.
>
>
>>
>> On Tuesday, 22 April 2014 06:11:14 UTC+10, Reza Razavipour wrote:
>>>
>>> I see a lot of code that does
>>> var self = this
>>>
>>> and then they use self instead of this for the duration of the function?
>>>
>>> Why and when would you use the above pattern?
>>>
>>>
>>> --
>> --
>> Job Board: http://jobs.nodejs.org/
>> Posting guidelines:
>> https://github.com/joyent/node/wiki/Mailing-List-Posting-Guidelines
>> You received this message because you are subscribed to the Google
>> Groups "nodejs" group.
>> To post to this group, send email to nod...@googlegroups.com<javascript:>
>> To unsubscribe from this group, send email to
>> nodejs+un...@googlegroups.com <javascript:>
>> For more options, visit this group at
>> http://groups.google.com/group/nodejs?hl=en?hl=en
>>
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "nodejs" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to nodejs+un...@googlegroups.com <javascript:>.
>> For more options, visit https://groups.google.com/d/optout.
>>
>
>
--
--
Job Board: http://jobs.nodejs.org/
Posting guidelines:
https://github.com/joyent/node/wiki/Mailing-List-Posting-Guidelines
You received this message because you are subscribed to the Google
Groups "nodejs" group.
To post to this group, send email to nodejs@googlegroups.com
To unsubscribe from this group, send email to
nodejs+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/nodejs?hl=en?hl=en
---
You received this message because you are subscribed to the Google Groups
"nodejs" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to nodejs+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
