(初学者Java问题)为什么这段代码不起作用?

时间:2013-02-05 01:15:42

标签: java

(完整代码如下) 我刚刚上课实验室,我们不得不上一个描述某本书的课程。我无法弄清楚如何做两件事。 1.如果某人为'pages'或'suggestedRetailPrice'输入的值小于零,则为该值 必须设置为零。在此代码中,即使值为正,该值也会设置为0。在:


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

代码,如果我将第二个'0'设置为不同的数字,例如:


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

然后,您为“页面”输入的内容的值将为1.但是,如果您输入的值是负数,那么它不应该是1吗?我不知道我做错了什么。

我无法弄清楚的第二件事是在代码的底部,我们必须显示所有信息。我的老师希望我们将这本书的平装本显示为“是”或“否”而不是“真实”或“假”。我该怎么做呢?我尝试像这样放一个if / else语句: System.out.println(“平装:”+ if(平装=真){是} if(平装=假){无};)

没有用,无法搞清楚。请参阅下面的完整代码。


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);
    }
}

2 个答案:

答案 0 :(得分:4)

您在使用参数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
}

对于第二个问题,创建一个String,比如名为isPaperback,将其设置为“yes”如果平装是真的话,将字符串设置为输出字符串,如果不是,则设置为“否”,然后在需要时显示该字符串。或者将你的System.out.println(“是”)放在if块中测试平装的值。

即,

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

答案 1 :(得分:1)

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

if(numberOfPages <= 0) pages = 0

此外,如果你能弄清楚如何使用三元运算符,你可以使这一切变得整洁和漂亮,但我会把它留给你。

至于print语句,请在打印之外取“if”,这样就可以了。

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

另外看这是三元运营商的另一个亮点。