FLUID-3005: Refactor BuilderUtilityClass into utility functions (not a class)

Metadata

Source
FLUID-3005
Type
Task
Priority
Major
Status
Closed
Resolution
Fixed
Assignee
Laurel Williams
Reporter
Laurel Williams
Created
2009-07-07T11:45:50.000-0400
Updated
2009-08-24T11:25:05.000-0400
Versions
N/A
Fixed Versions
  1. 1.2
Component
  1. Infusion Builder

Description

Comments

  • Laurel Williams commented 2009-07-07T11:46:36.000-0400

    Colin comments: I can't quite tell why you've structured your BuilderUtilityClass as a full-fledged class with instance methods. There seems to be no state here, just a couple of standalone utility functions. Indeed the naming suggests as much--it's not an object with any real identity, just a collection of functions. Maybe it's easier to ditch the class and just invoke these functions directly? Or perhaps there's a bit more thinking about how to structure the various bits of logic in your code?

  • Laurel Williams commented 2009-07-13T10:39:33.000-0400

    Created GroupClass and ModuleClass to encapsulate details.