feat(default-params): replace || cond with default params (#153)

Author: etroynov

lib/layer.js

@@ -17,8 +17,8 @@ module.exports = Layer;
  * @private
  */
 
-function Layer(path, methods, middleware, opts) {
-  this.opts = opts || {};
+function Layer(path, methods, middleware, opts = {}) {
+  this.opts = opts;
   this.name = this.opts.name || null;
   this.methods = [];
   this.paramNames = [];
@@ -60,14 +60,12 @@ Layer.prototype.match = function (path) {
  *
  * @param {String} path
  * @param {Array.<String>} captures
- * @param {Object=} existingParams
+ * @param {Object=} params
  * @returns {Object}
  * @private
  */
 
-Layer.prototype.params = function (path, captures, existingParams) {
-  const params = existingParams || {};
-
+Layer.prototype.params = function (path, captures, params = {}) {
   for (let len = captures.length, i = 0; i < len; i++) {
     if (this.paramNames[i]) {
       const c = captures[i];

lib/router.js

@@ -47,10 +47,10 @@ module.exports = Router;
  * @constructor
  */
 
-function Router(opts) {
+function Router(opts = {}) {
   if (!(this instanceof Router)) return new Router(opts);
 
-  this.opts = opts || {};
+  this.opts = opts;
   this.methods = this.opts.methods || [
     'HEAD',
     'OPTIONS',
@@ -430,8 +430,7 @@ Router.prototype.routes = Router.prototype.middleware = function () {
  * @returns {Function}
  */
 
-Router.prototype.allowedMethods = function (options) {
-  options = options || {};
+Router.prototype.allowedMethods = function (options = {}) {
   const implemented = this.methods;
 
   return function allowedMethods(ctx, next) {
@@ -562,9 +561,7 @@ Router.prototype.redirect = function (source, destination, code) {
  * @private
  */
 
-Router.prototype.register = function (path, methods, middleware, opts) {
-  opts = opts || {};
-
+Router.prototype.register = function (path, methods, middleware, opts = {}) {
   const router = this;
   const stack = this.stack;