Пытаясь реорганизовать это с помощью Ramda.js, вот как далеко я получил - PullRequest
Новая
       7

Пытаясь реорганизовать это с помощью Ramda.js, вот как далеко я получил

0 голосов
/ 22 декабря 2018

Я изучаю Ramda.js.Функция, которую я пытаюсь изменить с помощью Ramda, приведена ниже.Он вызывает функцию firestore метод базы данных, чтобы получить некоторые данные.Но firestore возвращает данные внутри snapshot метода, и нам нужно вызвать .data().В зависимости от результата, я хочу построить разные ответы.

Хотелось бы здесь, чтобы ваш мыслительный процесс 

const queryForUsersQuizResults = async (request, response) => {
  try {
    const snapshot = await firestore
      .collection("quizResults")
      .doc(request.user.uid)
      .collection("courses_chapters")
      .doc(request.params.courseId + "_" + request.params.chapterId)
      .get();

    let data = snapshot.data();
    if (!data) {
      data = {
        message: "result not found for this user for this course and chapter"
      };
    }

    return response.send(data);
  } catch (error) {
    return response.status(500).send(error);
  }
}

... и вот что я былвозможность рефакторинга, хотел бы видеть лучше / другие способы сделать это (я не уверен, если это вообще работает).Я борюсь с методом sendResult. 

//get user id
export const getUserId = pathOr('', ['user', 'uid']);

// get chapter id
export const getChapterId = concat(pathOr('', ['params', 'courseId']), '_', pathOr('', ['params', 'chapterId']));

//queryQuizResult
export const query = curry(async (firestore, request) => {
  return tryCatch(() =>
    firestore
    .collection("quizResults")
    .doc(getUserId(request))
    .collection("courses_chapters")
    .doc(getChapterId(request))
    .get(), F)();
});

//Receives "response" object and calls response.status with the value passed to the new status function
export const status = invoker(1, "status");

//Receives "response" object and calls response.send witht he value passed to the new send function
export const send = invoker(1, "send");

//Create {"message",Your_Error} object
export const constructError = objOf('message');

//Returns JSON from Firestore's snapshot object
export const getDataFromSnapshot = (snapshot) => snapshot.data();

//Actual error message
const QUIZ_RESULTS_NOT_FOUND = "Quiz results not found for this user for this course and chapter";

//Returns error message
export const quizResultsNotFoundError = constructError(QUIZ_RESULTS_NOT_FOUND);

//Receives "response" object and calls response.status and then respose.send
export const sendError = pipe(
  status(401),
  send(quizResultsNotFoundError)
);
//Sends valid result  //IS there any better way of doing this?
export const sendResult = (snapshot, response) => {
  const data = getDataFromSnapshot(snapshot);
  response.send(data); //Especially this, I wanted to use the "send" method and pipe things
}

//Main Method
export const  queryForUsersQuizResults = async (firestore, request, response) => {
 const snapshot = await query(firestore, request);
 snapshot ? sendResult(snapshot, response) :sendError(response);
}

Ответы [ 2 ]

0 голосов
/ 25 декабря 2018

Вот движущиеся части, которые я вижу.Это всего лишь один из многих возможных способов написания программы - 

const tryCatch = f =>
  new Promise
    ( (resolve, reject) =>
        { try { resolve (f ()) }
          catch (err) { reject (err) }
        }
    )

const handleSuccess = res => data =>
  res .send (data)

const handleError = res => err =>
  res .status (500) .send (err)

const getSnapshot = (uid, docid) =>
  firestore
    .collection ("quizResults")
    .doc (uid)
    .collection ("courses_chapters")
    .doc (docid)
    .get ()
    .data ()

const queryForUsersQuizResults = (req, res) =>
  tryCatch
    ( () =>
        getSnapshot
          ( req.user.uid
          , req.params.courseId + "_" + req.params.chapterId
          )
    )
    .catch
      ( _ => Promise .reject (Error ("result not found for this user for this course and chapter"))
      )

const main = (req, res) =>
  queryForUsersQuizResults (req, res)
    .then
      ( handleSuccess (res)
      , handleError (res)
      )

А вот рабочая демонстрация, демонстрирующая концепции в действии - 

const DB =
  { 1: "ant"
  , 2: "bear"
  , 3: "cuttlefish"
  }

const queryForUsersQuizResults = (id = 0) =>
{ if (DB[id] === undefined)
    throw Error (`no record for id: ${id}`)
  else
    return DB[id]
}

const tryCatch = f =>
{ try
  { return Promise .resolve (f ()) }
  catch (err)
  { return Promise .reject (err) }
}

const fakeResponse = (status = 200) =>
  ({ send: data =>
       console .log ("=>", status, data)
   , status: n =>
       fakeResponse (n)
  })

const demo = (req, res) =>
  tryCatch (() => queryForUsersQuizResults (req.params.uid))
    .then
      ( data => res .send (data)
      , err => res .status (500) .send (err)
      )

demo
  ( { params: { uid: 1 } }
  , fakeResponse ()
  ) // => 200 ant

demo
  ( { params: { uid: 2 } }
  , fakeResponse ()
  ) // => 200 bear

demo
  ( { params: { uid: 3 } }
  , fakeResponse ()
  ) // => 200 cuttlefish

demo
  ( { params: { uid: 4 } }
  , fakeResponse ()
  ) // => 500 Error: no record for id: 4
0 голосов
/ 24 декабря 2018

Я также думаю, что то, что у вас уже есть, хорошо, если вы действительно хотите использовать композицию, это может быть что-то вроде этого: 

export const sendResult = (snapshot, response) =>
  pipe(getDataFromSnapshot, flip(send)(response))
    (snapshot);

Вам нужно использовать flip, поскольку вы этого не делаетеиметь (на момент определения канала) результат getDataFromSnapshot(snapshot), но у вас уже есть объект, содержащий реализацию вашего метода send.

...