Termine in Kalender einfügen

[Robert B.]

Moin,

gestatte mir ein paar Fragen und Anmerkungen:

$datumTermine = Kalender_Termine($mysqli, $date);

    if($datumTermine > 0) {
        foreach($datumTermine as $array){

Was gibt denn deine Funktion Kalender_Termine für einen Datentyp zurück? Ich sehe hier einmal einen numerischen Vergleich und dann eine Array-Iteration. Damit das wie gewünscht funktioniert, darf im Fall keiner Daten in $datumTermine kein Array stehen, sondern z.B. NULL, denn

$datumTermine = array();
$datumTermine > 0 == TRUE;

Das heißt du kannst explizit auf NULL testen.

Im folgenden Code fallen verschiedene Dinge auf:

• Du benutzt sehr viele Inline-Styles für mehrere Elemente. Benutze besser CSS-Klassen.
• Werte aus PHP werden direkt in den HTML-Code eingefügt. Hast du den Kontextwechsel nach HTML berücksichtigt?
• target=_blank ist nicht die feine englische Art, denn der Nutzer möchte vielleicht gerne entscheiden, wie viele Browserfenster sich öffnen. Außerdem funktioniert in einem neuen Fenster das Zurück nicht mehr.
• Du benutzt recht viele div und span. Aber im Ausgangsposting sieht man doch eine Tabelle. Warum also nicht td?

                $week .= '<div style="background:'.$array["farbe"].'; 
                                      margin-bottom:5px; 
                                      padding-left:8px;">
                        <a href="/kalender-datenblatt.php?code='.$array["id"].'"
                           target="_blank" 
                           style="color:#ffffff">'.$array["bezeichnung"]. "</a>
                        
                        <span style='color:#ffffff; 
                                     font-size:10px; 
                                     display: block; 
                                     padding-bottom: 2px;'>". $array["wer"]. '</span>
                           </div>';

Viele Grüße
Robert