ควรตรวจสอบ `c> = '0'` หรือ` c> = 48` หรือไม่?

หลังจากการสนทนากับเพื่อนร่วมงานของฉันฉันได้รับคำถาม 'ปรัชญา' เกี่ยวกับวิธีจัดการกับชนิดข้อมูลถ่านใน Java ตามแนวทางปฏิบัติที่ดีที่สุด

สมมติว่าสถานการณ์ง่าย ๆ(เห็นได้ชัดว่านี่เป็นเพียงตัวอย่างง่ายๆเพื่อให้ความหมายในการฝึกฝนสำหรับคำถามของฉัน)โดยที่ String 's' เป็นอินพุตคุณต้องนับจำนวนอักขระที่เป็นตัวเลขในนั้น

นี่คือโซลูชันที่เป็นไปได้ 2 แบบ:

1)

    for(int i=0; i<s.length(); i++) {
        if(s.charAt(i) >= 48 && s.charAt(i) <= 57) {
            n++;
        }
    }

2)

    for(int i=0; i<s.length(); i++) {
        if(s.charAt(i) >= '0' && s.charAt(i) <= '9' ) {
            n++;
        }
    }

ข้อใดที่สอง 'สะอาด' และสอดคล้องกับแนวทางปฏิบัติที่ดีที่สุดของ Java?

ทั้งคู่เป็นคนที่น่ากลัว แต่คนแรกนั้นน่ากลัวกว่ากัน

ทั้งสองไม่สนใจความสามารถในตัวของ Java ในการตัดสินใจว่าอักขระใดเป็น "ตัวเลข" (ผ่านวิธีการในCharacter) แต่สิ่งแรกไม่เพียง แต่ละเว้นลักษณะ Unicode ของสายอักขระสมมติว่ามีเพียง 0123456789 มันยังปิดบังแม้เหตุผลที่ไม่ถูกต้องนี้โดยใช้รหัสอักขระที่เหมาะสมถ้าคุณรู้อะไรเกี่ยวกับประวัติของการเข้ารหัสอักขระ

ทั้ง ปล่อยให้คลาสตัวละครในตัวของ Java คิดออกมาเพื่อคุณ

for (int i = 0; i < s.length(); ++i) {
  if (Character.isDigit(s.charAt(i))) {
    ++n;
  }
}

มีช่วงอักขระไม่กี่ตัวมากกว่าตัวเลข ASCII ที่นับเป็นตัวเลขและไม่มีตัวอย่างที่คุณโพสต์ไว้ JavaDocสำหรับCharacter.isDigit()รายการเหล่านี้ช่วงตัวอักษรเป็นตัวเลขที่ถูกต้อง:

ช่วงอักขระ Unicode บางตัวที่มีตัวเลข:

• '\ u0030' ถึง '\ u0039', ตัวเลข ISO-LATIN-1 ('0' ถึง '9')
• '\ u0660' ถึง '\ u0669', ตัวเลขอารบิคภาษาอาหรับ
• '\ u06F0' ถึง '\ u06F9', ตัวเลขอารบิคภาษาอาหรับแบบขยาย
• '\ u0966' ถึง '\ u096F', ตัวเลข Devanagari
• '\ uFF10' ถึง '\ uFF19', ตัวเลขเต็มความกว้าง

ช่วงอักขระอื่น ๆ จำนวนมากมีตัวเลขเช่นกัน

ที่ถูกกล่าวว่าหนึ่งควรมอบให้Character.isDigit()กับรายการนี้ เนื่องจากมีการเติมข้อมูล Unicode ใหม่รหัส Java จะถูกปรับปรุง การอัพเกรด JVM สามารถทำให้โค้ดเก่าทำงานด้วยอักขระตัวเลขใหม่ได้อย่างราบรื่น นอกจากนี้ยังเป็นDRY : โดยการแปลรหัส "นี่คือตัวเลข" ให้เป็นสถานที่เดียวที่อ้างอิงถึงที่อื่นจะสามารถหลีกเลี่ยงแง่ลบของการทำสำเนารหัส (เช่นข้อบกพร่อง) ได้ สุดท้ายให้สังเกตบรรทัดสุดท้าย: รายการนี้ไม่ครบถ้วนสมบูรณ์และมีตัวเลขอื่น ๆ

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

ข้อยกเว้นสำหรับกฎนี้คือถ้าคุณจำเป็นต้องทดสอบหลัก ASCII ตามตัวอักษรจริงๆไม่ใช่ตัวเลขอื่น ๆ ตัวอย่างเช่นถ้าคุณกำลังแยกกระแสและเพียงตัวเลข ASCII (เมื่อเทียบกับตัวเลขอื่น ๆ ) มีความหมายพิเศษแล้วมันจะไม่ได้Character.isDigit()มีความเหมาะสมกับการใช้งาน

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

ถ้าคุณเคยเขียนใบสมัครใน C ที่ใช้ EBCDIC เป็นชุดตัวอักษรพื้นฐานและความต้องการที่จะอักขระ ASCII กระบวนการแล้วใช้และ48 57คุณกำลังทำอย่างนั้น? ฉันไม่คิดอย่างนั้น

เกี่ยวกับการใช้isDigit(): มันขึ้นอยู่กับ คุณกำลังเขียนโปรแกรมวิเคราะห์คำ JSON หรือไม่ เท่านั้น0ที่จะ9ได้รับการยอมรับเป็นหลักจึงไม่ใช้isDigit()ตรวจสอบและ>= '0' <= '9'คุณกำลังประมวลผลอินพุตของผู้ใช้หรือไม่ ใช้isDigit()ตราบเท่าที่ส่วนที่เหลือของรหัสของคุณสามารถจัดการกับสตริงและเปลี่ยนเป็นตัวเลขได้อย่างถูกต้อง

ตัวอย่างที่สองดีกว่าอย่างเห็นได้ชัด ความหมายของตัวอย่างที่สองนั้นชัดเจนทันทีเมื่อคุณดูโค้ด ความหมายของตัวอย่างแรกนั้นชัดเจนหากคุณจดจำตาราง ASCII ทั้งหมดในหัวของคุณ

คุณควรแยกแยะระหว่างการตรวจสอบอักขระเฉพาะหรือตรวจสอบช่วงหรือคลาสของอักขระ

1) ตรวจสอบอักขระเฉพาะ

สำหรับตัวอักษรธรรมดาให้ใช้ตัวอักษรตามตัวอักษรเช่นif(ch=='z').... if (ch=='\n')...หากคุณตรวจสอบกับตัวอักษรพิเศษเช่นแท็บหรือแบ่งบรรทัดคุณควรใช้หนีเช่น หากตัวละครที่คุณกำลังตรวจสอบนั้นผิดปกติ (เช่นไม่สามารถจดจำได้ทันทีหรือไม่สามารถใช้กับแป้นพิมพ์มาตรฐานได้) คุณอาจใช้รหัสตัวอักษรฐานสิบหกแทนตัวอักษร แต่เนื่องจากรหัสเลขฐานสิบหกเป็น "ค่าเวทย์มนตร์" คุณจะแยกมันเป็นค่าคงที่และบันทึก:

const char snowman = 0x2603; // snowman char used to detect encoding issues
...
if (ch==showman)...

Hex codes เป็นวิธีมาตรฐานในการระบุรหัสอักขระ

2) ตรวจสอบคลาสอักขระหรือช่วง

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

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

มันจะดีกว่าที่จะใช้c >= '0'เพราะc >= 48คุณจำเป็นต้องแปลง c ในรหัส ascii

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

public static int countDigits(String str) {
    str = Objects.requireNonNull(str).trim();

    return str.replaceAll("[^\\d]", "").length();
}

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