>No, that's a magic number.
>
>The constant needs to be defined earlier
>
>const short maxAdmissionDays = 1;
>DateTime admission_end = this.dtScan.Date.AddDays(Math.Max(admissions, maxAdmissionDays).AddSeconds(-1);
>
>
Hmm, is it good practice? I can probably use this idea (the constant name should be min, though).
Thanks again.
Also, where and how should I declare it? At the top of the class code?
If it's not broken, fix it until it is.
My Blog