Uploaded image for project: 'Struts 1'
  1. Struts 1
  2. STR-2965

avoid calling member methods that potentially can be overridden by subclasses

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Future
    • Component/s: Core
    • Labels:
      None
    • Flags:
      Patch

      Description

      The class ForwardConfig has two constructors that call other member method to set the class instances variables. This is generally a poor programming style. Since the member methods are not final and public. They can be overridden by subclasses. It will potntially end up with a object of class is constructed in invalid state

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              yusheng Yusheng Wang
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated: