【问题标题】:Simplifying Persistent & Esqueleto code简化 Persistent & Esqueleto 代码
【发布时间】:2015-04-27 16:32:10
【问题描述】:

我有一个相当简单的查询,它执行两个外连接。 (一顿饭有很多食谱,而食谱又反过来有很多食物)。

getMeals :: (MonadIO m) => Key DbUser -> SqlPersistT m [Meal]
getMeals user =
  fmap deserializeDb $ E.select $
        E.from $ \(m `E.InnerJoin` u `E.LeftOuterJoin` r `E.LeftOuterJoin` f) -> do
          E.on     (r ?. DbRecipeId E.==. f ?. DbFoodRecipeId)
          E.on     (E.just (m ^. DbMealId) E.==. r ?. DbRecipeMealId)
          E.on     (m ^. DbMealUserId      E.==. u ^. DbUserId)
          E.where_ (m ^. DbMealUserId      E.==. E.val user )
          return (m, r, f)

这个查询很棒,它说明了它需要什么,没有任何其他内容。但是,由于 SQL 的工作方式,它为我返回了一个包含许多重复餐点的表,对于每个匹配的外连接。

例如,一顿饭有两个菜谱,每个菜谱有两种食物变成 4 个元组。

(m1, r1, f1)
(m1, r1, f2)
(m1, r2, f3)
(m1, r2, f4)

我想将这些回滚到单个 Meal 数据类型中。 (这里简化显示结构,其他字段当然存储在数据库中)。

data Meal   = Meal   { recipes :: [Recipe] }
data Recipe = Recipe { foods :: [Food]   }
data Food   = Food   { name :: String }

我似乎必须完全手动进行此合并,而对于这个单个查询,它最终需要 2 页左右的代码。

忽略不应该这样使用类型类的事实,它看起来像很多(愚蠢的)类型类的实例DeserializeDb

class DeserializeDb a r | a -> r where
  deserializeDb :: a -> r

instance DeserializeDb [(Entity DbMeal, Maybe (Entity DbRecipe))] [Meal] where
  deserializeDb items = let grouped = groupBy (\a b -> entityKey (fst a) == entityKey (fst b)) items
                            joined  = map (\list -> ( (fst . head) list
                                                    ,  mapMaybe snd list
                                                    )) grouped
                        in (map deserializeDb joined)

截取大量各种复杂的实例(代码:https://gist.github.com/cschneid/2989057ec4bb9875e2ae

instance DeserializeDb (Entity DbFood) Food where
  deserializeDb (Entity _ val) = Food (dbFoodName val)

问题:

我唯一想公开的是查询签名。其余的都是实施垃圾。有没有我没有注意到的使用 Persistent 的技巧?我是否必须手动将联接合并回 haskell 类型?

【问题讨论】:

  • 您的问题似乎与stackoverflow.com/questions/21686579/… 非常相似,但我认为您不会喜欢答案 (-:
  • 我不介意手动执行此操作,除非它的复杂性似乎增长得很快。这里的 3 表连接是 7 行查询和 80 行反序列化。必须有更好的方法来处理这个问题。
  • 我认为你应该只有属性的反序列化代码,并在元组中获得一个没有食谱的膳食,一个没有食物的食谱:(膳食,食谱,食物)然后将所有结果分组到一个方法中,它会简单得多。
  • 对不起?我不跟。问题是元组的(Meal 部分重复,每个独特的食谱/食物一次。所以大部分groupBy 垃圾都在解决这个问题。有更好的方法吗?
  • 我只是在想您将数据库的反序列化和分组结果混为一谈。具有从实体 DbMeal 转换为 Meal 等的基本功能可能会更容易,然后具有无需任何 DB ([(Meal,Recipe,Food)]->[Meal] 即可执行分组的功能。您可能是能够抽象更多并减少样板的数量。

标签: haskell yesod persistent esqueleto


【解决方案1】:

感谢@JPMoresmau 的暗示,我最终得到了一个更短且我认为更简单的方法。由于nub,它在大型数据集上可能会更慢,但在小型数据集上,它的返回速度比我需要的要快得多。

我仍然讨厌我有这么多的手动管道来根据从数据库返回的数据构建树结构。我想知道是否有一种通用的好方法?

module Grocery.Database.Calendar where

import Grocery.DatabaseSchema
import Grocery.Types.Meal
import Grocery.Types.Recipe
import Grocery.Types.Food
import Database.Persist
import Database.Persist.Sqlite
import qualified Database.Esqueleto      as E
import           Database.Esqueleto      ((^.), (?.))
import Data.Time
import Control.Monad.Trans -- for MonadIO
import Data.List
import Data.Maybe
import Data.Tuple3

getMeals :: (MonadIO m) => Key DbUser -> SqlPersistT m [Meal]
getMeals user =
  fmap deserializeDb $ E.select $
        E.from $ \(m `E.InnerJoin` u `E.LeftOuterJoin` r `E.LeftOuterJoin` f) -> do
          E.on     (r ?. DbRecipeId E.==. f ?. DbFoodRecipeId)
          E.on     (E.just (m ^. DbMealId) E.==. r ?. DbRecipeMealId)
          E.on     (m ^. DbMealUserId      E.==. u ^. DbUserId)
          E.where_ (m ^. DbMealUserId      E.==. E.val user )
          return (m, r, f)

deserializeDb :: [(Entity DbMeal, Maybe (Entity DbRecipe), Maybe (Entity DbFood))] -> [Meal]
deserializeDb results = makeMeals results
  where
    makeMeals :: [(Entity DbMeal, Maybe (Entity DbRecipe), Maybe (Entity DbFood))] -> [Meal]
    makeMeals dupedMeals = map makeMeal (nub $ map fst3 dupedMeals)

    makeMeal :: Entity DbMeal -> Meal
    makeMeal (Entity k m) = let d = dbMealDay m
                                n = dbMealName m
                                r = makeRecipesForMeal k
                            in  Meal Nothing (utctDay d) n r

    makeRecipesForMeal :: Key DbMeal -> [Recipe]
    makeRecipesForMeal mealKey = map makeRecipe $ appropriateRecipes mealKey

    appropriateRecipes :: Key DbMeal -> [Entity DbRecipe]
    appropriateRecipes mealKey = nub $ filter (\(Entity _ v) -> dbRecipeMealId v == mealKey) $ mapMaybe snd3 results

    makeRecipe :: Entity DbRecipe -> Recipe
    makeRecipe (Entity k r) = let n = dbRecipeName r
                                  f = makeFoodForRecipe k
                              in  Recipe Nothing n f

    makeFoodForRecipe :: Key DbRecipe -> [Food]
    makeFoodForRecipe rKey = map makeFood $ appropriateFoods rKey

    appropriateFoods :: Key DbRecipe -> [Entity DbFood]
    appropriateFoods rKey = nub $ filter (\(Entity _ v) -> dbFoodRecipeId v == rKey) $ mapMaybe thd3 results

    makeFood :: Entity DbFood -> Food
    makeFood (Entity _ f) = Food (dbFoodName f)

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2011-08-09
    • 2014-10-10
    • 2015-01-03
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多