Gunnar Bittersmann: Prüfung schlägt fehl

Beitrag lesen

@@Matthias Apsel

dedlfixens Hilfe ist um vieles wertvoller.

So ist es. Die ganzen Werte in mittendrin in if-Abfragen zu haben ist Spaghetti-Code.

Sinnvoll ist, Daten und Programmlogik voneinander zu trennen. Bspw. so:

$statusInserts = [
	1 => [
		'07bc9a80a0da7031218f1a9514a327c3',
		'8f9bdc964d3b7496587b8161e2c43d1f',
		'904895f93ccb9f1f44479582c9d4bd4e',
	],
	2 => [
		'54a556b805869244d18fe0dc0c19c4ce',
		'3018babe7d2a1fbeb43c3bae0fcd532f',
		'2afeb8b8f86e17862748b6402da01db6',
		'6e623de7c366c92accd2b4a7ee49ad2a',
		'602c5602d0a86003a925951b46a012b4',
		'5555d1be56a4b410da755026783b4d9e',
	],
];

function getStatusInsert($status, $statusInserts)
{
	foreach ($statusInserts as $statusInsert => $statusArray)
	{
		if (in_array($status, $statusArray))
		{
			return $statusInsert;
		}
	}

	return 3;
}

var_dump(getStatusInsert('8f9bdc964d3b7496587b8161e2c43d1f', $statusInserts));	// 1
var_dump(getStatusInsert('6e623de7c366c92accd2b4a7ee49ad2a', $statusInserts));	// 2
var_dump(getStatusInsert('33333333333333333333333333333333', $statusInserts));	// 3

oder die Datenstruktur andersrum:

$statusInserts = [
	'07bc9a80a0da7031218f1a9514a327c3' => 1,
	'8f9bdc964d3b7496587b8161e2c43d1f' => 1,
	'904895f93ccb9f1f44479582c9d4bd4e' => 1,
	'54a556b805869244d18fe0dc0c19c4ce' => 2,
	'3018babe7d2a1fbeb43c3bae0fcd532f' => 2,
	'2afeb8b8f86e17862748b6402da01db6' => 2,
	'6e623de7c366c92accd2b4a7ee49ad2a' => 2,
	'602c5602d0a86003a925951b46a012b4' => 2,
	'5555d1be56a4b410da755026783b4d9e' => 2,
];

function getStatusInsert($status, $statusInserts)
{
	if (array_key_exists($status, $statusInserts))
	{
		return $statusInserts[$status];
	}
	else
	{
		return 3;
	}
}

var_dump(getStatusInsert('8f9bdc964d3b7496587b8161e2c43d1f', $statusInserts));	// 1
var_dump(getStatusInsert('6e623de7c366c92accd2b4a7ee49ad2a', $statusInserts));	// 2
var_dump(getStatusInsert('33333333333333333333333333333333', $statusInserts));	// 3

LLAP 🖖

--
„Wer durch Wissen und Erfahrung der Klügere ist, der sollte nicht nachgeben. Und nicht aufgeben.“ —Kurt Weidemann