MOD amélioration de la complexité de la fonction euphorik_bd:nouveau_message/3 mais...

diff --git a/modules/erl/euphorik_bd.erl b/modules/erl/euphorik_bd.erl
index e5afe74..81a36a0 100755 (executable)
--- a/modules/erl/euphorik_bd.erl
+++ b/modules/erl/euphorik_bd.erl
@@ -276,11 +276,14 @@ user_by_mess(Id) ->
 % retourne soit l'id du message soit {erreur, <raison>}.\r
 nouveau_message(Mess, Auteur_id, Repond_A_ids) ->\r
    % regarde si les id 'Repond_A' existent\r
-   F = fun() ->   \r
-      Repond_a = e(\r
-         q([M || M <- mnesia:table(minichat), lists:member(M#minichat.id, Repond_A_ids)]),\r
-         [{tmpdir, ?KEY_SORT_TEMP_DIR}]\r
-      ), \r
+   F = fun() ->         \r
+      Repond_a = lists:map(\r
+         fun(Repond_a_id) ->\r
+            [M] = mnesia:wread({minichat, Repond_a_id}),\r
+            M\r
+         end,\r
+         Repond_A_ids\r
+      ),      \r
       Racine_id = case Repond_a of\r
          [] -> undefined;\r
          [M | _] -> \r
@@ -297,37 +300,38 @@ nouveau_message(Mess, Auteur_id, Repond_A_ids) ->
          {erreur, E} -> {erreur, E};\r
          _ ->\r
             % est-ce que l'auteur existe ?\r
-            case e(q([E || E <- mnesia:table(user), E#user.id =:= Auteur_id]), [{tmpdir, ?KEY_SORT_TEMP_DIR}]) of\r
+            case mnesia:wread({user, Auteur_id}) of\r
                [#user{profile = Profile} = Auteur] ->\r
                   if length(Repond_a) =/= length(Repond_A_ids) ->\r
                         {erreur, "Un ou plusieurs messages introuvable"};\r
                      true ->\r
                         % comparaison entre la date du dernier poste et maintenant (gestion du flood)\r
-                        Delta = euphorik_common:delta_date_ms(Auteur#user.date_derniere_connexion, now()),\r
+                        Now = now(),\r
+                        Delta = euphorik_common:delta_date_ms(Auteur#user.date_derniere_connexion, Now),\r
                         Nouvel_indice_flood = Auteur#user.indice_flood + if Delta =< ?DUREE_SPAM -> 2; true -> -1 end,\r
                         Auteur_maj = Auteur#user{\r
                            indice_flood = if Nouvel_indice_flood > ?INDICE_SPAM_MAX -> ?INDICE_SPAM_MAX; Nouvel_indice_flood < 0 -> 0; true -> Nouvel_indice_flood end,\r
-                           date_derniere_connexion = now()\r
+                           date_derniere_connexion = Now\r
                         },\r
                         % est-ce que l'auteur à trop floodé ?\r
                         if Auteur#user.indice_flood =/= ?INDICE_SPAM_MAX, Auteur_maj#user.indice_flood =:= ?INDICE_SPAM_MAX, Delta =< ?DUREE_BLOCAGE_SPAM ->\r
-                           mnesia:write(Auteur#user{indice_flood = Auteur_maj#user.indice_flood}),\r
-                           nouveau_message_sys("\"" ++ Profile#profile.pseudo ++ if Auteur#user.login =/= [] -> " (" ++ Auteur#user.login ++ ")"; true -> "" end ++ "\" est bloqué pour " ++ integer_to_list(trunc(?DUREE_BLOCAGE_SPAM / 1000)) ++ " secondes pour cause de flood.");\r
-                        Auteur#user.indice_flood =:= ?INDICE_SPAM_MAX, Delta =< ?DUREE_BLOCAGE_SPAM ->\r
-                           {erreur, "Bloqué pour cause de flood"};\r
-                        true ->     \r
-                           mnesia:write(Auteur_maj),\r
-                           Id = nouvel_id(minichat),\r
-                           inserer_reponses(Id, Repond_A_ids),\r
-                           mnesia:write(#minichat{\r
-                              id = Id,\r
-                              auteur_id = Auteur#user.id,\r
-                              date = now(),\r
-                              pseudo = Profile#profile.pseudo,\r
-                              contenu = Mess,\r
-                              racine_id = if Racine_id =:= undefined -> Id; true -> Racine_id end\r
-                           }),\r
-                           Id\r
+                              mnesia:write(Auteur#user{indice_flood = Auteur_maj#user.indice_flood}),\r
+                              nouveau_message_sys("\"" ++ Profile#profile.pseudo ++ if Auteur#user.login =/= [] -> " (" ++ Auteur#user.login ++ ")"; true -> "" end ++ "\" est bloqué pour " ++ integer_to_list(trunc(?DUREE_BLOCAGE_SPAM / 1000)) ++ " secondes pour cause de flood.");\r
+                           Auteur#user.indice_flood =:= ?INDICE_SPAM_MAX, Delta =< ?DUREE_BLOCAGE_SPAM ->\r
+                              {erreur, "Bloqué pour cause de flood"};\r
+                           true ->     \r
+                              mnesia:write(Auteur_maj),\r
+                              Id = nouvel_id(minichat),\r
+                              inserer_reponses(Id, Repond_A_ids),\r
+                              mnesia:write(#minichat{\r
+                                 id = Id,\r
+                                 auteur_id = Auteur#user.id,\r
+                                 date = Now,\r
+                                 pseudo = Profile#profile.pseudo,\r
+                                 contenu = Mess,\r
+                                 racine_id = if Racine_id =:= undefined -> Id; true -> Racine_id end\r
+                              }),\r
+                              Id\r
                         end\r
                   end;\r
                _ ->\r

diff --git a/modules/erl/euphorik_test.erl b/modules/erl/euphorik_test.erl
index f07941c..c1bf49b 100644 (file)
--- a/modules/erl/euphorik_test.erl
+++ b/modules/erl/euphorik_test.erl
@@ -28,7 +28,11 @@
    bench_get_messages_avec_2_conversations/0
 ]).
 -include("../include/euphorik_bd.hrl").
-
+\r
+% les intervalles en seconde min en max entre deux postes de message d'un utilisateur\r
+% le temps d'attente est choisi au hasard entre ces deux valeurs\r
+-define(INTERVALLE_MIN, 2).\r
+-define(INTERVALLE_MAX, 5).
 
 % N est le nombre d'utilisateur
 % M est le nombre de message que chaque utilisateur va poster
@@ -163,10 +167,11 @@ tire_element_rand(N, L, Elements) ->
 loop(User_id, 0) ->
    io:format("~p a fini~n", [User_id]);
 loop(User_id, M) -> 
-   % attend un temp aléatoire compris entre 2 sec et 10 sec
-   timer:sleep(2000 * random:uniform(10)),
+   % attend un temp aléatoire compris entre INTERVALLE_MIN sec et INTERVALLE_MAX sec
+   timer:sleep(1000 * (random:uniform(?INTERVALLE_MAX - ?INTERVALLE_MIN + 1) + ?INTERVALLE_MIN - 1)),
    % poste un message aléatoire par une personne aléatoire répondant à des messages aléatoires
-   {Message, Repond_a} = {message_rand(), messages_id_rand()},
+   {Message, Repond_a} = {message_rand(), messages_id_rand()},\r
+   %{Message, Repond_a} = {"blablablablablabla", []},
    io:format("~p poste ~p et repond a ~w~n", [User_id, Message, Repond_a]),
    case euphorik_bd:nouveau_message(Message, User_id, Repond_a) of
       {erreur, E} -> 

diff --git a/tools/start_yaws.sh b/tools/start_yaws.sh
index 1a2a144..87e6e4e 100755 (executable)
--- a/tools/start_yaws.sh
+++ b/tools/start_yaws.sh
@@ -1,2 +1,3 @@
-#!/bin/bash
-screen -S yaws_preprod yaws --conf ./yaws.conf --sname yaws_dev --mnesiadir "../var/BD/" -I debian_yaws_dev
+#!/bin/bash\r
+# screen est utilisé par exemple pour lancé une version de preproduction et permettre de la faire tourner après un délog
+screen -S yawspreprod yaws --conf ./yaws.conf --sname yaws_dev --mnesiadir "../var/BD/" -I debian_yaws_dev