HNHacker News
TopNewBestAskShowJobs

devillighter

27 karma · joined February 6, 2013

submissionscomments
devillighter··on The Source Code for NYC's Forensic DNA Statistical Analysis Tool
Code in "Comparison.cs" on line 1307

   // if we don't have frequencies, use the default (this should never happen, but here it is)
                    if (tblFreq == null || tblFreq.Rows.Count == 0)
                    {
                        a = (float)0.02;
                        b = (float)0.02;
                    }
is kind of fishy. Why not bomb out instead of failing silently for things that are never supposed to happen? Not saying this is inherently wrong - I'm certainly not capable of actually understanding the details of the analysis