java - 接口(interface)的更高级用法

Question

说实话，我不太确定我自己是否理解这个任务:) 我被告知创建类 MySimpleIt，它实现 Iterator 和 Iterable，并允许运行提供的测试代码。对象的参数和变量不能是集合或数组。
代码:

 MySimpleIt msi=new MySimple(10,100,
                           MySimpleIt.PRIME_NUMBERS);

 for(int el: msi)
   System.out.print(el+" ");    
 System.out.println();

 msi.setType(MySimpleIterator.ODD_NUMBERS);
 msi.setLimits(15,30);
 for(int el: msi)
   System.out.print(el+" ");    
 System.out.println();

 msi.setType(MySimpleIterator.EVEN_NUMBERS);
 for(int el: msi)
   System.out.print(el+" ");    
 System.out.println();

我应该得到的结果:

11 13 17 19 23 29 31 37 41 43 47 53 59 61 67 71 73 79 83 89 97 


15 17 19 21 23 25 27 29 


16 18 20 22 24 26 28 30

这是我的代码:

import java.util.Iterator; 
interface MySimpleIterator{
    static int ODD_NUMBERS=0;
    static int EVEN_NUMBERS = 1;
    static int PRIME_NUMBERS = 2;

    int setType(int i);
}

public class MySimpleIt implements Iterable, Iterator, MySimpleIterator {    
    public MySimple my;

    public MySimpleIt(MySimple m){ 
        my = m;      
    }

    public int setType(int i){
        my.numbers = i;
        return my.numbers;
    }

    public void setLimits(int d, int u){
        my.down = d;
        my.up = u;
    }

    public Iterator iterator(){
        Iterator it = this.iterator();
        return it;
    }

    public void remove(){  
    }

    public Object next(){
        Object o = new Object();
        return o;
    }

    public boolean hasNext(){
        return true;
    }

}

class MySimple {
    public int down;
    public int up;
    public int numbers;

    public MySimple(int d, int u, int n){
        down = d;
        up = u;
        numbers = n;
    }
}

在测试代码中，创建 MySimpleIt msi 对象时出现错误，因为它找到的是 MySimple 而不是 MySimpleIt。另外，我在 for-each 循环中遇到错误，因为编译器需要“int”而不是“Object”。有人知道如何解决这个问题吗？

Answer 1

这个作业的设计有很多错误。

使用enum

测试代码包含以下片段:

MySimpleIt(erator?).PRIME_NUMBERS
MySimpleIt(erator?).ODD_NUMBERS
MySimpleIt(erator?).EVEN_NUMBERS

在一个地方，类型是 MySimpleIt ，在另一个中是 MySimpleIterator 。不管怎样，顾名思义，就是使用接口(interface)来定义一堆常量。这不是 interface 的正确使用!!!

使用 enum 会是一个更好的设计相反:

enum SequenceType {
  PRIME_NUMBERS, ODD_NUMBERS, EVEN_NUMBERS;
}

参见:Effective Java 第二版 第 30 项:使用枚举代替 int常数。

<小时/>

考虑 interface 的多种实现而不是 setType 的整体

看起来你的音序器应该能够随心所欲地切换音序类型。这将导致该类成为一个巨大的 Blob ，它必须知道如何生成每种类型的序列。它可能仅适用于此处给出的 3 种类型，但如果您稍后想要添加更多类型的序列，这绝对是一个糟糕的设计。

考虑对不同类型的序列使用同一接口(interface)的不同实现。您可能想要定义 AbstractIntegerSequencer它定义了委托(delegate)给 hasNext() 的基本功能(重置边界、回答 iterator() 、 abstract protected int generateNext() 等)。对于子类 @Override 。这样，要生成的序列类型的细节就可以很好地封装到每个子类中。

您仍然可以保留enum SequenceType对于static实例化这些不同子类的工厂方法，每个子类对应一种序列类型，但这些序列本身可能不应该能够随心所欲地切换类型。

<小时/>

使用泛型

而不是输入 implements Iterator ，你应该做到 implements Iterator<Integer> .

来自JLS 4.8 Raw Types (强调他们的):

The use of raw types is allowed only as a concession to compatibility of legacy code. The use of raw types in code written after the introduction of genericity into the Java programming language is strongly discouraged. It is possible that future versions of the Java programming language will disallow the use of raw types.

另请参阅《Effective Java 第 2 版》第 32 条:不要在新代码中使用原始类型。

<小时/>

不要混淆Iterator<T>与 Iterable<T> .

假设你有这样的东西:

IntegerSequencer seq = new PrimeSequencer(0, 10);

for (int i : seq) {
  System.out.println(i);
} // prints "2", "3", "5", 7"

for (int i : seq) {
  System.out.println(i);
} // what should it print???

如果你做seq implements Iterable<Integer>, Iterator<Integer>和@Override Iterator<Integer> iterator()至return this; ，那么第二个循环不会打印任何内容，因为 seq是它自己的iterator() ，此时不再有 hasNext()对于 seq .

Iterable<Integer>的正确实现应该能够生成尽可能多的独立Iterator<Integer>根据用户的需要，这样的实现将再次打印 0 之间的素数和10在上面的代码中。

<小时/>

有关 stackoverflow 的进一步阅读

• Iterator performance contract (and use on non-collections)
  • 此问题询问的是 PrimeGenerator implements Iterator<Integer> (除其他外)
• Why is Java’s Iterator not an Iterable?
• What is the Iterable interface used for?
• Why can't I iterate over an iterator?