From 8395e958817fb99c17acc36e72375a74e1d7a38f Mon Sep 17 00:00:00 2001 From: Roland Haeder Date: Mon, 10 Aug 2015 15:27:42 +0200 Subject: [PATCH] =?utf8?q?Don't=20do=20this,=20having=20chars=20for=20data?= =?utf8?q?base=20keys,=20you=20can=20directly=20use=20the=20enum=20name=20?= =?utf8?q?itself.=20Signed-off-by:Roland=20H=C3=A4der=20?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit --- .../mxchange/jcore/contact/BaseContact.java | 2 +- src/org/mxchange/jcore/contact/Gender.java | 111 +----------------- 2 files changed, 6 insertions(+), 107 deletions(-) diff --git a/src/org/mxchange/jcore/contact/BaseContact.java b/src/org/mxchange/jcore/contact/BaseContact.java index 465289a..d5dc47b 100644 --- a/src/org/mxchange/jcore/contact/BaseContact.java +++ b/src/org/mxchange/jcore/contact/BaseContact.java @@ -280,7 +280,7 @@ public class BaseContact extends BaseFrameworkSystem implements Contact { String csvString = String.format( "\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\";\"%s\"", //NOI18N this.isOwnContact(), - this.getGender().getDatabaseValue(), + this.getGender().name(), this.getSurname(), this.getFamilyName(), this.getCompanyName(), diff --git a/src/org/mxchange/jcore/contact/Gender.java b/src/org/mxchange/jcore/contact/Gender.java index 46e7f49..598b697 100644 --- a/src/org/mxchange/jcore/contact/Gender.java +++ b/src/org/mxchange/jcore/contact/Gender.java @@ -28,108 +28,19 @@ public enum Gender { /** * Unknown enum */ - UNKNOWN("U", "BaseContact.gender.unknown.text"), + UNKNOWN("BaseContact.gender.unknown.text"), /** * Male enum */ - MALE("M", "BaseContact.gender.male.text"), + MALE("BaseContact.gender.male.text"), /** * Female enum */ - FEMALE("F", "BaseContact.gender.female.text"), + FEMALE("BaseContact.gender.female.text"), /** * Company enum */ - COMPANY("C", "BaseContact.gender.company.text"); - - /** - * Cache for valid chars - */ - private static char[] validChars; - - /** - * Getter for Gender enum from given character - * - * @param gender Gender character - * @return Gender enum - */ - public static Gender fromChar (final char gender) { - Gender g = null; - switch (gender) { - case 'U': // Unknown - g = UNKNOWN; - break; - - case 'M': // Male - g = MALE; - break; - - case 'F': // Female - g = FEMALE; - break; - - case 'C': // Company - g = COMPANY; - break; - - default: // Unsupported - throw new IllegalArgumentException("gender " + gender + " is invalid."); - } - - // Return it - //* NOISY-DEBUG: */ System.out.println("gender=" + g.getClass().getName()); - return g; - } - - /** - * Valid chars - * - * @return Valid chars - */ - public static char[] validChars () { - // Is cache set? - if (validChars != null) { - // Return it - return validChars; - } - - // Init array - char[] valid = new char[3]; - - // Get values - int i = 0; - for (Object value : values()) { - // First cast - Gender gender = (Gender) value; - - // Debug message - //* NOISY-DEBUG: */ System.out.println("gender=" + gender); - // Is it UNKNOWN? - if (gender.equals(Gender.UNKNOWN)) { - // Skip this - continue; - } - - // Debug message - //* NOISY-DEBUG: */ System.out.println(MessageFormat.format("gender={0} - adding at pos {1} ...", gender, i)); - // Get database value as this is also the access - valid[i] = gender.getDatabaseValue().charAt(0); - - // Increment index - i++; - } - - // Set it here - validChars = valid; - - // Return finialized array - return valid; - } - - /** - * Database value - */ - private final String databaseValue; + COMPANY("BaseContact.gender.company.text"); /** * Output value (for messages) @@ -139,24 +50,12 @@ public enum Gender { /** * Constructor * - * @param databaseValue Value being stored in database * @param messageKey Message key for resource file */ - private Gender (final String databaseValue, final String messageKey) { - // Set both - this.databaseValue = databaseValue; + private Gender (final String messageKey) { this.messageKey = messageKey; } - /** - * Database value - * - * @return the databaseValue - */ - public String getDatabaseValue () { - return this.databaseValue; - } - /** * Output value (for messages) * -- 2.39.5