การขว้างและจับข้อยกเว้นในฟังก์ชั่น / วิธีการเดียวกัน

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

ฉันสงสัยว่านักพัฒนาคนอื่นคิดอย่างไรกับเรื่องนี้ ฉันอาจใช้ข้อยกเว้นในทางที่ผิดในฟังก์ชั่น นี่คือรหัสใน Java:

private static int sideInput()
{
    int side = 0;
    String input;
    Scanner scanner = new Scanner(System.in);

    do {
        System.out.print("Side length: ");
        input = scanner.nextLine();
        try {
            side = Integer.parseInt(input);
            if (side <= 0) {
                // probably a misuse of exceptions
                throw new NumberFormatException();
            }
        }
        catch (NumberFormatException numFormExc) {
            System.out.println("Invalid input. Enter a natural number.");
        }
    } while (side <= 0);

    return side;
}

ฉันสนใจสองสิ่ง:

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

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

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

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

นี่เป็นการใช้ข้อยกเว้นที่ไม่ดี ในการเริ่มต้นด้วยหมายเลขที่ไม่ใช่บวกไม่ใช่ข้อยกเว้นรูปแบบ

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

while (true)
{
   // Get user input.
   String input = scanner.nextLine();

   try
   {
      side = Integer.parseInt(input);

      break;
   }
   catch (NumberFormatException ex)
   {
      // Inform user of invalid input.
      System.out.println("Invalid input. Enter a natural number.");
   }
}

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

หนึ่งสามารถกำจัดลอง / จับที่นี่และเพียงเอกสารการเรียกวิธีการ:

//Throws NumberFormatException if read input is less than 0
private static int sideInput()

หนึ่งยังคงต้องจัดการข้อยกเว้นนั้นขึ้นสายโซ่ / กอง!

คุณไม่ควรโยนและตรวจจับข้อยกเว้นเดียวกันในวิธีใดวิธีหนึ่งฉันยังคิดว่าบล็อกจับจะจับข้อยกเว้นเดียวกับที่คุณขว้างดังนั้นคุณจึงไม่ได้ขว้าง

หากประสบความสำเร็จแล้วก็ไม่ได้เป็นparseIntNumberFormatException

ถ้าด้านข้างมีค่าน้อยกว่าศูนย์คุณควรโยน a NegativeSideLengthException;

สร้างข้อยกเว้นแบบกำหนดเอง / ธุรกิจ NegativeSideLengthException

public class NegativeSideLengthException extends Exception
{


    public NegativeSideLengthException(Integer i)
    {
        super("Invalid negative side length "+i);        
    }

}

จากนั้นจึงsideInputขว้าง NegativeSideLengthException

private static int sideInput() throws NegativeSideLengthException
{
    int side = 0;
    String input;
    Scanner scanner = new Scanner(System.in);

    do {
        System.out.print("Side length: ");
        input = scanner.nextLine();
        try {
            side = Integer.parseInt(input);
            if (side <= 0) {
                throw new NegativeSideLengthException(side);
            }
        }
        catch (NumberFormatException numFormExc) {
            System.out.println("Invalid input. Enter a natural number.");
        }
    } while (side <= 0);

    return side;
}

คุณสามารถแม้กระทั่ง (ถ้าคุณต้องการ) เพิ่มบล็อก catch อื่นเพื่อจับNegativeSideLengthExceptionและไม่มีวิธีการโยน

do {
    System.out.print("Side length: ");
    input = scanner.nextLine();
    try {
        side = Integer.parseInt(input);
        if (side <= 0) {
            throw new NegativeSideLengthException(side);
        }
    }
    catch (NumberFormatException numFormExc) {
        System.out.println("Invalid input. Enter a natural number.");
    } catch (NegativeSideLengthException e){
        System.out.println("Invalid input. Enter a non-negative number.");
    }
} while (side <= 0);

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

ข้อยกเว้นเป็นสิ่งที่น่าหวาดเสียวพวกเขานำความซับซ้อนมากกว่าที่พวกเขาแก้ปัญหา

ประการแรกหากคุณไม่ได้รับการยกเว้นผู้โทรสามารถทำได้on error resume nextนั่นคือหลังจากผ่านไปหนึ่งสัปดาห์คุณจะไม่ทราบว่าฟังก์ชันของคุณสามารถส่งสัญญาณอะไรได้บ้างและจะทำอย่างไรกับมัน:

{
    ...
}
catch(OutOfMemory, CorruptedMemory, BadData, DanglingPointers, UnfinishedCommit)
{
    Console.WriteLine("Nothing to see here, move on.");
    Console.WriteLine("The app is very stable, see, no crashing!");
}

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

นอกจากนี้สิ่งที่ตลกคือถ้าคุณมีโอกาสจัดการข้อยกเว้นใด ๆ คุณต้องใช้ภาษา RAII จริงซึ่งเป็นเรื่องตลกเนื่องจาก Java และ. NET เป็นข้อยกเว้น ...

ทำซ้ำอีกครั้ง แต่ ... :

http://blogs.msdn.com/b/oldnewthing/archive/2004/04/22/118161.aspx

http://blogs.msdn.com/b/oldnewthing/archive/2005/01/14/352949.aspx

http://www.joelonsoftware.com/items/2003/10/13.html