Problem mit Javascript Variable Scope

Question

Ich habe eine Variable muteedUser, die ich gerne zu einer anderen Funktion erhalten würde. Ich habe ein bisschen Probleme mit der Variablen, die außerhalb des Klickereignisses besteht. Was wäre der beste Weg, um es so zu haben, dass der "return muteedUser" die "gedämpfte" String-Addition basierend auf den Bedingungen der if-Anweisung beibehalten würde? Vielen Dank!

* Die console.log des prüften mich, um zu sehen, wo die persistance

this.isUserMuted = function isUserMuted(payload) { 
    var mutedUser = ''; 
    // If mute button is clicked place them into muted users list 
    // check for duplicates in list 
    $("#messages-wrapper").off('click', '.message button.muteButton'); 
    $("#messages-wrapper").on('click', '.message button.muteButton', function(e) { 

     $('#unMute').show(); 

     //create userId reference variable 
     var chatUserID = parseInt($(this).parent().parent().attr("data-type")); 

     //store userId in muted user object 
     mutedUsers[chatUserID] = {}; 
     mutedUsers[chatUserID].id = chatUserID; 
     mutedUsers[chatUserID].muted = true; 

     if (mutedUsers[chatUserID] !== null && mutedUsers[chatUserID].id === payload.a) { 
      console.log("user is now muted"); 
      mutedUser += ' muted'; 
      console.log(mutedUser + 1); 
     } 
     console.log(mutedUser + 2); 
    }); 
    return mutedUser; 
}; 

Answer 1

Wenn ich verstand, was Sie (mit Blick auf dem Code) zu tun versuchen, dies wäre der beste Ansatz:

// If mute button is clicked place them into muted users list 
// check for duplicates in list 
$("#messages-wrapper").off('click', '.message button.muteButton'); 
$("#messages-wrapper").on('click', '.message button.muteButton', function(e) { 
    $('#unMute').show(); 

    //create userId reference variable 
    var chatUserID = parseInt($(this).parent().parent().attr("data-type")); 

    //store userId in muted user object 
    mutedUsers[chatUserID] = {}; 
    mutedUsers[chatUserID].id = chatUserID; 
    mutedUsers[chatUserID].muted = true; 
}); 

this.isUserMuted = function isUserMuted(payload) { 
    var mutedUser = ''; 

    if (mutedUsers[payload.a] !== null) { 
     mutedUser += ' muted'; 
    } 

    return mutedUser; 
}; 

Die Code behält das Array von mutedUsers und isUserMuted Funktion überprüft, ob Benutzer in diesem Array zur Verfügung gestellt wird. Im Code, den Sie zur Verfügung gestellt, würden Sie einen neuen Event-Handler jedes Mal isUserMuted Funktion aufgerufen wird ..

Die isUserMuted Funktion befestigen sogar verkürzt werden konnte:

this.isUserMuted = function isUserMuted(payload) { 
    return mutedUsers[payload.a] !== null ? ' muted' : ''; 
}; 

Answer 2

bearbeiten

Sorry, mein Fehler stoppt. Ein anderer Weg ist in diesen Variablen zu übergeben, das heißt

this.isUserMuted = function isUserMuted(payload, isMuted) { 
    isMuted = ''; 
    // If mute button is clicked place them into muted users list 
    // check for duplicates in list 
    $("#messages-wrapper").off('click', '.message button.muteButton'); 
    $("#messages-wrapper").on('click', '.message button.muteButton', function(e) { 

     $('#unMute').show(); 

     //create userId reference variable 
     var chatUserID = parseInt($(this).parent().parent().attr("data-type")); 

     //store userId in muted user object 
     mutedUsers[chatUserID] = {}; 
     mutedUsers[chatUserID].id = chatUserID; 
     mutedUsers[chatUserID].muted = true; 

     if (mutedUsers[chatUserID] !== null && mutedUsers[chatUserID].id === payload.a) { 
      console.log("user is now muted"); 
      isMuted += ' muted'; 
      console.log(mutedUser + 1); 
     } 
     console.log(mutedUser + 2); 
    }); 
    return isMuted; 
}; 

Answer 3

Sie können nicht. Wenn Sie eine Zeichenfolge von der Funktion zurückgeben, wird sie immer als Wert übergeben, d. H. Kopiert; und sein Wert wird sich nicht mehr ändern. Sie müssten eine Funktion zurückgeben, die auf den aktuellen Wert der lokalen Variablen zugreifen kann, oder ein Objekt mit einer Eigenschaft, die sich ändert.

Wie Sie bereits scheinen, ein Objekt zu haben, Option # 2 wird hier gut passen:

function User() { // or whatever you have 
    … 

    var user = this; 
    // If mute button is clicked place them into muted users list 
    // check for duplicates in list 
    $("#messages-wrapper").on('click', '.message button.muteButton', function(e) { 

     $('#unMute').show(); 

     //store userId in muted user object 
     mutedUsers[user.id] = user; 
     user.muted = true; 
    }); 
    this.muted = false; 
    this.isUserMuted = function() { 
     return this.muted ? ' muted' : ''; 
    } 
}