error stackoverflow en Java

Question

Soy un novato en Java. Estoy escribiendo una clase donde el constructor debe verificar el parámetro de precio y asegurarse de que no sea un número negativo. Y si es negativo, debe establecer el precio en cero. Obtengo un error de stackoverflow cuando miro el precio. ¿Puedo obtener ayuda con lo que hice mal?

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

Su método getprice() llama a sí mismo en lugar de comprobar price. Esto está llevando a una recursión infinita en este caso.

Answer 2

Ignacio ha explicado la causa y la solución:

Cambie la línea

if(getprice() < 0) 

a esto:

if(price < 0) 

Answer 3

Su conseguir una recursión infinita, porque su condición if comprueba su getprice() método, no su variable price.

Muchos compiladores modernos le advertirán cuando haya codificado algo que dé como resultado recursión infinita.

A veces me cruzo con este error también, especialmente con los IDE que tienen intellisense.

¡Buena suerte aprendiendo Java! :)

Answer 4

Cuando escribe un bean, generalmente quiere comprobar si el precio establecido es < 0, en lugar de hacer este cálculo cada vez que intenta obtener la variable.

Answer 5

No es cure para el problema de recursión, pero también debería considerar verificar el precio en el momento de la construcción.
A veces (¿la mayoría de las veces?) Es mejor que su constructor falle con una excepción en lugar de permitir la construcción de un objeto incoherente. De esta forma, es más fácil localizar dicho error.
Ejemplo:

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

El riesgo es que su aplicación puede fallar cuando en el entorno de producción, que puede ser un desastre. Para evitar esto, puede usar un assert o, al menos, dar o registrar el error y usar alguna alternativa. La comprobación assert se debe activar para el desarrollo y se puede desactivar en producción. Para más detalles véase 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

Su getprice simplemente debe escribirse como:

return price < 0 ? 0 : price; 

Por cierto, bonito ver que un error de stackoverflow se resuelve por stackoverflow.com