Ожидаемый catch () или возврат (обещание / catch-or-return)

Question

Ожидаемый catch () или возврат (обещание / catch-or-return)

1

Я новичок в JavaScript. Это моя первая функция в javascript для развертывания функции на базе firebase.

Получил эту ошибку:

 - [eslint] Unexpected function expression. (prefer-arrow-callback)
 - [eslint] Expected catch() or return (promise/catch-or-return)

Что не так с этой функцией?

const functions = require('firebase-functions');
const admin = require('firebase-admin');
admin.initializeApp(functions.config().firebase);

exports.grantSignupReward = functions.database.ref('/users/{uid}/last_signin_at')
  .onCreate(event => {
  var uid = event.params.uid;

  admin.database().ref('users/${uid}/referred_by')
    .once('value').then(function(data) {
    var referred_by_somebody = data.val();

    if (referred_by_somebody) {
      var moneyRef = admin.database()
      .ref('/users/${uid}/inventory/pieces_of_eight');

      moneyRef.transaction(function(current_value) {
        return (current_value || 0) + 50;
      });
    }
  });
});

Shweta Chauhan 28 апр. 2018, в 17:00

Источник

0

Это не ошибки, это предупреждения от линтера.
Bergi 28 апр. 2018, в 14:41
0

Вы можете найти объяснения правил на eslint.org/docs/rules
Bergi 28 апр. 2018, в 14:43

Теги:

javascript

firebase

google-cloud-functions

eslint

1 ответ

Ещё вопросы

Это не ошибки, это предупреждения от линтера.
Вы можете найти объяснения правил на eslint.org/docs/rules

31piy · Accepted Answer · 2018-04-28T13-21-00.000Z

Первая ошибка предполагает использование функции стрелки в качестве обратного вызова. Поэтому вам нужно заменить обычные функции, (function() {... }), со стрелочными функциями, (() => {... }).

Вторая ошибка предполагает, что вам либо нужно уловить отказ обещаний, либо вернуть само обещание. Я не слишком уверен в вашем коде, но я считаю, что этот метод:

admin.database().ref('users/${uid}/referred_by').once('value')

возвращает обещание. Поэтому его нужно вернуть так:

return admin.database().ref('users/${uid}/referred_by').once('value')

или обрабатывать ошибку следующим образом:

admin.database().ref('users/${uid}/referred_by').once('value')
  // ... your code here
  .catch(error => { ... });

Поскольку @Bergi отметил в комментариях, что возвращение обещания здесь не предпочтительнее, вы можете просто добавить блок catch в свое обещание.

Вы можете добавить, что return обещания здесь не подходит.
@ Берги - боюсь, я не знаком с функциями, используемыми здесь. Вот почему выложили дипломатический ответ.
спасибо @ 31piy. На самом деле я ничего не знаю о javascript. Но ваш ответ мне очень помог.