Comments on: Fun with AOP - reducing redundant code - the ‘for’ loop http://blogs.plexibus.com/2008/07/15/fun-with-aop-reducing-redundant-code-the-for-loop/ Thoughts, questions and opinions - some from the trenches, a few from the Ivory Tower Thu, 04 Dec 2008 02:01:06 +0000 http://wordpress.org/?v=2.5.1 By: ronen http://blogs.plexibus.com/2008/07/15/fun-with-aop-reducing-redundant-code-the-for-loop/#comment-330 ronen Sun, 20 Jul 2008 14:46:28 +0000 http://blogs.plexibus.com/?p=21#comment-330 The main problem with your code is that the number of loops is set in static time, better loop abstraction could be achieved by using something like Commons Collection Utils (they use anonymous methods and abstract the iteration away), still its a nice intro to AOP. The main problem with your code is that the number of loops is set in static time, better loop abstraction could be achieved by using something like Commons Collection Utils (they use anonymous methods and abstract the iteration away), still its a nice intro to AOP.

]]> By: roshanallan http://blogs.plexibus.com/2008/07/15/fun-with-aop-reducing-redundant-code-the-for-loop/#comment-328 roshanallan Fri, 18 Jul 2008 14:54:09 +0000 http://blogs.plexibus.com/?p=21#comment-328 GM>> You are right for the above *single* example - HelloWorldClient - it may not make sense to refactor the <i>for loop</i> into an aspect. But the idea of refactoring the <i>for loop</i> (identified by an annotation) into an aspect was so that this aspect (run a method multiple times) could be used by classes other than just HelloWorldClient. So in the future if you need to code <i>for loops</i> which call methods, all you got to do is: - tag the method with the RunMultiple annotation - run the code with the javaagent jvm option. Hope this makes sense. Thanks, RAS GM>>
You are right for the above *single* example - HelloWorldClient - it may not make sense to refactor the for loop into an aspect.

But the idea of refactoring the for loop (identified by an annotation) into an aspect was so that this aspect (run a method multiple times) could be used by classes other than just HelloWorldClient. So in the future if you need to code for loops which call methods, all you got to do is:
- tag the method with the RunMultiple annotation
- run the code with the javaagent jvm option.

Hope this makes sense.

Thanks,
RAS

]]> By: Ivan http://blogs.plexibus.com/2008/07/15/fun-with-aop-reducing-redundant-code-the-for-loop/#comment-327 Ivan Fri, 18 Jul 2008 09:39:36 +0000 http://blogs.plexibus.com/?p=21#comment-327 This is interesting... Should've factored out a timing aspect too just for a total before and after :) This is interesting… Should’ve factored out a timing aspect too just for a total before and after :)

]]> By: GM http://blogs.plexibus.com/2008/07/15/fun-with-aop-reducing-redundant-code-the-for-loop/#comment-326 GM Fri, 18 Jul 2008 01:39:29 +0000 http://blogs.plexibus.com/?p=21#comment-326 Um - I'm new to this, so, hey, forgive me if I'm way off base, but it looks to me like you started with a file 38 lines long, and then you wrote 85 lines of code along with 9 lines of additional configuration so that you could end up with a file that is 38 lines long. Dude. Um - I’m new to this, so, hey, forgive me if I’m way off base, but it looks to me like you started with a file 38 lines long, and then you wrote 85 lines of code along with 9 lines of additional configuration so that you could end up with a file that is 38 lines long.

Dude.

]]>