750 Line function
-
Where i work, we have an aspx page with a single 750 line function for page load. It works by posting querystring data back to itself, and It's order of execution is backwards, i.e. the first bit of executed logic is at the end of the function, and in some cases it goes 12 branch statements deep. Fun :S
------------------------------- Carrier Bags - 21st Century Tumbleweed.
Sounds like some sort of automated test cases being run one after the other.
-Prakash
-
no, it posts back to the user each time, and then doing stuff posts BACK to the page again with whatever QS arguments were attatched to the link.
------------------------------- Carrier Bags - 21st Century Tumbleweed.
Oh, so basically the whole page is implemented in Page_Load? Hey, Ajax is overrated any way, and so is OO for that matter. Even structured programming pales before the Monolith.
-
Where i work, we have an aspx page with a single 750 line function for page load. It works by posting querystring data back to itself, and It's order of execution is backwards, i.e. the first bit of executed logic is at the end of the function, and in some cases it goes 12 branch statements deep. Fun :S
------------------------------- Carrier Bags - 21st Century Tumbleweed.
Oh wow. Who needs method calls anyway? Goto and Gosub should be more than enough for real programmers.
Deja View - the feeling that you've seen this post before.
-
Wow - that sounds like two problems in one. 750 lines ? Apart from the fact that no function should come within cooee of that size, how do you get 750 lines in the code behind, unless there's no business logic layer ?
Christian Graus - Microsoft MVP - C++ Metal Musings - Rex and my new metal blog "I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )
Layers? Now that's a funny idea :(
------------------------------- Carrier Bags - 21st Century Tumbleweed.
-
Oh wow. Who needs method calls anyway? Goto and Gosub should be more than enough for real programmers.
Deja View - the feeling that you've seen this post before.
Pete O'Hanlon wrote:
Goto and Gosub should be more than enough for real programmers
:laugh:
-
Where i work, we have an aspx page with a single 750 line function for page load. It works by posting querystring data back to itself, and It's order of execution is backwards, i.e. the first bit of executed logic is at the end of the function, and in some cases it goes 12 branch statements deep. Fun :S
------------------------------- Carrier Bags - 21st Century Tumbleweed.
Wow, maybe you should submit that to Guinness, it may be a new world record for a page_load!
-
Where i work, we have an aspx page with a single 750 line function for page load. It works by posting querystring data back to itself, and It's order of execution is backwards, i.e. the first bit of executed logic is at the end of the function, and in some cases it goes 12 branch statements deep. Fun :S
------------------------------- Carrier Bags - 21st Century Tumbleweed.
-
Where i work, we have an aspx page with a single 750 line function for page load. It works by posting querystring data back to itself, and It's order of execution is backwards, i.e. the first bit of executed logic is at the end of the function, and in some cases it goes 12 branch statements deep. Fun :S
------------------------------- Carrier Bags - 21st Century Tumbleweed.
I disagree with what everyone said. I can think of a lot of things that would justify a long Page_Load function like that. There is nothing inherently wrong with a really long function. It usually indicates that something needs to be re-factored, but that is not always the case. If everything in the function is atomic, unique, and well-formed... then there's no reason to make it even longer by breaking it up into multiple parts, which would only serve to decrease performance by the amount of the function-calling overhead. I would rather read an if..else structure that had 50 branches, than dive into 50 separate functions which are only called one time.
"Quality Software since 1983!"
http://www.smoothjazzy.com/ - see the "Programming" section for freeware tools and articles. -
I disagree with what everyone said. I can think of a lot of things that would justify a long Page_Load function like that. There is nothing inherently wrong with a really long function. It usually indicates that something needs to be re-factored, but that is not always the case. If everything in the function is atomic, unique, and well-formed... then there's no reason to make it even longer by breaking it up into multiple parts, which would only serve to decrease performance by the amount of the function-calling overhead. I would rather read an if..else structure that had 50 branches, than dive into 50 separate functions which are only called one time.
"Quality Software since 1983!"
http://www.smoothjazzy.com/ - see the "Programming" section for freeware tools and articles.I hope you are kidding! It usually indicates that it is way too complicated and needs to be broken down in to smaller chunks. Of course there are acceptations, but they are far and few between.
INTP "Program testing can be used to show the presence of bugs, but never to show their absence."Edsger Dijkstra
-
I hope you are kidding! It usually indicates that it is way too complicated and needs to be broken down in to smaller chunks. Of course there are acceptations, but they are far and few between.
INTP "Program testing can be used to show the presence of bugs, but never to show their absence."Edsger Dijkstra
No I wasn't joking. It's rare, but a long a function isn't always wrong. If everything is only done once, why push the stack and everything else you have to do? You're right though, it usually indicates something that hasn't been thought out properly.
"Quality Software since 1983!"
http://www.smoothjazzy.com/ - see the "Programming" section for freeware tools and articles.