16

JavaIteratorとIterableインターフェースを理解しようとしています

私はこのクラスを書いています

class MyClass implements Iterable<String> {
    public String[] a = null;
    public MyClass(String[] arr) {
        a = arr;    
    }

    public MyClassIterator iterator() {
        return new MyClassIterator(this);
    }

    public class MyClassIterator implements Iterator<String> {
        private MyClass myclass = null;
        private int count = 0;
        public MyClassIterator(MyClass m) {
            myclass = m;    
        }

        public boolean hasNext() {
            return count < myclass.a.length;
        }
        public String next() {
            int t = count;
            count++;
            return myclass.a[t];
        }

        public void remove() {
            throw new UnsupportedOperationException();
        }
    }   
}

動作しているようです。

私が持っている必要があります：

Myclass implements Iterable<Stirng>, Iterator<String> {

}

または私はMyClassIterator外MyClassに置く必要があります

class MyClass implements Iterable<String> {
    public String[] a = null;
    public MyClass(String[] arr) {
        a = arr;    
    }
    public MyClassIterator iterator() {
        return new MyClassIterator(this);
    }
}


    public class MyClassIterator implements Iterator<String> {
        private MyClass myclass = null;
        private int count = 0;
        public MyClassIterator(MyClass m) {
            myclass = m;    
        }

        public boolean hasNext() {
            return count < myclass.a.length;
        }
        public String next() {
            int t = count;
            count++;
            return myclass.a[t];
        }

        public void remove() {
            throw new UnsupportedOperationException();
        }
    }

どちらがいいですか？

4

3 に答える 3

35

と の両方を同じクラスに実装することはほとんどありません。彼らはさまざまなことをします。イテレータは自然にステートフルです。イテレータを使用して反復すると、世界観を更新する必要があります。ただし、イテラブルは、新しいイテレータを作成できる必要があるだけです。特に、複数のイテレータが同じ元の iterable に対して同時に動作する可能性があります。IterableIterator

あなたの現在のアプローチはほとんど大丈夫です - 私が変更したい実装の側面がありますが、責任の分離に関しては問題ありません。

于 2011-04-29T19:00:10.387 に答える
9

あなたは最初の試みで順調に進んでいました。MyClassを実装するだけでよく、そのためには から返す実装をIterable<String>提供する必要があります。Iterator<String>Iterable<String>.iterator()

ほとんどの場合、を直接使用する必要さえないため(s の構文によって暗黙的に使用されます)、他のすべての場合では、実際に実装に追加の動作を追加しない限り、インターフェイスで十分であるため、をMyClassIterator外側に配置する必要はありません。 （これはおそらく行う必要はありません）。MyClassIterator<String>for .. in ..Iterable<String>

これが私が行う方法です。コメントをインラインで参照してください。

import java.util.Iterator;

class MyClass implements Iterable<String>{
    public String[] a=null; //make this final if you can
    public MyClass(String[] arr){
        a=arr; //maybe you should copy this array, for fear of external modification
    }

    //the interface is sufficient here, the outside world doesn't need to know
    //about your concrete implementation.
    public Iterator<String> iterator(){
        //no point implementing a whole class for something only used once
        return new Iterator<String>() {
            private int count=0;
            //no need to have constructor which takes MyClass, (non-static) inner class has access to instance members
            public boolean hasNext(){
                //simplify
                return count < a.length;
            }
            public String next(){
                return a[count++]; //getting clever
            }

            public void remove(){
                throw new UnsupportedOperationException();
            }
        };
    }
}
于 2011-04-29T19:29:57.993 に答える
2

Myclass implements Iterable<String>,Iterator<String>{イテレータは使い捨てなので、そうすべきではありません。リスト反復子を除いて、それらを先頭に戻す方法はありません。

ちなみに、スキップできます

MyClass myClass;
public MyClassInterator(MyClass m){
  myclass=m;  
}

参照する代わりに

myClass

参照

MyClass.this

内部クラスは静的ではないため、それMyClass.thisを作成した外側のクラスのインスタンスを参照します。

于 2011-04-29T19:01:30.620 に答える