stackoverflow Fehler in Java

Question

Ich bin ein Neuling in Java. Ich schreibe eine Klasse, in der der Konstruktor den Preisparameter überprüfen und sicherstellen muss, dass es keine negative Zahl ist. Und wenn es negativ ist, muss es den Preis auf Null setzen. Ich bekomme einen Stackoverflow-Fehler, wenn ich den Preis überprüfe. Kann ich Hilfe bekommen mit dem, was ich falsch gemacht habe?

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

Ihre getprice() Methode nennt sich stattdessen price zu prüfen. Dies führt in diesem Fall zu einer unendlichen Rekursion.

Answer 2

Ignacio hat erklärt, die Ursache und die Lösung:

Ändern Sie die Zeile

if(getprice() < 0) 

dazu:

if(price < 0) 

Answer 3

Ihr immer eine unendliche Rekursion, weil Ihr if Zustand Ihrer getprice() prüft Methode, nicht Ihre price Variable.

Viele moderne Compiler warnen Sie, wenn Sie etwas codiert haben, das zu einer unendlichen Rekursion führt.

Ich bin immer noch manchmal auf diesen Fehler, vor allem mit IDEs, die Intellisense haben.

Viel Glück beim Lernen von Java! :)

Answer 4

Wenn Sie eine Bean schreiben, möchten Sie im Allgemeinen überprüfen, ob der Preis < 0 ist, anstatt diese Berechnung jedes Mal vorzunehmen, wenn Sie versuchen, die Variable zu erhalten.

Answer 5

Nicht ein Heilung für das Rekursionsproblem, aber Sie sollten auch in Betracht ziehen, den Preis zur Bauzeit zu überprüfen.
Manchmal (am häufigsten?) Ist es besser, dass Ihr Konstruktor mit einer Ausnahme fehlschlägt, anstatt die Konstruktion eines inkonsistenten Objekts zuzulassen. Auf diese Weise ist es einfacher, einen solchen Fehler zu lokalisieren.
Beispiel:

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

Das Risiko besteht darin, dass Ihre Anwendung, wenn in der Produktionsumgebung scheitern kann, was für ein Chaos sein kann. Um dies zu vermeiden, können Sie einen assert verwenden oder zumindest den Fehler ausgeben oder protokollieren und eine Alternative verwenden. Die Prüfung assert muss für die Entwicklung aktiviert und möglicherweise bei der Produktion deaktiviert werden. siehe Details Programming With Assertions

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

oder

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

Ihre getprice sollte einfach geschrieben werden als:

return price < 0 ? 0 : price; 

Btw, schön zu sehen, dass ein Fehler von Stackoverflow stackoverflow.com gelöst