Gunnar Bittersmann: CSS-Problem

Beitrag lesen

@@Friedel

      $dateiname = basename($_SERVER['SCRIPT_NAME'],".php");
      if (file_exists($dateiname.".css")) echo "<link rel=\"stylesheet\" type=\"text/css\" href=\"".$dateiname.".css\">";
      if (file_exists($dateiname.".js")) echo "<script src=\"".$dateiname.".js\"></script>";

Von der Schreibweise mal abgesehen: du willst für jede deiner Seiten eine eigene Stylesheet-Datei und eine eigene Script-Datei laden?

Macht für mich keinen Sinn. Dinge, die auf der ganzen Website einheitlich aussehen sollen, sollten in einem Stylesheet angegeben werden, welches einmal geladen beim Aufruf einer anderen Seite schon im Browsercache ist.

Wenn einzelne Seiten zusätzlich Stile brauchen:

    <link rel="stylesheet" href="default.css"/>
<?php if (file_exists($dateiname.'.css')): ?>
    <link rel="stylesheet" href="<?= $dateiname ?>.css"/>
<?php endif; ?>

Aber auch das macht wenig Sinn. Die zusätzlichen Stile und Scripte müssen wohl nicht gecachet werden. Stattdessen kann man sie gleich ins HTML-Dokument schreiben und so HTTP-Requests sparen:

    <link rel="stylesheet" href="default.css"/>
<?php if (file_exists($dateiname.'.css')): ?>
    <style><?php readfile($dateiname.'css'); ?></style>
<?php endif; ?>
<?php if (file_exists($dateiname.'.js')): ?>
    <script><?php readfile($dateiname.'js'); ?></script>
<?php endif; ?>

LLAP 🖖

--
„Man kann sich halt nicht sicher sein“, sagt der Mann auf der Straße, „dass in einer Gruppe Flüchtlinge nicht auch Arschlöcher sind.“
„Stimmt wohl“, sagt das Känguru, „aber immerhin kann man sich sicher sein, dass in einer Gruppe Rassisten nur Arschlöcher sind.“

—Marc-Uwe Kling