アンギュラ - 未定義

Question

として戻り値Iは、次の機能があります：私はonNgInitに次のようにしようとすると

redirect() { 
    this.afs.collection('links').doc(this.path).ref.get().then(function(doc) { 
     if(doc.exists) { 
     // define data and clicks 
     var data = doc.data(); 
     var clicks = doc.data().clicks; 
     // Update clicks 
     doc.ref.update({ 
      'clicks': (clicks + 1) 
     }) 
     .then(function() { 
      if(data.landing == false) { 
      // Redirect to url 
      return false; 
      } else { 
      // Stay for Landing Page 
      return true; 
      } 
     }); 
     } else { 
     this.router.navigate(['/404']); 
     } 
    }).catch(function(error) { 
     console.log("Error getting document:", error); 
    }); 
    } 

を：

console.log(this.redirect()); 

それはundefinedを返します。 trueまたはfalseを返すために値をtrueまたはfalseに設定できるようにするにはどうすればよいか分かりません。

Answer 1

機能redirectは、現在何も返されません。しかし、プロミスと仕事をしているようです。その約束を返すようにリファクタリングを試してみてください：

redirect() { 

    return this.afs.collection('links').doc(this.path).ref.get().then(function(doc) { 
     if(doc.exists) { 
     // define data and clicks 
     var data = doc.data(); 
     var clicks = doc.data().clicks; 
     // Update clicks 
     doc.ref.update({ 
      'clicks': (clicks + 1) 
     }) 
     .then(function() { 
      if(data.landing == false) { 
      // Redirect to url 
      return false; 
      } else { 
      // Stay for Landing Page 
      return true; 
      } 
     }); 
     } else { 
     this.router.navigate(['/404']); 
     } 
    }).catch(function(error) { 
     console.log("Error getting document:", error); 
    }); 
    } 

をだから今約束の解決時にあなたが値をプリントアウトすることができます。

this.redirect().then(value => console.log(value))