Java初学者问题：为什么这段代码没有运行？

Question

（以下是完整代码） 我刚刚为课堂提交了一个实验，我们必须制作一个描述某本书的类。我无法弄清楚如何做到两件事。 1. 如果有人为“页数”或“建议零售价”输入小于零的值，则该值必须设置为零。在此代码中，即使该值为正，该值也被设置为0。在：

---

if ( pages <= 0 )
    {
        pages = 0;
    }

如果我将第二个“0”设置为其他数字，例如：

if ( pages <= 0 )
    {
        pages = 1;
    }

那么您输入的“页面”所对应的值将为1。但是，只有在您输入的值为负数时，它才应该为1。我不知道我错在哪里。

第二件我解决不了的事情是，在代码底部，我们需要显示所有信息。我的老师要求我们将书籍是否为平装显示为“是”或“否”，而不是“true”或“false”。我该怎么做？我尝试添加一个if / else语句像这样：

System.out.println（“平装：”+ if（paperback = true）{Yes} if（paperback = false）{no};）

没有奏效，我无法解决。下面是整个代码

---

public class Book {
    // Instance variables
    private String title;
    private String author;
    private int isbn;
    private int pages;
    private boolean paperback;
    private int suggestedRetailPrice;

    /**
     * Default contructor
     */
    public Book() {
        title = "";
        author = "";
        isbn = 0;
        pages = 0;
        paperback = false;
        suggestedRetailPrice = 0;
    }

    /**
     * book information
     */
    public Book(String whatIsTitle, String whoIsAuthor, int isbnCode,
            int numberOfPages, boolean isItPaperback,
            int theSuggestedRetailPrice) {
        title = whatIsTitle;
        author = whoIsAuthor;
        isbn = isbnCode;
        if (pages <= 0) {
            pages = 0;
        } else {
            pages = numberOfPages;
        }
        paperback = isItPaperback;
        if (suggestedRetailPrice <= 0) {
            suggestedRetailPrice = 0;
        } else {
            suggestedRetailPrice = theSuggestedRetailPrice;
        }
    }

    /**
     * @return title
     */
    public String getTitle() {
        return title;
    }

    /**
     * @return author
     */
    public String getAuthor() {
        return author;
    }

    /**
     * @return isbn
     */
    public int getIsbn() {
        return isbn;
    }

    /**
     * @return pages
     */
    public int getPages() {
        return pages;
    }

    /**
     * @return paperback
     */
    public boolean getPaperback() {
        return paperback;
    }

    /**
     * @return suggestedRetailPrice
     */
    public int getSuggestedRetailPrice() {
        return suggestedRetailPrice;
    }

    /**
     * title
     */
    public void setTitle(String whatIsTitle) {
        title = whatIsTitle;
    }

    /**
     * author
     */
    public void setAuthor(String whoIsAuthor) {
        author = whoIsAuthor;
    }

    /**
     * isbn code
     */
    public void setIsbn(int isbnCode) {
        isbn = isbnCode;
    }

    /**
     * number of pages
     */
    public void setPages(int numberOfPages) {
        if (pages <= 0) {
            pages = 0;
        } else {
            pages = numberOfPages;
        }
    }

    /**
     * is it paperback
     */
    public void setPaperback(boolean isItPaperback) {
        paperback = isItPaperback;
    }

    /**
     * suggested retail price
     */
    public void setSuggestedRetailPrice(int theSuggestedRetailPrice) {
        if (suggestedRetailPrice <= 0) {
            suggestedRetailPrice = 0;
        } else {
            suggestedRetailPrice = theSuggestedRetailPrice;
        }
    }

    /**
     * displays information
     */
    public void displayBook() {
        System.out.println("Title : " + title);
        System.out.println("Author : " + author);
        System.out.println("ISBN : " + isbn);
        System.out.println("Pages : " + pages);
        System.out.println("Paperback : " + paperback);
        System.out.println("Suggested price : " + suggestedRetailPrice);
    }
}

- Joshua Baker

Answer 1

在使用参数 numberOfPages 设置页面值之前，您需要检查当前页面的值：

 title = whatIsTitle;
 author = whoIsAuthor;
 isbn = isbnCode;

 // pages is still at its initialzed value of 0 here.
 if ( pages <= 0 )
 {
     pages = 0;
 }
 else
 {
     pages = numberOfPages; // this will *never* be called
 } 

将顺序颠倒。或者更好的方法是，检查参数值并使用它来设置您的页面值：

if (numberOfPages < 0) {
   pages = 0;
} else {
   pages = numberOfPages
}

针对你的第二个问题，创建一个名为isPaperback的字符串变量，并将其设置为"yes"（如果paperback为true）或"no"（如果不是），然后在需要时显示该字符串。或者，可以将System.out.println("yes")放在测试paperback值的if块中。

即：

if (paperback) {
   System.out.println(...);
} else {
   System.out.pringln(...);
}

- Hovercraft Full Of Eels

Answer 2

我有点不确定你想做什么，但是我注意到在Book类中，当参数为"numberOfPages"时，你正在使用"pages"。尝试这样做：

if(numberOfPages <= 0) pages = 0

此外，如果你能想出如何使用三目运算符，将所有内容弄得整洁漂亮，那就再加分不过了，但这个我会留给你自己完成。

至于打印语句，把 “if” 放在 print 外面即可，代码如下：

if(paperback) System.out.println("Paperback: Yes")
else System.out.println("Paperback: no)

另外，这也是三元运算符的另一个好用之处。

- jbeverid