On Apr 4, 4:28 pm, cd34 <[email protected]> wrote: > While the controller is cleaner, does that really make it more > readable? From a maintainability standpoint, would someone new to the > code be able to see what that action was doing reasonably quickly?
If i were going for code readability, I don't know. Usually I go for application maintainability. Looking at the original code , this bigtask() class looked to be a logging functionality that is used across multiple controllers and actions. i'm inclined to refactor code like that into helper functions , so I only need to update them in the smallest amount of places when the objects interface / methods change. assuming bigtask() adds a new column, and is called in a dozen places, I'd rather only edit 1 invocation of the method to pull the request params -- not 12. -- You received this message because you are subscribed to the Google Groups "pylons-discuss" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/pylons-discuss?hl=en.
