Monaden / automatabuilder

application for building automata
MIT License
0 stars 0 forks source link

implementet interator for Word #21

Closed Oliv95 closed 9 years ago

Oliv95 commented 9 years ago

Första gången jag gjort en iterator så vet inte om jag gjorde grejer korrekt

Oliv95 commented 9 years ago

Kan du iterators @artog ?

almenjonatan commented 9 years ago

Det saknas test till det du lägger till! Jag kan kösa senare idag.

artog commented 9 years ago

Ser bra ut, men det kollas inte hurvida den ändras / sparar inte strukturen. Nu är det inte superviktigt att vi göra det då vi inte kommer hafsa runt med trådar (än)..

Kanske skall implementera en copy-constructor så kan man göra en kopia inne i iteratorn som man använder.

Eller så kan vi bara stoppa in allt i en list och sen iterera den, eller tom använda dess iterator:

@Override
public Iterator<Symbol> iterator() {
    List<Symbol> list = new LinkedList();
    Symbol currentHead = head;
    IWord currentTail = tail;
    while (currentTail != null) {
        list.add(currentHead);
        currentHead = currentTail.head();
        currentTail = currentTail.tail();
    }
    return list.iterator();
}
almenjonatan commented 9 years ago

Stänger denna då den är mergad redan! :dancer: eller inte?

Vad menas med.

Merge branch 'master' into issue-#16 …
Conflicts:
    src/main/java/automatabuilder/Word.java

@artog @Oliv

är det en rebase som skett här?

artog commented 9 years ago

Det är väl bara en vanlig konflikt

artog commented 9 years ago

Tror det är fixat nu, iaf merge master -> issue-#16, dvs denna är uppdaterad till senaste master