Błąd stackoverflow w java

Question

Jestem nowicjuszem w Javie. Piszę klasę, w której konstruktor musi sprawdzić parametr ceny i upewnić się, że nie jest liczbą ujemną. A jeśli jest ujemny, musi ustawić cenę na zero. Dostaję błąd stackoverflow, kiedy sprawdzam cenę. Czy mogę uzyskać pomoc dotyczącą tego, co zrobiłem źle?

public class Book 
{ 
    private String title; 
    private String author; 
    private String isbn; 
    private int pages; 
    private boolean pback; 
    private double price; 

    /** 
    * Constructor for objects of class Book 
    */ 
    public Book(String bookTitle, String bookAuthor, String bookCode, int bookPages, boolean paperback, double bookRetail) 
    { 
     title = bookTitle; 
     author = bookAuthor; 
     isbn = bookCode; 
     pages = bookPages; 
     pback = paperback; 
     price = bookRetail; 
    } 

    /** 
    * @returns title 
    */ 

    public String gettitle() 
    { 
     return title; 
    } 

    /** 
    * @returns author 
    */ 

    public String getauthor() 
    { 
     return author; 
    } 

    /** 
    * @returns ISBN# 
    */ 

    public String getisbn() 
    { 
     return isbn; 
    } 

    /** 
    * @return number of pages 
    */ 

    public int getpages() 
    { 
     return pages; 
    } 

    /** 
    * @return is book paperback 
    */ 

    public boolean getpback() 
    { 
     return pback; 
    } 

    /** 
    * @return retail price 
    */ 

    public double getprice() 
    { 
     if(getprice() < 0) 
     { 
      return 0; 
     } 
     else 
     { 
      return price; 
     } 

    } 
} 

Answer 1

Twoja metoda getprice() nazywa się zamiast sprawdzania price. Prowadzi to do nieskończonej rekurencji w tym przypadku.

Answer 2

Ignacio wyjaśnił przyczynę i rozwiązanie:

zmienić linię

if(getprice() < 0) 

do tego:

if(price < 0) 

Answer 3

Twój uzyskanie nieskończonej rekurencji, ponieważ stan if sprawdza getprice() metoda, a nie zmienna price.

Wiele współczesnych kompilatorów ostrzega, gdy zakoduje się coś, co powoduje nieskończoną rekurencję.

Nadal czasami napotykam ten błąd, szczególnie w przypadku IDE, które mają intellisense.

Powodzenia w nauce Java! :)

Answer 4

Podczas pisania fasoli generalnie chcesz sprawdzić, czy ustalana cena to < 0, zamiast wykonywać te obliczenia za każdym razem, gdy próbujesz uzyskać zmienną.

Answer 5

Nie jest to utwardzanie dla problemu rekursji, ale należy również rozważyć sprawdzenie ceny w czasie budowy.
Czasami (większość razy?) Lepiej jest, jeśli konstruktor zawiedzie z wyjątkiem, zamiast pozwolić na konstrukcję niespójnego obiektu. W ten sposób łatwiej jest zlokalizować taki błąd.
Przykład:

public Book(String bookTitle, String bookAuthor, String bookCode, int bookPages, boolean paperback, double bookRetail) 
{ 
    if (bookRetail < 0.0) 
     throw new IllegalArgumentException("negative bookRetail: " + bookRetail); 
    ... 
} 

Istnieje ryzyko, że aplikacja może zawieść, gdy w środowisku produkcyjnym, które mogą być bałagan. Aby tego uniknąć, możesz użyć numeru assert lub przynajmniej podać lub zarejestrować błąd i użyć innej opcji. Kontrola assert musi być włączona do programowania i może być wyłączona podczas produkcji. O szczegóły patrz Programming With Assertions

public Book(String bookTitle, String bookAuthor, String bookCode, int bookPages, boolean paperback, double bookRetail) 
{ 
    assert bookRetail >= 0.0 : bookRetail; 
    ... 
} 

lub

public Book(String bookTitle, String bookAuthor, String bookCode, int bookPages, boolean paperback, double bookRetail) 
{ 
    if (bookRetail >= 0.0) { 
     price = bookRetail; 
    } else { 
     price = 0.0; 
     // display or log the "illegal argument" 
     Exception ex = new IllegalArgumentException("negative bookRetail: " + bookRetail); 
     ex.printStackTrace(); 
    } 
    ... 
} 

Answer 6

Twój getprice powinien po prostu być zapisany jako:

return price < 0 ? 0 : price; 

Btw, miło zobaczyć, że to błąd stackoverflow jest rozwiązany przez stackoverflow.com