Changeset 5270


Ignore:
Timestamp:
Aug 12, 2016, 1:44:03 AM (6 years ago)
Author:
Christian Schulte
Message:

o Updated to add 'final' modifier to fields.
o Updated to use 'CopyOnWriteArrayList?' instead of 'ArrayList?'.

See #102

File:
1 edited

Legend:

Unmodified
Added
Removed
  • jomc-modlet/trunk/src/main/java/org/jomc/modlet/DefaultModelContext.java

    r5268 r5270  
    5454import java.util.StringTokenizer;
    5555import java.util.TreeMap;
     56import java.util.concurrent.CopyOnWriteArrayList;
    5657import java.util.jar.Attributes;
    5758import java.util.jar.Manifest;
     
    19081909     * The context of the instance.
    19091910     */
    1910     private ModelContext context;
     1911    private final ModelContext context;
    19111912
    19121913    /**
    19131914     * The report of the instance.
    19141915     */
    1915     private ModelValidationReport report;
     1916    private final ModelValidationReport report;
    19161917
    19171918    /**
     
    19221923    ModelErrorHandler( final ModelContext context )
    19231924    {
    1924         this( context, null );
     1925        this( context, new ModelValidationReport() );
    19251926    }
    19261927
     
    19451946    public ModelValidationReport getReport()
    19461947    {
    1947         if ( this.report == null )
    1948         {
    1949             this.report = new ModelValidationReport();
    1950         }
    1951 
    19521948        return this.report;
    19531949    }
     
    20352031     * The {@code Marshaller.Listener}s of the instance.
    20362032     */
    2037     private List<Marshaller.Listener> listeners;
     2033    private final List<Marshaller.Listener> listeners = new CopyOnWriteArrayList<Marshaller.Listener>();
    20382034
    20392035    /**
     
    20572053    List<Marshaller.Listener> getListeners()
    20582054    {
    2059         if ( this.listeners == null )
    2060         {
    2061             this.listeners = new ArrayList<Marshaller.Listener>();
    2062         }
    2063 
    20642055        return this.listeners;
    20652056    }
     
    20682059    public void beforeMarshal( final Object source )
    20692060    {
    2070         for ( int i = 0, s0 = this.getListeners().size(); i < s0; i++ )
    2071         {
    2072             this.getListeners().get( i ).beforeMarshal( source );
     2061        for ( final Marshaller.Listener listener : this.getListeners() )
     2062        {
     2063            listener.beforeMarshal( source );
    20732064        }
    20742065    }
     
    20772068    public void afterMarshal( final Object source )
    20782069    {
    2079         for ( int i = 0, s0 = this.getListeners().size(); i < s0; i++ )
    2080         {
    2081             this.getListeners().get( i ).afterMarshal( source );
     2070        for ( final Marshaller.Listener listener : this.getListeners() )
     2071        {
     2072            listener.afterMarshal( source );
    20822073        }
    20832074    }
     
    20982089     * The {@code Unmarshaller.Listener}s of the instance.
    20992090     */
    2100     private List<Unmarshaller.Listener> listeners;
     2091    private final List<Unmarshaller.Listener> listeners = new CopyOnWriteArrayList<Unmarshaller.Listener>();
    21012092
    21022093    /**
     
    21202111    List<Unmarshaller.Listener> getListeners()
    21212112    {
    2122         if ( this.listeners == null )
    2123         {
    2124             this.listeners = new ArrayList<Unmarshaller.Listener>();
    2125         }
    2126 
    21272113        return this.listeners;
    21282114    }
     
    21312117    public void beforeUnmarshal( final Object target, final Object parent )
    21322118    {
    2133         for ( int i = 0, s0 = this.getListeners().size(); i < s0; i++ )
    2134         {
    2135             this.getListeners().get( i ).beforeUnmarshal( target, parent );
     2119        for ( final Unmarshaller.Listener listener : this.getListeners() )
     2120        {
     2121            listener.beforeUnmarshal( target, parent );
    21362122        }
    21372123    }
     
    21402126    public void afterUnmarshal( final Object target, final Object parent )
    21412127    {
    2142         for ( int i = 0, s0 = this.getListeners().size(); i < s0; i++ )
    2143         {
    2144             this.getListeners().get( i ).afterUnmarshal( target, parent );
     2128        for ( final Unmarshaller.Listener listener : this.getListeners() )
     2129        {
     2130            listener.afterUnmarshal( target, parent );
    21452131        }
    21462132    }
Note: See TracChangeset for help on using the changeset viewer.