response formatter

.eslintrc

@@ -1,20 +1,3 @@
 {
-  "extends": "sactive",
-  "rules": {
-    "space-unary-ops":
-      ["error", {
-        "words": true,
-        "nonwords": true,
-        "overrides": {
-          "!": false
-        }
-      }],
-    "no-useless-call": 0,
-    "space-before-function-paren": ["error", {
-      "anonymous": "never",
-      "named": "never",
-      "asyncArrow": "ignore"
-    }],
-    "new-cap": ["error", { "newIsCap": false }]
-  }
+  "extends": "sactive"
 }

lib/application.js

@@ -6,9 +6,17 @@ const _ = require('lodash');
 const {Sactive} = require('./di');
 const {Route} = require('./route');
 const Log = require('./logger');
+const {
+  ResponseProcessorFactory,
+  JsonProcessor,
+  HtmlProcessor
+} = require('./response_formatter/response');
 const binder = new Sactive();
 binder.bindClass('logger', Log);
 binder.bindClass('route', Route);
+binder.bindClass('jsonProcessor', JsonProcessor);
+binder.bindClass('htmlProcessor', HtmlProcessor);
+binder.bindClass('responseProcessor', ResponseProcessorFactory);
 
 /**
  * Create a new `Application`.

lib/response_formatter/response.js

@@ -1,29 +1,28 @@
 const _ = require('lodash');
 const {UnifiedError} = require('./error');
 
-class responseProcessorFactory {
-  constructor(processors) {
-    this.processors = processors;
+class ResponseProcessorFactory {
+  constructor($$jsonProcessor, $$htmlProcessor) {
+    this.processors = [$$jsonProcessor, $$htmlProcessor];
   }
 
   determine(ctx) {
-    ctx.accepts('text/html');
     return _.find(this.processors, processor => {
       return processor.determine(ctx);
     });
   }
 }
 
 class JsonProcessor {
-  processSuccess(response, ctx) {
+  processSuccess(ctx, response) {
     ctx.body = {
       code: 200,
       msg: 'success.',
       data: response
     };
   }
 
-  processFail(response, ctx) {
+  processFail(ctx, response) {
     let result = null;
     if (!response) {
       response = new UnifiedError();
@@ -42,16 +41,16 @@ class JsonProcessor {
     if (_.isEmpty(contentType)) {
       return false;
     }
-    return ctx.is(contentType).includes('json');
+    return ctx.accepts('json') && ctx.is(contentType).includes('json');
   }
 }
 
 class HtmlProcessor {
-  processSuccess(response, ctx) {
+  processSuccess(ctx, response) {
     ctx.body = response;
   }
 
-  processFail(response, ctx) {
+  processFail(ctx, response) {
     let result = null;
     if (!response) {
       response = new UnifiedError();
@@ -67,11 +66,11 @@ class HtmlProcessor {
   }
 
   determine(ctx) {
-    return true;
+    return ctx.accepts('html');
   }
 }
 module.exports = {
-  responseProcessorFactory,
+  ResponseProcessorFactory,
   JsonProcessor,
   HtmlProcessor
 };