Expose more detailed analysis of individual words

[mortterna]

No description provided.

[komu]

Tää vaikuttaa vähän epäilyttävältä. Jos ei muuten, niin ainakin nimeämiseltään, koska jotenkin nimen perusteella olettaisi että toteutus olisi ekvivalentti toString().replace("-", "") kanssa, mutta ilmeisesti näin ei ole.

Muutenkin, tuonne else-haaraanhan ei ole mahdollista mennä, mutta kääntäjä ei vain tajuta sitä koodin rakenteesta enkä minäkään ensin. Mitä jos toi olisikin:

if (s.isEmpty() || s.equals("-")) 
    return "";

int startOffset = startsWithChar(s, '-') ? 1 : 0;
int endOffset = endsWithChar(s, '-') ? 1 : 0;

return s.substring(startOffset, s.length() - endOffset);

Eikös tämä ole semantiikaltaan sama? Ainakin mulle kommunikoi paremmin mistä on kyse. Toi jopa on allokoinneiltaan identtinen, koska String.substring sisältää optimoinnin:

if (beginIndex == 0 && endIndex == length) {
    return this;
}

Muutenkin mietin tarvitseeko tuon olla rajapinnassa default-toteutuksena. Ei kai kellään rajapinnan toteuttajalla ole tarvetta overridetä sitä? Harkitsisin ihan vain jotain removeLeadingAndTrailing(s, '-') -funktiota. Tuollaisella funktiolle olisi helpompi kirjoittaa testitkin kun ei tule koko muuta roskaa mukana. Ja jos haluaa sen default-toteutuksen sinne rajapintaan, niin sekin voisi olla vain kutsu tuohon staattiseen funktioon.

[mortterna]

Kyseistä logiikkaa tarvitsee myös pluginin algoritmeissa, joissa käsitellään yhdyssanojen palasia, mutta sen verran pieni se on myös monistaa, joten ehkä se on parempi vain ottaa pois tästä rajapinnasta.

[komu]

Tää vähän sattuu muhun. Sen sijaan että allokoidaan ja kopioidaan, eikö voisi vain pitää indeksiä ensimmäiseen konsumoimattomaan baseFormiin. Eli olisi:

private int baseFormPartIndex = 0;

private @NotNull String popBaseForm() {
    if (isConsumed())
        throw new IllegalStateException("Base form parts empty");

    return baseFormParts[baseFormPartIndex++];
}

private boolean isConsumed() {
    return baseFormPartsIndex >= baseFormParts.length;
}

Tai jos ei halua noin matalan tason juttua, niin wrapata koko paska ArrayDeque-luokkaan ja kutsua sille removeFirst:iä. Se tekee sisäisesti saman jutun, että manipuloi vain indeksejä eikä kopioi mitään.

[komu]

Mä en tekisi omaa not-metodiaan vain yhden käyttökerran takia. Kyllä se if (! currentStrongMorpheme.isConsumed()) olisi ihan tarpeeksi selkeä. Itse asiassa sanoisin että se on jopa selkeämpi, koska tässä koodissa tulee korostuu selkeämmin se että eka ja toinen if-testaavat täsmälleen saman asian arvoja.

if (!currentStrongMorpheme.isConsumed()) {
    currentPart.addBaseForm(currentStrongMorpheme.popBaseForm());
    currentStrongMorpheme.addPart(currentPart.toWordPart());

    if (currentStrongMorpheme.isConsumed()) {
        wordParts.add(currentStrongMorpheme.toWordPart());
        currentStrongMorpheme.reset();
    }
}