Chord.setOctave（x）在jFugue 5中返回错误值的注释。我错过了什么？

Question

以下是示例代码：

public class TestNoteValue {
    public static void main(String[] args) {
        Chord c = new Chord("C5maj");
        DevLog.debug(MusicAnnotationUtil.reportChord(c));
        // ----------------
        c.setOctave(4);
        DevLog.debug(MusicAnnotationUtil.reportChord(c));
    }
}

结果输出：

第一次的结果是对的：     和弦{Note {C5,60}，Note {E5,64}，Note {G5,67}，}

C5的价值确实是60。 但是在Chord.setOctave（4）之后，它并没有改变第一个音符的表达式，但改变了它的值。导致正确性受损的音符数组：     和弦{注意{C5,48} ，注意{E4,52}，注意{G4,55}，}

我在这里错过了什么吗？

感谢您的帮助！

当David正在帮助查看代码时，我会发布我的临时解决方法。希望它有助于其他人。

    public static Chord setOctave(Chord c, byte octave) {
    DevLog.super_trace("setting octave for chord "+c+" to "+octave);
    c.setOctave(octave);
    Note[] nA = new Note[3];
    for(int i=0; i<c.getNotes().length; i++){
        nA[i] = new Note(c.getNotes()[i].getValue());
    }
    return Chord.fromNotes(nA);
}

Answer 1

我在Note类中添加了一个新方法，用于更新＆＃34;原始字符串＆＃34;当该音符的值发生变化时，会创建一个音符。

我在解决其他一些问题后会发布新版本。与此同时，这里是Note.java中的变化：

更新方法：

public Note setValue(byte value) {
    this.value = value;
    this.updateOriginalString(); // New line that calls the method below
    return this;
}

新方法：

private void updateOriginalString() {
    if (this.getOriginalString() != null) {
        String oldOriginalString = this.getOriginalString();
        StringBuilder newOriginalString = new StringBuilder();
        newOriginalString.append(getToneString());
        if ((oldOriginalString.length() > 1) && (oldOriginalString.substring(oldOriginalString.length()-2, oldOriginalString.length()-1).matches("\\d"))) {
            newOriginalString.append(getOctave());
        }
        setOriginalString(newOriginalString.toString());
    }
}

NoteTest.java中的新测试：

@Test
public void testOriginalStringForNotes() {
    assertTrue(new Note("C").getOriginalString().equals("C"));
    assertTrue(new Note("C5").getOriginalString().equals("C5"));
    assertTrue(new Note("C").changeValue(+1).getOriginalString().equals("C#"));
    assertTrue(new Note("C5").changeValue(+1).getOriginalString().equals("C#5"));
    assertTrue(new Note("C").setValue((byte)48).getOriginalString().equals("C"));
    assertTrue(new Note("C5").setValue((byte)48).getOriginalString().equals("C4"));
}