It sort of sorts...
-
So we use a product here at work called "Code Reviewer" by Smart Bear
Software. The UI for this product appears to have been created by
a bunch of blind monkeys typing on an infinite number of
keyboards. They don't believe in menus, so everything is a
button. But the real fun starts when you try to view a list of
outstanding code reviews. The list is a normal Windoze-style
list, with a series of columns containing dates, owners, descriptions,
IDs, etc. As is usual for this type of list, you can click on the
headers to sort the list by different columns.
Now let's try to sort by "Time in State":
Not only does it not sort chronologically, which is almost excusable,
but it doesn't even sort numerically - it is sorting a human-readable
time interval field alphabetically!
-
OK, so the picture insertion worked about as well as I expected.
Has anyone complained about this forum software yet? Anyhow,
here's a text excerpt representation of the "before" and "after" that I
was trying to do graphically:
Before:
Time in State
3 weeks
3 hours
10 days
11 days
20 days
31 minutes
After:
Time in State
10 days
11 days
20 days
3 hours
3 weeks
31 minutes
-
It's actually a much more complicated algorithm than you might think:
The list is sorted in increasing order of importance. At the very bottom of the list is the most important code reviews, the ones submitted within the last hour. Up higher in the list you have the people who have been waiting the longest for a response, which is why you see it go from "31 minutes" to "3 weeks". If you don't have to deal with it immediately, then it can wait 3 weeks.
The rest of the list is the people who don't need a response yet because they haven't been waiting long enough.
That "3 hours" item mixed in the middle randomly so you deal with some of them in a timely manner, thereby giving the illusion that you do it with all of them. As long as you take care of at one every so often...
This software is AWESOME.
-
@Manni said:
It's actually a much more complicated algorithm than you might think:
The list is sorted in increasing order of importance. At the very bottom of the list is the most important code reviews, the ones submitted within the last hour. Up higher in the list you have the people who have been waiting the longest for a response, which is why you see it go from "31 minutes" to "3 weeks". If you don't have to deal with it immediately, then it can wait 3 weeks.
The rest of the list is the people who don't need a response yet because they haven't been waiting long enough.
That "3 hours" item mixed in the middle randomly so you deal with some of them in a timely manner, thereby giving the illusion that you do it with all of them. As long as you take care of at one every so often...
Are you sure that is how it is sorting? From the example given, it seems to be sorting on the string value of the field (3 weeks, 31 minutes) rather than the underlying "date" ie. number of seconds since Jan 1, 1970 (in unix) or some floating point number (in excel and vb.) However, that's just from the example .
-
@triso said:
@Manni said:
It's actually a much more complicated algorithm than you might think:
The list is sorted in increasing order of importance. At the very bottom of the list is the most important code reviews, the ones submitted within the last hour. Up higher in the list you have the people who have been waiting the longest for a response, which is why you see it go from "31 minutes" to "3 weeks". If you don't have to deal with it immediately, then it can wait 3 weeks.
The rest of the list is the people who don't need a response yet because they haven't been waiting long enough.
That "3 hours" item mixed in the middle randomly so you deal with some of them in a timely manner, thereby giving the illusion that you do it with all of them. As long as you take care of at one every so often...
Are you sure that is how it is sorting? From the example given, it seems to be sorting on the string value of the field (3 weeks, 31 minutes) rather than the underlying "date" ie. number of seconds since Jan 1, 1970 (in unix) or some floating point number (in excel and vb.) However, that's just from the example .I believe that Manni is being sarcastic :)
-
@Enric Naval said:
@triso said:
@Manni said:
It's actually a much more complicated algorithm than you might think:
The list is sorted in increasing order of importance. At the very bottom of the list is the most important code reviews, the ones submitted within the last hour. Up higher in the list you have the people who have been waiting the longest for a response, which is why you see it go from "31 minutes" to "3 weeks". If you don't have to deal with it immediately, then it can wait 3 weeks.
The rest of the list is the people who don't need a response yet because they haven't been waiting long enough.
That "3 hours" item mixed in the middle randomly so you deal with some of them in a timely manner, thereby giving the illusion that you do it with all of them. As long as you take care of at one every so often...
Are you sure that is how it is sorting? From the example given, it seems to be sorting on the string value of the field (3 weeks, 31 minutes) rather than the underlying "date" ie. number of seconds since Jan 1, 1970 (in unix) or some floating point number (in excel and vb.) However, that's just from the example .I believe that Manni is being sarcastic :)
Not so obviously, triso was, too.
I hope.
Maybe.
-
@sas said:
Not so obviously, triso was, too.
I hope.
Maybe.
Sorry, I am blind to sarcasm that is over one sentence long--a quick
comeback is all I know. I fell for the "uppercase lowercase space
in VB" thing this spring. I made a fool of myself in front of all
the volks at work. I was finally convinced that it was a gag when
someone showed me that the only hits for "uppercase space" were here on
thedailywtf.com.
-
@stevekj said:
Has anyone complained about this forum software yet?
I believe that everyone has. It's a rite of passage of sorts to posting here. ^.~
The routine is probably:
ListHeader_Click(column)
ListHeader.SortColumn(column)
End Function
Or some BS like that. (Hey, if windows explorer can do it why shouldn't we! Assuming they ever even tested for sanity.)
-
@triso said:
Sorry, I am blind to sarcasm that is over one sentence long--a quick
comeback is all I know. I fell for the "uppercase lowercase space
in VB" thing this spring. I made a fool of myself in front of all
the volks at work. I was finally convinced that it was a gag when
someone showed me that the only hits for "uppercase space" were here on
thedailywtf.com.
This is the upprcase space:
This is the lower case space:
See?
-
@dhromed said:
@triso said:
Sorry, I
am blind to sarcasm that is over one sentence long--a quick
comeback is all I know. I fell for the "uppercase lowercase space
in VB" thing this spring. I made a fool of myself in front of all
the volks at work. I was finally convinced that it was a gag when
someone showed me that the only hits for "uppercase space" were here on
thedailywtf.com.
This is the upprcase space:
This is the lower case space:
See?
Oooh, I totally see the difference, now.
-
Splendid!
Now let me show you our fine selection of invisible apples. They look great!
-
@dhromed said:
Splendid! Now let me show you our fine selection of invisible apples. They look great!
<FONT face="Courier New" size=2>"appels"</FONT>
-
@dhromed said:
Splendid!
Now let me show you our fine selection of invisible apples. They look great!
They sure do. I want the absent one, right there. It looks so tasty.
-
I have Hens teeth.
(not in my head, dont be so stupid. they are on my right big toe.)
-
@foxyshadis said:
@stevekj said:
Has anyone complained about this forum software yet?
I believe that everyone has. It's a rite of passage of sorts to posting here. ^.~
I knew that. That was my own attempt at sarcasm, but everyone's sarcasm detectors seem to be broken today. *-)
@foxyshadis said:
The routine is probably:
ListHeader_Click(column)
ListHeader.SortColumn(column)
End Function
Yeah, that's what I'm guessing. I'm not too surprised that they
actually coded it this way, at least for a first attempt; what's a bit
more of a WTF is that it got past their excuse for QC that way.
-