Skip to content

Commit 4bfd3fd

Browse files
committed
🚧 updated routes and controller
1 parent c8e0c42 commit 4bfd3fd

File tree

3 files changed

+14
-21
lines changed

3 files changed

+14
-21
lines changed

src/controllers/movie.js

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,9 @@
1-
class MovieController {
2-
constructor(Movie) {
3-
this.Movie = Movie;
4-
}
1+
import Movie from '../models/movie';
52

3+
class MovieController {
64
async get(req, res) {
75
try {
8-
const movies = await this.Movie.find({});
6+
const movies = await Movie.find({});
97
res.send(movies);
108
} catch (err) {
119
res.status(400).send(err.message);
@@ -18,15 +16,15 @@ class MovieController {
1816
params: { id },
1917
} = req;
2018

21-
const movie = await this.Movie.find({ _id: id });
19+
const movie = await Movie.find({ _id: id });
2220
res.send(movie);
2321
} catch (err) {
2422
res.status(400).send(err.message);
2523
}
2624
}
2725

2826
async create(req, res) {
29-
const movie = new this.Movie(req.body);
27+
const movie = new Movie(req.body);
3028
try {
3129
await movie.save();
3230
res.status(201).send(movie);
@@ -37,7 +35,7 @@ class MovieController {
3735

3836
async update(req, res) {
3937
try {
40-
await this.Movie.updateOne({ _id: req.params.id }, req.body);
38+
await Movie.updateOne({ _id: req.params.id }, req.body);
4139
res.sendStatus(200);
4240
} catch (err) {
4341
res.status(422).send(err.message);
@@ -46,12 +44,12 @@ class MovieController {
4644

4745
async delete(req, res) {
4846
try {
49-
await this.Movie.deleteOne({ _id: req.params.id });
47+
await Movie.deleteOne({ _id: req.params.id });
5048
res.sendStatus(204);
5149
} catch (err) {
5250
res.status(400).send(err.message);
5351
}
5452
}
5553
}
5654

57-
export default MovieController;
55+
export default new MovieController();

src/routes/index.js

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,9 @@
1-
import express from 'express';
1+
import { Router } from 'express';
22
import movieRouter from './movie';
33

4-
const router = express.Router();
4+
const router = new Router();
55

6+
router.get('/healths', (req, res) => res.status(200).json({ status: 'UP' }));
67
router.use('/movies', movieRouter);
78

8-
router.get('/', (req, res) => {
9-
res.send('Hello World!');
10-
});
11-
129
export default router;

src/routes/movie.js

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
1-
import express from 'express';
2-
import MovieController from '../controllers/movie';
3-
import Movie from '../models/movie';
1+
import { Router } from 'express';
2+
import movieController from '../controllers/movie';
43

5-
const router = express.Router();
6-
const movieController = new MovieController(Movie);
4+
const router = new Router();
75

86
router.get('/', (req, res) => movieController.get(req, res));
97
router.get('/:id', (req, res) => movieController.getById(req, res));

0 commit comments

Comments
 (0)