www.sgleimbach.de

veritas696 · 20 April 2010

Hallo zusammen,

http://www.sgleimbach.de

hab euch vor einiger Zeit schon ein bissl genervt um mir bei meiner Seite zu helfen.
Nun hab ich sie größtenteils fertiggestellt und würde mich freuen wenn ihr Sie nach den Gesichtspunkten:

Layout/Design
Sicherheit
Semantik
quelltext etc.

bewerten könntet und ihr mir noch ein paar Tipps für die Weiterfühurng der Seite geben könnt.

Vielen dank im Vorraus :-o

Edit: weiß jemand warum das kontaktformular net funktioniert?

veritas696 · 20 April 2010

44 hits, keiner schreibt was... nichts zu beanstanden? oder seit ihr alle blind geworden auf der seite oder es ist hopfen und malz verloren :mrgreen: ?

ky_fr34k · 21 April 2010

ich glaub das mit dem blind werden trifft zu ;)
nein war nur Spaß.
so schlimm ist es nicht.
Aber wirklich schön ist es auch nicht!
Ich mag ja blau, aber das ist eindeutig zu viel.
Logo (Header) und Navi sind ok, aber der Hintergrund der Textboxen ist dann einfach zu viel.
Versuche doch mal diese komischen Schriftrollen-Dinger zu entfernen (stattdessen vll nur abgerundete Ecken), und Hintergrundfarbe weiß mit "blauer" (dunkel) Schrift.
Mit dem Hintergrund kann ich mich irgendwie nicht anfreunden (kann dir aber nicht sagen wieso).
Ladezeiten sind Top.
Quelltext kann ich nicht genau sagen, da ich meine Tools grad net da hab, sieht aber einigermaßen ok aus.

Soweit erstmal.
Schau mal was dir hilft.

Gruß KY

Cryptogott · 21 April 2010

Hallo,

die Seite sieht aus als hätte ich bei meinem Monitorkalibrierungsdaten den Rot - und Blaukanal entfernt. Schau dir mal ein paar seriöse Sportverein Seiten an und vergleiche. Die Seite ist total überladen von Grafiken und Effekten aus den 90ern: Angefangen beim Hintergrund bis hin zu den "Glow"-Effekten beim Text, die sich alles andere als harmonisch einfügen und kein bisschen zum Serifen-Font passen. Dass der Verein die Farben rot und blau als Markenzeichen hat, muss man nicht in jedem kleinsten Detail demonstrieren. Auch störend ins Auge fällt das mit Effekten überladene "Logo".

Semantik und Code sind dafür nicht schlecht.

Greetz

Gimli · 21 April 2010

Wikipedia schrieb:
Die erste Webseite wurde am 13. November 1990 von Tim Berners-Lee auf einem NeXT-Rechner verfasst und veröffentlicht – einen Tag, nachdem dieser gemeinsam mit Robert Cailliau ein neugefasstes Konzept für ein weltweites Hypertext-Projekt veröffentlicht hatte.

Hmm ja in den 90ern gabs ja schon soo viele Webseiten. Schlechter vergleich! Aber die Website wirkt wirklich sehr Kitschig

vitus37 · 21 April 2010

Heyho,

das Design der Seite finde ich persönlich schlecht. Überall ist ein Blau-Ton drin. Diese Boxen von Navi, Header und Überschrift finde ich auch nicht schön, da mich dieser Rahmen dort stört. Auch mit dem Hintergrund kann ich mich nicht anfreunden, der passt nicht so recht dazu. Was ich auch noch schrecklich finde, sind diese Schriftrollen auf der Startseite, sie passen überhaupt nicht zum Thema. Das Wappen/Logo begegnet mir beim ersten Blick auf der Startseite 4 mal, das finde ich übertrieben.

Den Aufbau der Seite finde ich in Ordnung, wobei ich die 4 Links unter dem Header noch zur Hauptnavigation hinzugefügt hätte.

Den Code finde ich gut.

Fazit:
Ansich garnicht so schlecht, nur am Design würde ich noch arbeiten. ;)

Liebe Grüße

ARatedChris · 21 April 2010

Hey, da ich ja eher ein HTML-Anfänger bin sage ich mal was zum Design was mir mehr liegt:

Die Farben eures Vereins sind Rot & Blau - die kann man gut kombinieren.

Das Blöde ist leider dass diese beiden Töne die du verwendet hast nicht wirklich harmonieren.
Was ich machen würde wäre, was auch "in" bzw. "most-used" ist auf Pastell-Töne zu setzen.

Anstatt der Farben die jetzt verwendet werden vielleicht diese hier:

Habe das Logo auch draufgepackt aber is leider zu gross zum hochladen, kann dir aber auch so sagen dass euer Logo bzw Wappen zu den Farben passen würde.

Wie gesagt, es ist meine Meinung, ich denke solche Pastelltöne kommen mehr an als die direkten Farben wie diese oder diese

Loon3y · 21 April 2010

Hallo.

Man muss ja auch mal anderen die gelegenheit geben zu a ntworten. Sonst antworten immer nur die gleichen personen. Das wird auf dauer anstrengend und langweilig. Ist ja dann doch immer mit recht viel Zeitaufwand verbunden, etwas auf herz und nieren zu prüfen.

Über das Design sag ich mal nichts, wirkt recht altmodisch und irgendwie...nicht professionell (was keine beleidigung ist, sondern einfach ein eindruck meinerseits).

Zu dem Code würde ich gerne mehr sagen...

anfänglich verspricht er viel (doctype xhtml 1.0 strict sagt schon einiges aus).

Allerdings lässts dann auch Stark nach.

- Fehlende Überschrift <h1>
- ein paar Syntakfehler
- ein paar logische Fehler
- ein <h6> ohne das ein <h1>, 2, 3, 4, <h5> davor kommt
- viele viele Klassen (versuchen über selektoren zu arbeiten)
- teilweise keine logischen klassen / id's.
- fehlende alt-attribute bei <img>
- kommentare im Quelltext
- Text wandert aus Boxen / Bilder beim Zoom
- Bilder die Inhalt enthalten als Background eingebunden und keine alternative angeboten
- Anschrift ist eher <address> wie tabellarische Daten
- typische table-fehler wie align="irgendwas" und valgin="nochwas"...das ist css.

hmmm..sry muss in 5mins los zu einen Termin. Ich führe das gerne weiter, wenn du oben die Fehler dir mal angeschaut hast :-)

Gruß
Loon3y

veritas696 · 21 April 2010

Erstmal vielen dank fürs Feedback :)
muss dazusagen, daß das meine erste selbsterstellte Website ist, die mag das eine oder andere vielleicht entschuldbar sein ;)
Das mit dem "Kitschig" is mir auch aufgefallen, aber ich habs mittlerweile ein wenig "entschärft" hoffe ich :( Hab also die serifen schrfit rausgenommen.
Das mit den Pasteltönen probier ich mal aus. Der Hintergrund gefällt mir auch noch nicht so recht, mir is aber noch keine alternative eingefallen.

Als H1 versteht sich im Prinzip "SG Leimbach" aus dem Header, leider weiß ich nicht wie ich es trotzdem als h1 definiere wenn das Bild selbst ja über css eingebunden ist. Sollt ich es lieber als "img" ein binden? macht man daß dann so: <h1><img src="..." alt="SG Leimbach"/><h1> ?

zu "alt" der images: Was gibt man denn als alternativtext für beispielsweise das Wappen ein. Wenn jemand keine Bilder im Browser anzeigen kann, will ich eigentlich, daß dann nix anstatt des Wappens angezeigt wird, ähnlich wie bei anderen bildern, hmm...

vitus37 · 21 April 2010

ich kann dir sagen, dafür, dass das deine erste Seite ist, ist die verdammt gut! Du scheinst HTML mehr oder weniger verstanden zu haben. Andere kommen hier mit Frames, Tabellenlayouts und DIV-Suppe an, im Gegensatz zu dir. ;)

Wenn du Text als eine Überschrift hast, dann füge ihn mit <h1>überschrift</h1> ein. Ein Logo oder ähnliches, kann man (finde ich) als Inhalt einstufen, und somit mit dem <img>-tag einbinden. Alles andere, die Zirde usw., wird per background eingebunden.

Als Alternativtext gibt man das an, was auf dem Bild zu sehen ist. Im Falle des Wappens eben "Wappen des SG Leimbach" und evtl. noch genauere Bildbschreibung.

Liebe Grüße
und gute Nacht
Vitus

Loon3y · 22 April 2010

vitus37 schrieb:
Wenn du Text als eine Überschrift hast, dann füge ihn mit <h1>überschrift</h1> ein. Ein Logo oder ähnliches, kann man (finde ich) als Inhalt einstufen, und somit mit dem <img>-tag einbinden. Alles andere, die Zirde usw., wird per background eingebunden.

es muss sogar als <img> eingebunden werden, da es contentrelevante inhalte enthält. Was man alternativ machen könnte wäre, dass man es als background einbindet und den Text in ein <span> und mit z-index dahinter setzt.

Gruß
Loon3y

MasteR ChieF · 22 April 2010

Erster Eindruck der Seite: (wie bereits erwähnt) zuviel Blau
ausserdem habe ich das Gefühl, das die einzelnen Elemente zwar strukturiert sind, aber irgendwie "stehen sie da einfach rum". Ein Rahmen um den Inhalt fände ich noch passend, das man das ganze ein wenig eingrenzen könnte.

Was mich persönlich auch "stört" ist, das der Background Fix ist, ich mag das irgendwie nicht.

Versch doch noch ein Footer einzubauen, ich habe das Gefühlt, das die Seite nach unten einfach aufhört... und das da noch was kommen sollte. Ein einfacher Footer mit wenigen Link, evtl. Copyright angaben. Würde schon reichen. Dann wirkt die ganze Seite schon stukturierter.

Code sieht sehr sauber aus und nicht schlecht für das erste mal :mrgreen:

MFG
MasterChief

JensB · 28 April 2010

Also ich finde den Hintergrund auch etwas zu blau, versuch doch mal, den weiß zu machen mit gaaaaanz leichter Blauspur, das reicht oft, denn die anderen blauen Elemente verstärken dann die Farbwirkung. Sähe schicker aus finde ich.

veritas696 · 24 März 2011

Hallo zusammen,

möchte den Thread mal kurz wiederbeleben da ich am design der seite noch ein bissl rumgeschreubt hab. Könnt ihr mir vielleicht ein paar weitere tipps geben oder was eventuell noch nicht gut rüberkommt!?

SG Leimbach

aJunkie · 24 März 2011

Den /includes Ordner hast du geschützt. Das ist gut.
Hätte nicht gewusst, dass es diesen gibt, wenn du die Fehlermeldungen zur Sicherheit unterdrücken würdest.
Habe eine Fehlermeldung provoziert, indem ich es mit SQL Injections versucht habe. Deine Bildergalerie scheint da noch paar Lücken zu haben.

Warum dein Kontaktformular nicht funktioniert, sage ich dir, wenn du deinen Code zeigst.

Oder hier: http://www.sgleimbach.de/teams.php?content=&id=

Da erscheinen Fehlermeldungen. Da wären paar if und is_ Abfragen angebracht. Ganz wichtig: Escapen. Du holst $id direkt in den Script.

veritas696 · 25 März 2011

vielen Danke erstmal,

die Fehlermeldungen hab ich jetzt soweit unterdrückt (Saison, Teams, Spielberichte).

kannst du mir das mit der Bildergalerie genauer erklären, wo können dabei Probleme auftreten? Was sind denn SQL Injections? Muss dazu sagen daß ich keine MYSql Datenbank benutze.

Also momentan tätige ich folgende Abfrage für die Variablen (beispiel teams.php):

Code:

<?php
          if (isset($_GET['content']) and isset($_GET['id']))
          {
            if(($_GET['id'] < 0) or ($_GET['id'] > 3))
            {
              $id = 1;
            }
            else
            {
              $id = $_GET['id'];
            }
              $datei = 'includes/team'.$id; // dateiname konstruieren

              switch($_GET['content'])
              {
                case "kader" : $datei = $datei.'kader.php'; break;
                case "foto"  : $datei = $datei.'foto.php';  break;
                default      : $datei = $datei.'foto.php';  break;
              }
              @include($datei);


          }
          else
          {
            include("includes/team1foto.php");
          }
        ?>

wie kann ich denn nach einer leeren Variable fragen?

Du holst $id direkt in den Script.

mach ich das? und is das gut oder schlecht?
Ich muss dazu sagen, daß ich ziemlicher Anfänger auf dem Gebiet von php bin :-?

das Kontakformular:

Code:

<?php

$mail_to   = '[email protected]'; // Ihre Mailadresse (Empfängeradresse)

$from_name    = GetParam('fromname');
$from_mail    = strtolower(GetParam('frommail'));
$mail_subject = GetParam('mailsubject');
$mail_text    = GetParam('mailtext');
$send         = GetParam('s');
$err_text='';

if(trim($from_name)=='') $err_text.='Bitte geben Sie Ihren Namen an.<br>';
if(trim($from_mail)=='') $err_text.='Bitte geben Sie Ihre E-Mail-Adresse an.<br>';
else
  if(!preg_match('/^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,6})$/i',$from_mail))
    $err_text.='Bitte geben Sie eine gültige E-Mail-Adresse an.<br>';

if(trim($mail_subject)=='')  $err_text.='Bitte geben Sie einen Betreff ein.<br>';
if(trim($mail_text)=='')     $err_text.='Bitte geben Sie einen Nachrichtentext ein.<br>';

// *** Zu langen Text abschneiden ***
if(strlen($mail_text)>1000)
{
  $mail_text=substr($mail_text,0,1000).'... (Text wurde gekürzt!)';
}
$from_name    = str_replace(chr(34),"''",$from_name);
$mail_subject = str_replace(chr(34),"''",$mail_subject);
$from_name    = stripslashes($from_name);
$from_mail    = stripslashes($from_mail);
$mail_subject = stripslashes($mail_subject);
$mail_text    = stripslashes($mail_text);

if (($send == '1') && ($err_text != '')) {
  echo '<p><big><b>Fehler:</b></big><br>';
  echo $err_text.'</p>';
}

if (($send != '1') || ($err_text != ''))
{
?>

<form action="<?=GetParam('PHP_SELF','S')?>" method="post">
  <table class="kontaktformular" border="0" cellspacing="2" cellpadding="0" >
    <tr>
      <td class="kleft">Ihr Name:</td>
      <td class="kright">
        <input type="text" name="fromname" size=53 maxlength=120 value="<?=$from_name?>">
      </td>
    </tr>
    <tr>
      <td class="kleft">Ihre E-Mail-Adresse:</td>
      <td class="kright">
        <input type="text" name="frommail" size=53 maxlength=120 value="<?=$from_mail?>">
      </td>
    </tr>
    <tr>
      <td class="kleft">Betreff:</td>
      <td class="kright">
        <input type="text" name="mailsubject" size=53 maxlength=120 value="<?=$mail_subject?>">
      </td>
    </tr>
    <tr>
      <td class="kleft" valign=top>Text:<br /><span class="kursiv">(max. 1.000 Zeichen)</span></td>
      <td class="kright">
        <textarea cols=40 rows=10 name="mailtext"><?=$mail_text?></textarea>
      </td>
    </tr>
    <tr>
      <td></td>
      <td>
        <input type="hidden" value="1" name="s">
        <input type="submit" value="Nachricht versenden" name="submit">
      </td>
    </tr>
  </table>
</form>

<?php
} else {
  $header="From: $from_name <$from_mail>\n";
  $header.="Reply-To: $from_mail\n";
  $header.="X-Mailer: PHP-ContactForm-Script\n";
  $header.="Content-Type: text/plain";
  $mail_date=gmdate('D, d M Y H:i:s').' +0000';
  $send=0;
  /*echo "$mail_to, $mail_subject, $mail_text, $header";*/
  if(@mail($mail_to,$mail_subject,$mail_text,$header))
  {
    echo "<p>Die Nachricht wurde erfolgreich abgesendet.</p>";
    echo "<p><a class=\"vereinslink\" href=\"kontakt.php\">Zurück zum Formular</a></p>";
  }else
  {
    echo "<p>Beim Versenden der Nachricht ist ein Fehler aufgetreten!</p>";
    echo "<p><a class=\"vereinslink\" href=\"kontakt.php\">Zurück zum Formular</a></p>";
  }
}

function GetParam($ParamName, $Method = 'P', $DefaultValue = '') {
  if ($Method == 'P') {
    if (isset($_POST[$ParamName])) return $_POST[$ParamName]; else return $DefaultValue;
  } else if ($Method == 'G') {
    if (isset($_GET[$ParamName])) return $_GET[$ParamName]; else return $DefaultValue;
  } else if ($Method == 'S') {
    if (isset($_SERVER[$ParamName])) return $_SERVER[$ParamName]; else return $DefaultValue;
  }
}
?>

beste grüße

aJunkie · 26 März 2011

PHP:

if(empty($var) { echo 'Variable ist leer!' }

Schreib' deine Funktion GetParam mal ganz oben hin. Noch über dem Script.

www.sgleimbach.de

veritas696

Neues Mitglied

veritas696

Neues Mitglied

ky_fr34k

Mitglied

Cryptogott

Neues Mitglied

Gimli

Mitglied

vitus37

Senior HTML'ler

ARatedChris

Guest

Loon3y

Neues Mitglied

veritas696

Neues Mitglied

vitus37

Senior HTML'ler

Loon3y

Neues Mitglied

MasteR ChieF

Mitglied

JensB

Neues Mitglied

veritas696

Neues Mitglied

aJunkie

Mitglied

veritas696

Neues Mitglied

aJunkie

Mitglied

Neueste Beiträge