Coding style
bearbeitet von Matthias ApselHallo Achot,
> ~~~
> // Begrüßung festlegen
> if(date("G") < 10) { $begruessung= "Guten Morgen"; }
> if(date("G") <= 18 && date("G") >= 10) { $begruessung= "Guten Tag"; }
> if(date("G") >= 19) { $begruessung= "Guten Abend"; }
> ~~~
Wenn die erste Bedingung zutrifft, brauchen die anderen nicht mehr geprüft zu werden.
~~~php
if(date("G") < 10) :
$begruessung= "Guten Morgen";
elseif(date("G") <= 18 && date("G") >= 10) :
$begruessung= "Guten Tag";
else:
$begruessung= "Guten Abend";
endif;
~~~
> `<link rel="stylesheet" type="text/css" href="include/css/style.css">`{: .language-html .bad}
text/css ist der Standardwert, kann deshalb weg.
> `<h2><?php echo htmlspecialchars($begruessung) ?></h2>`{: .language-html .bad}
Eine Begrüßung ist meiner Meinung nach keine Überschrift. `p` sollte die bessere Wahl sein.
> `<form form method="post" action="">`{: .language-html .bad}
Tippfehler.
> `<?php echo htmlspecialchars($array['name']); ?>`{: .language-php .bad}
`<?= htmlspecialchars($array['name']); ?>`
> `<a href="">Neuen Ansprechpartner hinzufügen</a>`{: .language-html .bad}
Ein a-Element scheint mir hier falsch zu sein. Was passiert bei Klick darauf?
> `<input type="text" `{: .language-html .bad}
text ist der Standardwert, kann also weg.
> `<input type="submit"`{: .language-html .bad}
[button existiert](https://blog.selfhtml.org/2015/02/09/input-oder-button-fuer-submit-elemente/).
Bis demnächst
Matthias
--
Dieses Forum nutzt Markdown. Im [Wiki](https://wiki.selfhtml.org){: style="color:#3481cd"} erhalten Sie [Hilfe bei der Formatierung Ihrer Beiträge](https://wiki.selfhtml.org/wiki/SELFHTML:Forum/Formatierung_der_Beitr%C3%A4ge){: style="color:#3481cd"}.