Meine Homepage! ;)

Thonixx

@Pion:

Bei einzeiligen PHP IF Anweisungen kannst du die { und } Zeichen auch weglassen, das finde ich leicht unnötig und unübersichtlich.
Und if(empty($content)){$content = ''; } ist auch totaler Schwachsinn, das braucht man nicht, denn du fragst ob sie leer ist und dann deklarierst du sie nochmal als leer !?

Und du hast ein paar Fehler drin.

So ist es richtig:

PHP

<?
$content = $_GET['content'];


if(!$content) include("startseite.php");
if($content == "bandit") include("dencontentfürdiebanditseite.php");
if($content == "xy") include("xy.php");
//etc
?>

driver

wobei ich bei sowas dann lieber ein switch nehme...

Bandit

Zitat von firefox5.0

So ist es richtig:

PHP

<?
$content = $_GET['content'];
 
if(!$content) include("startseite.php");
if($content == "bandit") include("dencontentfürdiebanditseite.php");
if($content == "xy") include("xy.php");
//etc
?>

Nein, so ist es nicht richtig! Was passiert, wenn kein Parameter übergeben wird oder dieser 0 ist? Und wie driver schon sagte, switch ist besser.

Thonixx

Zitat von bandit600

Nein, so ist es nicht richtig! Was passiert, wenn kein Parameter übergeben wird oder dieser 0 ist? Und wie driver schon sagte, switch ist besser.

Schritt für Schritt:
1. Zeile Einleitung PHP Code
3. Zeile Variable $content wird definiert und mit dem Wert von $_POST['conten'] gefüllt. Wenn diese 0 ist oder nichts hat, dann steht auch in der $content nichts.
4. Zeile Wenn nichts drin ist, dann wird startseite.php includiert
5. Zeile Wenn $content bandit drin hat, dann wird die Contentseite includiert
6. Zeile Wenn $content xy beinhaltet, dann wird xy.php includiert.

Na stimmt doch alles?? Was is daran falsch?

Und ich finde die Variante switch is einfach ne Alternative oder weswegen soll sie konkret besser sein?

Bandit

Wenn content nicht in der Url angegeben wird, bekommst du eine Notice-Meldung. Das ist unsauber programmiert! Und switch ist deutlich übersichtlicher und auch schneller. Und Short-Opentags sollte man auch nicht mehr nehmen.

PHP

<?php
   $content = isset($_GET['content']) ? $_GET['content'] : 'home';
 
   switch ($content)
   {
      case 'home':      $file = 'home.php';      break;
      case 'kontakt':   $file = 'kontakt.php';   break;
      case 'impressum': $file = 'impressum.php'; break;
      default:          $file = 'home.php';      break;
   }
 
   include($file);
 ?>

Alles anzeigen

Pion

Mensch leute die Diskussion haben wir doch JEDESMAL, die Lösung die ich zeige ist die einfachste zum erlernen , das es zig besser gibt weis ich, zudem finde ich switch auch nicht den hit, lieber nen reg Ausdruck

@bandit600
DEUTLICH SCHNELLER <- 0 Argument
DEUTLICH ÜBERSICHTLICHER <- Ich finde es unübersichtlicher

@minder
Ach ne;)

mfg

minder

sorry aber ist es nicht übersichtlicher wenn man die { } immer macht?

Coe

Hi,
also ich hab mir mal zeit genommen und ein neues Layout für meine Homepage gebastelt.
Ich würd mich freuen wenn Ihr vorbei gugen würdet.
Und vill schreibt ihr ja wie es euch so gefällt. Ihr könnt auch gerne was ins Gästebuch schreiben. Würde mich freuen.

http://www.unser-bandit.de

Danke!

lg Coe

Pion

Schau mal die Seite im ff an da kommt der Content ganz weit unten erst , unbedingt ändern

Coe

Jap das hatte ich bemerkt,
ich habs auch gleich geändert, ich hoffe das es jetzt richtig dargestellt wird.
Wäre nett wenn Du mir nochmal bescheid sagen könntest ob es auch bei Dir funktioniert.

Über andere Tipps oder Meinungen würde ich mich natürlich auch sehr freuen.

also dann
lg Coe

http://www.unser-bandit.de

Sweet_Angel

also ich habe wie auch vorher schon an dieser seite nichts auszusetzten, finde den hund süss

Coe

dankeschön! ;))))

yeti66

Trotz allem hat die Seite leider sehr viele technische Mängel!! Da wirst Du Dich nocheinmal mit auseinandersetzen müssen. codeseitig habe ich etwas aufgeräumt und die Seite horizontal und vertikal zentriert. Als nächstes sollte die Navigation sauber programmiert werde. Hier hast Du mal ein Ansatz zum Webseite basteln.

HTML

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta content="text/html; charset=iso-8859-1" http-equiv="Content-Type" />
<title>Unser-Bandit.de</title>
<style type="text/css">
<!--
html, body, div, span, applet, object, iframe, h1, h2, h3, h4, h5, h6, p, blockquote, pre, a, abbr, acronym, address, big, cite, code, del, dfn, em, font, img, ins, kbd, q, s, samp, small, strike, strong, sub, sup, tt, var, dl, dt, dd, ol, ul, li, fieldset, form, label, legend, table, caption, tbody, tfoot, thead, tr, th, td {
    margin: 0;
    padding: 0;
    border: 0;
    outline: 0;
    font-weight: inherit;
    font-style: inherit;
    font-size: 100%;
    font-family: inherit;
    vertical-align: baseline;
}
/*reset styles end*/
body {
    text-align:center;
    background-color:#ffffcc;
    scrollbar-face-color: #cccc99;
    scrollbar-shadow-color: #cccc99;
    scrollbar-highlight-color: #e1e2b8;
    scrollbar-3dlight-color: #e1e2b8;
    scrollbar-darkshadow-color: #e1e2b8;
    scrollbar-track-color: #e1e2b8;
    scrollbar-arrow-color: #e1e2b8;
}
#page {
    background: url(pic/background.jpg) no-repeat;
    width: 989px;
    height: 555px;/* fuer horizontale und vertikale zentrierung */
    position: absolute;
    left:50%;
    margin-left:-494px;  /* Um die Hälfte der Breite nach links schieben */
    top:50%;
    margin-top:-277px;  /* Um die Hälfte der Höhe nach oben schieben */
}
#navi {
    font-family: AaronBecker;
    position:absolute;
    margin-left: 368px;
    height:25px;
    line-height:25px;
    margin-top: 85px;
    width: 605px;
}
#content {
    position:absolute;
    width: 625px;
    height: 350px;
    overflow: auto;
    margin-top: 130px;
    margin-left: 345px;
}
#counter {
    position:absolute;
    margin-top: 490px;
    margin-left: 20px;
}
-->
</style>
</head>
<body >
<div id="page">
  <div id="navi"> <a href="">Home</a> <a href="">Neues</a> <a href="">Bandit</a> <a href="">Tagebuch</a> <a href="">Galerie</a> <a href="">Freunde</a> <a href="">Webmiss</a> <a href="">Of
    Angelhearts</a> <a href="http://650326.guestbook.onetwomax.de/?vd=013005" >G&auml;stebuch</a> <a href="">Kontakt</a> <a href="">Links</a></div>
  <div id="content">
    <p> Herzlich Willkommen auf Unser-Bandit.de!<br />
      Hier gibt es eine Menge &uuml;ber mich zu lesen. Fotos gibt es auch immer die neuesten.<br />
      Viel Spa&szlig; beim Herumschnuppern!<br />
      &Uuml;ber einen Pfotenabdruck im G&auml;stebuch w&uuml;rden wir uns freuen!<br />
      lg Corinna und Bandit</p>
    <img style="width: 401px; height: 354px;" alt="" src="pic/home.JPG" /><br />
  </div>
  <div id="counter">
    <!--Instant Counter Anfang-->
    <script language="JavaScript" type="text/javascript" src="55201-graylight-de.js"></script>
    <noscript>
    <a
href="http://www.phpwelt.net/instantcounterstats-55201-0-0.html"><img
border="0"
src="http://instant-counter.phpwelt.net/55201-graylight-de.png"></a>
    </noscript>
    <!--Instant Counter Ende-->
  </div>
</div>
</body>
</html>

Alles anzeigen

Die Fotolinks stimmen nicht mit Deinem überein, da ich sie mir zu besseren Ansicht heruntergeladen hatte.8) Vielleicht hilft es Dir ja weiter.........?

Coe

Neues Layout,
ich hab mal wieder ein neues Layout gebastelt und wollte wissen wie ihr es findet. Diesmal habe ich es auch mittig gesetzt.
Über eure Meinungen und Verbesserungsvorschläge würde ich mich freuen.
Danke schon mal im vorraus.
lg Coe

Elvis

Quelltext: 6-

Abzüge im Frontend für:

Fehlende mittige Ausrichtung
Unnötige Scrollbalken
Überflüssigen Counter
Unterschiedliche Schriftgrößen/-arten
Schlechte Kontraste (Lesbarkeit)
Zu kleine Schrift

Garry

Nettes Thema, süßer Hund, schönes Bild auf der Startseite. Aber...

Unbedingt eine andere Schriftart verwenden. Meine Vorschläge: Verdana, Tahoma, Arial
Seite so optimieren, dass der User nicht unnötig nach rechts scrollen muss um dann zu merken, dass dis unnötig war
Links (Navigation) mehr hervorheben, die geht bei der Schriftgröße fast unter!
Achte ein wenig auf die Rechtschreibung, das schreckt sonst unnötig ab

Coe

hey hey,

also ich hab mal wieder ein neues layout für meine Homepage online gestellt. Da hier immer wieder gesagt wurde, die layouts mit i-frame effekt nicht so gut sind, habe ich diesmal ein anderes gemacht.
Vielleicht gugt ihr einfach mal vorbei und schreibt mir dann wie euch das neue layout gefällt.

also dann

liebe grüße
Coe

http://www.unser-bandit.de

Alienx

Hio,

finde das Layout sehr ansprechend. Würde aber dringend empfehlen, dem text links und rechts ca. 20px Abstand vom Rand zu lassen. Die Navis könnten auch nen Hover-Effekt vertragen. Der Counter sollte ebenfalls da weg. Sonst gut. : )

MfG Alienx

Dommix

Die Farben passen gut zusammen, aber mir ist der gesamte Header (Menüs, Hund) einfach zu groß.

yeti66

Optisch würde ich sagen hast Du mit Deiner Seite einen Sprung gemacht. Leider hinkt der Quellcode der Optik meilenweit hinterher. Du hast die Struktur einer Webseite in eine andere kopiert und die Trennung von Inhalt und Layout noch nicht verinnerlicht.

HTML

...<big><big><big><span style="font-family: Times New Roman,Times,serif;">&nbsp;Name:</span></big></big></big></span><big><big style="font-family: Times New Roman,Times,serif;"><big> Corinna </big></big></big></span></span></span></span></small></small></small><br style="color: rgb(0, 0, 0);">

Deine Seite ließe sich dabei so einfach darstellen. Die Verwendung von Verweis-sensitiven Grafiken ist auch nicht das Mittel der Wahl für Navigation dieser Art. Das große Wunder ist aber, das die Seite in allen IE ab5.5 gleich dargestellt wird, soweit ich sehen konnte. Deshalb würde ich sagen ein sauberes Layout und aus der Seite wird noch was.

Meine Homepage! ;)

Tags

Benutzer online in diesem Thema