Как мне вернуть правильный result?

Question

[sinevik]

import { ADD_USER} from './actionTypes';
import { HELLO} from './actionTypes';
import firebase from 'react-native-firebase';

export const addUser = (surname, name, patronymic, phone) => {
    return dispatch => {
      var db = firebase.firestore();
      let user = {
        surname: surname,
        name: name,
        patronymic: patronymic,
        phone: phone
      }
      var result = 'начальный';
      var docRef = db.collection('users').doc(user.phone);
      docRef.get().then(function(doc) {
      if (doc.exists) {
          result = 'Такой номер уже есть';
          console.log('Такой номер уже есть');
      }else{
          db.collection('users').doc(user.phone).set(user);
          result = true;
          console.log(true);
      }
      }).catch(function(error) {
          result = 'Произошла ошибка';
          console.log('Произошла ошибка');
      });
      return result;
    };
};

При вызове данной функции из store redux

let result = this.props.onAddUser(
this.state.controls.surname.value,
this.state.controls.name.value,
this.state.controls.patronymic.value,
this.state.controls.phone.value
)
console.log(result);

result всегда равен значению 'начальный'. Хотя в консоле показывает ('Такой номер уже есть'). Как мне правильно перезаписать result?

Answer 1

[Анатолий Жаров]

import {ADD_USER} from './actionTypes';
import {HELLO} from './actionTypes';
import firebase from 'react-native-firebase';

export const addUser = (surname, name, patronymic, phone) => {
    return async dispatch => {
        let db = firebase.firestore();
        let user = {
            surname: surname,
            name: name,
            patronymic: patronymic,
            phone: phone,
        };
        let result = 'начальный';
        let docRef = db.collection('users').doc(user.phone);
        try {
            let doc = await docRef.get()
            if (doc.exists) {
                result = 'Такой номер уже есть';
                console.log('Такой номер уже есть');
            } else {
                db.collection('users').doc(user.phone).set(user);
                result = true;
                console.log(true);
            }
        } catch (e) {
            result = 'Произошла ошибка';
            console.log('Произошла ошибка');
        }
        return result;
    };
};

let result = this.props.onAddUser(
    this.state.controls.surname.value,
    this.state.controls.name.value,
    this.state.controls.patronymic.value,
    this.state.controls.phone.value,
).then(result => console.log(result))

Answer 2

[Alexey Bille]

Попробуйте так

docRef.get().then((doc) => {
      if (doc.exists) {
          result = 'Такой номер уже есть';
          console.log('Такой номер уже есть');
      }else{
          db.collection('users').doc(user.phone).set(user);
          result = true;
          console.log(true);
      }
      }).catch((error) => {
          result = 'Произошла ошибка';
          console.log('Произошла ошибка');
      });

Comments

[Alexey Bille]

Сори бред написал, промис выполниться после выполнения функции (скорее в консоли вы сначала увидете

console.log(result);

И только потом
console.log('Такой номер уже есть');

[sinevik]

Так у меня же так и есть

[Alexey Bille]

Сразу скажу я не знаком с реактом, но единственный вариант который я вижу:

Из функции вернуть промис, затем обработать его:

import { ADD_USER} from './actionTypes';
import { HELLO} from './actionTypes';
import firebase from 'react-native-firebase';

export const addUser = (surname, name, patronymic, phone) => {
    return dispatch => {
      var db = firebase.firestore();
      let user = {
        surname: surname,
        name: name,
        patronymic: patronymic,
        phone: phone
      }
      var result = 'начальный';
      var docRef = db.collection('users').doc(user.phone);
      return docRef.get().then(function(doc) {
      if (doc.exists) {
          result = 'Такой номер уже есть';
          console.log('Такой номер уже есть');
      }else{
          db.collection('users').doc(user.phone).set(user);
          result = true;
          console.log(true);
      }
      return result;
      }).catch(function(error) {
          result = 'Произошла ошибка';
          console.log('Произошла ошибка');
          return result;
     });
    };
};



this.props.onAddUser(
this.state.controls.surname.value,
this.state.controls.name.value,
this.state.controls.patronymic.value,
this.state.controls.phone.value
).then(result => {
    console.log(result);
})

[Alexey Bille]

Ну или через async\await как написал Анатолий Жаров

[sinevik]

А где в вашем коде возвращается из функции промис?

[Alexey Bille]

return docRef.get().then(function(doc) {
          //...
          return result;
      }).catch(function(error) {
          //...
          return result;
     });