Enrutamiento avanzado - respondiendo una pregunta

En el helper del conteo, pudiéramos validar si el arreglo de respuestas viene en null para devolver 0, y así no tendríamos que incluir el tag #if en la plantilla.

no sería mejor crear un modelo para “answer” ?

Para evitar el if dentro del archivo index.hbs podemos poner el siguiente código en el index.js

handlerbars.registerHelper('answerNumber', (answers) => {
  if (!answers) return 0;
  const keys = Object.keys(answers).length;
  return keys;
});

en el archivo index.hbs solo agregamos esto

{{answerNumber question.answers}}

✌

😊✌

A la funcion answerQuestion agregué un if() que evalúa si el usuario está o no está logueado. Si no lo esta, lo redirecciona al login.

async function answerQuestion (req, h) {
  if (!req.state.user) {
    return h.redirect('/login')
  }
  let result
  try {
    result = await questions.answer(req.payload, req.state.user)
    console.log('Respuesta creada' + result);
  } catch (error) {
    console.error(error);
  }

  return h.redirect('/question/' + req.payload.id)
}```

Me sigue gustando más como se ve con pug…
El punto es que pug acepta bloques de código dentro de la vista

extends ./layout/layout

block content
  main.container-fluid(role="main")
    .jumbotron.bg-white
      .container
        h1.display-3 FAQ
        p
          | The best place to learn and for those who are learning, ask a question
          | to clarify your doubts and share your knowledge if you know the answer

    .container
      h2.mb-4 Last Questions
      if questions 
        each q, key in questions
          .card.bg-white.mb-3
            .card-body
              .row
                .col-2.text-center
                  a.text-dark(href="#")
                    -
                      let numAns = 0;
                      if(q.answers){
                       numAns = Object.keys(q.answers).length
                      }
                    h3=numAns
                    | Answers
                .col-10.d-flex.align-items-stretch
                  h4.mt-3
                    a.text-dark(href="/question/"+key)= q.title
          hr
      else 
        .card.bg-white.mb-3
          .card-body
            .row
              .col-2.text-center
                h3 
              .col-10.d-flex.align-items-stretch
                h4.mt-3= error

block footer
  include ./partials/footer
block scripts
  include ./partials/scripts

controller.question.js

'use strict'

const questions = require('../models/index').questions

async function createQuestion(req, h) {
    let result

    try {
        result = await questions.create(req.payload, req.state.user)
        console.log(`Pregunta creada con el ID: ${result}`)
        return h.response(`Pregunta creada con el ID: ${result}`)
    } catch (error) {
        console.error(`Ocurrio un error: ${error}`)

        return h.view('ask', {
            title: 'Crear pregunta',
            error: 'Problemas creado la pregunta'
        }).code(500).takeover()
    }
}

async function answerQuestion(req, h) {
    let result
    try {
        result = questions.answer(req.payload, req.state.user)
        console.log(`Respuesta creada: ${result}`)
    } catch (error) {
        console.error(error)
    }

    return h.redirect(`/question/${req.payload.id}`)
}

module.exports = {
    createQuestion: createQuestion,
    answerQuestion: answerQuestion
}

models.questions.js

'use strict'

class Quiestions {
    constructor(db) {
        this.db = db
        this.ref = this.db.ref('/')
        this.collection = this.ref.child('questions')
    }

    async create (data, user) {
        const ask = {
            ...data
        }
        ask.owner = user
        const question = this.collection.push()
        question.set(ask)

        return question.key
    }

    async getLast(amount) {
        const query = await this.collection.limitToLast(amount).once('value')
        const data = query.val()
        return data
    }

    async getOne (id) {
        const query = await this.collection.child(id).once('value')
        const data = query.val()
        return data
    }

    async answer (data, user) {
        const answers = await this.collection.child(data.id).child('answers').push()
        answers.set({
            text: data.answer, 
            user: user
        })
        return answers
    }
}

module.exports = Quiestions

no es malo tener el id en el input vacio , ya que se puede ver en inspeccionar > elementos ?, o si hay ?

Modifique un poco el código y quedo así:
index.js

handlebars.registerHelper('answerNumber', (answers) => {
  if (answers === undefined) {
    return 0;
  }
  const keys = Object.keys(answers);
  return keys.length;
});

index.hbs

<h3>
  {{answerNumber question.answers}}
</h3>

me agrado la funcion que tiene handlerbars, me gustaria aplicar esa parte con express js

En el modelo de questions, en la funcion getOne, pueden modificar data.answers para que quede ordenado, siguiendo el ejemplo de un compañero de la clase anterior quedaría así.

  async getOne(id){
    const query = await this.collection.child(id).once('value');
    const data = query.val();
    const orderedAnswers = {};
    Object.keys(data.answers).reverse().map(key => orderedAnswers[key] = data.answers[key]);
    return {
      ...data,
      answers: orderedAnswers,
    };
  }```