Aus alt mach Neu

Aus alt mach Neu - Legacy Code modernisieren

Jeder kennt es. Software wächst und wächst und es entstehen immer neue Anforderungen. Dabei bleiben technische Schulden leider oft auf der Strecke. Wie geht es nun weiter? Die Legacy-Codebasis komplett ersetzen und von neu Programmieren? Das wird extrem schwer eurem Product Owner zu erklären.

In diesem Talk möchte ich euch zeigen, wie bei uns ein zentrales Modul modernisiert wurde. Ein paar Eckdaten: 3900 Zeilen Code, 19 Abhängigkeiten und eine NPath Komplexität jenseits von gut und böse. Und das alles natürlich in nur einer der dazugehörigen Klassen. Hier wird gezeigt wie man so ein Projekt umsetzen kann, sodass Entwickler und Product Owner zufrieden sind. Die Software wird modernisiert und neue Funktionen können während der Umbaumaßnahmen implementiert werden.

Habt keine Angst mehr vor eurem alten, ungetesteten und fehleranfälligen Code. Beginnt Ihn zu modernisieren!

B	I	N	G	O
Perfomant	ASAP	Agil	Modular	Tests
Pflegbar	wenig Aufwand	Sicher	Microservice	Dokumentation
Kosten	Anforderungen	Kundenorientiert	Conversion Rate	Qualität
Fehlerfrei	Pflegbar	Dynamisch	Konfigurierbar	Stabilität
Technisch sauber	Zukunftsfähig	Flexibilität	Schnell	Testabdeckung

	public function substractPositiveNumbers(int $a, int $b, int $c, int $d): int
	{
	1 2 3 4 5
	if ($a < 0) { $a = 0; } 0 1 0 0 0
	if ($b < 0) { $b = 0; } 0 0 1 0 0
	if ($c < 0) { $c = 0; } 0 0 0 1 0
	if ($d < 0) { $d = 0; } 0 0 0 0 1

	return $a - $b - $c - $d;
	}

	public function substractPositiveNumbers(): int
	{
	1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6
	if ($this->a < 0) { $this->a = 0; } 0 1 1 1 1 1 1 1 0 0 0 0 0 0 0 0
	if ($this->b < 0) { $this->b = 0; } 0 0 1 0 0 0 1 1 1 0 0 0 0 1 1 1
	if ($this->c < 0) { $this->c = 0; } 0 0 0 1 0 1 1 1 0 1 0 1 0 1 0 1
	if ($this->d < 0) { $this->d = 0; } 0 0 0 0 1 1 0 1 0 0 1 1 1 0 1 1
	return $this->a - $this->b - $this->c - $this->d;
	}

	public function substractPositiveNumbers(int $a, int $b, int $c, int $d): int
	{
	if ($this->a < 0) { $this->a = 0; }
	if ($this->b < 0) { $this->b = 0; }
	if ($this->c < 0) { $this->c = 0; }
	if ($this->d < 0) { $this->d = 0; }
	return $this->a - $this->b - $this->c - $this->d;
	}

	public function doWithA(): void
	{
	if ($this->a === 0) { $this->blackMagic(); }
	}

	// before
	class SearchResulthandler {
	private function getSqlHotelFilter(ResultsRequest $request, array &$params): string
	{
	$sql = '';
	$this->distanceSetByFilter = false; // Interner State

	// filter by hotel id
	if ($request->getHotel_id()) {
	$sql .= ' AND cstc.cluster_id = :clusterId'; // SQL String ohne Kontext
	$params['clusterId'] = $request->getHotel_id();
	}
	// [...]
	return $sql;
	}
	}

Aus alt mach Neu

Ausgangssituation

Search Result Handler

SRH - Aufgaben

SRH - Datenbank Ablaufdiagramm

Anforderungen analysieren

Grobe Anforderungen

Detailierte Anforderungen

Lösung

Löschen und neu Machen?!

"Nein"

Löschen und neu Machen?

Refaktoring des Projekts

Challenge Accepted

Metriken erfassen

Metriken erfassen - Beispiel

Metriken erfassen - Beispiel

Metriken erfassen - Beispiel

Problemstellen identifzieren

Testbarkeit Herstellen

Perfomance Reports erstellen

Trennung der Logik

Extrahieren der Methoden

Code duplizieren

Beispiel Refactoring

Beispiel Refactoring

Neue Funktionen implementieren

Neue Funktionen implementieren

Vergleich der Metriken

PHPMetrics Analyse

Übersicht

Perfomance Monitoring

Lessons Learned

Motivierend

Überblick bekommen

Umbau nicht liegen lassen

Kleine Schritte gehen

Wartet nicht zu lange

	class ResultsFilter { // Eigene Klasse
	private function addResultFilter(ResultsRequest $request, QueryBuilder $selectQuery): void
	{
	if ($request->getHotel_id()) {
	$selectQuery->andWhere('cluster_id = :clusterId'); // Query Builder
	$selectQuery->setParameter('clusterId', $request->getHotel_id(), Type::INTEGER);
	}
	}
	}
	class SearchResultHandler {
	private function getSqlHotelFilter(ResultsRequest $request, array &$params): string
	{
	$clusterIdSelect = $this->resultsFilter->getResultFilter($request);
	$params = array_merge($params, $clusterIdSelect->getParameters());
	// Minimal Eingriff in die Legacy Codebase
	return ' cluster_id IN ( ' . $clusterIdSelect->getSQL() . ' )';
	}
	}

	// Before
	class ResultsRequest {
	public function getCancelableBitmask(): ?int;
	}
	class CompactResultsRequest extends ResultsRequest {
	}

	class SearchResultHandler {
	function getSqlHotelFilter($sql, &$params): string {
	// Check every possible Request type
	if ($this->request->getCancelableBitmask()) {
	if ($this->request instanceof ResultsRequest) {
	$sql .= ' AND cancelable & :cancelableBitmask > 0';
	$params['cancelableBitmask'] = $this->request->getCancelableBitmask();
	}
	if ($this->request instanceof CompactResultsRequest) {
	$sql .= ' AND cancelable & :cancelableBitmask = :cancelableBitmask';
	$params['cancelableBitmask'] = $this->request->getCancelableBitmask();
	}
	}
	return $sql;
	}
	}

	// After
	interface ResultsClusterOfferFilterableInterface {
	public function getCancelableBitmask(): ?int;
	}
	interface OfferFilterableInterface {
	public function getCancelableBitmask(): ?int;
	}

	class ResultsOfferClusterFilter {
	function addResultFilter(QueryBuilder $selectQuery, ResultsRequest $request): void {
	if ($request->getCancelableBitmask()) {
	$selectQuery->andWhere('cancelable & :cancelableBitmask > 0');
	$selectQuery->setParameter('cancelableBitmask', $request->getCancelableBitmask(), Type::INTEGER);
	}
	}
	}

	class OffersFilter {
	function addResultFilter(QueryBuilder $selectQuery, ResultsRequest $request): void {
	if ($request->getCancelableBitmask()) {
	$selectQuery->andWhere('cancelable & :cancelableBitmask = :cancelableBitmask');
	$selectQuery->setParameter('cancelableBitmask', $request->getCancelableBitmask(), Type::INTEGER);
	}
	}
	}

	Vorher	Nachher
Average cyclomatic complexity by class	23.31	11.84
Lines of Code	5823	7922
Classes	35	91

# Requests Listenansicht	~ 300k
Beispielwerte	05.05	01.09
AVG Response Time Listenansicht	316ms	↑ 263ms
# Requests Detailansicht	~ 610k
AVG Response Time Detailansicht	168ms	↑ 155ms
# Requests Vertriebspartner	~ 900k
AVG Response Vertriebspartner	110ms	↑ 59ms
# Requests Kartenansicht	~ 250k
AVG Response Kartenansicht	264ms	↓ 343ms