Click here to Skip to main content
15,868,016 members

Comments by Robert Rohde (Top 8 by date)

Robert Rohde 23-Sep-13 9:23am View    
Reason for my vote of 1 \n You are testing so wrong that I don't know where to start.
First of all you should know that you are primary testing the performance of Console.WriteLine. The rest (especially your "measured" difference) are pure JIT effects. Try switching the test (first HashTable then Dictionary) and you will most probably see that Dictionary wins.
To make your test better do the following:
1. Remove the Console.WriteLine and make something else which clearly doesn't consume much processing power.
2. Always test several times and calculate an average time.
3. Make the test base bigger. 1000 Elements are nothing.
Robert Rohde 6-Dec-11 15:22pm View    
"I'm assuming his query is properly formed and that he's done everything else he needs to do to make it work" sounds a bit naive. If he asks such a simple question do you really think he knows what he's doing...?
Robert Rohde 6-Dec-11 7:34am View    
Even though your solution is "a bit" better than from Uma and Derek this is still dangerous. Just because it starts with "select" it doesn't mean that it cannot modify the database.
Like mjbohn said this should be made with a specialized database user with read only access rights.
Robert Rohde 31-Aug-11 3:54am View    
Why do you not just use one Combobox where all infos (id, name and location) are shown together?
Robert Rohde 17-May-11 1:45am View    
You forgot to set isAdded to true.