Skip to content

Scott Tenefrancia 311 Checkpoint 1 #84

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
tenefra wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Scott Tenefrancia 311 Checkpoint 1 #84

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
bcd7edf
init commit
tenefra May 24, 2021
86de7b5
Error handling added
tenefra May 24, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Error handling added
  • Loading branch information
@tenefra
tenefra committed May 24, 2021
commit 86de7b54365776e3f6b07b2d22b408c8a38fc239
17 changes: 16 additions & 1 deletion controllers/users.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ const listUsers = (req, res) => {
}

const showUser = (req, res) => {
if (req.params.id > users.length) {
res.sendStatus(404)
}
users.forEach(user => {
if (user.id == req.params.id) {
res.json(user)
Expand All @@ -17,12 +20,21 @@ const createUser = (req, res) => {
let counter = ++users.length
users.push({
id: ++counter,
body: req.body.message
name: req.body.name,
username: req.body.username,
email: req.body.email,
address: req.body.address,
phone: req.body.phone,
website: req.body.website,
company: req.body.company
})
res.json(users[users.length - 1])
}

const updateUser = (req, res) => {
if (req.params.id > users.length) {
res.sendStatus(404)
}
users.forEach(user => {
if (user.id == req.params.id) {
let index = users.indexOf(user)
Expand All @@ -33,6 +45,9 @@ const updateUser = (req, res) => {
}

const deleteUser = (req, res) => {
if (req.params.id > users.length) {
res.sendStatus(404)
}
users.forEach(user => {
if (user.id == req.params.id) {
let index = users.indexOf(user)
Expand Down