Genarate Images on fly in ASP.Net

Happy holidays, I do not want to be negative, but here are three things to learner: 1) First of foremost; this is not article, it is a listing of source code without any comments. I could have accepted this as an article if the code had been well documented as all code should be. 2) Hard coding of the “ImageSavePath” is an absolute no-no. If hard coded you cannot move and reuse the code without first changing the code and that means Code Once, Test Once, Reuse Many Time goes out the door. The use if either calling the OS for relative folder position or the simple use of “~” would have been a better solution. 3) The last one is real bad. We have in two different places (the ASPX and the CS) code definition of constants so if I have misspelled one of the constants the code will not work, and second if I would at a later time add another constant, I have to remember to do it in both places. I have one more, but that falls in under religion with regards to how one codes. I like to not have intelligent directly under an event but build an assembly where the intelligent is stored; hence splitting the smart stuff from the pretty stuff. Why do I like this? Well the intelligent can now be reused and expanded as my life goes on without having to think about what could break. And on the pretty side I just have to think CCS if I want to change the look and feel, hence making a Website dynamic. See if some of the above would have been introduced in the text we would have had an article. Again I am not here to make someone upset and be negative; I am here to help us all to be better designers / programmers. May 2010 become a better year with regards to articles and codes! Bjarne

Good One Rahul, Keep it up.