]> git.scottworley.com Git - reliable-chat/commitdiff
Keep messages sorted by timestamp
authorScott Worley <sworley@chkno.net>
Fri, 3 Aug 2012 18:30:52 +0000 (11:30 -0700)
committerScott Worley <sworley@chkno.net>
Fri, 3 Aug 2012 18:30:52 +0000 (11:30 -0700)
This causes messages that appear on some servers and not others to be
correctly interleaved on initial load.

Note: this is incomplete. rcupdatemessagetime() can break history's
always-in-sorted-order invariant by changing Time after messages have
been inserted.  Fixing this is next.

Consider replacing rcaddmessagetohistory()'s simple backwards linear scan
with a binary search in the unlikely event that its performance is ever
noticed.

webclient/rc.html

index f3abaf3faadee799136099b6eaf9b685695db784..c3284da03b1480583a40cfb4eb93b0d332d22196 100644 (file)
@@ -82,6 +82,7 @@
        var session = Math.random();  // For outgoing message IDs
        var since = {};    // server -> time: For fetch?since=
        var seen = {};     // seen_key -> message
        var session = Math.random();  // For outgoing message IDs
        var since = {};    // server -> time: For fetch?since=
        var seen = {};     // seen_key -> message
+       var history = [];  // List of messages sorted by Time
        // Messages have these fields:
        //   Time: The timestamp.  Median of ServerTimes
        //   ID: Some unique string for deduping
        // Messages have these fields:
        //   Time: The timestamp.  Median of ServerTimes
        //   ID: Some unique string for deduping
        }
 
        function rcaddmessagetohistory(message) {
        }
 
        function rcaddmessagetohistory(message) {
-               message.UI = document.createElement("div");
+               var message_i;
+               if (message.Time) {
+                       for (var i = history.length - 1; ; i--) {
+                               if (i < 0 || (history[i].Time && message.Time >= history[i].Time)) {
+                                       message_i = i+1;
+                                       history.splice(message_i, 0, message);
+                                       break;
+                               }
+                       }
+               } else {
+                       history.push(message);
+                       message_i = history.length-1;
+               }
+
                var text = (message.Time ? rcformattime(message.Time) : "") + " " + message.Text;
                var text = (message.Time ? rcformattime(message.Time) : "") + " " + message.Text;
+               message.UI = document.createElement("div");
                message.UI.appendChild(document.createTextNode(text));
                var h = document.getElementById("history");
                message.UI.appendChild(document.createTextNode(text));
                var h = document.getElementById("history");
-               h.appendChild(message.UI);
+               if (message_i + 1 < history.length) {
+                       h.insertBefore(message.UI, history[message_i + 1].UI);
+               } else {
+                       h.appendChild(message.UI);
+               }
                window.scrollTo(0, document.body.scrollHeight);
        }
 
                window.scrollTo(0, document.body.scrollHeight);
        }
 
        }
 
        function rcsend(d, message) {
        }
 
        function rcsend(d, message) {
-               var id = new Date().getTime() + "-" + session + "-" + Math.random();
-               seen[make_seen_key(id, message)] = message;
+               message.ID = new Date().getTime() + "-" + session + "-" + Math.random();
+               seen[make_seen_key(message.ID, message.Text)] = message;
                var path = "/speak" +
                var path = "/speak" +
-                       "?id=" + encodeURIComponent(id) +
-                       "&text=" + encodeURIComponent(message);
+                       "?id=" + encodeURIComponent(message.ID) +
+                       "&text=" + encodeURIComponent(message.Text);
                for (var i in servers) {
                        var uri = rcserverbase(servers[i]) + path;
                        var img = document.createElement("img");
                for (var i in servers) {
                        var uri = rcserverbase(servers[i]) + path;
                        var img = document.createElement("img");
                        message = "<" + rcnick() + "> " + input;
                }
 
                        message = "<" + rcnick() + "> " + input;
                }
 
-               var m = {'Text': message};
+               var m = {'Text': message, 'ServerTimes': {}};
                rcaddmessagetohistory(m);
                rcsend(m.UI, m);
        }
                rcaddmessagetohistory(m);
                rcsend(m.UI, m);
        }