Undefined variable in $.each() jquery

Question

I'm using this class in javascript:

function rMotoristaList() {
}

rMotoristaList.prototype.permInsert = false;
rMotoristaList.prototype.permSelect = false;
rMotoristaList.prototype.permUpdate = false;
rMotoristaList.prototype.permDelete = false;

rMotoristaList.prototype.init = function() {
    // Pega as permissoes
    var perm = new cPermissao();
    this.permInsert = perm.verificaPermissao(ID_CAD_MOTORISTAS, "insert");
    this.permUpdate = perm.verificaPermissao(ID_CAD_MOTORISTAS, "update");
    this.permDelete = perm.verificaPermissao(ID_CAD_MOTORISTAS, "delete");

    if (this.permInsert == false) {
        $("#btn-add-novo").hide();
    }
};

rMotoristaList.prototype.renderGrid = function(data) {
    var html = "";

    // Faz o loop em todos os elementos retornados
    $.each(data,function(index, motorista) {
        if (this.permUpdate != false) {
            //Do something
        }
    }
};

The attrbitue permUpdate is false, but, when i compare him, inside of $.each(), don't work, i receive a undefined.

How can i get the value of this.permUpdate inside $.each()?

Inside the each callback, this refers to the current item in the loop not the class. Halfway through writing and saw that @lgupta had posted the answer! See that — Adam
– Adam, Commented Apr 16, 2014 at 20:12
Would it make more sense to have that if statment outside of the each? does the result of the conditional change on each iteration? — user400654
– user400654, Commented Apr 16, 2014 at 20:13
I recommend you reading something about closures in javascript and how the context for the function works... stackoverflow.com/questions/111102/… — FredyC
– FredyC, Commented Apr 16, 2014 at 20:13
@KevinB - +1, It sure would, what's the point of iterating if this.permUpdate is false and nothing happens anyway ? — adeneo
– adeneo, Commented Apr 16, 2014 at 20:15

score 2 · Accepted Answer · 2014-04-16 20:46:41Z

2

Inside an anonymous function this will not refer rMotoristaList. You can cache this and use it.

rMotoristaList.prototype.renderGrid = function(data) {
    var html = "",
        self = this;

    // Faz o loop em todos os elementos retornados
    $.each(data,function(index, motorista) {
        if (self.permUpdate != false) {
            //Do something
        }
};

edited Apr 16, 2014 at 20:46

answered Apr 16, 2014 at 20:10

user2616876

Sign up to request clarification or add additional context in comments.

1 Comment

FredyC Over a year ago

You could have at least explain what have changed and why...like @sweetamylase.

Amy · Accepted Answer · 2014-04-16 20:14:31Z

1

The this inside $.each() refers to the function context of the .each() and not the one in your code. What you can do is save your context outside of the .each() block, in JavaScript it's called a Closure, see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Closures.

rMotoristaList.prototype.renderGrid = function(data) {
    var html = "";
    var that = this;
    // Faz o loop em todos os elementos retornados
    $.each(data,function(index, motorista) {
        if (that.permUpdate != false) {
        //Do something
        }
    }

} ;

answered Apr 16, 2014 at 20:14

Amy

7,5262 gold badges22 silver badges31 bronze badges

Comments

user400654 · Accepted Answer · 2014-04-16 20:21:00Z

1

There is also another approach, which might be somewhat better in situation like this...

rMotoristaList.prototype.renderGrid = function(data) {
    var html = "";

    // Faz o loop em todos os elementos retornados
    $.each(data,function(index, motorista) {
        if (this.permUpdate != false) {
            //Do something
        }
    }.bind(this)); // <-- Bind the function to the right context.
};

This way you are effectively saying, that this function should be executed with the context of rMotoristaList instance.

edited Apr 16, 2014 at 20:21

user400654

95.1k16 gold badges168 silver badges188 bronze badges

answered Apr 16, 2014 at 20:19

FredyC

4,0874 gold badges32 silver badges41 bronze badges

Collectives™ on Stack Overflow

Undefined variable in $.each() jquery

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related