Aynı işlev / yöntemde istisnalar atma ve yakalama

Kullanıcı bir pozitif tamsayı (doğal bir sayı) girene kadar bir kullanıcı giriş için soran bir işlev yazdım. Birisi fonksiyonumda istisnalar atmamalı ve yakalamamam gerektiğini ve fonksiyonumun arayanının onları idare etmesine izin vermesi gerektiğini söyledi.

Diğer geliştiricilerin bu konuda ne düşündüğünü merak ediyorum. Ayrıca muhtemelen işlevdeki istisnaları kötüye kullanıyorum. Java kodu:

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;
}

İki şeyle ilgileniyorum:

1. Arayanın istisnalar hakkında endişelenmesine izin vermeli miyim? İşlevin amacı, kullanıcı doğal bir sayı girene kadar kullanıcıyı sektirmesidir. İşlevin noktası kötü mü? Kullanıcı arabirimi (kullanıcı doğru girdi olmadan döngüden çıkamıyorum) hakkında değil, istisnalar işlenen döngüsel girdi hakkında konuşmuyorum.
2. Fırlatma ifadesinin (bu durumda) istisnaların kötüye kullanılması olduğunu söyleyebilir misiniz? Numaranın geçerliliğini kontrol etmek için kolayca bir bayrak oluşturabilir ve bu bayrağa göre uyarı mesajı verebilirim. Ama bu koda daha fazla satır eklemek istiyorum ve ben olduğu gibi mükemmel okunabilir düşünüyorum.

Mesele şu ki ayrı bir giriş fonksiyonu yazıyorum. Kullanıcı birden çok kez bir sayı girmek zorundaysa, girdi için tüm biçimlendirme istisnalarını ve sınırlamalarını işleyen ayrı bir işlev oluştururum.

İstisnaların amacı, bir yöntemin sizi arayan değere normal bir şekilde devam edemediği bir duruma giren, sizi dönüş değerine bir hata kodu gömmeye zorlamadan söylemesine izin vermesidir.

Sizin durumunuzda, yönteminiz giriş 0'dan büyük olmadığında ne yapacağınızı tam olarak bilir. Burada satır kaydetmenin tek nedeni, giriş bir sayı olmasaydı alacağınız istisnayı atmanızdır. Ancak, attığınız istisna, kodunuzun neden girişi sevmediğini doğru bir şekilde göstermiyor. Başka biri gelip bu kodu görseydi, işlerin tam olarak nasıl çalıştığını görmek için fazladan zaman harcamak zorunda kalacaklardı.

Bu istisnaların kötü kullanımıdır. Başlangıç ​​olarak, pozitif olmayan bir sayı biçim istisnası değildir.

Neden istisnalar kullanılmalı? Hangi girişe izin verilmediğini biliyorsanız, kullanıcıdan geçerli bir girdi elde edene kadar döngüden çıkmayın, aşağıdakine benzer:

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.");
   }
}

Yalnızca geçerli yöntem çağrısı ile ilgili bir şey yapmak istiyorsanız istisnaları yakalayın; temizleme, hata mantığı vb. Bu durumda, yakalama yalnızca konsola bir mesaj gönderir, sideInput yöntemiyle ilgili değildir, bu nedenle çağrı zinciri / yığını üzerinde daha fazla ele alınabilir.

Burada denemek / yakalamak kurtulmak ve sadece yöntem çağrısı belgelemek:

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

Hala bu istisnayı çağrı zinciri / yığını daha ileri ele gerekir!

Bir yöntemde aynı istisnayı hem atmalı hem de yakalamamalısınız, hatta catch bloğunun attığınız aynı istisnayı yakalayacağını düşünüyorum, bu yüzden gerçekten atmıyorsunuz.

parseIntBaşarılı olsaydı , o değil NumberFormatException.

taraf sıfırdan küçükse, a atmalısınız NegativeSideLengthException;

Adlı bir özel / iş istisnası oluşturun NegativeSideLengthException

public class NegativeSideLengthException extends Exception
{


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

}

Sonra sideInputNegativeSideLengthException atar

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;
}

Hatta (isterseniz) yakalamak için başka bir catch bloğu ekleyebilir NegativeSideLengthExceptionve yöntemin atmasını sağlayamazsınız.

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);

Bayraklar istisnaları ele almak için iyi bir yol değildir.

İstisnalar oldukça kabusuz şeyler, çözdüklerinden daha fazla karmaşıklık getiriyorlar.

İlk olarak, istisnalarınızı yakalamazsanız, arayan sadece yapabilir on error resume next, yani bir hafta sonra işlevinizin ne atabileceğini ve onunla ne yapacağını bilemezsiniz:

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

Temel olarak, onları yakalarsanız, sözleşmeler ve istisna garantileri hakkında çok iyi bir anlayışa sahip olmanız gerekir. Bu nadiren gerçek bir dünyada olur. Ayrıca kodunuzu okumak zor olacaktır.

Ayrıca, komik olan şey, gerçekten istisnaları ele alma şansınız varsa, Java ve .NET'in tamamen istisnalarla ilgili olması nedeniyle, bir çeşit mizahi olan gerçek RAII diline ihtiyacınız var ...

Bunu bir kez daha tekrarlıyorum, ama ...:

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