]> git.mxchange.org Git - jaddressbook-lib.git/commitdiff
Added 'final' keyword to getters/setters as it makes no sense overwriting them.
authorRoland Haeder <roland@mxchange.org>
Thu, 23 Jul 2015 11:39:32 +0000 (13:39 +0200)
committerRoland Haeder <roland@mxchange.org>
Thu, 23 Jul 2015 11:39:32 +0000 (13:39 +0200)
Signed-off-by: Roland Häder <roland@mxchange.org>
14 files changed:
Addressbook/src/org/mxchange/addressbook/BaseFrameworkSystem.java
Addressbook/src/org/mxchange/addressbook/FrameAlreadyInitializedException.java
Addressbook/src/org/mxchange/addressbook/FrameworkInterface.java
Addressbook/src/org/mxchange/addressbook/client/BaseClient.java
Addressbook/src/org/mxchange/addressbook/client/console/ConsoleClient.java
Addressbook/src/org/mxchange/addressbook/client/gui/AddressbookFrame.java
Addressbook/src/org/mxchange/addressbook/client/gui/ClientFrame.java
Addressbook/src/org/mxchange/addressbook/contact/BaseContact.java
Addressbook/src/org/mxchange/addressbook/contact/user/UserContact.java
Addressbook/src/org/mxchange/addressbook/database/backend/csv/CsvBackend.java
Addressbook/src/org/mxchange/addressbook/database/backend/csv/CsvDatabaseBackend.java
Addressbook/src/org/mxchange/addressbook/database/frontend/BaseDatabaseFrontend.java
Addressbook/src/org/mxchange/addressbook/manager/BaseManager.java
Addressbook/src/org/mxchange/addressbook/manager/contact/ContactManager.java

index 680474db064288d1de4d9e47f98835ed158060be..49ac02e968a1ed103fb7c2cda3cfdfe1997d8c12 100644 (file)
@@ -160,7 +160,7 @@ public class BaseFrameworkSystem implements FrameworkInterface {
      * \r
      * @return Resource bundle\r
      */\r
-    protected ResourceBundle getBundle () {\r
+    protected final ResourceBundle getBundle () {\r
        return this.bundle;\r
     }\r
 }\r
index 1709f0b1b192db25876c0282dd318ba6724abf9d..0a4be3cc08f2fa64ab9548c36f67047f107c2fc0 100644 (file)
@@ -17,7 +17,8 @@
 package org.mxchange.addressbook;
 
 /**
- *
+ * This exception is thrown when initFrame() is called twice.
+ * 
  * @author Roland Haeder
  */
 public class FrameAlreadyInitializedException extends Exception {
index 2158bdbbd6d19f22f1e6b04fd423702d3c498e6e..5236877890177bf661c51483c5f88ec50ec1fe2a 100644 (file)
@@ -21,6 +21,7 @@ import org.mxchange.addressbook.client.Client;
 import org.mxchange.addressbook.manager.contact.ManageableContact;\r
 \r
 /**\r
+ * A general interface which should be always expanded\r
  *\r
  * @author Roland Haeder\r
  */\r
index 0282c91f6f65760462868118d1f7fbdf0813b638..399a6c1b9bedb891adc79f848e9ce61269ab9777 100644 (file)
@@ -69,7 +69,7 @@ public abstract class BaseClient extends BaseFrameworkSystem {
     /**\r
      * Enables the client\r
      */\r
-    public void enableIsRunning () {\r
+    public final void enableIsRunning () {\r
        this.isRunning = true;\r
     }\r
 \r
@@ -78,7 +78,7 @@ public abstract class BaseClient extends BaseFrameworkSystem {
      * \r
      * @return the currentMenu\r
      */\r
-    public String getCurrentMenu () {\r
+    public final String getCurrentMenu () {\r
        return this.currentMenu;\r
     }\r
 \r
@@ -86,7 +86,7 @@ public abstract class BaseClient extends BaseFrameworkSystem {
      * Current menu choice\r
      * @param currentMenu the currentMenu to set\r
      */\r
-    public void setCurrentMenu (final String currentMenu) {\r
+    public final void setCurrentMenu (final String currentMenu) {\r
        this.currentMenu = currentMenu;\r
     }\r
 \r
@@ -116,7 +116,7 @@ public abstract class BaseClient extends BaseFrameworkSystem {
      * \r
      * @return Whether the application is still active\r
      */\r
-    public boolean isRunning () {\r
+    public final boolean isRunning () {\r
        // In console client, 0 may have been used\r
        return this.isRunning;\r
     }\r
@@ -124,7 +124,7 @@ public abstract class BaseClient extends BaseFrameworkSystem {
     /**\r
      * Disables running state, so the main loop can abort.\r
      */\r
-    protected void disableIsRunning () {\r
+    protected final void disableIsRunning () {\r
        this.isRunning = false;\r
     }\r
 \r
index b97600e42d1643e88202ac0868b2f37454524a39..8d553096afae3f2e33f1cbdf2e36ee441cc8789b 100644 (file)
@@ -414,25 +414,25 @@ public class ConsoleClient extends BaseClient implements Client {
     public void userChooseChangeContactData (final Contact contact) throws UnhandledUserChoiceException {\r
        // Ask the user for editing [name], [a]ddress or [other] data\r
        char choice = this.enterChar(new char[]{'n', 'a', 'o', 'x'}, "Welchen Daten möchten Sie ändern? (n=Namensdaten, a=Anschriftsdaten, o=Andere, x=Zurück zur Hauptauswahl) ");\r
-       \r
+\r
        // @TODO Get rid of this ugly switch block, too\r
        switch (choice) {\r
            case 'n': // Name data\r
                this.getContactManager().doChangeNameData(contact, this);\r
                break;\r
-               \r
+\r
            case 'a': // Address data\r
                this.getContactManager().doChangeAddressData(contact, this);\r
                break;\r
-               \r
+\r
            case 'o': // Other data\r
                this.getContactManager().doChangeOtherData(contact, this);\r
                break;\r
-               \r
+\r
            case 'x': // Exit this menu\r
                // Ignored as it should go back\r
                break;\r
-               \r
+\r
            default:\r
                // @TODO throw own exception\r
                throw new UnhandledUserChoiceException(MessageFormat.format("Choice '{0}' not handled yet.", choice));\r
index 29151b1a1460ec185dd549b5bea0a20b6f2983c1..fe9d47b09ad23b721d4a1717c8bb3a2fc4669f12 100644 (file)
@@ -16,6 +16,7 @@
  */\r
 package org.mxchange.addressbook.client.gui;\r
 \r
+import java.text.MessageFormat;\r
 import javax.swing.JFrame;\r
 import org.mxchange.addressbook.BaseFrameworkSystem;\r
 import org.mxchange.addressbook.FrameAlreadyInitializedException;\r
@@ -62,11 +63,12 @@ public class AddressbookFrame extends BaseFrameworkSystem implements ClientFrame
 \r
     /**\r
      * Creates an instance of this frame with a client instance\r
+     * \r
      * @param client\r
      */\r
     private AddressbookFrame (final Client client) {\r
        // Debug line\r
-       this.getLogger().debug("Initializing Swing frame ...");\r
+       this.getLogger().trace(MessageFormat.format("client={0}: CALLED!", client));\r
 \r
        // Set frame instance\r
        this.frame = new JFrame(AddressbookApplication.printableTitle());\r
@@ -75,18 +77,16 @@ public class AddressbookFrame extends BaseFrameworkSystem implements ClientFrame
        this.setClient(client);\r
     }\r
 \r
-    @Override\r
-    public void setVisible (final boolean visible) {\r
-       throw new UnsupportedOperationException("Not supported yet.");\r
-    }\r
-\r
     /**\r
-     * Setups the frame\r
+     * Setups the frame, do not set isInitialized here\r
      * \r
      * @param client Client instance\r
      */\r
     @Override\r
     public void setupFrame (final Client client) {\r
+       // Debug line\r
+       this.getLogger().trace(MessageFormat.format("client={0}: CALLED!", client));\r
+\r
        // Has the user entered own data?\r
        if (this.getClient().getContactManager().isOwnContactAdded()) {\r
            // Debug message\r
@@ -109,8 +109,11 @@ public class AddressbookFrame extends BaseFrameworkSystem implements ClientFrame
      */\r
     @Override\r
     public void initFrame () throws FrameAlreadyInitializedException {\r
+       // Debug line\r
+       this.getLogger().trace("CALLED!");\r
+\r
        // Has this frame been initialized?\r
-       if (this.isInitialized) {\r
+       if (this.isInitialized()) {\r
            // Throw exception\r
            throw new FrameAlreadyInitializedException();\r
        }\r
@@ -122,10 +125,22 @@ public class AddressbookFrame extends BaseFrameworkSystem implements ClientFrame
        this.isInitialized = true;\r
     }\r
 \r
+    /**\r
+     * Returns field isInitialized. This flag indicates whether this frame has been initialized or not.\r
+     * \r
+     * @return Field isInitialized\r
+     */\r
+    @Override\r
+    public final boolean isInitialized () {\r
+       return this.isInitialized;\r
+    }\r
+\r
     /**\r
      * Initialize components\r
      */\r
     private void initComponents () {\r
+       // Debug line\r
+       this.getLogger().trace("CALLED!");\r
     }\r
 \r
 }\r
index 06af9bb57af2cb05b262f18aa1172d7212c61eed..0c5ab4dfa365d580cd7d769f253f5aec4880c671 100644 (file)
@@ -21,18 +21,11 @@ import org.mxchange.addressbook.FrameworkInterface;
 import org.mxchange.addressbook.client.Client;\r
 \r
 /**\r
+ * An interface for applications with a frame\r
  *\r
  * @author Roland Haeder\r
  */\r
 public interface ClientFrame extends FrameworkInterface {\r
-\r
-    /**\r
-     * From JFrame\r
-     * \r
-     * @param visible Set visibility\r
-     */\r
-    public void setVisible (boolean visible);\r
-\r
     /**\r
      * Setups the frame (and starts it). You have to call initFrame() before you\r
      * can call this method.\r
@@ -47,4 +40,11 @@ public interface ClientFrame extends FrameworkInterface {
      * @throws org.mxchange.addressbook.FrameAlreadyInitializedException If this method has been called twice\r
      */\r
     public void initFrame () throws FrameAlreadyInitializedException;\r
+\r
+    /**\r
+     * Returns field isInitialized. This flag indicates whether this frame has been initialized or not.\r
+     * \r
+     * @return Field isInitialized\r
+     */\r
+    public boolean isInitialized ();\r
 }\r
index 2a1ecb73ffd0bbf9d1f3605b063a71355379bf39..975ce3f516826741bba66a66cc95f308fde2de96 100644 (file)
@@ -152,15 +152,6 @@ public class BaseContact extends BaseFrameworkSystem {
        return this.birthday;\r
     }\r
 \r
-    /**\r
-     * Birth day\r
-     *\r
-     * @param birthday the birthday to set\r
-     */\r
-    public void setBirthday (final String birthday) {\r
-       this.birthday = birthday;\r
-    }\r
-\r
     /**\r
      * Cellphone number\r
      *\r
@@ -170,15 +161,6 @@ public class BaseContact extends BaseFrameworkSystem {
        return this.cellphoneNumber;\r
     }\r
 \r
-    /**\r
-     * Cellphone number\r
-     *\r
-     * @param cellphoneNumber the cellphoneNumber to set\r
-     */\r
-    public void setCellphoneNumber (final String cellphoneNumber) {\r
-       this.cellphoneNumber = cellphoneNumber;\r
-    }\r
-\r
     /**\r
      * City\r
      *\r
@@ -193,7 +175,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param city the city to set\r
      */\r
-    public void setCity (final String city) {\r
+    private void setCity (final String city) {\r
        this.city = city;\r
     }\r
 \r
@@ -211,7 +193,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param comment the comment to set\r
      */\r
-    public void setComment (final String comment) {\r
+    private void setComment (final String comment) {\r
        this.comment = comment;\r
     }\r
 \r
@@ -229,7 +211,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param companyName the companyName to set\r
      */\r
-    public void setCompanyName (final String companyName) {\r
+    private void setCompanyName (final String companyName) {\r
        this.companyName = companyName;\r
     }\r
 \r
@@ -247,7 +229,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param countryCode the countryCode to set\r
      */\r
-    public void setCountryCode (final String countryCode) {\r
+    private void setCountryCode (final String countryCode) {\r
        this.countryCode = countryCode;\r
     }\r
 \r
@@ -295,7 +277,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param emailAddress the emailAddress to set\r
      */\r
-    public void setEmailAddress (final String emailAddress) {\r
+    private void setEmailAddress (final String emailAddress) {\r
        this.emailAddress = emailAddress;\r
     }\r
 \r
@@ -313,7 +295,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param familyName the familyName to set\r
      */\r
-    public void setFamilyName (final String familyName) {\r
+    private void setFamilyName (final String familyName) {\r
        this.familyName = familyName;\r
     }\r
 \r
@@ -331,7 +313,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param faxNumber the faxNumber to set\r
      */\r
-    public void setFaxNumber (final String faxNumber) {\r
+    private void setFaxNumber (final String faxNumber) {\r
        this.faxNumber = faxNumber;\r
     }\r
 \r
@@ -349,7 +331,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param gender the gender to set\r
      */\r
-    public void setGender (final char gender) {\r
+    private void setGender (final char gender) {\r
        this.gender = gender;\r
     }\r
 \r
@@ -362,15 +344,6 @@ public class BaseContact extends BaseFrameworkSystem {
        return this.houseNumber;\r
     }\r
 \r
-    /**\r
-     * House number\r
-     *\r
-     * @param houseNumber the houseNumber to set\r
-     */\r
-    public void setHouseNumber (final int houseNumber) {\r
-       this.houseNumber = houseNumber;\r
-    }\r
-\r
     /**\r
      * Phone number\r
      *\r
@@ -380,15 +353,6 @@ public class BaseContact extends BaseFrameworkSystem {
        return this.phoneNumber;\r
     }\r
 \r
-    /**\r
-     * Phone number\r
-     *\r
-     * @param phoneNumber the phoneNumber to set\r
-     */\r
-    public void setPhoneNumber (final String phoneNumber) {\r
-       this.phoneNumber = phoneNumber;\r
-    }\r
-\r
     /**\r
      * Street\r
      *\r
@@ -403,7 +367,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param street the street to set\r
      */\r
-    public void setStreet (final String street) {\r
+    protected final void setStreet (final String street) {\r
        this.street = street;\r
     }\r
 \r
@@ -412,19 +376,10 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @return the surname\r
      */\r
-    public String getSurname () {\r
+    public final String getSurname () {\r
        return this.surname;\r
     }\r
 \r
-    /**\r
-     * Surname\r
-     *\r
-     * @param surname the surname to set\r
-     */\r
-    public void setSurname (final String surname) {\r
-       this.surname = surname;\r
-    }\r
-\r
     /**\r
      * Some "getter" for a translated/human-readable gender\r
      * @return gender Human-readable gender\r
@@ -461,7 +416,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @return the zipCode\r
      */\r
-    public long getZipCode () {\r
+    public final long getZipCode () {\r
        return this.zipCode;\r
     }\r
 \r
@@ -470,10 +425,10 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @param zipCode the zipCode to set\r
      */\r
-    public void setZipCode (final long zipCode) {\r
+    protected final void setZipCode (final long zipCode) {\r
        this.zipCode = zipCode;\r
     }\r
-\r
+    \r
     @Override\r
     public int hashCode () {\r
        int hash = 7;\r
@@ -488,7 +443,7 @@ public class BaseContact extends BaseFrameworkSystem {
      *\r
      * @return Own data?\r
      */\r
-    public boolean isOwnContact () {\r
+    public final boolean isOwnContact () {\r
        return this.ownContact;\r
     }\r
 \r
@@ -500,11 +455,11 @@ public class BaseContact extends BaseFrameworkSystem {
     public void show (final Client client) {\r
        // Display name "box"\r
        client.displayNameBox((Contact) this);\r
-\r
-       // Display address "box"\r
+       \r
+       // Display address "box"\r
        client.displayAddressBox((Contact) this);\r
-\r
-       // Display other data "box"\r
+       \r
+       // Display other data "box"\r
        client.displayOtherDataBox((Contact) this);\r
     }\r
 \r
@@ -589,7 +544,52 @@ public class BaseContact extends BaseFrameworkSystem {
      * Enables the flag "own data" which signals that this contact is the user's\r
      * own data.\r
      */\r
-    protected void enableFlagOwnContact () {\r
+    protected final void enableFlagOwnContact () {\r
        this.ownContact = true;\r
     }\r
+\r
+    /**\r
+     * Surname\r
+     *\r
+     * @param surname the surname to set\r
+     */\r
+    protected final void setSurname (final String surname) {\r
+       this.surname = surname;\r
+    }\r
+\r
+    /**\r
+     * Phone number\r
+     *\r
+     * @param phoneNumber the phoneNumber to set\r
+     */\r
+    protected final void setPhoneNumber (final String phoneNumber) {\r
+       this.phoneNumber = phoneNumber;\r
+    }\r
+\r
+    /**\r
+     * House number\r
+     *\r
+     * @param houseNumber the houseNumber to set\r
+     */\r
+    protected final void setHouseNumber (final int houseNumber) {\r
+       this.houseNumber = houseNumber;\r
+    }\r
+\r
+    /**\r
+     * Cellphone number\r
+     *\r
+     * @param cellphoneNumber the cellphoneNumber to set\r
+     */\r
+    protected final void setCellphoneNumber (final String cellphoneNumber) {\r
+       this.cellphoneNumber = cellphoneNumber;\r
+    }\r
+\r
+    /**\r
+     * Birth day\r
+     *\r
+     * @param birthday the birthday to set\r
+     */\r
+    protected final void setBirthday (final String birthday) {\r
+       this.birthday = birthday;\r
+    }\r
 }\r
index 1f9b14186dfcb59d56ac65398b01cc54dba91f16..bf0e5228f9102318bcc0b1e42c6e1d0bfe4b1e6e 100644 (file)
@@ -40,10 +40,8 @@ public class UserContact extends BookContact implements Contact, StoreableCsv {
        // Make sure all constructors are called\r
        this();\r
 \r
-       this.setGender(gender);\r
-       this.setSurname(surname);\r
-       this.setFamilyName(familyName);\r
-       this.setCompanyName(companyName);\r
+       // Update all data\r
+       this.updateNameData(gender, surname, familyName, companyName);\r
     }\r
 \r
     /**\r
index 704cbb6d726cfee2f0d14d7945aa900bda0a2a91..f310588e0b334cab63ee88602fe400144b416d3f 100644 (file)
@@ -31,6 +31,7 @@ public interface CsvBackend extends DatabaseBackend {
      * Gets an iterator for contacts\r
      * \r
      * @return Iterator for contacts\r
+     * @throws org.mxchange.addressbook.BadTokenException If the CSV token is badly formulated\r
      */\r
     public Iterator<Contact> contactIterator () throws BadTokenException;\r
 }\r
index 0bc97f3a78b9d12436195296e223d2ce7c44639e..9f3e8329fbd41f04df9372490969ed228ca76d93 100644 (file)
@@ -100,7 +100,7 @@ public class CsvDatabaseBackend extends BaseDatabaseBackend implements CsvBacken
     public void doShutdown () {\r
        try {\r
            // Close file\r
-           this.storageFile.close();\r
+           this.getStorageFile().close();\r
        } catch (final IOException ex) {\r
            this.getLogger().catching(ex);\r
            System.exit(1);\r
@@ -117,7 +117,7 @@ public class CsvDatabaseBackend extends BaseDatabaseBackend implements CsvBacken
        long length = 0;\r
        \r
        try {\r
-           length = this.storageFile.length();\r
+           length = this.getStorageFile().length();\r
            this.getLogger().debug(MessageFormat.format("length={0}", length));\r
        } catch (final IOException ex) {\r
            // Length cannot be determined\r
@@ -139,7 +139,7 @@ public class CsvDatabaseBackend extends BaseDatabaseBackend implements CsvBacken
 \r
        try {\r
            // Rewind underlaying database file\r
-           this.storageFile.seek(0);\r
+           this.getStorageFile().seek(0);\r
        } catch (final IOException ex) {\r
            this.getLogger().catching(ex);\r
            System.exit(1);\r
@@ -169,7 +169,7 @@ public class CsvDatabaseBackend extends BaseDatabaseBackend implements CsvBacken
        this.getLogger().debug(MessageFormat.format("str({0})={1}", str.length(), str));\r
 \r
        // The string is now a valid CSV string\r
-       this.storageFile.writeBytes(str);\r
+       this.getStorageFile().writeBytes(str);\r
     }\r
 \r
     /**\r
@@ -198,6 +198,15 @@ public class CsvDatabaseBackend extends BaseDatabaseBackend implements CsvBacken
        }\r
     }\r
 \r
+    /**\r
+     * Returns storage file\r
+     * \r
+     * @return Storage file instance\r
+     */\r
+    private RandomAccessFile getStorageFile () {\r
+       return this.storageFile;\r
+    }\r
+\r
     /**\r
      * Checks whether end of file has been reached\r
      * \r
@@ -208,7 +217,7 @@ public class CsvDatabaseBackend extends BaseDatabaseBackend implements CsvBacken
        boolean isEof = true;\r
 \r
        try {\r
-           isEof = (this.storageFile.getFilePointer() >= this.length());\r
+           isEof = (this.getStorageFile().getFilePointer() >= this.length());\r
        } catch (final IOException ex) {\r
            // Length cannot be determined\r
            this.getLogger().catching(ex);\r
@@ -481,7 +490,7 @@ public class CsvDatabaseBackend extends BaseDatabaseBackend implements CsvBacken
        String input = null;\r
 \r
        try {\r
-           input = this.storageFile.readLine();\r
+           input = this.getStorageFile().readLine();\r
        } catch (final IOException ex) {\r
            this.getLogger().catching(ex);\r
        }\r
index de208ba582d5f69ebab454c2936e7492210a66c7..509dd66a2e4f10bfe558b65d9a8e8690cbd9aa12 100644 (file)
@@ -43,7 +43,7 @@ public class BaseDatabaseFrontend extends BaseFrameworkSystem {
      * \r
      * @return the backend\r
      */\r
-    protected DatabaseBackend getBackend () {\r
+    protected final DatabaseBackend getBackend () {\r
        return this.backend;\r
     }\r
 \r
@@ -52,7 +52,7 @@ public class BaseDatabaseFrontend extends BaseFrameworkSystem {
      * \r
      * @param backend the backend to set\r
      */\r
-    protected void setBackend (final DatabaseBackend backend) {\r
+    protected final void setBackend (final DatabaseBackend backend) {\r
         this.backend = backend;\r
      }\r
 \r
index 27900753d30d566977cf34481dce44b5f8303ef5..d845d92a36fb1d73ce95c0e8980062ad62bf070e 100644 (file)
@@ -25,7 +25,7 @@ import org.mxchange.addressbook.BaseFrameworkSystem;
  * @version 0.0\r
  * @since 0.0\r
  */\r
-public class BaseManager extends BaseFrameworkSystem {\r
+public class BaseManager extends BaseFrameworkSystem implements Manageable {\r
     /**\r
      * No instances can be created of this class\r
      */\r
index fefd2c1410e9473d13d7aadaf2e78764e1228931..709f6f9369bd8d9f4e140a15382896086ce5c7b8 100644 (file)
@@ -241,7 +241,7 @@ public class ContactManager extends BaseManager implements ManageableContact {
     @Override\r
     public void doShutdown () {\r
        // Shut down the database layer\r
-       this.contactDatabase.doShutdown();\r
+       this.getContactDatabase().doShutdown();\r
     }\r
 \r
     /**\r
@@ -373,7 +373,7 @@ public class ContactManager extends BaseManager implements ManageableContact {
     }\r
 \r
     @Override\r
-    public int getColumnCount () {\r
+    public final int getColumnCount () {\r
        return this.columnNames.size();\r
     }\r
 \r
@@ -465,7 +465,7 @@ public class ContactManager extends BaseManager implements ManageableContact {
      * @return size of contact "book"\r
      */\r
     @Override\r
-    public int size () {\r
+    public final int size () {\r
        return this.contacts.size();\r
     }\r
 \r
@@ -473,6 +473,9 @@ public class ContactManager extends BaseManager implements ManageableContact {
      * Fills the column names array with strings from bundle\r
      */\r
     private void fillColumnNamesFromBundle () {\r
+       // Debug message\r
+       this.getLogger().trace("CALLED!");\r
+\r
        // First get an iterator from key set to iterate over\r
        Iterator<String> iterator = this.getBundle().keySet().iterator();\r
 \r
@@ -490,6 +493,9 @@ public class ContactManager extends BaseManager implements ManageableContact {
                this.columnNames.add(this.getBundle().getString(key));\r
            }\r
        }\r
+\r
+       // Debug message\r
+       this.getLogger().trace(MessageFormat.format("getColumnCount()={0}: EXIT!", this.getColumnCount()));\r
     }\r
 \r
     /**\r