errore stackoverflow in java

Question

Sono un novizio in java. Sto scrivendo una classe in cui il costruttore deve controllare il parametro del prezzo e assicurarsi che non sia un numero negativo. E se è negativo, deve impostare il prezzo a zero. Ottengo un errore StackOverflow quando controllo il prezzo. Posso avere aiuto con quello che ho fatto di sbagliato?

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

Il tuo metodo getprice() si definisce invece di controllare price. Questo sta portando a una ricorsione infinita in questo caso.

Answer 2

Ignacio ha spiegato la causa e la soluzione:

Modificare la linea

if(getprice() < 0) 

a questo:

if(price < 0) 

Answer 3

vostro ottenere ricorsione infinita, perché la sua condizione if controlla il vostro getprice() metodo, non la tua variabile price.

Molti compilatori moderni ti avviseranno quando hai codificato qualcosa che si traduce in ricorsione infinita.

Ancora a volte mi imbatto anche in questo errore, specialmente con IDE che hanno intellisense.

Buona fortuna per imparare Java! :)

Answer 4

Quando si scrive un bean in genere si desidera verificare se il prezzo impostato è < 0, invece di effettuare questo calcolo ogni volta che si tenta di ottenere la variabile.

Answer 5

Non è un cure per il problema di ricorsione, ma si dovrebbe anche considerare di controllare il prezzo al momento della costruzione.
A volte (la maggior parte delle volte?) È meglio che il costruttore non riesca a eseguire con un'eccezione anziché consentire la costruzione di un oggetto incoerente. In questo modo è più facile localizzare un tale errore.
Esempio:

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

Il rischio è che l'applicazione può riuscire quando nell'ambiente di produzione, che può essere un pasticcio. Per evitare questo, è possibile utilizzare uno assert o, almeno, dare o registrare l'errore e utilizzare qualche alternativa. Il controllo assert deve essere attivato per lo sviluppo e può essere disattivato in fase di produzione. Per i dettagli vedi Programming With Assertions

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

o

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

tuo getprice dovrebbe semplicemente essere scritto come:

return price < 0 ? 0 : price; 

Btw, bello vedere che un errore di StackOverflow è risolto stackoverflow.com