ควรกำหนดค่าคงที่สตริงถ้าจะใช้เพียงครั้งเดียวหรือไม่

เรากำลังใช้งานอะแดปเตอร์สำหรับ Jaxen (ไลบรารี XPath สำหรับ Java) ที่ช่วยให้เราใช้ XPath เพื่อเข้าถึงโมเดลข้อมูลของแอปพลิเคชันของเรา

สิ่งนี้ทำได้โดยการใช้คลาสที่จับคู่สตริง (ส่งถึงเราจาก Jaxen) ไปยังองค์ประกอบของโมเดลข้อมูลของเรา เราประเมินว่าเราต้องการประมาณ 100 คลาสโดยมีการเปรียบเทียบสตริงมากกว่า 1,000 รายการ

ฉันคิดว่าวิธีที่ดีที่สุดในการทำเช่นนี้คือง่ายถ้าคำสั่ง / else พร้อมสตริงที่เขียนลงในโค้ดโดยตรงแทนที่จะกำหนดแต่ละสตริงเป็นค่าคงที่ ตัวอย่างเช่น:

public Object getNode(String name) {
    if ("name".equals(name)) {
        return contact.getFullName();
    } else if ("title".equals(name)) {
        return contact.getTitle();
    } else if ("first_name".equals(name)) {
        return contact.getFirstName();
    } else if ("last_name".equals(name)) {
        return contact.getLastName();
    ...

อย่างไรก็ตามฉันถูกสอนเสมอว่าเราไม่ควรฝังค่าสตริงลงในรหัสโดยตรง แต่สร้างค่าคงที่สตริงแทน ที่จะมีลักษณะเช่นนี้:

private static final String NAME = "name";
private static final String TITLE = "title";
private static final String FIRST_NAME = "first_name";
private static final String LAST_NAME = "last_name";

public Object getNode(String name) {
    if (NAME.equals(name)) {
        return contact.getFullName();
    } else if (TITLE.equals(name)) {
        return contact.getTitle();
    } else if (FIRST_NAME.equals(name)) {
        return contact.getFirstName();
    } else if (LAST_NAME.equals(name)) {
        return contact.getLastName();
    ...

ในกรณีนี้ฉันคิดว่ามันเป็นความคิดที่ไม่ดี ค่าคงที่จะถูกใช้ครั้งเดียวเท่านั้นในgetNode()วิธีการ การใช้สตริงโดยตรงนั้นง่ายต่อการอ่านและทำความเข้าใจเหมือนกับการใช้ค่าคงที่และช่วยให้เราเขียนโค้ดอย่างน้อยหนึ่งพันบรรทัด

ดังนั้นมีเหตุผลใดที่จะกำหนดค่าคงที่สตริงสำหรับการใช้งานครั้งเดียว? หรือเป็นที่ยอมรับในการใช้สตริงโดยตรง

PS ก่อนใครแนะนำให้ใช้ enums แทนเราทำต้นแบบว่า แต่การแปลง enum ช้ากว่าการเปรียบเทียบสตริงธรรมดา 15 เท่าดังนั้นจึงไม่ได้รับการพิจารณา

สรุป: คำตอบด้านล่างขยายขอบเขตของคำถามนี้นอกเหนือจากค่าคงที่สตริงดังนั้นฉันจึงมีข้อสรุปสองข้อ:

• อาจเป็นไปได้ที่จะใช้สตริงโดยตรงแทนที่จะเป็นค่าคงที่สตริงในสถานการณ์นี้แต่
• มีวิธีหลีกเลี่ยงการใช้สตริงเลยซึ่งอาจจะดีกว่า

ดังนั้นฉันจะลองใช้เทคนิค wrapper ซึ่งหลีกเลี่ยงสตริงอย่างสมบูรณ์ น่าเสียดายที่เราไม่สามารถใช้คำสั่ง string switch ได้เนื่องจากเรายังไม่ได้ใช้บน Java 7 ในท้ายที่สุดฉันคิดว่าคำตอบที่ดีที่สุดสำหรับเราคือการลองใช้เทคนิคแต่ละอย่างและประเมินประสิทธิภาพ ความจริงก็คือถ้าเทคนิคหนึ่งเร็วขึ้นอย่างเห็นได้ชัดเราอาจเลือกโดยไม่คำนึงถึงความงามหรือการยึดมั่นในการประชุม

ลองสิ่งนี้ การสะท้อนครั้งแรกนั้นมีราคาแพงอย่างแน่นอน แต่ถ้าคุณจะใช้มันหลายครั้งซึ่งฉันคิดว่าคุณจะต้องใช้นี่เป็นวิธีแก้ปัญหาที่ดีที่สุดในสิ่งที่คุณเสนอ ฉันไม่ชอบใช้การสะท้อน แต่ฉันพบว่าตัวเองใช้มันเมื่อฉันไม่ชอบทางเลือกในการสะท้อนกลับ ฉันคิดว่าสิ่งนี้จะช่วยให้ทีมของคุณปวดหัวมาก แต่คุณต้องผ่านชื่อของวิธีการ (เป็นตัวพิมพ์เล็ก)

กล่าวอีกนัยหนึ่งแทนที่จะผ่าน "ชื่อ" คุณจะต้องผ่าน "ชื่อเต็ม" เพราะชื่อของวิธีการรับคือ "getFullName ()"

Map<String, Method> methodMapping = null;

public Object getNode(String name) {
    Map<String, Method> methods = getMethodMapping(contact.getClass());
    return methods.get(name).invoke(contact);
}

public Map<String, Method> getMethodMapping(Class<?> contact) {
    if(methodMapping == null) {
        Map<String, Method> mapping = new HashMap<String, Method>();
        Method[] methods = contact.getDeclaredMethods();
        for(Method method : methods) {
            if(method.getParameterTypes().length() == 0) {
                if(method.getName().startsWith("get")) {
                    mapping.put(method.getName().substring(3).toLower(), method);
                } else if (method.getName().startsWith("is"))) {
                    mapping.put(method.getName().substring(2).toLower(), method);
                }
            }
        }
        methodMapping = mapping;
    }
    return methodMapping;
}

หากคุณต้องการเข้าถึงข้อมูลที่มีอยู่ภายในสมาชิกของผู้ติดต่อคุณอาจพิจารณาสร้างคลาส wrapper สำหรับผู้ติดต่อที่มีวิธีการทั้งหมดในการเข้าถึงข้อมูลที่จำเป็น สิ่งนี้จะเป็นประโยชน์สำหรับการรับประกันว่าชื่อของฟิลด์การเข้าถึงจะยังคงเหมือนเดิมเสมอ (เช่นถ้าคลาส wrapper มี getFullName () และคุณโทรด้วยชื่อเต็มมันจะใช้ได้เสมอแม้ว่าชื่อ getFullName () ของผู้ติดต่อจะถูกเปลี่ยน - จะทำให้เกิดข้อผิดพลาดในการรวบรวมก่อนที่จะให้คุณทำเช่นนั้น)

public class ContactWrapper {
    private Contact contact;

    public ContactWrapper(Contact contact) {
        this.contact = contact;
    }

    public String getFullName() {
        return contact.getFullName();
    }
    ...
}

โซลูชันนี้ช่วยฉันหลายครั้งคือเมื่อฉันต้องการให้มีการแสดงข้อมูลเดี่ยวเพื่อใช้ใน DataTables jsf และเมื่อข้อมูลนั้นจำเป็นต้องส่งออกไปยังรายงานโดยใช้ jasper (ซึ่งไม่จัดการกับ access object ที่ซับซ้อนในประสบการณ์ของฉัน) .

หากเป็นไปได้ให้ใช้ Java 7 ซึ่งอนุญาตให้คุณใช้ Strings ในswitchข้อความสั่ง

จากhttp://docs.oracle.com/javase/tutorial/java/nutsandbolts/switch.html

public class StringSwitchDemo {

    public static int getMonthNumber(String month) {

        int monthNumber = 0;

        if (month == null) {
            return monthNumber;
        }

        switch (month.toLowerCase()) {
            case "january":
                monthNumber = 1;
                break;
            case "february":
                monthNumber = 2;
                break;
            case "march":
                monthNumber = 3;
                break;
            case "april":
                monthNumber = 4;
                break;
            case "may":
                monthNumber = 5;
                break;
            case "june":
                monthNumber = 6;
                break;
            case "july":
                monthNumber = 7;
                break;
            case "august":
                monthNumber = 8;
                break;
            case "september":
                monthNumber = 9;
                break;
            case "october":
                monthNumber = 10;
                break;
            case "november":
                monthNumber = 11;
                break;
            case "december":
                monthNumber = 12;
                break;
            default: 
                monthNumber = 0;
                break;
        }

        return monthNumber;
    }

    public static void main(String[] args) {

        String month = "August";

        int returnedMonthNumber =
            StringSwitchDemo.getMonthNumber(month);

        if (returnedMonthNumber == 0) {
            System.out.println("Invalid month");
        } else {
            System.out.println(returnedMonthNumber);
        }
    }
}

ฉันไม่ได้วัด แต่ฉันเชื่อว่าคำสั่ง switch รวบรวมไปยังตารางกระโดดแทนที่จะเป็นรายการเปรียบเทียบแบบยาว ควรเร็วกว่านี้

เกี่ยวกับคำถามที่แท้จริงของคุณ: หากคุณจะใช้เมื่อคุณไม่จำเป็นต้องทำให้มันคงที่ พิจารณาว่าค่าคงที่สามารถจัดทำเป็นเอกสารและแสดงใน Javadoc สิ่งนี้อาจมีความสำคัญสำหรับค่าสตริงที่ไม่สำคัญ

หากคุณจะรักษาสิ่งนี้ไว้ (ทำการเปลี่ยนแปลงใด ๆ ที่ไม่ใช่เรื่องไร้สาระ ) ฉันอาจลองใช้การสร้างรหัสกำกับด้วยคำอธิบายประกอบ (อาจจะผ่านCGLib ) หรือแม้แต่สคริปต์ที่เขียนโค้ดทั้งหมดให้คุณ ลองนึกภาพจำนวนข้อผิดพลาดและข้อผิดพลาดที่อาจคืบคลานเข้ามาด้วยวิธีการที่คุณกำลังพิจารณา ...

ฉันยังคงใช้ค่าคงที่ที่กำหนดไว้ที่ด้านบนของชั้นเรียนของคุณ มันทำให้โค้ดของคุณสามารถบำรุงรักษาได้มากขึ้นเนื่องจากง่ายต่อการดูสิ่งที่สามารถเปลี่ยนแปลงได้ในภายหลัง (ถ้าจำเป็น) ตัวอย่างเช่น"first_name"อาจกลายเป็น"firstName"ในภายหลัง

หากการตั้งชื่อของคุณสอดคล้องกัน (aka "some_whatever"ถูกแมปเสมอgetSomeWhatever()) คุณสามารถใช้การสะท้อนเพื่อกำหนดและเรียกใช้เมธอด get

ฉันเดาว่าการประมวลผลคำอธิบายประกอบอาจเป็นวิธีแก้ปัญหาแม้ว่าจะไม่มีคำอธิบายประกอบก็ตาม มันเป็นสิ่งที่สามารถสร้างรหัสที่น่าเบื่อสำหรับคุณ ข้อเสียคือคุณจะได้รับคลาส N ที่สร้างขึ้นสำหรับคลาสโมเดล N คุณไม่สามารถเพิ่มสิ่งใด ๆ ลงในคลาสที่มีอยู่เดิมได้ แต่เขียนสิ่งที่ต้องการ

public Object getNode(String name) {
    return SomeModelClassHelper.getNode(this, name);
}

หนึ่งครั้งต่อคลาสไม่น่าจะมีปัญหา หรือคุณสามารถเขียนบางอย่างเช่น

public Object getNode(String name) {
    return getHelper(getClass()).getNode(this, name);
}

ในซูเปอร์คลาสทั่วไป

คุณสามารถใช้การไตร่ตรองแทนการประมวลผลคำอธิบายประกอบสำหรับการสร้างรหัส ข้อเสียคือคุณต้องรวบรวมรหัสของคุณก่อนจึงจะสามารถใช้การสะท้อนกลับได้ ซึ่งหมายความว่าคุณไม่สามารถพึ่งพาโค้ดที่สร้างในคลาสโมเดลของคุณได้เว้นแต่คุณจะสร้างสตับบางตัว

ฉันจะพิจารณาใช้การสะท้อนโดยตรง แน่นอนว่าการสะท้อนกลับช้า แต่ทำไมช้า? เป็นเพราะมันต้องทำทุกสิ่งที่คุณต้องทำเช่นเปิดชื่อฟิลด์