ошибка stackoverflow в java

Question

Я новичок в java. Я пишу класс, где конструктор должен проверить параметр цены и убедиться, что это не отрицательное число. И если он отрицательный, он должен установить цену на ноль. Когда я проверяю цену, я получаю ошибку stackoverflow. Могу ли я получить помощь от того, что я сделал неправильно?

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

Ваш метод getprice() называет себя вместо проверки price. В этом случае это приводит к бесконечной рекурсии.

Answer 2

Игнасио объяснил причину и решение:

Изменение линии

if(getprice() < 0) 

к этому:

if(price < 0) 

Answer 3

Ваше получение бесконечной рекурсии, потому что ваш if условие проверяет getprice() метод, а не переменную price.

Многие современные компиляторы предупреждают вас, когда вы закодировали что-то, что приводит к бесконечной рекурсии.

Я до сих пор иногда сталкиваюсь с этой ошибкой, особенно с IDE, которые имеют intellisense.

Удачи в изучении Java! :)

Answer 4

Когда вы пишете фасоль, вы обычно хотите проверить, установлена ​​ли цена < 0, вместо того, чтобы делать это вычисление каждый раз, когда вы пытаетесь получить переменную.

Answer 5

Не проблема cure для проблемы рекурсии, но вы также должны рассмотреть возможность проверки цены во время строительства.
Иногда (чаще?) Лучше, чтобы ваш конструктор сменил с Исключением, вместо того чтобы разрешить конструкцию несогласованного объекта. Таким образом, легче локализовать такую ​​ошибку.
Пример:

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

Риск заключается в том, что ваше приложение может произойти сбой при в производственной среде, которая может быть беспорядок. Чтобы этого избежать, вы можете использовать assert или, по крайней мере, выдавать или регистрировать ошибку и использовать какую-либо альтернативу. Проверка assert должна быть включена для разработки и может быть отключена при производстве. Подробно см Programming With Assertions

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

или

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

Ваш getprice нужно просто записать в виде:

return price < 0 ? 0 : price; 

Btw, приятно видеть, что ошибка StackOverflow решается

stackoverflow.com