Question

我对Java很陌生并且遇到了一种奇怪的行为，我无法解释为什么会发生这种情况或我的代码中出现错误。

以下是代码：

import java.util.ArrayList;
import java.util.Arrays;
import java.util.Iterator;

abstract class Shape {
    public abstract void printMe(String no);
}

final class Circle extends Shape {
    @Override
    public void printMe(String no){
        System.out.println("This is Circle no: " + no);
    }
}

final class Square extends Shape {
    @Override
    public void printMe(String no) {
        System.out.println("This is Square no: " + no);
    }
}

final class Triangle extends Shape {
    @Override
    public void printMe(String no) {
        System.out.println("This is Triangle no: " + no);
    }
}

public class Foo {
    private ArrayList<Shape> shapes;

    public Foo(){
        this.shapes   = new ArrayList<Shape>();

        this.shapes.add(new Circle());
        this.shapes.add(new Square());
        this.shapes.add(new Triangle());
    }

    public void printShapes(ArrayList<String> numbers){
        for(String s:numbers){
            Iterator<Shape> iter = this.shapes.iterator();
            Shape shape = iter.next();
            shape.printMe(s);
        }
    }

    public static void main(String[] args) {
        ArrayList<String> numbers = new ArrayList<String>(Arrays.asList("1", "2", "3"));
        Foo foo = new Foo();
        foo.printShapes(numbers);
    }
}

我期望的输出是：

This is Circle no: 1
This is Square no: 2
This is Triangle no: 3

然而，我得到的输出是：

This is Circle no: 1
This is Circle no: 2
This is Circle no: 3

我做错了什么？

Answer 1

将此行拉出循环：

 Iterator<Shape> iter = this.shapes.iterator();

Answer 2

你总是得到一个新的迭代器 - 而不是使用相同的迭代器。

我不清楚为什么你这样做呢？传递整数并循环直到它用完，或迭代形状并保持计数器。传递一个字符串数组让我觉得笨拙。

public void printShapes() {
    int i = 1;
    for (Shape shape : shapes) {
        shape.printMe(i++); // And modify the method to take an int.
    }
}

我对一个需要意识到它可以有位置的形状感到不舒服。如果这是一个要求，创建一个“PositionalShape”或其他东西（但是ew），或者让形状输出一个字符串表示，可以与列表位置等附加信息合成，或者创建一个形状装饰器等。

// (If you're really trying to print the first n shapes)
public void printShapes(int n) {
    Iterator<Shape> iter = shapes.iterator();
    for (int i = 0; i < n; i++) {
        Shape shape = iter.next();
        shape.printMe("" + i+1);
    }
}

Answer 3

查看你的循环中的Iterator<Shape> iter 。

public void printShapes(ArrayList<String> numbers){ for(String s:numbers){ Iterator<Shape> iter = this.shapes.iterator(); Shape shape = iter.next(); shape.printMe(s); } }

你总是抓住第一个形状（初始化迭代器，然后抓住它）

Answer 4

我怀疑你需要在调试器中看到这个，但是

Iterator<Shape> iter = this.shapes.iterator();
Shape shape = iter.next();
shape.printMe(s);

您每次都使用第一个共享（这是一个圆圈）

您可以在循环外移动iter声明来修复它。

Answer 5

以下两行需要循环： Shape shape = iter.next(); shape.printMe(s);

Answer 6

您始终重置迭代器：

Iterator<Shape> iter = this.shapes.iterator();

为什么多态不能像我在我的代码中所期望的那样工作？

6 个答案: