Styling variabler

En "sexet" måde er at ændre baggrundsfarven på det tekst-felt der er galt.

Lidt hurtigt og utestet:

Som TOM skriver er JS en oplagt løsning, men skulle det være PHP er her et bud:

Mange tak for hjælpen. Men jeg fandt en meget mere enkel løsning :)

  {
    echo '<text style="color:red";>'."De 2 kodeord er ikke ens!".'</text>';
    $error=1;
    }

Enklere for dig, ikke for brugeren som skal vente på at svaret kommer tilbage fra serveren, og derefter bakke, og taste forfra.
- med js får hun at vide hvad der er galt med det samme, uden at forlade formen.

Men ronols version kan også klare de tilfælde hvor fejlen ikke kan afsløres "på forhånd" (med js)
- ideelt set skal du bruge begge løsninger : )

Kunne godt lyde rigtigt. Gav det et forsøg og endte ud med:


<?php

$submit= $_POST['register'];
if($submit=="Registrer")
{
    include('connect.php');
    $username = mysql_real_escape_string(strip_tags(htmlspecialchars($_POST['username'])));
    $password1 = md5($_POST['password1']);
    $pw1 = $_POST['password1'];
    $pw2 = $_POST['password2'];
    $error=0;
?>
<script type="text/javascript>
    function valider()
      {
      var fejl = false
      if (document.loginForm.navn == "")
        {
        fejl = "\nDu glemte navn"
        document.getElementById("NAVN").style.backgroundColor="#F88" // lyserød
        }
      if (document.loginForm.pw1.value != document.loginForm.pw2.value)
        {
        fejl = "\nDe 2 kodeord er ikke ens"
        document.getElementById("PW1").style.backgroundColor="#F88"
        document.getElementById("PW2").style.backgroundColor="#F88"
        }
      if (fejl != "")
        {
        alert("Fejl:"+fejl)
        return false
        }
      return true
      }

    <form name="loginForm" action="login.php" onsubmit="return valider()">
    <input type="text" name="navn" id="NAVN" onfocus="this.style.backgroundColor='#FFF'">
    <input type="password" name="pw1" id="PW1" onfocus="this.style.backgroundColor='#FFF'">
    <input type="password" name="pw2" id="PW2" onfocus="this.style.backgroundColor='#FFF'">
    </form>
    </script>

<?php
    $sql = "SELECT id FROM users WHERE username LIKE '".$username."'";
    $rs  = mysql_query($sql);
    if($rs)
    {
        if (mysql_num_rows($rs) == 1)
        {   
            echo  '<p style="color:red";>'."Brugernavnet er optaget!".'</p>';
        }   
    }
    if(!$error)
    {       
        $sql="INSERT INTO `users` (`username`,`password`) VALUES ('".$username."','".$password1."')";
        mysql_query($sql) or die(mysql_error());
        echo "Tillykke! Du er nu klar til at afslutte sætninger!";   

        echo "</body></html>";
        exit;
    }
}
?>

Det fungerer altså ikke. Når jeg ikke skriver noget i formen, skriver den bare "Tillykke! Du er nu klar til at afslutte sætninger!";  . Og jeg bliver redirectet til en ny hvid side. Var det meningen at jeg skulle blive i facebox formen? :)

Feltnavnene er helt hen i vejret.
Formen ligger inden i script-blokken.
- til at begynde med : )

#4 Man kan jo bare kode det, så man ikke skal "bakke". :)
Det vil sige, ved at lave det i 1 fil. Jeg laver altid min validering med PHP og outputtet heraf via CSS, som så meget andet styles med, istedet for JavaScript, da det ikke er nogen garanti, at det kører :)

<ole>

Brug JavaScript for brugerens skyld og PHP for din og din servers sikkerheds skyld. Er der en klient, der tilgår en af dine sider, og klienten ikke understøtter JavaScript, er der 99% sikkerhed for, det er en robot.

Det kan dog også være en af nettes 48 brugere, som slår JavaScript fra, men odds er næppe gode  *o)

/mvh
</bole>

Kan ikke finde ud af om jeg skal bruge PHP eller Javascript ?

Har prøvet begge ting, men ingen af delene ser ud til at virke.
<div style="display:none" id="register">
    <form action='<?php echo $_SERVER['PHP_SELF']?>' method='POST'>
        <table>
            <tr>
                <td>Brugernavn: </td>
                <td><input type="text" name="username" class="<?php echo $usernamecolor; ?>" value="<?php echo $username; ?>" ><br></td>
            </tr>
            <tr>
                <td>Password: </td>
                <td><input type="password" name="password1" class="<?php echo $pwcolor; ?>" value="<?php echo $pw1; ?>"><br></td>
            </tr>
            <tr>
                <td>Gentag password: </td>
                <td><input type="password" name="password2" class=" <?php echo $pwcolor; ?>" value="<?php echo $pw2; ?>"><br></td>
            </tr>
            <tr>
                <td><input name="register" type="submit" value="Registrer"></td>
                <td></td>
            </tr>
        </table>
        </form>
    <?php
        echo $error;
    ?>
</div>
<?php

$submit= $_POST['register'];
if($submit=="Registrer")
{
    include('connect.php');
    $username = mysql_real_escape_string(strip_tags(htmlspecialchars($_POST['username'])));
    $password1 = md5($_POST['password1']);
    $pw1 = $_POST['password1'];
    $pw2 = $_POST['password2'];
    $pwcolor="DefaultBgColor";
    $usernamecolor="DefaultBgColor";
    $error=0;

    if($pw1 != $pw2)
    {
    echo "De 2 kodeord er ikke ens!";
    $pwcolor = "ErrorBgColor";
    $error=1;
    }
    if($pw1 == "")
    {
      echo "Du har ikke indtastet et kodeord!";
    $pwcolor = "ErrorBgColor";
    $error=1;
    }
    if($username == "")
    {
      echo "Du har ikke indtastet et brugernavn!";
    $usernamecolor="ErrorBgColor";
    $error=1;
    }
    $sql = "SELECT id FROM users WHERE username LIKE '".$username."'";
    $rs  = mysql_query($sql);
    if($rs)
    {
        if (mysql_num_rows($rs) == 1)
        {   
            echo  "Brugernavnet er optaget!";
        }   
    }
    if(!$error)
    {       
        $sql="INSERT INTO `users` (`username`,`password`) VALUES ('".$username."','".$password1."')";
        mysql_query($sql) or die(mysql_error());
        echo "Tillykke! Du er nu klar til at afslutte sætninger!";   

        echo "</body></html>";
        exit;
    }
}
?>

Der er ingen rød/gul error nu. Samtidig bliver man redirectet over til en ny hvid side hvor der står "Tillykke! Du er nu klar til at afslutte sætninger!";?

//TumseMM

Og ja, jeg har sat css'en ind :) no worries ^^

det der skriger i mine øjne er :
du starter med at angive du ikke vil vise siden (<div style="display:none" id="register">)
du udskriver din html code, før du begynder at teste om der er fejl.

så start med at fjerne din style, og bytte om på test blokken og html blokken


nb: dine indlæg vl blive nemmere at læse / overskue, hvis du indrammer din code i en blå box

lukker

- bare sådan helt uden kommentarer af nogen art?