MOD optimisation des fonctions est_une_reponse_a_user et a_repondu_a_message du modul...

diff --git a/modules/erl/euphorik_bd.erl b/modules/erl/euphorik_bd.erl
index 196a26e..1cd534e 100755 (executable)
--- a/modules/erl/euphorik_bd.erl
+++ b/modules/erl/euphorik_bd.erl
@@ -190,7 +190,11 @@ user_by_cookie(Cookie) ->
 user_by_id(ID) ->\r
    resultat_transaction(mnesia:transaction(\r
       fun() ->\r
-         case e(q([E || E <- mnesia:table(user), E#user.id =:= ID]), [{tmpdir, ?KEY_SORT_TEMP_DIR}]) of\r
+%         case e(q([E || E <- mnesia:table(user), E#user.id =:= ID]), [{tmpdir, ?KEY_SORT_TEMP_DIR}]) of\r
+%            [User] -> {ok, User};\r
+%            _ -> erreur\r
+%         end\r
+         case mnesia:read({user,  ID}) of\r
             [User] -> {ok, User};\r
             _ -> erreur\r
          end\r
@@ -378,15 +382,14 @@ message_by_id(Id) ->
    resultat_transaction(mnesia:transaction(\r
       fun() ->\r
          case mnesia:read({minichat, Id}) of\r
-            [] -> erreur;\r
-            [M] ->\r
-               {ok, M#minichat{contenu = contenu_message(M)}}\r
+            [M] -> {ok, M#minichat{contenu = contenu_message(M)}};\r
+            _ -> erreur\r
          end\r
       end\r
    )).\r
    \r
    \r
-% Renvoie le contenu d'un message donnée, à utiliser à l'intérieur d'une transaction.\r
+% Renvoie le contenu d'un message donnée en fonction du troll associé, à utiliser à l'intérieur d'une transaction.\r
 % TODO : Cette fonction pourrait être remplacé par un "outer-join", est-ce possible avec qlc ?\r
 contenu_message(E) ->\r
    case mnesia:read({troll, E#minichat.troll_id}) of\r
@@ -483,33 +486,47 @@ enfants_id(M) ->
      \r
 \r
 % Est-ce que le message Id_mess est une réponse d'une message de Id_user ?\r
+% On evite d'utiliser qlc (ce qui était fait avant) pour des raisons de performance.\r
 est_une_reponse_a_user(Id_user, Id_mess) ->\r
-   case mnesia:transaction(\r
-            fun() ->\r
-               e(q([\r
-                  M#minichat.auteur_id || M <- mnesia:table(minichat), R <- mnesia:table(reponse_minichat),\r
-                  M#minichat.auteur_id =:= Id_user, M#minichat.id =:= R#reponse_minichat.cible, R#reponse_minichat.repondant =:= Id_mess\r
-               ]), [{unique_all, true}, {tmpdir, ?KEY_SORT_TEMP_DIR}])\r
-            end\r
-         ) of\r
-      {atomic, [_]} -> true;\r
-      _ -> false\r
-   end.\r
+   resultat_transaction(mnesia:transaction(\r
+      fun() ->\r
+         case mnesia:read({reponse_minichat, Id_mess}) of\r
+            [] -> false;\r
+            Cibles ->\r
+               lists:any(\r
+                  fun(#reponse_minichat{cible = Cible}) ->\r
+                     case mnesia:read({minichat, Cible}) of\r
+                        [#minichat{auteur_id = Id_user}] -> true;\r
+                        _ -> false\r
+                     end\r
+                  end,\r
+                  Cibles\r
+               )\r
+         end\r
+      end\r
+   )).\r
 \r
    \r
 % Est-ce que Id_user à répondu au message Id_mess\r
+% On evite d'utiliser qlc (ce qui était fait avant) pour des raisons de performance.\r
 a_repondu_a_message(Id_user, Id_mess) ->\r
-   case mnesia:transaction(\r
-            fun() ->\r
-               e(q([\r
-                  M#minichat.auteur_id || M <- mnesia:table(minichat), R <- mnesia:table(reponse_minichat),\r
-                  R#reponse_minichat.cible =:= Id_mess, R#reponse_minichat.repondant =:= M#minichat.id, M#minichat.auteur_id =:= Id_user\r
-               ]), [{unique_all, true}, {tmpdir, ?KEY_SORT_TEMP_DIR}])\r
-            end\r
-         ) of\r
-      {atomic, [_]} -> true;\r
-      _ -> false\r
-   end.\r
+   resultat_transaction(mnesia:transaction(\r
+      fun() ->\r
+         case mnesia:match_object({reponse_minichat, '_', Id_mess}) of\r
+            [] -> false;\r
+            Reponses ->\r
+               lists:any(\r
+                  fun(#reponse_minichat{repondant = Reponse}) ->\r
+                     case mnesia:read({minichat, Reponse}) of\r
+                        [#minichat{auteur_id = Id_user}] -> true;\r
+                        _ -> false\r
+                     end\r
+                  end,\r
+                  Reponses\r
+               )\r
+         end\r
+      end\r
+   )).\r
    \r
    \r
 % Est-ce que Id_user possède Id_mess ?\r

diff --git a/modules/erl/euphorik_requests.erl b/modules/erl/euphorik_requests.erl
index efe2e06..f497382 100755 (executable)
--- a/modules/erl/euphorik_requests.erl
+++ b/modules/erl/euphorik_requests.erl
@@ -23,7 +23,7 @@
 \r
 \r
 -module(euphorik_requests).\r
--export([out/1]).\r
+-export([out/1, traiter_donnees/2]).\r
 -include_lib("xmerl/include/xmerl.hrl").\r
 -include_lib("yaws/include/yaws_api.hrl").\r
 -include("../include/euphorik_defines.hrl").\r

diff --git a/modules/erl/euphorik_test.erl b/modules/erl/euphorik_test.erl
index 8b8dbfb..b705fc6 100644 (file)
--- a/modules/erl/euphorik_test.erl
+++ b/modules/erl/euphorik_test.erl
@@ -73,18 +73,18 @@ mot_rand(L, Mot) ->
 
 % Tire au hasard de 0 à 3 messages sur les 10 derniers postés, renvoie une liste de int()
 % répartition : 
-%  0 : 0.5
-%  1 : 0.3
+%  0 : 0.1
+%  1 : 0.7
 %  2 : 0.15
 %  3 : 0.05
 messages_id_rand() ->
    R = random:uniform(),
-   if R =< 0.5 ->
+   if R =< 0.1 ->
          [];
       true ->
          Messages = lists:map(fun(#minichat{id = Id}) -> Id end, euphorik_bd:messages(8)),
          if
-            R > 0.5 andalso R =< 0.8 ->
+            R > 0.1 andalso R =< 0.8 ->
                tire_element_rand(1, Messages);
             R > 0.8 andalso R =< 0.95 ->
                tire_element_rand(2, Messages);