PHP Daten in einem Tag auf Überschneidung prüfien und dementsprechend anzeigen

[Whiz-zarD]

Vielen Danke schonmal.
Ich habe mir überlegt, dass ich die Spalte für den Tag je nach Anzahl der Überschneidungen einfach breiter mache.
Also zuerst überprüfe, wieviele maximale Überschneidungen es an diesem Tag gibt dann einfach mehr <td> anzeige.
Wenn bei einem anderen Termin keine Überschneidungen sind, mach ich einfach eine Colspan mit der Anzahl der Überschneidungen.
Das sollte meinen Aufbau nicht ganz so sehr über den Haufen werfen.

...

Musst du wissen. Finde ich aber sehr suboptimal.
Ich finde, das macht das Ganze sehr unübersichtlich, wenn die Tage eine unterschiedliche Breite haben. Man muss also schon genau hinschauen, welcher Termin zu welchem Tag gehört, wenn z.B. der Montag plötzlich die selbe Breite hat, wie der Dienstag und Mittwoch zusammen. Oder wenn ich mir schon einige Terminpläne meiner Arbeitskollegen anschaue, dann gäbe es oft der Fall, dass ein Tag mal den Platz einer gesamten Woche benötigt.

Man kann den "Zoom"-Faktor einstellen. Möchte man einen Auftrag anlegen, Wir die Zeit aus der Tabelle übernommen und man gibt einen Zeitraum in 10-Minuten Intervallen an.

...

Bringt dir ja trotzdem nichts.
Wie willst du ein Termin korrekt anzeigen lassen, der z.B. um 08:25 Uhr anfängt?
Auch wenn man rein- oder rauszoomen kann, muss ja dennoch der Termin korrekt in der Spalte angezeigt werden.
Es kann ja nicht sein, dass der Termin um 08:00 oder 8:30 Uhr anfängt, wenn die Einteilung halb-stündlich ist, und dann plötzlich um 08:00 oder 09:00 Uhr anfängt, wenn die Einteilung stündlich ist.
Darum sage ich ja, dass du mit einer festen Einteilung (auch mit Zoom-Funktion ist die Einteilung fest) nicht weit kommen wirst, du musst schon was haben, um die Termine in deiner Zeitleiste variabel verschieben zu können.

[Raildex]

Hey, vielen Dank für die Hilfe!
Ich hab allerdings Probleme mit der Umsetzung
Ich hab hier mal meine Funktion:

Code:

function getNumberOfMaxOverlaps(DateTime $date,array $tasks){
	$tasksOfThisDay = array();
	foreach($tasks as $task)
	{
		if($task->start->format('Y-m-d') == $date->format('Y-m-d'))
		{
			$tasksOfThisDay[] = $task;
		}
	}
	$overlap = array();
	foreach ($tasksOfThisDay as $task1) 
	{
		foreach ($tasksOfThisDay as $task2) 
		{


			if (strtotime($task1->start->format("Y-m-d HH:ii:ss")) <  strtotime($task2->end->format("Y-m-d HH:ii:ss")) &&
				strtotime($task1->start->format("Y-m-d HH:ii:ss")) >= strtotime($task2->start))
			{
				$array = array($task1->id, $task2->id);
				sort($array);
				if (!in_array($array, $overlap)) $overlap[] = $array;
			}
		}
	}
}

$overlap zeigt am Ende allerdings immer 0 an. Gearbeitet habe ich mit folgenden Daten

Code:

$rows               = array();$class              = new stdClass();
$class->id          = 1;
$class->name        = 'Test';
$class->description = 'Test Description';
$class->start       = '2014-06-17 06:05:00';
$class->end         = '2014-06-17 07:00:00';
$rows[]             = $class;
$class              = new stdClass();
$class->id          = 2;
$class->name        = 'Test 2';
$class->description = 'Test Description 2';
$class->start       = '2014-06-17 07:00:00';
$class->end         = '2014-06-17 07:40:00';
$rows[]             = $class;
$class              = new stdClass();
$class->id          = 3;
$class->name        = 'Test 2';
$class->description = 'Test Description 2';
$class->start       = '2014-06-17 09:00:00';
$class->end         = '2014-06-17 09:40:00';
$rows[]             = $class;
$class              = new stdClass();
$class->id          = 4;
$class->name        = 'Test 2';
$class->description = 'Test Description 2';
$class->start       = '2014-06-17 10:00:00';
$class->end         = '2014-06-17 11:40:00';
$rows[]             = $class;

[Whiz-zarD]

Kapsel deinen Code weiter in Funktionen. Hab keine Scheu kleine Funktionen zu schreiben.
Für start und end kannst auch gleich DateTime Objekte erstellen. Dann sparst du dir das ganze rumkonvertiere.

Außerdem ist das Ergebnis von 0 doch richtig mit deinen Testdaten und es fehlt ein return in deiner Funktion.

[Raildex]

Wenn ich

Code:

count($overlap);

anwende, zeigt er 0 0 0 0 0 0
an.
Die IDs haben aber einen Wert, der nicht 0 ist.

Für start und end kannst auch gleich DateTime Objekte erstellen. Dann sparst du dir das ganze rumkonvertiere.

...

Alles Klar

€dit:
Ok, die ganze Sache mit assoziativen Arrays verwirrt mich.

Code:

foreach($overlap as $key => $value)
			{
				echo $key;
				echo $value;
			}

gibt aber auch nichts aus...

[Whiz-zarD]

Drei Fehler:
1. Erst mal hast du bei deiner if-Abfrage bei $task2->start vergessen den Formatstring mitzugeben.
2. Der Formatstring für die Uhrzeit muss H:i:s lauten.

Korrigierst du diese beiden Fehler, dann wirst du auch ein Ergebnis bekommen, welches aber noch falsch ist, denn:

3. Du testest die IDs nicht auf Gleichheit. Das sorgt dafür, dass er eine Task mit sich selbst überprüft.

Edit:
Ich würde den Code aber auch so in etwas aufteilen:

Code:

function getOverlaps(DateTime $date,array $tasks)
{
	$result = array();
	$tasksOfThisDay = getTasksOfDate($date, $tasks);
	
	foreach ($tasksOfThisDay as $task1) 
	{
		foreach ($tasksOfThisDay as $task2) 
		{
			if (isOverlay($task1, $task2)) 
			{
				$array = array($task1->id, $task2->id);
				sort($array);
				if (!in_array($array, $result)) $result[] = $array;
			}
		}
	}
	
	return $result;
}

function getTasksOfDate(DateTime $date, array $tasks)
{
	$result = array();
	foreach($tasks as $task)
	{
		if($task->start->format('Y-m-d') == $date->format('Y-m-d'))
		{
			$result[] = $task;
		}
	}
	
	return $result;
}

function isOverlay(stdClass $task1, stdClass $task2)
{
	return  ($task1->id != $task2->id)
			&& ($task1->start->format('H:i:s') < $task2->end->format('H:i:s')) 
			&& ($task1->start->format('H:i:s') >= $task2->start->format('H:i:s'));
}

Meine php Kenntnisse sind aber auch sehr eingerostet. Vermutlich geht der Code noch hübscher. Ich hab schon seit etlichen Jahren nichts mit mit php am Hut gehabt.

[Raildex]

Puh, vielen Dank!

Ich habe mich im Übrigen für deine Variante entschieden, allein weil es wirklich schöner ist, wenn die Daten genauer angezeigt werden.