Stack Exchange network consists of 183 Q&A communities including Stack Overflow, the largest, most trusted online community for developers to learn, share their knowledge, and build their careers.
I was also considering that in Java exception that can be thrown are specified in the code. Like to say: this method can throw this exception for a reason.. I think a NullReferenceException should never occurr in a consolidated piece of code..
@ChrisHayes: thanks, I know.. even if it's not very intuitive and could not be perceived as something fair. Some other website use a similar politic on images and there were a lot of protests. However in the specific case is not a problem, I'm simply saying that something which is by law can be perceived as strongly bad. But this community seems to have reached a good compromise.
@Vogel612: ok.. but I really think the meaning wasn't changed. And it sound a bit weird to lose the ownership of a question after posting it online (moderator gets the right to decide at my place on a content I wrote, it's not very intuitive). But I strongly believe in this community and I appreciate especially the work you are doing here on code review.
I can try to explain my point of view but it's strictly personal of course. Take it for just for it. In the first approach I have to read every Line and think: "what do he want to do?". For example he declares a DateTime called eventDateWithTime.. My first tought is: "why?". And then I see something about cultureinfo.. and again I have to wonder why the solution he used required to access culture. The only wanted operation was: "EndDate = EvenDate; EndTime = EndDate.ToHour();" this is what I would like to realize immediately at first glance without any effort.