XMLWordPrintable

Details

    • Sub-task
    • Resolution: Unresolved
    • Major
    • 3.x Future
    • 2.0.0.Alpha14
    • Resources, Scaffold
    • None

    Description

      Discussion on : https://community.jboss.org/message/842429

      Developers look at Forge as the "way to write Java EE code" or if you like "if those guys write code like this, then we should".

      I am digging into some details of the generated code (I am writing a blog about several architectural styles starting with Forge) and I feel coding convention should be homogenized. I know extensions are written by different individuals, but some basic coding conventions should be applied. For example, when you generate a web app with REST and Faces scaffolding, you get some difference :

      • Faces Backing Bean use query builder (e.g getAll method is entityManager.createQuery(criteria.select(criteria.from(Book.class))).getResultList(); and
      • REST Endpoint use dynamic queries (the listAll method is "SELECT DISTINCT b FROM Book b ORDER BY b.id"))

      Method names are different and do the same :

      • JSF : getAll
      • REST : listAll

      Attributes

      • private EntityManager em;
      • private EntityManager entityManager; // em would be better
        Or the use of this keyword (JSF beans use this.entityManager instead of directly em in REST)

      And there are several examples like this. If Forge is seen as "the way of writing code" maybe something should be created to get homogenized code. PMD, Checkstyle, human review and so one.....

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              agoncal Antonio Goncalves (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: