Exercise Tracker Stub

JavaScript
1

I don’t know how to deal with this story in the Exercise Tracker Project. The story: I can add an exercise to any user by posting form data userId(_id), description, duration, and optionally date to /api/exercise/add. If no date supplied it will use current date. App will return the user object with the exercise fields added. Please Help Me.

Your code so far

const express = require('express')
const app = express()
const bodyParser = require('body-parser')
const shortid = require('shortid');

const cors = require('cors')

app.use(cors())

app.use(bodyParser.urlencoded({extended: false}))
app.use(bodyParser.json())


app.use(express.static('public'))
app.get('/', (req, res) => {
  res.sendFile(__dirname + '/views/index.html')
});


const users = [];
const exercises = [];

const getUsernameById = (id) => users.find(user => user._id === id).username;
const getExercisesFromUserWithId = (id) => exercises.filter(exe => exe._id === id);


app.post('/api/exercise/new-user', (req, res) => {
  const { username } = req.body;
  
  const newUser = {
    username,
    _id: shortid.generate()
  }
  
  users.push(newUser);
  
  return res.json(newUser);
});

app.get('/api/exercise/users', (req, res) => {
  return res.json(users);
});


app.post('/api/exercise/add', (req, res) => {
  const { userId, description, duration, date } = req.body;
  
  
  const dateObj = date === '' ? new Date() : new Date(date);
  
  const newExercise = {
    _id: userId,
    description,
    duration: +duration,
    date: dateObj.toString()
  }
  
  exercises.push(newExercise);
  
  res.json(newExercise);
});


app.get('/api/exercise/log', (req, res) => {
  const { userId, from, to, limit } = req.query;
  
  let log = getExercisesFromUserWithId(userId);
  
  
  if(from) {
    const fromDate = new Date(from);
    log = log.filter(exe => new Date(exe.date) >= fromDate);
  }
  
  if(to) {
    const toDate = new Date(to);
    log = log.filter(exe => new Date(exe.date) <= toDate);
  }
  
  if(limit) {
    log = log.slice(0, +limit);
  }
  
  res.json({
    _id: userId,
    username: getUsernameById(userId),
    count: log.length,
    log
  });
});

// Not found middleware
app.use((req, res, next) => {
  return next({status: 404, message: 'not found'})
})

// Error Handling middleware
app.use((err, req, res, next) => {
  let errCode, errMessage

  if (err.errors) {
    // mongoose validation error
    errCode = 400 // bad request
    const keys = Object.keys(err.errors)
    // report the first validation error
    errMessage = err.errors[keys[0]].message
  } else {
    // generic or custom error
    errCode = err.status || 500
    errMessage = err.message || 'Internal Server Error'
  }
  res.status(errCode).type('txt')
    .send(errMessage)
});

const listener = app.listen(process.env.PORT || 3000, () => {
  console.log('Your app is listening on port ' + listener.address().port)
})

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.138 Safari/537.36.

Challenge: Exercise Tracker

Link to the challenge:

2

As written your app will always assign the dateObj to the current timestamp.
you are strictly comparing an undefined variable to an empty string, which are not strictly equal.

1 Like