Ignore:
Timestamp:
08/17/12 09:46:55 (12 years ago)
Author:
pharms
Message:
  • adaptations to remove some find bugs warnings
Location:
trunk/quest-core-events/src/main/java/de/ugoe/cs/quest/eventcore
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • trunk/quest-core-events/src/main/java/de/ugoe/cs/quest/eventcore/gui/InteractionEventList.java

    r550 r564  
    77package de.ugoe.cs.quest.eventcore.gui; 
    88 
     9import java.io.Serializable; 
    910import java.util.ArrayList; 
    1011import java.util.Collection; 
     
    1920 * @author 2011, last modified by $Author: $ 
    2021 */ 
    21 @SuppressWarnings("serial") 
    2222public class InteractionEventList extends ArrayList<Event> implements List<Event> { 
     23     
     24    /**  */ 
     25    private static final long serialVersionUID = 1L; 
    2326     
    2427    /** stores the events to be handled later in the order, they should be processed */ 
     
    116119     */ 
    117120    private void addEventPair(KeyEventPair eventPair) { 
    118         super.add(eventPair.mFirst); 
    119  
    120         for (KeyEventPair child : eventPair.mChildren) { 
     121        super.add(eventPair.first); 
     122 
     123        for (KeyEventPair child : eventPair.children) { 
    121124            addEventPair(child); 
    122125        } 
    123126 
    124         super.add(eventPair.mSecond); 
     127        super.add(eventPair.second); 
    125128    } 
    126129 
     
    128131     * 
    129132     */ 
    130     private static class KeyEventPair { 
     133    private static class KeyEventPair implements Serializable { 
    131134         
     135        /**  */ 
     136        private static final long serialVersionUID = 1L; 
     137 
    132138        /** the first key interaction event */ 
    133         private Event mFirst; 
     139        private Event first; 
    134140 
    135141        /** the second key interaction event */ 
    136         private Event mSecond; 
     142        private Event second; 
    137143 
    138144        /** the children, this event pair may have */ 
    139         private List<KeyEventPair> mChildren = new ArrayList<KeyEventPair>(); 
     145        private List<KeyEventPair> children = new ArrayList<KeyEventPair>(); 
    140146 
    141147        /** 
     
    148154                   "event pair"); 
    149155            } 
    150             mFirst = first; 
     156            this.first = first; 
    151157        } 
    152158 
     
    166172 
    167173            if ((event.getType() instanceof KeyReleased) && 
    168                 (((KeyInteraction) mFirst.getType()).getKey() == 
     174                (((KeyInteraction) first.getType()).getKey() == 
    169175                 ((KeyInteraction) event.getType()).getKey()) && 
    170                  (mSecond == null)) 
     176                (second == null)) 
    171177            { 
    172178                // this is the release of the represented key. Store it and notify the processing. 
    173                 mSecond = event; 
     179                second = event; 
    174180                return true; 
    175181            } 
    176182            else if ((event.getType() instanceof KeyPressed) && 
    177                 (((KeyInteraction) mFirst.getType()).getKey() == 
    178                  ((KeyInteraction) event.getType()).getKey()) && 
    179                  (mSecond == null)) 
     183                     (((KeyInteraction) first.getType()).getKey() == 
     184                      ((KeyInteraction) event.getType()).getKey()) && 
     185                     (second == null)) 
    180186            { 
    181187                // the key was pressed before it was released again. This happens, if the key 
    182188                // stays pressed for a long time. Generate a key released event but do not mark 
    183189                // the new event as processed 
    184                 mSecond = new Event 
     190                second = new Event 
    185191                    (new KeyReleased(((KeyInteraction) event.getType()).getKey()), 
    186                      mFirst.getTarget()); 
     192                     first.getTarget()); 
    187193                 
    188194                return false; 
    189195            } 
    190             else if (((KeyInteraction) mFirst.getType()).getKey().isCombinationKey() && 
    191                      (((KeyInteraction) mFirst.getType()).getKey() != 
     196            else if (((KeyInteraction) first.getType()).getKey().isCombinationKey() && 
     197                     (((KeyInteraction) first.getType()).getKey() != 
    192198                      ((KeyInteraction) event.getType()).getKey())) 
    193199            { 
     
    195201                // doesn't work, add the event as a new child pair, if it is a new key pressed 
    196202 
    197                 for (KeyEventPair child : mChildren) { 
     203                for (KeyEventPair child : children) { 
    198204                    if (child.process(event)) { 
    199205                        return true; 
     
    202208 
    203209                if (event.getType() instanceof KeyPressed) { 
    204                     mChildren.add(new KeyEventPair(event)); 
     210                    children.add(new KeyEventPair(event)); 
    205211                    return true; 
    206212                } 
     
    219225         */ 
    220226        private boolean isComplete() { 
    221             if ((mFirst != null) && (mSecond != null)) { 
    222                 for (KeyEventPair child : mChildren) { 
     227            if ((first != null) && (second != null)) { 
     228                for (KeyEventPair child : children) { 
    223229                    if (!child.isComplete()) { 
    224230                        return false; 
  • trunk/quest-core-events/src/main/java/de/ugoe/cs/quest/eventcore/gui/KeyPressed.java

    r544 r564  
    8080    } 
    8181 
     82    /* (non-Javadoc) 
     83     * @see java.lang.Object#hashCode() 
     84     */ 
     85    @Override 
     86    public int hashCode() { 
     87        return super.getKey().hashCode(); 
     88    } 
     89 
    8290} 
  • trunk/quest-core-events/src/main/java/de/ugoe/cs/quest/eventcore/gui/KeyReleased.java

    r544 r564  
    8080    } 
    8181 
     82    /* (non-Javadoc) 
     83     * @see java.lang.Object#hashCode() 
     84     */ 
     85    @Override 
     86    public int hashCode() { 
     87        return super.getKey().hashCode(); 
     88    } 
     89 
    8290} 
  • trunk/quest-core-events/src/main/java/de/ugoe/cs/quest/eventcore/gui/ValueSelection.java

    r550 r564  
    9494    } 
    9595 
     96    /* (non-Javadoc) 
     97     * @see java.lang.Object#hashCode() 
     98     */ 
     99    @Override 
     100    public int hashCode() { 
     101        return selectedValue.hashCode(); 
     102    } 
     103 
    96104} 
  • trunk/quest-core-events/src/main/java/de/ugoe/cs/quest/eventcore/guimodel/AbstractDefaultGUIElement.java

    r550 r564  
    1818    public static final long serialVersionUID = 1L; 
    1919 
    20     /** the parent interaction element */ 
    21     private IGUIElement parent; 
    22  
    2320    /** the information about the original type before the mapping */ 
    2421    private String originalTypeInfo; 
    25  
    26     /* 
    27      * (non-Javadoc) 
    28      *  
    29      * @see GUIElement#getParent() 
    30      */ 
    31     public IGUIElement getParent() { 
    32         return parent; 
    33     } 
    3422 
    3523    /* 
  • trunk/quest-core-events/src/main/java/de/ugoe/cs/quest/eventcore/guimodel/AbstractDefaultGUIElementFactory.java

    r550 r564  
    4040        } 
    4141        else { 
    42             return object2 == null; 
     42            // object 1 is null but object 2 not --> return false 
     43            return false; 
    4344        } 
    4445    } 
Note: See TracChangeset for help on using the changeset viewer.