php - måder at skrive koden på - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php

Bruger #16176 @ 01.11.10 14:14

Jeg har denne her kodestump liggende i min indsættelses fil.

Men kan den skrives kortere? jeg har flere liggende i filen, en for hver variabel i registreringen. måske de kan blive flettet sammen på en måde så det ikke fylder så meget.

Vil gerne have det på denne måde med at den kommer med de fejlmeddelelser og de enkelte regler for hver variabel. Så måske der ikke er andre muligheder.

Kode

if(empty($brugernavn)) {
    echo ("<script type=\"text/javascript\">
         alert('Du skal indtaste brugernavn!');
         </script>");
    $errorCount++;
}       
        
if(!isset($brugernavn)) return false;
    function bnavn_check($brugernavn) {
    $bnavn = '/^[A-zÆØÅæøå0-9]{2,30}$/';
            
    if(!preg_match($bnavn, $brugernavn))
    return false;
    return true;
}
                
if(!bnavn_check($_POST['brugernavn'])) {
    echo("<script type=\"text/javascript\">
        alert('Dit brugernavn må kun indeholde bogstaver og tal!');
        </script>");
        $errorCount++;            
}

Indlæg senest redigeret d. 01.11.2010 14:18 af Bruger #16176

Bruger #10216 @ 01.11.10 19:43

4.283

Eftersom at man altid skal holde sig fra at bland PHP logik med HTML, så vil jeg nok gå med noget der ligner følgende:

PHP kode

 <?php
 $variables = array(
     'brugernavn' => array(
         'match' => '/^[A-zÆØÅæøå0-9]{2,30}$/',
         'error_message' => 'Dit brugernavn må kun indeholde bogstaver og tal!',
         'empty_message' => 'Du skal indtaste brugernavn!'
     ),
     'password' => array(
         'match' => '/^[A-zÆØÅæøå0-9]{2,30}$/',
         'error_message' => 'Din adgangskode må kun indeholde bogstaver og tal!',
         'empty_message' => 'Du skal indtaste adgangskode!'
     )
 );
 
 foreach($variables as $ident => $meta) {
     if(!isset($_POST[$ident]) || empty($_POST[$ident])) {
         $errors[] = $meta['empty_message'];
     }
     if(!preg_match($meta['match'],$_POST[$ident])) {
         $errors[] = $meta['error_message'];
     }
 }
 
 $error_count = sizeof($errors);
 ?>
 
 <html>
     <body>
         <?php if($error_count > 0): ?>
             <?php foreach($errors as $err): ?>
                 <script> alert('<?php echo $err?>');</script>
             <?php endforeach; ?>
         <?php endif; ?>
     </body>
 </html>

Indlæg senest redigeret d. 02.11.2010 05:49 af Bruger #10216

måder at skrive koden på

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler