mirror of
				https://github.com/goldbergyoni/nodebestpractices.git
				synced 2025-10-31 09:38:39 +08:00 
			
		
		
		
	Constructor function in error should be uppercase
Its perhaps a small thing but important for newbies I guess as constructor classes (functions) are usually uppercased in javascript: https://stackoverflow.com/questions/1564398/javascript-method-naming-lowercase-vs-uppercase
This commit is contained in:
		| @ -36,20 +36,20 @@ if(!productToAdd) | ||||
|  | ||||
| ```javascript | ||||
| // centralized error object that derives from Node’s Error | ||||
| function appError(name, httpCode, description, isOperational) { | ||||
| function AppError(name, httpCode, description, isOperational) { | ||||
|     Error.call(this); | ||||
|     Error.captureStackTrace(this); | ||||
|     this.name = name; | ||||
|     //...other properties assigned here | ||||
| }; | ||||
|  | ||||
| appError.prototype.__proto__ = Error.prototype; | ||||
| AppError.prototype.__proto__ = Error.prototype; | ||||
|  | ||||
| module.exports.appError = appError; | ||||
| module.exports.AppError = AppError; | ||||
|  | ||||
| // client throwing an exception | ||||
| if(user == null) | ||||
|     throw new appError(commonErrors.resourceNotFound, commonHTTPErrors.notFound, "further explanation", true) | ||||
|     throw new AppError(commonErrors.resourceNotFound, commonHTTPErrors.notFound, "further explanation", true) | ||||
| ``` | ||||
|  | ||||
| ### Blog Quote: "I don’t see the value in having lots of different types" | ||||
|  | ||||
		Reference in New Issue
	
	Block a user
	 Mitja O
					Mitja O