約束でサービスレイヤーをコーディング

Question

CRUD Logicをルートから取り出して、サービスレイヤーに配置したいと考えています。

だから、基本的に私はそのようなサービス層を呼び出したい：

function createPost(titel, description, tags, callback) { 

    const post = { 
     titel: titel, 
     description: description, 
     tags: tags, 
     createdAt: new Date(), 
     deleted: false, 
    } 
    console.log("Create Post: " + post.titel + " " + post.description + " " + post.tags + " " + post.createdAt + " " + post.deleted) 

    knex("posts").insert(post, "id").then(id => { 
     console.log(id) 
     callback(id[0]) 
    }) 
} 

は現在、私はこの機能を処理するためにcallbackを使用しています：私は、以下の機能を持っている私のpostService.jsファイルで

const service = require("../service/post") 

router.post("/new", (req, res) => { 
    service.createPost(req.body.titel, req.body.description, req.body.tags, function(id){ 
     console.log("Created post with id: " + id) 
     res.redirect("index") 
    }) 
}) 

。

idを返すために、より有望なベースのスタイルを使用する方法と、ルータのコードが約束が終わるのを待っていますか？

あなたの返信ありがとうございます！あなたの例では

Answer 1

、あなたがあれば、あなたのルートでは、あなたが、代わりに

router.post('/new', async (req, res) => { 
    const id = await service.createPost(...); 
    console.log("Created post with id: " + id[0]); 
    res.redirect("index"); 
}); 

結果をawaitことができますcallbackパラメータを取り除くとknex

createPost(...) { 
    ... 
    return knex('posts').insert(post, "id"); 
} 

によって返された約束を返す取得することができますknex（配列を返します）の応答を前処理したい場合は、新しいPromiseを返すことができます

async createPost(...) { 
    ... 
    const result = await knex('posts').insert(...); 
    return result[0]; 
} 

FWIW後者はレイヤ間をきれいに分離するので、後者をお勧めします。