Click here to Skip to main content
14,392,692 members

Welcome to the Lounge

   

For discussing anything related to a software developer's life but is not for programming questions. Got a programming question?

The Lounge is rated Safe For Work. If you're about to post something inappropriate for a shared office environment, then don't post it. No ads, no abuse, and no programming questions. Trolling, (political, climate, religious or whatever) will result in your account being removed.
 
GeneralRe: Long Lines Pin
PIEBALDconsult22-Oct-19 3:57
professionalPIEBALDconsult22-Oct-19 3:57 
GeneralRe: Long Lines Pin
Member 916705721-Oct-19 21:52
MemberMember 916705721-Oct-19 21:52 
GeneralRe: Long Lines Pin
Sander Rossel21-Oct-19 22:55
professionalSander Rossel21-Oct-19 22:55 
GeneralRe: Long Lines Pin
Rolf Borchmann22-Oct-19 1:14
MemberRolf Borchmann22-Oct-19 1:14 
GeneralRe: Long Lines Pin
Richard Deeming22-Oct-19 2:35
communityengineerRichard Deeming22-Oct-19 2:35 
GeneralRe: Long Lines Pin
PeejayAdams22-Oct-19 2:45
MemberPeejayAdams22-Oct-19 2:45 
GeneralRe: Long Lines Pin
Gary Wheeler22-Oct-19 3:10
MemberGary Wheeler22-Oct-19 3:10 
GeneralRe: Long Lines Pin
Dan Neely22-Oct-19 3:53
MemberDan Neely22-Oct-19 3:53 
class ConfigWrapper
{
    public static double LowerBound { get { return double.Parse(ConfigurationManager.AppSettings["LowerBound"].ToString()); } }
    public static double UpperBound { get { return double.Parse(ConfigurationManager.AppSettings["UpperBound"].ToString()); } }
    ...
}

 ...

var face = allFaces.Where(f => f.Proportion > lowerBound && f.Proportion < upperBound).ToList()
                   .OrderBy(f => f.Proportion).ThenByDescending(f => f.Rectangle.Height).FirstOrDefault();
OutputImage(file, face);


Wrap the config so that you can be sure it's always accessed correctly, and can potentially do lazy loading and data validation. This also makes changing hard coded app.config settings into user configurable ones later on a lot less painful.

Unless I was doing something that LINQ couldn't translate to SQL or that caused it to do something terribly inefficient, I'd drop the `ToList()` in the middle of the method chain as useless overhead.
Did you ever see history portrayed as an old man with a wise brow and pulseless heart, weighing all things in the balance of reason?
Is not rather the genius of history like an eternal, imploring maiden, full of fire, with a burning heart and flaming soul, humanly warm and humanly beautiful?
--Zachris Topelius

Training a telescope on one’s own belly button will only reveal lint. You like that? You go right on staring at it. I prefer looking at galaxies.
-- Sarah Hoyt

GeneralRe: Long Lines Pin
obermd22-Oct-19 4:35
Memberobermd22-Oct-19 4:35 
GeneralRe: Long Lines Pin
Thomas James22-Oct-19 5:36
MemberThomas James22-Oct-19 5:36 
GeneralRe: Long Lines Pin
MikeTheFid22-Oct-19 6:51
MemberMikeTheFid22-Oct-19 6:51 
GeneralRe: Long Lines Pin
Jan Heckman22-Oct-19 13:11
professionalJan Heckman22-Oct-19 13:11 
GeneralThought of the Day Pin
OriginalGriff21-Oct-19 5:38
mveOriginalGriff21-Oct-19 5:38 
GeneralRe: Thought of the Day Pin
lopatir21-Oct-19 6:12
Memberlopatir21-Oct-19 6:12 
GeneralRe: Thought of the Day Pin
Kschuler21-Oct-19 6:44
MemberKschuler21-Oct-19 6:44 
GeneralRe: Thought of the Day Pin
jeron121-Oct-19 6:46
Memberjeron121-Oct-19 6:46 
GeneralRe: Thought of the Day Pin
Ravi Bhavnani21-Oct-19 6:48
professionalRavi Bhavnani21-Oct-19 6:48 
GeneralRe: Thought of the Day Pin
dandy7221-Oct-19 10:28
Memberdandy7221-Oct-19 10:28 
GeneralRe: Thought of the Day Pin
digimanus21-Oct-19 22:16
Memberdigimanus21-Oct-19 22:16 
GeneralAre you still alive... Pin
glennPattonWork21-Oct-19 4:09
professionalglennPattonWork21-Oct-19 4:09 
GeneralRe: Are you still alive... Pin
  Forogar  21-Oct-19 4:24
professional  Forogar  21-Oct-19 4:24 
GeneralRe: Are you still alive... Pin
phil.o21-Oct-19 4:32
mvephil.o21-Oct-19 4:32 
GeneralRe: Are you still alive... Pin
ZurdoDev21-Oct-19 6:59
professionalZurdoDev21-Oct-19 6:59 
GeneralRe: Are you still alive... Pin
Mike Hankey21-Oct-19 4:33
professionalMike Hankey21-Oct-19 4:33 
GeneralRe: Are you still alive... Pin
AFell221-Oct-19 6:19
MemberAFell221-Oct-19 6:19 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.