Question

请注意：我之前创建了一个有这个问题和其他几个问题的帖子，但被告知，由于我在同一篇文章中提出了这么多问题，所以最好将其分解为个别问题。所以请不要将其标记为重复，是的说明是相同的，是的，使用相同的代码，但问题本身是不同的。感谢。

我正在制作一个程序，其中包含以下说明：

编写一个名为Octagon的类，它扩展了GeometricObject并实现了Comparable和Cloneable接口。假设八边形的所有8个边都具有相同的大小。可以使用以下公式计算面积

面积=（2 + 4/2的平方根）*边*侧

编写程序（驱动程序）以从文件中读取一系列值，显示区域和周边，创建克隆并比较对象及其克隆（基于区域）。此外，您的程序应该将当前对象（刚刚读入）与读入的第一个对象进行比较。当从文件中读取负数时，程序结束。

这是我到目前为止的代码，这是我的GeometricObject类：

public abstract class GeometricObject {

    public abstract double getArea();
    public abstract double getPerimeter(); 
}

我的八角形课程：

public class Octagon extends GeometricObject implements Comparable<Octagon>, Cloneable {

    private double side;

    public Octagon() {
    }

    public Octagon(double side) {
        this.side = side;
    }

    public double getSide() {
        return side;
    }

    public void setSide(double side) {
        this.side = side;
    }

    public double getArea() {
        return (2 + (4 / (Math.sqrt(2))) * side * side); 
    }

    public double getPerimeter() {
        return side * 8;
    }

    public String toString() {
        return "Area: " + getArea() + "\nPerimeter: " 
            + getPerimeter() + "\nClone Compare: " +  "\nFirst Compare: ";
    }

    public int compareTo(Octagon octagon) {
        if(getArea() > octagon.getArea()) 
            return 1;
        else if(getArea() < octagon.getArea()) 
                return -1;
        else
            return 0;
    }

    @Override 
    public Octagon clone() {
        return new Octagon(this.side);
    } 
}

我的司机或测试员班:(这是我需要最多帮助的地方）：

import java.util.*;

public class Driver {
    public static void main(String[] args) throws Exception {
        java.io.File file = new java.io.File("prog7.dat");
        Scanner fin = new Scanner(file);
        Octagon first = null;
        int i = 1;
        Octagon older;

        while(fin.hasNext())
        {
            double side = fin.nextDouble();
            if(side < 0.0)
                break;
            Octagon oct = new Octagon(side);
            System.out.print("Octagon " + i + ": \"" + oct.toString() + "\"");
            if (first == null) {
                first = oct;
                System.out.print("Equal");
            }
            else {
                int comparison = oct.compareTo(first);
                if (comparison < 0)
                    System.out.print("less than first");
                else if (comparison > 0)
                    System.out.print("greater than first");
                else 
                    System.out.print("equal");
            }
            String cloneComparison = "Clone Compare: ";
            older = oct;
            Octagon clone = oct.clone();
            if ( older.getArea() == clone.getArea() ){
                cloneComparison = cloneComparison + "Equal";
            } else {
                cloneComparison = cloneComparison + "Not Equal";
            }
            //System.out.println(cloneComparison);
            i++; 
            first = new Octagon(side);
            System.out.println();
        }
        fin.close();
    }
}

这是用于获取输入的文件。每一行是一个八边形：

5.0
7.5
3.26
0.0
-1.0

程序正在正确地对原始对象和它的克隆进行比较，但是，我无法按照我希望的方式进行打印。

在编写上面的代码时，这是我的输出：

Octagon 1: "Area: 72.71067811865474
Perimeter: 40.0
Clone Compare: 
First Compare: "EqualClone Comparision: Equal

Octagon 2: "Area: 161.09902576697317
Perimeter: 60.0
Clone Compare: 
First Compare: "greater than firstClone Comparision: Equal

Octagon 3: "Area: 32.0593921109526
Perimeter: 26.08
Clone Compare: 
First Compare: "less than firstClone Comparision: Equal

Octagon 4: "Area: 2.0
Perimeter: 0.0
Clone Compare: 
First Compare: "less than firstClone Comparision: Equal

但我需要它看起来像这样：

Octagon 1: Area: 2.0
Perimeter: 0.0
Clone Compare: Equal
First Compare: less than first

我觉得我的Octagon类中的toString（）方法可能有问题，因为那是我从中打印getArea（）和getPerimeter（）的地方，但是我无法用它在cloneComparison中编译退货声明。

我尝试使用toString（）方法的return语句包括：

"\nClone Compare: " + cloneComparison + ...
"\nClone Compare: " + oct.cloneComparison + ...
"\nClone Compare: " + Driver.cloneComparison + ...
And
"\nClone Compare: " + Octagon.cloneComparison + ...

非常感谢任何输入。

Answer 1

您的toString()方法返回太多了。您想要的部分输出将由您的main方法输出，因此它不属于toString()。哪部分代码负责什么？如果您的主程序正在打印比较信息，请让它打印比较信息（以及Clone compare:)等标签，并且不要将其放入toString()。

一般而言，定义每个类的职责是有帮助的。考虑它的一种方法是：＆＃34;如果除了Driver之外还有其他一些类，使用我的Octagon类，那么如果要调用toString，该类会期望什么？在一个Octagon对象上？＆＃34;它当然不会期望结果包含"Clone compare"之类的字符串，因为这与Octagon无关;它更多地与Driver想要做的事情有关。将责任分成这样的逻辑位置是了解面向对象编程最重要的事情之一 - 而且，我说，这是发明面向对象编程的主要原因之一。

此外，您想要的输出中没有引号，因此您不应将它们包含在您要打印的字符串中（其中您有反斜杠后跟{{1 }}）。

打印clone（）方法的结果

1 个答案: