Page History

Det sentrale her er å rigge opp et test-objekt og å bruke relevante testdata, inkludert bokstaver som ikke har noen oversettelse og ikke-bokstaver. Som nevnt er vi ikke opptatt at JUnit-spesifikke detaljer, men dersom en ikke bruker JUnit-metoder, så bør en selv definere egne assert-aktige hjelpemetoder.

private RadioAlphabet radioAlphabet;
@Override
protected void setUp() throws Exception {
	// TODO Auto-generated method stub
	super.setUp();
	radioAlphabet = new RadioAlphabet(alphabet, "-");
}
public void testConvertChar() {
	assertEquals("alfa", radioAlphabet.convert('a'));
	assertEquals("hotel", radioAlphabet.convert('h'));
	assertEquals("zulu", radioAlphabet.convert('z'));
	assertNull(radioAlphabet.convert('!'));
}
public void testConvertString() {
	assertEquals("hotel alfa lima lima victor alfa romeo delta", radioAlphabet.convert("hallvard"));
	assertEquals("hotel alfa lima alfa lima", radioAlphabet.convert("!h!a!l!a!l?"));
	radioAlphabet.setWord("åring");
	assertEquals("hotel victor alfa romeo delta", radioAlphabet.convert("håvard"));
}

Det vesentlige her er at en må teste oppførselen både før og etter bruken av endringsmetoden, i praksis at convert fungerer ulikt før og etter bruk av setWord og removeWord. Det kan synes som om det kun er nødvendig å teste oppførselen etterpå, fordi en tross alt har andre tester for oppførsel også. Men det er riktig å test både før og etter, for at testen skal dekke tilfellet alene.

public void testSetWord() {
	assertEquals("hotel alfa lima lima victor alfa romeo delta", radioAlphabet.convert("hallvard"));
	radioAlphabet.setWord("lala");
	assertEquals("hotel alfa lala lala victor alfa romeo delta", radioAlphabet.convert("hallvard"));
}
public void testRemoveWord() {
	assertEquals("hotel alfa lima lima victor alfa romeo delta", radioAlphabet.convert("hallvard"));
	radioAlphabet.removeWord('l');
	assertEquals("hotel alfa victor alfa romeo delta", radioAlphabet.convert("hallvard"));
}

Besvarelser vurderes ut fra korrekthet og grad av gjenbruk. Noen løsninger på del 3, vil egne seg bedre til gjenbruk enn andre, så vi må delvis vurdere ut fra det vi oppfatter som potensialet for gjenbruk. Det teller positivt at en reflekterer over om en kunne økt graden av gjenbruk, dersom en hadde skrevet om RadioAlphabet slik og slik, selv om en ikke har hatt tid til å gjøre det.

Generelt så er det best om en kan gjenbruke logikk som allerede er public. protected-metoder foretrekkes fremfor protected-felt.

Med vår løsning blir det lite ekstra kode, siden den eksisterende representasjonen med forbokstav og ord i ett kan også brukes for morsealfabetet. Hvert element i lista vil være tegnet foran prikkene og strekene, f.eks. ”s...” og ”o---”. Trikset er å utnytte superklassens metoder, ved å justere på argument/returverdi. convert(char) kaller super-metoden og fjerner første bokstav. Den nye setWord-metoden kaller superklassens setWord-metode med tegnet lagt foran ordet. Convert(String) og remove-metoden arves i sin helhet.

public class MorseAlphabet extends RadioAlphabet {
	private final static String morseAlphabet = "s... o--- e. t-";
	public MorseAlphabet() {
		super(morseAlphabet, " ");
	}
	public String convert(char c) {
		String word = super.convert(c);
		return (word != null ? word.substring(1) : null);
	}
	public void setWord(char c, String word) {
		super.setWord(c + word);
	}
}

Lenke til løsningskode: MorseAlphabet.java (MorseRadioAlphabetTest.java)

Observerbarhet krever

1. at en holder styr på lytterne og
2. at alle endringsmetoder kaller lytternes lyttermetode.

Med vår løsning trenger en faktisk ikke å endre MorseAlphabet, siden metodene for 1) arves og all endring 2) skjer gjennom arvede metoder.

1. Holder styr på lytterne

   Code Block
   theme Eclipse language java
   private List<RadioAlphabetListener> listeners = new ArrayList<RadioAlphabetListener>(); public void addRadioAlphabetListener(RadioAlphabetListener listener) { listeners.add(listener); } public void removeRadioAlphabetListener(RadioAlphabetListener listener) { listeners.remove(listener); }

2.  Alle endringsmetoder kaller lytternes lyttermetode:

   Code Block
   theme Eclipse language java
   protected void fireRadioAlphabet(char c) { for (RadioAlphabetListener listener : listeners) { listener.radioAlphabetChanged(this, c); } } public void setWord(String word) { int pos = pos(word.charAt(0)); if (pos >= 0) { alphabet.set(pos, word); // added fireRadioAlphabet(word.charAt(0)); } } public void removeWord(char c) { int pos = pos(c); if (pos >= 0) { alphabet.remove(pos); // added fireRadioAlphabet(c); } }