（Java初学者问题）为什么这段代码不起作用？

Question

（下面是完整的代码）我刚刚交了一个课堂实验，在那里我们必须做一个描述某本书的课堂。我不知道怎么做两件事。1.如果有人为“页面”或“SuggestedMetalPrice”输入的值小于零，则该值必须设置为零。在该代码中，即使值为正，该值也设置为0。在：

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

如果我将第二个“0”设置为不同的数字，比如说:

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

那么你输入的“页面”的值将是1。但是如果你输入的值是负数，它不应该只有1吗？我不明白我做错了什么。

我不明白的第二件事是在代码的底部，我们必须显示所有信息。我的老师想让我们用“是”或“否”而不是“对”或“错”来显示这本书是否是平装本。我该怎么做？我试着这样放一个if/else语句进去:System.out.println("平装:" if(平装= true) {Yes} if(平装= 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);
    }
}

Answer 1

我有点不确定你想做什么，但我注意到在Book类中，当参数为“numberOfPages”时，你正在使用“页面”。尝试做

if(numberOfPages <= 0) pages = 0

另外，如果你能弄清楚如何使用三元运算符，你就能使这一切变得整洁漂亮，但是我将把这个留给你自己。

至于print语句，请在print之外加上“if”。

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

另外，这是三元运算符的另一个亮点。

Answer 2

在用参数numberOfPages设置pages值之前，您正在检查它:

 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
 } 

颠倒这个顺序。或者更好的方法是，检查参数值并使用它来设置您的pages值:

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

对于您的第二个问题，创建一个字符串，比方说称为isPaperback，如果平装本为true，则将其设置为" yes" String，否则设置为" no ",然后在需要时显示该字符串。或者将System.out.println("yes ")放在if块中测试平装书的价值。

即。，

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