Home > Articles > Software Development & Management

  • Print
  • + Share This
This chapter is from the book

This chapter is from the book

Move Creation Knowledge to Factory

Move Creation Knowledge to Factory

Data and code used to instantiate a class is sprawled across numerous classes.

Move the creation knowledge into a single Factory class.

Motivation

When the knowledge for creating an object is spread out across numerous classes, you have creation sprawl: the placement of creational responsibilities in classes that ought not to be playing any role in an object’s creation. Creation sprawl, which is a case of the Solution Sprawl smell (43), tends to result from an earlier design problem. For example, a client needed to configure an object based on some preferences yet lacked access to the object’s creation code. If the client can’t easily access the object’s creation code, say, because it exists in a system layer far removed from the client, how can the client configure the object?

A typical answer is by using brute force. The client passes its configuration preferences to one object, which hands them off to another object, which holds onto them until the creation code, by means of still more objects, obtains the information for use in configuring the object. While this works, it spreads creation code and data far and wide.

The Factory pattern is helpful in this context. It uses one class to encapsulate both creation logic and a client’s instantiation/configuration preferences. A client can tell a Factory instance how to instantiate/configure an object, and then the same Factory instance may be used at runtime to perform the instantiation/configuration. For example, a NodeFactory creates StringNode instances and may be configured by clients to embellish those instances with a DecodingStringNode Decorator:

A Factory need not be implemented exclusively by a concrete class. You can use an interface to define a Factory and make an existing class implement that interface. This approach is useful when you want other areas of a system to communicate with an instance of the existing class exclusively through its Factory interface.

If the creation logic inside a Factory becomes too complex, perhaps due to supporting too many creation options, it may make sense to evolve it into an Abstract Factory [DP]. Once that’s done, clients can configure a system to use a particular ConcreteFactory (i.e., a concrete implementation of an Abstract Factory) or let the system use a default ConcreteFactory. While the above NodeFactory is certainly not complicated enough to merit such an evolution, the diagram on the next page shows what it would look like as an Abstract Factory.

I’ve seen numerous systems in which the Factory pattern was overused. For example, if every object in a system is created by using a Factory, instead of direct instantiation (e.g., new StringNode(ノ)),the system probably has an overabundance of Factories. Overusing this pattern often occurs when people always decouple client code from code that chooses between which classes to instantiate or how to instantiate them. For example, the following createQuery() method makes a choice about which of two query classes to instantiate:

public class Query...
   public void createQuery() throws QueryException...
      if (usingSDVersion52()) {
         query = new QuerySD52(); 
         ...
      } else {
         query = new QuerySD51(); 
         ...
      } 

To eliminate the conditional logic in the above code, some would refactor it to use a QueryFactory:

public class Query...
   public void createQuery() throws QueryException...
      query = queryFactory.createQuery();
      ...

QueryFactory now encapsulates the choice of what concrete query class to instantiate. Yet does QueryFactory improve the design of this code? It certainly doesn’t consolidate creation sprawl, and if it only decouples the Query class from the code that instantiates one of the two concrete queries, it most definitely does not add enough value to merit its existence. This illustrates the point that it’s best not to implement a Factory unless it really improves the design of your code or enables you to create/configure objects in a way that wasn’t possible with direct instantiation.

Mechanics

These mechanics assume your Factory will be implemented as a class, rather than as an interface implemented by a class. If you need a Factory interface that is implemented by a class, you must make minor modifications to these mechanics.

  1. An instantiator is a class that collaborates with other classes to instantiate a product (i.e., an instance of some class). If the instantiator doesn’t instantiate the product using a Creation Method, modify it and, if necessary, also modify the product’s class, so the instantiation occurs through a Creation Method.
  2. →Compile and test.

  3. Create a new class that will become your factory. Name your factory based on what it creates (e.g., NodeFactory, LoanFactory).
  4. →Compile.

  5. Apply Move Method [F] to move the Creation Method to the factory. If the Creation Method is static, you can make it nonstatic after moving it to the factory.
  6. →Compile.

  7. Update the instantiator to instantiate the factory and call the factory to obtain an instance of the class.
  8. →Compile and test that the instantiator still functions correctly.

    Repeat this step for any instantiators that could no longer compile because of changes made during step 3.

  9. Data and methods from the other classes are still being used in the instantiation. Move whatever makes sense into the factory, so it handles as much of the creation work as possible. This may involve moving where the factory gets instantiated and who instantiates it.
  10. →Compile and test.

Example

This example comes from the HTML Parser project. As described in Move Embellishment to Decorator (144), a user of the parser can instruct it to handle string parsing in different ways. If a user doesn’t want parsed strings to contain encoded characters, like &amp; (which represents an ampersand, &) or &lt; (which represents an opening angle bracket, <), the user can call the parser’s setStringNodeDecoding(shouldDecode: boolean) method, which turns the string decoding option on or off. As the sketch at the beginning of this Move Creation Knowledge to Factory refactoring illustrates, the parser’s StringParser actually creates StringNode objects, and when it does so, it configures them to decode or not decode, based on the value of the decoding field in Parser.

While this code worked, StringNode creation knowledge was now spread across the Parser, StringParser, and StringNode classes. This problem worsened as new string parsing options were added to the Parser. Each new option required the creation of a new Parser field with corresponding getters and setters, as well as new code in the StringParser and StringNode to handle the new option. The boldface code in the diagram on the following page illustrates some of the changes made to classes that resulted from adding an escape character (e.g., \n or \r) removal option.

The fields, getters, and setters that were added to Parser to support different parsing options for StringNodes didn’t belong on the Parser class. Why? Because Parser has the responsibility of kicking off a parsing session, not controlling how StringNodes (which represent just one of numerous Node and Tag types) ought to be parsed. In addition, the StringNode class also had no good reason to know anything about decoding or escape character removal options, which have already been modeled using the Decorator pattern (see the example for Move Embellishment to Decorator, 144).

Based on my earlier definition, we can say that StringNode is already a Factory because it implements a Creation Method. The trouble is, StringNode isn’t helping consolidate all knowledge used in instantiating/configuring a StringNode, nor do we actually want it to because it is better to keep StringNode small and simple. A new Factory class will be better able to consolidate the instantiation/configuration, so I will refactor to one. For simplicity, the following code includes only one parsing option—the one for decoding nodes—and doesn’t include the option for escape character removal.

  1. StringParser instantiates StringNode objects. The first step in this Move Creation Knowledge to Factory refactoring is to make StringParser perform its instantiation of StringNode objects by using a Creation Method. It already does this, as the following code shows.
  2. public class StringParser...
      public Node find(...) {
        ...
        return StringNode.createStringNode(
          textBuffer, textBegin, textEnd,
          parser.shouldDecodeNodes()
        );
      }
       
    public class StringNode...
      public static Node createStringNode(
        StringBuffer textBuffer, int textBegin, int textEnd, boolean shouldDecode) {
        if (shouldDecode)
          return new DecodingStringNode(
            new StringNode(textBuffer, textBegin, textEnd)
          );
        return new StringNode(textBuffer, textBegin, textEnd);
      }
  3. Now I create a new class that will become a factory for StringNode objects. Because a StringNode is a type of Node, I name the class NodeFactory:
  4. public class NodeFactory {
    }
  5. Next, I apply Move Method [F] to move StringNode’s Creation Method to NodeFactory. I decide to make the moved method nonstatic because I don’t want client code statically bound to one Factory implementation. I also decide to delete the Creation Method in StringNode:
  6. public class NodeFactory {
      public static Node createStringNode(
        StringBuffer textBuffer, int textBegin, int textEnd, boolean shouldDecode) {
        if (shouldDecode)
          return new DecodingStringNode(
            new StringNode(textBuffer, textBegin, textEnd));
        return new StringNode(textBuffer, textBegin, textEnd);
      }
    } 
       
    public class StringNode...
      public static Node createStringNode(...
      }

    After this step, StringParser and other clients that used to call the StringNode’s Creation Method no longer compile. I’ll fix that next.

  7. Now I modify the StringParser to instantiate a NodeFactory and call it to create a StringNode:
  8. public class StringParser...
      public Node find(...) {
       ...
       NodeFactory nodeFactory = new NodeFactory();
       return nodeFactory.createStringNode(
         textBuffer, textBegin, textEnd, parser.shouldDecodeNodes()
       );  
      }

    I perform a similar step for any other clients that no longer compile because of work done in step 3.

  9. Now comes the fun part: eliminating or reducing creation sprawl by moving the appropriate creation code from other classes into the NodeFactory. In this case the other class is the Parser, which the StringParser calls to pass an argument to the NodeFactory during StringNode creation:
  10. public class StringParser...
      public Node find(...) {
        ...
        NodeFactory nodeFactory = new NodeFactory();
        return nodeFactory.createStringNode(
          textBuffer, textBegin, textEnd, parser.shouldDecodeNodes()
        );  
      }

    I’d like to move the following Parser code to the NodeFactory:

    public class Parser...
      private boolean shouldDecodeNodes = false;
       
      public void setNodeDecoding(boolean shouldDecodeNodes) {
        this.shouldDecodeNodes = shouldDecodeNodes;
      }
       
      public boolean shouldDecodeNodes() {
        return shouldDecodeNodes;
      }

However, I can’t simply move this code into the NodeFactory because clients of this code are clients of the parser, which call Parser methods like setNodeDecoding(ノ) to configure the parser for a given parse. Meanwhile, NodeFactory is not even visible to parser clients: it is instantiated by StringParser, which itself is not visible to parser clients. This leads me to conclude that the NodeFactory instance must be accessible to both Parser clients and the StringParser. To make that happen, I take the following steps.

  1. I first apply Extract Class [F] on the Parser code I want to eventually merge with the NodeFactory. This leads to the creation of the String-NodeParsingOption class:
  2. public class StringNodeParsingOption {
      private boolean decodeStringNodes;
       
      public boolean shouldDecodeStringNodes() {
        return decodeStringNodes;
      }
       
      public void setDecodeStringNodes(boolean decodeStringNodes) {
        this.decodeStringNodes = decodeStringNodes;
      }
    }

    This new class replaces the shouldDecodeNodes field, getter, and setter with a StringNodeParsingOption field and its getter and setter:

    public class Parser....
      private StringNodeParsingOption stringNodeParsingOption = 
        new StringNodeParsingOption();
       
      private boolean shouldDecodeNodes = false;
       
      public void setNodeDecoding(boolean shouldDecodeNodes) {
        this.shouldDecodeNodes = shouldDecodeNodes;
      }
       
      public boolean shouldDecodeNodes() {
        return shouldDecodeNodes;
      }
       
      public StringNodeParsingOption getStringNodeParsingOption() {
        return stringNodeParsingOption;
      }
       
      public void setStringNodeParsingOption(StringNodeParsingOption option) {
        stringNodeParsingOption = option;
      }

    Parser clients now turn StringNode decoding on by instantiating and configuring a StringNodeParsingOption instance and passing it to the parser:

    class DecodingNodeTest...
      public void testDecodeAmpersand() {
        ...
        StringNodeParsingOption decodeNodes =
          new StringNodeParsingOption();
        decodeNodes.setDecodeStringNodes(true);
        parser.setStringNodeParsingOption(decodeNodes);
        parser.setNodeDecoding(true); 
        ...
      }

    The StringParser now obtains the state of the StringNode decoding option by means of the new class:

    public class StringParser...
      ...  
      public Node find(...) {
        NodeFactory nodeFactory = new NodeFactory();
        return nodeFactory.createStringNode(
          textBuffer, 
          textBegin, 
          textEnd,
          parser.getStringNodeParsingOption().shouldDecodeStringNodes()
        );  
      }
  3. Now I apply Inline Class [F] to merge NodeFactory with StringNodeParsing-Option. This leads to the following changes in StringParser:
  4. public class StringParser...
      public Node find(...) {
        ...
        return parser.getStringNodeParsingOption().createStringNode(
          textBuffer, textBegin, textEnd,
          parser.getStringNodeParsingOption().shouldDecodeStringNodes()
       );
      }

    And the following changes in StringNodeParsingOption:

    public class StringNodeParsingOption...
      private boolean decodeStringNodes;
       
      public Node createStringNode(
        StringBuffer textBuffer, int textBegin, int textEnd, boolean shouldDecode) {
        if (decodeStringNodes)
        return new DecodingStringNode(
          new StringNode(textBuffer, textBegin, textEnd));
        return new StringNode(textBuffer, textBegin, textEnd);
      }
    }
  5. The final step is to rename the class StringNodeParsingOption to NodeFactory and then perform a similar renaming on the NodeFactory field, getter, and setter in Parser:
  6. public class StringNodeParsingOption NodeFactory...
       
    public class Parser...
      private NodeFactory nodeFactory = new NodeFactory();
       
      public NodeFactory getNodeFactory() {
        return nodeFactory;
      }
       
      public void setNodeFactory(NodeFactory nodeFactory) {
        this.nodeFactory = nodeFactory;
      }

And that does it. NodeFactory has helped tame creation sprawl by handling the work associated with instantiating and configuring StringNode objects.

  • + Share This
  • 🔖 Save To Your Account

InformIT Promotional Mailings & Special Offers

I would like to receive exclusive offers and hear about products from InformIT and its family of brands. I can unsubscribe at any time.

Overview


Pearson Education, Inc., 221 River Street, Hoboken, New Jersey 07030, (Pearson) presents this site to provide information about products and services that can be purchased through this site.

This privacy notice provides an overview of our commitment to privacy and describes how we collect, protect, use and share personal information collected through this site. Please note that other Pearson websites and online products and services have their own separate privacy policies.

Collection and Use of Information


To conduct business and deliver products and services, Pearson collects and uses personal information in several ways in connection with this site, including:

Questions and Inquiries

For inquiries and questions, we collect the inquiry or question, together with name, contact details (email address, phone number and mailing address) and any other additional information voluntarily submitted to us through a Contact Us form or an email. We use this information to address the inquiry and respond to the question.

Online Store

For orders and purchases placed through our online store on this site, we collect order details, name, institution name and address (if applicable), email address, phone number, shipping and billing addresses, credit/debit card information, shipping options and any instructions. We use this information to complete transactions, fulfill orders, communicate with individuals placing orders or visiting the online store, and for related purposes.

Surveys

Pearson may offer opportunities to provide feedback or participate in surveys, including surveys evaluating Pearson products, services or sites. Participation is voluntary. Pearson collects information requested in the survey questions and uses the information to evaluate, support, maintain and improve products, services or sites, develop new products and services, conduct educational research and for other purposes specified in the survey.

Contests and Drawings

Occasionally, we may sponsor a contest or drawing. Participation is optional. Pearson collects name, contact information and other information specified on the entry form for the contest or drawing to conduct the contest or drawing. Pearson may collect additional personal information from the winners of a contest or drawing in order to award the prize and for tax reporting purposes, as required by law.

Newsletters

If you have elected to receive email newsletters or promotional mailings and special offers but want to unsubscribe, simply email information@informit.com.

Service Announcements

On rare occasions it is necessary to send out a strictly service related announcement. For instance, if our service is temporarily suspended for maintenance we might send users an email. Generally, users may not opt-out of these communications, though they can deactivate their account information. However, these communications are not promotional in nature.

Customer Service

We communicate with users on a regular basis to provide requested services and in regard to issues relating to their account we reply via email or phone in accordance with the users' wishes when a user submits their information through our Contact Us form.

Other Collection and Use of Information


Application and System Logs

Pearson automatically collects log data to help ensure the delivery, availability and security of this site. Log data may include technical information about how a user or visitor connected to this site, such as browser type, type of computer/device, operating system, internet service provider and IP address. We use this information for support purposes and to monitor the health of the site, identify problems, improve service, detect unauthorized access and fraudulent activity, prevent and respond to security incidents and appropriately scale computing resources.

Web Analytics

Pearson may use third party web trend analytical services, including Google Analytics, to collect visitor information, such as IP addresses, browser types, referring pages, pages visited and time spent on a particular site. While these analytical services collect and report information on an anonymous basis, they may use cookies to gather web trend information. The information gathered may enable Pearson (but not the third party web trend services) to link information with application and system log data. Pearson uses this information for system administration and to identify problems, improve service, detect unauthorized access and fraudulent activity, prevent and respond to security incidents, appropriately scale computing resources and otherwise support and deliver this site and its services.

Cookies and Related Technologies

This site uses cookies and similar technologies to personalize content, measure traffic patterns, control security, track use and access of information on this site, and provide interest-based messages and advertising. Users can manage and block the use of cookies through their browser. Disabling or blocking certain cookies may limit the functionality of this site.

Do Not Track

This site currently does not respond to Do Not Track signals.

Security


Pearson uses appropriate physical, administrative and technical security measures to protect personal information from unauthorized access, use and disclosure.

Children


This site is not directed to children under the age of 13.

Marketing


Pearson may send or direct marketing communications to users, provided that

  • Pearson will not use personal information collected or processed as a K-12 school service provider for the purpose of directed or targeted advertising.
  • Such marketing is consistent with applicable law and Pearson's legal obligations.
  • Pearson will not knowingly direct or send marketing communications to an individual who has expressed a preference not to receive marketing.
  • Where required by applicable law, express or implied consent to marketing exists and has not been withdrawn.

Pearson may provide personal information to a third party service provider on a restricted basis to provide marketing solely on behalf of Pearson or an affiliate or customer for whom Pearson is a service provider. Marketing preferences may be changed at any time.

Correcting/Updating Personal Information


If a user's personally identifiable information changes (such as your postal address or email address), we provide a way to correct or update that user's personal data provided to us. This can be done on the Account page. If a user no longer desires our service and desires to delete his or her account, please contact us at customer-service@informit.com and we will process the deletion of a user's account.

Choice/Opt-out


Users can always make an informed choice as to whether they should proceed with certain services offered by InformIT. If you choose to remove yourself from our mailing list(s) simply visit the following page and uncheck any communication you no longer want to receive: www.informit.com/u.aspx.

Sale of Personal Information


Pearson does not rent or sell personal information in exchange for any payment of money.

While Pearson does not sell personal information, as defined in Nevada law, Nevada residents may email a request for no sale of their personal information to NevadaDesignatedRequest@pearson.com.

Supplemental Privacy Statement for California Residents


California residents should read our Supplemental privacy statement for California residents in conjunction with this Privacy Notice. The Supplemental privacy statement for California residents explains Pearson's commitment to comply with California law and applies to personal information of California residents collected in connection with this site and the Services.

Sharing and Disclosure


Pearson may disclose personal information, as follows:

  • As required by law.
  • With the consent of the individual (or their parent, if the individual is a minor)
  • In response to a subpoena, court order or legal process, to the extent permitted or required by law
  • To protect the security and safety of individuals, data, assets and systems, consistent with applicable law
  • In connection the sale, joint venture or other transfer of some or all of its company or assets, subject to the provisions of this Privacy Notice
  • To investigate or address actual or suspected fraud or other illegal activities
  • To exercise its legal rights, including enforcement of the Terms of Use for this site or another contract
  • To affiliated Pearson companies and other companies and organizations who perform work for Pearson and are obligated to protect the privacy of personal information consistent with this Privacy Notice
  • To a school, organization, company or government agency, where Pearson collects or processes the personal information in a school setting or on behalf of such organization, company or government agency.

Links


This web site contains links to other sites. Please be aware that we are not responsible for the privacy practices of such other sites. We encourage our users to be aware when they leave our site and to read the privacy statements of each and every web site that collects Personal Information. This privacy statement applies solely to information collected by this web site.

Requests and Contact


Please contact us about this Privacy Notice or if you have any requests or questions relating to the privacy of your personal information.

Changes to this Privacy Notice


We may revise this Privacy Notice through an updated posting. We will identify the effective date of the revision in the posting. Often, updates are made to provide greater clarity or to comply with changes in regulatory requirements. If the updates involve material changes to the collection, protection, use or disclosure of Personal Information, Pearson will provide notice of the change through a conspicuous notice on this site or other appropriate way. Continued use of the site after the effective date of a posted revision evidences acceptance. Please contact us if you have questions or concerns about the Privacy Notice or any objection to any revisions.

Last Update: November 17, 2020