]> git.mxchange.org Git - addressbook-swing.git/commitdiff
Cleanup through inspection + updated jar(s)
authorRoland Haeder <roland@mxchange.org>
Thu, 15 Oct 2015 07:50:40 +0000 (09:50 +0200)
committerRoland Haeder <roland@mxchange.org>
Thu, 15 Oct 2015 07:50:40 +0000 (09:50 +0200)
Signed-off-by:Roland Häder <roland@mxchange.org>

18 files changed:
lib/jcontacts-business-core.jar
lib/jcontacts-core.jar
lib/jcore-logger-lib.jar
lib/jcore-swing.jar
lib/jcore.jar
lib/jcountry-core.jar
lib/jphone-core.jar
src/org/mxchange/addressbook/application/AddressbookApplication.java
src/org/mxchange/addressbook/client/AddressbookClient.java
src/org/mxchange/addressbook/client/BaseAddressbookClient.java
src/org/mxchange/addressbook/client/console/ConsoleClient.java
src/org/mxchange/addressbook/client/gui/AddressbookFrame.java
src/org/mxchange/addressbook/client/gui/SwingClient.java
src/org/mxchange/addressbook/facade/contact/ContactFacade.java
src/org/mxchange/addressbook/menu/BaseMenu.java
src/org/mxchange/addressbook/menu/Menu.java
src/org/mxchange/addressbook/menu/item/SelectableMenuItem.java
src/org/mxchange/addressbook/menu/item/console/ConsoleMenuItem.java

index b0ead65c4474b20f1881740895a946c1f63b4ae4..2314d5552061d032db5c327684ab92a975c5db13 100644 (file)
Binary files a/lib/jcontacts-business-core.jar and b/lib/jcontacts-business-core.jar differ
index 4bf2b793f990ceb4be6bc46a3fcbabd1234b7a7a..588f0e34e77622c6568476b940a2c8377d868844 100644 (file)
Binary files a/lib/jcontacts-core.jar and b/lib/jcontacts-core.jar differ
index 5049937263bc38db18ddd3a2d142e3f0d86881ed..ce30ba8ab061bc1a1e57a982ffaeea02ad9cb21f 100644 (file)
Binary files a/lib/jcore-logger-lib.jar and b/lib/jcore-logger-lib.jar differ
index 427a4b307cb3fb4df0f706da679b2d4e6c104cc6..b66098fd63eceb1e1d146121b9b302035308a665 100644 (file)
Binary files a/lib/jcore-swing.jar and b/lib/jcore-swing.jar differ
index a6eb677503c0d30b4193df4b45c57ed7216d2652..1c2e5700bc02644df1ffeab194e3f5068109e9bc 100644 (file)
Binary files a/lib/jcore.jar and b/lib/jcore.jar differ
index 9daf43639f781789907f1add123c492bbecd73a3..eaf6110dd5bb7767f1dd977ab8ec1a5ff5f8e05c 100644 (file)
Binary files a/lib/jcountry-core.jar and b/lib/jcountry-core.jar differ
index 97bdd3944061172815f3267f811233b4f4dce6c9..9637788dea980962faa7693e89d80d6fcd735f66 100644 (file)
Binary files a/lib/jphone-core.jar and b/lib/jphone-core.jar differ
index c84b7f462ebb2d69e396675978ff3d34b73ce4c3..20b26acc208903c371940f9924907ead37130c7f 100644 (file)
@@ -52,6 +52,25 @@ public class AddressbookApplication extends BaseApplication implements Applicati
         */
        public static final String APP_VERSION = "0.0"; //NOI18N
 
+       /**
+        * Main method (entry point)
+        * <p>
+        * @param args the command line arguments
+        */
+       public static void main (String[] args) {
+               // Start application
+               new AddressbookApplication().start(args);
+       }
+
+       /**
+        * Getter for printable application name
+        * <p>
+        * @return A printable name
+        */
+       public static String printableTitle () {
+               return MessageFormat.format("{0} v{1}", APP_TITLE, APP_VERSION); //NOI18N
+       }
+
        /**
         * Console client is enabled by default
         */
@@ -91,25 +110,6 @@ public class AddressbookApplication extends BaseApplication implements Applicati
                this.init();
        }
 
-       /**
-        * Main method (entry point)
-        * <p>
-        * @param args the command line arguments
-        */
-       public static void main (String[] args) {
-               // Start application
-               new AddressbookApplication().start(args);
-       }
-
-       /**
-        * Getter for printable application name
-        * <p>
-        * @return A printable name
-        */
-       public static String printableTitle () {
-               return MessageFormat.format("{0} v{1}", APP_TITLE, APP_VERSION); //NOI18N
-       }
-
        @Override
        public void doBootstrap () {
                this.getLogger().logDebug("Initializing application ..."); //NOI18N
@@ -289,19 +289,18 @@ public class AddressbookApplication extends BaseApplication implements Applicati
        private void parseArguments (final String[] args) {
                // Trace message
                this.getLogger().logTrace(MessageFormat.format("args()={0} - CALLED!", args.length)); //NOI18N
-
                // Debug message
                this.getLogger().logDebug(MessageFormat.format("Parsing {0} arguments ...", args.length)); //NOI18N
-
                for (final String arg : args) {
                        // Switch on it
                        switch (arg) {
-                               case "-console": //NOI18N
-                                       enableConsoleClient();
+                               case "-console":
+                                       //NOI18N
+                                       this.enableConsoleClient();
                                        break;
-
-                               case "-gui": //NOI18N
-                                       enableGuiClient();
+                               case "-gui":
+                                       //NOI18N
+                                       this.enableGuiClient();
                                        break;
                        }
                }
@@ -390,4 +389,5 @@ public class AddressbookApplication extends BaseApplication implements Applicati
                this.getLogger().logException(exception);
        }
 
+
 }
index 4961722c77e042d403e4ec79b68cfd19a6f5937e..4a4b67a702150e3c8a864f78d6c372cbb4352b20 100644 (file)
@@ -35,56 +35,56 @@ public interface AddressbookClient extends Client {
         * <p>
         * @param contact Contact to show address from
         */
-       public void displayAddressBox (final Contact contact);
+       void displayAddressBox (final Contact contact);
 
        /**
         * Displays a "box" for the name
         * <p>
         * @param contact Contact to show name from
         */
-       public void displayNameBox (final Contact contact);
+       void displayNameBox (final Contact contact);
 
        /**
         * Displays a "box" for other data
         * <p>
         * @param contact Contact to show other data from
         */
-       public void displayOtherDataBox (final Contact contact);
+       void displayOtherDataBox (final Contact contact);
 
        /**
         * Shows given contact instamce
         * <p>
         * @param contact Contact instance
         */
-       public void show (final Contact contact);
+       void show (final Contact contact);
 
        /**
         * The user changes own address data
         * <p>
         * @param contact Contact instance to change
         */
-       public void doChangeOwnAddressData (final Contact contact);
+       void doChangeOwnAddressData (final Contact contact);
 
        /**
         * The user changes own name data
         * <p>
         * @param contact
         */
-       public void doChangeOwnNameData (final Contact contact);
+       void doChangeOwnNameData (final Contact contact);
 
        /**
         * The user changes own other data
         * <p>
         * @param contact Constact instance to change
         */
-       public void doChangeOwnOtherData (final Contact contact);
+       void doChangeOwnOtherData (final Contact contact);
 
        /**
         * Allows the user to enter own data
         * <p>
         * @return Finished Contact instance
         */
-       public Contact doEnterOwnData ();
+       Contact doEnterOwnData ();
 
        /**
         * Asks the user for a choice and proceeds accordingly
@@ -93,7 +93,7 @@ public interface AddressbookClient extends Client {
         * @throws org.mxchange.jcore.exceptions.MenuInitializationException If the
         * menu cannot be initialized
         */
-       public void doUserMenuChoice () throws UnhandledUserChoiceException, MenuInitializationException;
+       void doUserMenuChoice () throws UnhandledUserChoiceException, MenuInitializationException;
 
        /**
         * Asks the the user to enter a single character which must match validChars
@@ -103,7 +103,7 @@ public interface AddressbookClient extends Client {
         * <p>
         * @return      Allowed character
         */
-       public char enterChar (final char[] validChars, final String message);
+       char enterChar (final char[] validChars, final String message);
 
        /**
         * Asks the user to enter his/her gender (M=Male, F=Female, C=Company)
@@ -112,7 +112,7 @@ public interface AddressbookClient extends Client {
         * <p>
         * @return Gender enum
         */
-       public Gender enterGender (final String message);
+       Gender enterGender (final String message);
 
        /**
         * Reads an integer (int) from the user
@@ -123,7 +123,7 @@ public interface AddressbookClient extends Client {
         * <p>
         * @return Entered string by user or null if empty string is allowed
         */
-       public int enterInt (final int minimum, final int maximum, final String message);
+       int enterInt (final int minimum, final int maximum, final String message);
 
        /**
         * Reads a string of minimum and maximum length from the user. An empty
@@ -137,14 +137,14 @@ public interface AddressbookClient extends Client {
         * <p>
         * @return Entered string by user or null if empty string is allowed
         */
-       public String enterString (final int minLength, final int maxLength, final String message, final boolean allowEmpty);
+       String enterString (final int minLength, final int maxLength, final String message, final boolean allowEmpty);
 
        /**
         * Setter for current menu choice
         * <p>
         * @param currentMenu Current menu choice
         */
-       public void setCurrentMenu (final String currentMenu);
+       void setCurrentMenu (final String currentMenu);
 
        /**
         * Some "Getter" for menu item
@@ -154,19 +154,19 @@ public interface AddressbookClient extends Client {
         * <p>
         * @return
         */
-       public SelectableMenuItem getMenuItem (final char accessKey, final String text);
+       SelectableMenuItem getMenuItem (final char accessKey, final String text);
 
        /**
         * Shows current menu selection to the user
         */
-       public void showCurrentMenu ();
+       void showCurrentMenu ();
 
        /**
         * Shows given menu entry in client
         * <p>
         * @param item Menu item to show
         */
-       public void showEntry (final SelectableMenuItem item);
+       void showEntry (final SelectableMenuItem item);
 
        /**
         * Let the user choose what to change on the address: [n]ame, [a]ddress,
@@ -176,5 +176,5 @@ public interface AddressbookClient extends Client {
         * <p>
         * @throws UnhandledUserChoiceException If choice is not supported
         */
-       public void userChooseChangeContactData (final Contact contact) throws UnhandledUserChoiceException;
+       void userChooseChangeContactData (final Contact contact) throws UnhandledUserChoiceException;
 }
index 3dced1bd5ea2743726bf00b1f00a25e193f3c89e..0091974372c6a337d83ebf6d451d4a399f6c5a40 100644 (file)
@@ -18,6 +18,7 @@ package org.mxchange.addressbook.client;
 
 import java.sql.SQLException;
 import java.text.MessageFormat;
+import java.util.Collections;
 import java.util.HashMap;
 import java.util.Map;
 import javax.naming.Context;
@@ -79,12 +80,12 @@ public abstract class BaseAddressbookClient extends BaseClient implements Addres
         * <p>
         * @return the currentMenu
         */
-       public final String getCurrentMenu () {
+       public String getCurrentMenu () {
                return this.currentMenu;
        }
 
        @Override
-       public final void setCurrentMenu (final String currentMenu) {
+       public void setCurrentMenu (final String currentMenu) {
                this.currentMenu = currentMenu;
        }
 
@@ -141,8 +142,8 @@ public abstract class BaseAddressbookClient extends BaseClient implements Addres
         * <p>
         * @return Map of all menus
         */
-       protected final Map<String, Menu> getMenus () {
-               return this.menus;
+       protected Map<String, Menu> getMenus () {
+               return Collections.unmodifiableMap(this.menus);
        }
 
        /**
index 686f1926cbd56bba1f90b326665628e69d7a3ae3..dd98ba2f70faa24f752687fe5c4e221306bcc8db 100644 (file)
@@ -643,7 +643,7 @@ public class ConsoleClient extends BaseAddressbookClient implements AddressbookC
        }
 
        @Override
-       protected final void fillMenuMap () {
+       protected void fillMenuMap () {
                // Trace message
                this.getLogger().logTrace("CALLED!"); //NOI18N
 
index c1c03468e319698de415c34c2db96626cfc89132..9a46532b38a2fd821c77fcc70bd1cf9862e3b319 100644 (file)
@@ -78,7 +78,7 @@ public class AddressbookFrame extends BaseAddressbookSystem implements ClientFra
         * <p>
         * @return Returns a singelton instance of this frame
         */
-       public static final ClientFrame getSelfInstance (final Client client) {
+       public static ClientFrame getSelfInstance (final Client client) {
                // Is it set?
                if (!(self instanceof ClientFrame)) {
                        // Create new instance
@@ -294,7 +294,7 @@ public class AddressbookFrame extends BaseAddressbookSystem implements ClientFra
         * @return Field isInitialized
         */
        @Override
-       public final boolean isInitialized () {
+       public boolean isInitialized () {
                return this.initialized;
        }
 
@@ -507,16 +507,16 @@ public class AddressbookFrame extends BaseAddressbookSystem implements ClientFra
 
                // Init 3 panels:
                // 1) "name" panel
-               initNameDataPanel(this.addContact);
+               this.initNameDataPanel(this.addContact);
 
                // 2) "address" panel
-               initAddressDataPanel(this.addContact);
+               this.initAddressDataPanel(this.addContact);
 
                // 3) "other" panel
-               initOtherDataPanel(this.addContact);
+               this.initOtherDataPanel(this.addContact);
 
                // 4) "Add" and "Cancel" buttons, combined they are unique for this dialog
-               initAddCancelButtons();
+               this.initAddCancelButtons();
 
                // x)Only for developing:
                /*
@@ -858,7 +858,7 @@ public class AddressbookFrame extends BaseAddressbookSystem implements ClientFra
 
                // Init other windows:
                // 1) Add contact
-               initAddContactDialog();
+               this.initAddContactDialog();
 
                // Trace message
                this.getLogger().logTrace("EXIT!"); //NOI18N
index 82cf84e9db11462403c9cfcbad0f8d83367fd341..a4b6e41290221af209a39e941d9c2c1b2fadff42 100644 (file)
@@ -233,7 +233,7 @@ public class SwingClient extends BaseAddressbookClient implements AddressbookCli
         * Fills menu map with swing menus
         */
        @Override
-       protected final void fillMenuMap () {
+       protected void fillMenuMap () {
                // Nothing to fill here as the Swing frame is handling this all
                throw new UnsupportedOperationException("Not implemented."); //NOI18N
        }
index 1e93bf4d5997e70f893fec31fcf1d6c1c8948e10..5879608b100ee54172a9cfacfdd8ea0f985fbace 100644 (file)
@@ -41,12 +41,12 @@ public interface ContactFacade extends Facade {
         * @throws org.mxchange.jcontacts.exceptions.ContactAlreadyAddedException If
         * the contact is already added
         */
-       public void addContact (final Contact contact) throws ContactAlreadyAddedException;
+       void addContact (final Contact contact) throws ContactAlreadyAddedException;
 
        /**
         * Let the user add a new other address
         */
-       public void doAddOtherAddress ();
+       void doAddOtherAddress ();
 
        /**
         * The user can change address data, like street, ZIP code, city and country
@@ -54,7 +54,7 @@ public interface ContactFacade extends Facade {
         * <p>
         * @param contact Instance to change data
         */
-       public void doChangeAddressData (final Contact contact);
+       void doChangeAddressData (final Contact contact);
 
        /**
         * The user can change name data, like gender, surname, family name and
@@ -62,147 +62,146 @@ public interface ContactFacade extends Facade {
         * <p>
         * @param contact Instance to change data
         */
-       public void doChangeNameData (final Contact contact);
+       void doChangeNameData (final Contact contact);
 
        /**
         * Let the user change other address
         */
-       public void doChangeOtherAddress ();
+       void doChangeOtherAddress ();
 
        /**
         * The user can change other data, like phone numbers or comments.
         * <p>
         * @param contact Instance to change data
         */
-       public void doChangeOtherData (final Contact contact);
+       void doChangeOtherData (final Contact contact);
 
        /**
         * Let the user change own data
         * <p>
         */
-       public void doChangeOwnData ();
+       void doChangeOwnData ();
 
        /**
         * Let the user delete other address
         */
-       public void doDeleteOtherAddress ();
+       void doDeleteOtherAddress ();
 
        /**
         * Asks user for own data
         * <p>
         * @throws org.mxchange.jcontacts.exceptions.ContactAlreadyAddedException If
         * own contact is already added
-        * @throws java.io.IOException If an IO error was found
         */
-       public void doEnterOwnData () throws ContactAlreadyAddedException;
+       void doEnterOwnData () throws ContactAlreadyAddedException;
 
        /**
         * List all contacts
         */
-       public void doListContacts ();
+       void doListContacts ();
 
        /**
         * Searches address book for a contact
         */
-       public void doSearchContacts ();
+       void doSearchContacts ();
 
        /**
         * Allows the user to enter own cellphone number.
         * <p>
         * @return Cellphone number
         */
-       public DialableCellphoneNumber enterOwnCellNumber ();
+       DialableCellphoneNumber enterOwnCellNumber ();
 
        /**
         * Allows the user to enter own city name.
         * <p>
         * @return City name
         */
-       public String enterOwnCity ();
+       String enterOwnCity ();
 
        /**
         * Allows the user to enter comment for own entry.
         * <p>
         * @return Comment
         */
-       public String enterOwnComment ();
+       String enterOwnComment ();
 
        /**
         * Allows the user to enter own company name.
         * <p>
         * @return Company name
         */
-       public String enterOwnCompanyName ();
+       String enterOwnCompanyName ();
 
        /**
         * Allows the user to enter own country code.
         * <p>
         * @return Country code
         */
-       public Country enterOwnCountryCode ();
+       Country enterOwnCountryCode ();
 
        /**
         * Allows the user to enter own email address.
         * <p>
         * @return Email address
         */
-       public String enterOwnEmailAddress ();
+       String enterOwnEmailAddress ();
 
        /**
         * Allows the user to enter own family name.
         * <p>
         * @return Family name
         */
-       public String enterOwnFamilyName ();
+       String enterOwnFamilyName ();
 
        /**
         * Allows the user to enter own fax number.
         * <p>
         * @return Fax number
         */
-       public DialableFaxNumber enterOwnFaxNumber ();
+       DialableFaxNumber enterOwnFaxNumber ();
 
        /**
         * Allows the user to enter own surname.
         * <p>
         * @return Surname
         */
-       public String enterOwnFirstName ();
+       String enterOwnFirstName ();
 
        /**
         * Allows the user to enter own gender.
         * <p>
         * @return Gender
         */
-       public Gender enterOwnGender ();
+       Gender enterOwnGender ();
 
        /**
         * Allows the user to enter own phone number.
         * <p>
         * @return Phone number
         */
-       public DialableLandLineNumber enterOwnPhoneNumber ();
+       DialableLandLineNumber enterOwnPhoneNumber ();
 
        /**
         * Allows the user to enter own street and house number.
         * <p>
         * @return Street and house number
         */
-       public String enterOwnStreet ();
+       String enterOwnStreet ();
 
        /**
         * Allows the user to enter own ZIP code.
         * <p>
         * @return ZIP code
         */
-       public int enterOwnZipCode ();
+       int enterOwnZipCode ();
 
        /**
         * Getter for column count
         * <p>
         * @return Column count TODO: This is needed for TableModel in Swing
         */
-       public int getColumnCount ();
+       int getColumnCount ();
 
        /**
         * Getter for column name at given index.
@@ -211,7 +210,7 @@ public interface ContactFacade extends Facade {
         * <p>
         * @return Database column name TODO: This is needed for TableModel in Swing
         */
-       public String getColumnName (final int columnIndex);
+       String getColumnName (final int columnIndex);
 
        /**
         * Getter for translated column name at given index.
@@ -221,7 +220,7 @@ public interface ContactFacade extends Facade {
         * @return Human-readable column name TODO: This is needed for TableModel in
         * Swing
         */
-       public String getTranslatedColumnName (final int columnIndex);
+       String getTranslatedColumnName (final int columnIndex);
 
        /**
         * Somewhat "getter" for value from given row and column index
@@ -231,7 +230,7 @@ public interface ContactFacade extends Facade {
         * <p>
         * @return Value from given row/column
         */
-       public Object getValueFromRowColumn (final int rowIndex, final int columnIndex);
+       Object getValueFromRowColumn (final int rowIndex, final int columnIndex);
 
        /**
         * Checks whether own contact is already added by checking all entries for
@@ -241,12 +240,12 @@ public interface ContactFacade extends Facade {
         * <p>
         * @throws java.io.IOException If an IO error occurs
         */
-       public boolean isOwnContactAdded () throws IOException;
+       boolean isOwnContactAdded () throws IOException;
 
        /**
         * Adds given contact to address book
         * <p>
         * @param contact Contact being added TODO Add check for book size
         */
-       public void registerContact (final Contact contact);
+       void registerContact (final Contact contact);
 }
index cf5b916ff17bc502bb76710ada4d1fb2afdee318..1d9dfcc38259af7315747c9ec74d1e18f580d1e5 100644 (file)
@@ -18,6 +18,7 @@ package org.mxchange.addressbook.menu;
 
 import java.text.MessageFormat;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
 import org.mxchange.addressbook.BaseAddressbookSystem;
@@ -87,8 +88,8 @@ public abstract class BaseMenu extends BaseAddressbookSystem implements Menu {
         * <p>
         * @return      menuList List of menu entries
         */
-       protected final List<SelectableMenuItem> getMenuList () {
-               return this.menuList;
+       protected List<SelectableMenuItem> getMenuList () {
+               return Collections.unmodifiableList(this.menuList);
        }
 
        /**
index a0ba62f9f1539f8b37bcf253f6cfbd9ffdf80b4d..f6055638f202c28890f96b0acdb98408ded04438 100644 (file)
@@ -32,19 +32,19 @@ public interface Menu {
         * <p>
         * @return
         */
-       public int getMenuItemsCount ();
+       int getMenuItemsCount ();
 
        /**
         * "Getter" for an iterator on all menu items of the current menu
         * <p>
         * @return Iterator on all menu items
         */
-       public Iterator<SelectableMenuItem> getMenuItemsIterator ();
+       Iterator<SelectableMenuItem> getMenuItemsIterator ();
 
        /**
         * Shows this menu
         * <p>
         * @param client Client instance
         */
-       public void show (final Client client);
+       void show (final Client client);
 }
index 59758f821b201f9619d514052a705b6259e0bbce..5f81076c9c20db0ac5754b1dd77688ecedbeff6a 100644 (file)
@@ -31,19 +31,19 @@ public interface SelectableMenuItem extends FrameworkInterface {
         * <p>
         * @return the accessKey
         */
-       public char getAccessKey ();
+       char getAccessKey ();
 
        /**
         * Text to user
         * <p>
         * @return the text
         */
-       public String getText ();
+       String getText ();
 
        /**
         * Shows this menu item
         * <p>
         * @param client Client instance
         */
-       public void show (final Client client);
+       void show (final Client client);
 }
index 74f8180171e45c128a3e30682538d380aaae19c8..339623d69c6784c69ff651e3bcd40f46f503ee5d 100644 (file)
@@ -51,15 +51,33 @@ public class ConsoleMenuItem extends BaseMenuItem implements SelectableMenuItem
        }
 
        @Override
-       public final char getAccessKey () {
+       public char getAccessKey () {
                return this.accessKey;
        }
 
+       /**
+        * Access key
+        * <p>
+        * @param accessKey the accessKey to set
+        */
+       private void setAccessKey (final char accessKey) {
+               this.accessKey = accessKey;
+       }
+
        @Override
        public String getText () {
                return this.text;
        }
 
+       /**
+        * Text to user
+        * <p>
+        * @param text the text to set
+        */
+       private void setText (final String text) {
+               this.text = text;
+       }
+
        @Override
        public void show (final Client client) {
                // Trace message
@@ -84,22 +102,4 @@ public class ConsoleMenuItem extends BaseMenuItem implements SelectableMenuItem
                this.getLogger().logTrace("EXIT!"); //NOI18N
        }
 
-       /**
-        * Text to user
-        * <p>
-        * @param text the text to set
-        */
-       private void setText (final String text) {
-               this.text = text;
-       }
-
-       /**
-        * Access key
-        * <p>
-        * @param accessKey the accessKey to set
-        */
-       private void setAccessKey (final char accessKey) {
-               this.accessKey = accessKey;
-       }
-
 }