Konwencja pisania wartości w asercjach

0

Lepiej tak:

final int DEFAULT_NULL_VALUE = 0;
	@Test
	public void testCalc1() {
		// Given:
		String text = "Wiek: 20 lat.";
		
		// When:
		AgeAlg a = new AgeAlg();
		Optional<Integer> result = a.calc(text).stream().findFirst();
		
		// Then:
		assertTrue(result.isPresent());
		assertEquals(20, result.orElse(DEFAULT_NULL_VALUE).intValue());
	}

czy tak:

final int DEFAULT_NULL_VALUE = 0;
	@Test
	public void testCalc2() {
		// Given:
		String text = "Wiek: 20 lat.";
		
		// When:
		AgeAlg a = new AgeAlg();
		Optional<Integer> result = a.calc(text).stream().findFirst();
		int resultInt = result.orElse(DEFAULT_NULL_VALUE).intValue();
		boolean isPresent = result.isPresent();
		
		// Then:
		assertTrue(isPresent);
		assertEquals(20, resultInt );
	}
2

Wg mnie pierwsza opcja.
result wykorzystujesz 2 razy. resultInt i resultInt tylko po raz.
Fakt, że nieco czytelniej, ale IMHO nie ma to wartego takiej zmiany impaktu.

1 użytkowników online, w tym zalogowanych: 0, gości: 1