FACN3 / spend500

week 7 project
0 stars 1 forks source link

Unnecessary passing around of req/res #48

Open finnhodgkin opened 6 years ago

finnhodgkin commented 6 years ago

A fair few of the functions in your application aren't making use of the arguments you give them.

Examples include:

handleAuth - doesn't need res handleBuy - doesn't need req handleCart - doesn't need req, but may do once you change the username to not be hard coded... handleError - doesn't need req

It's not a big deal and gives you more room to change functionality later, but still worth considering.