Alleen cijfers?

T.Nijborg

Heey allemaal,

Ik heb een website waar je gratis kan hosten. Maar bij het registeren heb ik een optie bij gemaakt. Namelijk je telefoon nummer invoeren. Dit om bijvoorbeeld een sms te sturen indien je website eruit ligt of iets. (dit gaat geheel automatisch).

Maar nu wil ik ervoor zorgen dat je alleen getallen mag invoeren. Maar dat lukt met niet. Als ik het doe zegt hij tijdens het registeren steeds mag alleen getallen invullen terwijl er getallen staan. Dit is het stukje waar het om gaat:( telefoonnummer invoeren in niet verplicht trouwens)

PHP

if( preg_match( '/^[0-9\-]+$/', $telefoonnummer ) == 0 ) {
	
		$error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.<br />'; 
	
	}

Als iemand weet hoe het wel werkt, hoor ik het graag.

Darsstar

PHP

if( ! empty( $telefoonnummer ) AND ! ctype_digit( $telefoonnummer ) ) {
    
        $error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.'; 
    
    }

Nu kom je pas bij de controlle of het alleen cijfers zijn als $telefoonnummer niet leeg is.
Zo is het dus optioneel.

T.Nijborg

Nee, zo krijg je dan het telefoonnummer verplicht is. Maar dat is niet de bedoeling. Het telefoon nummer moet niet verplicht zijn. Maar er mogen wel alleen maar cijfer in het input veld zijn.

L.Groot

Dan maak je een OR erin. Dus if((isset($_POST['veld'])) && ctype_digit($_POST['veld'])) !isset($_POST['veld'])){.

habib10

doe dit en het zal werken

PHP

if(! ctype_digit($_POST['HIER']))
 echo"alleen cijfers";

L.Groot

habib10,
Nee dat is hardstikke fout. Je controleert nu of het niet allemaal uit getallen bestaat, echter bestaat een lege input (niks ingevuld) niet uit een cijfer / integer dus werkt jouw manier voor de TS niet. Als het goed is werkt mijn manier wel.

Malik

PHP

if ( isset($_POST['telefoonnummer']) && !ctype_digit($_POST['telefoonnummer']) )
{
	$error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.'; 
}
else {
	// hier kan dan je dan verder gaan
}

Dan maar deze?

L.Groot

@Malik,
Ook verkeerd, nu geef je een error als de input uit allemaal cijfers bestaat...

Edit: ik zie dat je net een ! hebt toegevoegd, dit had je eerst niet nl.

Malik

Nee hoor

haha maar ok mijn code is goed!

Tarwewijk

Als je er echt niet uitkomt doe je ditxD

PHP

if(! ctype_digit($_POST['HIER']) AND $_POST['hier'] != '')
 echo"alleen cijfers";

Dan werkt hij als het goed is(K)

T.Nijborg

Ja dat zou normaal werken in een nogal normaal script:P, maar mijn script nogal anders in elkaar.

PHP

if( preg_match( '/^[0-9-]+$/', $telefoonnummer ) == 0 ) { 
     
        $error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.';  
     
    }

Zo ongeveer moet het, maar dan krijg je continu als je als nog een nummer invoert: je mag alleen cijfers invoeren. Die melding krijg je gewoon elke keer. Hoe dit op te lossen?

Tim

PHP

<?php
if(isset($_POST['telephone_number']) && !empty($_POST['telephone_number']) && !ctype_digit($_POST['telephone_number'])) {
	//invalid phone number
}
?>

Tarwewijk

PHP

if(! ctype_digit($telefoonnummer) AND $telefoonnummer != '')
 echo"alleen cijfers";

Zo doet hij het(K)

T.Nijborg

@brokenTrack, ten eerste word het telefoon nummer later pas opgeslagen in de database.
En ten tweede je telefoon nummer invoeren is niet verplicht. Zie hier onder mijn gehele aanmelden.php:(hoe die nu is en hoe die het dus niet doet;))

PHP

<?php
/************************************************************
 *    Om deze pagina te zien moet je uitgelogd zijn
************************************************************/
if( max_toegang( 10 ) ) {


/************************************************************
 *    Registreren
************************************************************/
if( isset( $_POST[ 'submit' ] ) ) {


    $core[ 'tpl' ]->assign( 'submit', 1 );
    
    $gebruikersnaam            = ( isset( $_POST[ 'gebruikersnaam' ] ) ) ? $_POST[ 'gebruikersnaam' ] : '';
    $telefoonnummer            = ( isset( $_POST[ 'telefoonnummer' ] ) ) ? $_POST[ 'telefoonnummer' ] : '';
    $wachtwoord                = ( isset( $_POST[ 'wachtwoord' ] ) ) ? $_POST[ 'wachtwoord' ] : '';
    $wachtwoord_controle    = ( isset( $_POST[ 'wachtwoord_controle' ] ) ) ? $_POST[ 'wachtwoord_controle' ] : '';
    $email                    = ( isset( $_POST[ 'email' ] ) ) ? $_POST[ 'email' ] : '';
    $type                    = ( isset( $_POST[ 'type' ] ) && is_numeric( $_POST[ 'type' ] ) && $_POST[ 'type' ] < 4 && $_POST[ 'type' ] > 0 ) ? $_POST[ 'type' ] : '1';
    $referral                = ( isset( $_POST[ 'referral' ] ) ) ? $_POST[ 'referral' ] : '';
    
    
    $error = '';
    
    if( preg_match( '/^[a-z0-9-]+$/', $gebruikersnaam ) == 0 ) {
    
        $error .= '- Je gebruikersnaam mag alleen uit a-z en 0-9 en bestaan. Dus geen hoofdletters!.'; 
    
    }
    if( empty( $gebruikersnaam ) ) {
    
        $error .= '- Vul een gebruikersnaam in.'; 
        
    }
    if( preg_match( '/^[0-9-]+$/', $telefoonnummer ) == 0 ) { 
     
        $error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.';  
     
    }
    if( empty( $wachtwoord ) ) {
    
        $error .= '- Vul een wachtwoord in.'; 
        
    }
    if( $wachtwoord != $wachtwoord_controle ) {
    
        $error .= '- De wachtwoorden komen niet overeen.'; 
        
    }
    if( preg_match( '/^.+@.+..+$/', $email ) == 0 ) {
    
        $error .= '- Vul een geldig emailadres in.'; 
    
    }
    
    $query = mysql_query( 'SELECT id FROM leden WHERE gebruikersnaam = "' . mysql_real_escape_string( $gebruikersnaam ) . '" LIMIT 1' ) or die( mysql_error( ) );
    
    if( mysql_num_rows( $query ) > 0 ) {
    
        $error .= '- Deze gebruikersnaam bestaat al.';
    
    }




    
    $query = mysql_query( 'SELECT id FROM leden WHERE ip = "' . getIPAddress( ) . '" LIMIT 1' ) or die( mysql_error( ) );
    
    if( mysql_num_rows( $query ) > 0 ) {
    
        $error .= '- Er heeft al eens iemand zich aangemeld met dit IP, er is maximaal 1 account per IP toegestaan.';
    
    }
    
    $query = mysql_query( 'SELECT id FROM leden WHERE email = "' . mysql_real_escape_string( $email ) . '" LIMIT 1' ) or die( mysql_error( ) );
    
    if( mysql_num_rows( $query ) > 0 ) {
    
        $error .= '- Er heeft al eens iemand zich aangemeld met dit email adres, er is maximaal 1 account per email adres toegestaan.';
    
    }
    
    if( $error != '' ) {
        
        $core[ 'tpl' ]->assign( 'fouten', $error );
        
    } else {
        
        $code = gen_code( );
        
        $referral_id = ( isset( $_SESSION[ 'ref_id' ] ) ) ? $_SESSION[ 'ref_id' ] : '0';
        
        $q1 = mysql_query( 'INSERT INTO leden ( gebruikersnaam, telefoonnummer, wachtwoord, email, geactiveerd, activatiecode, ip, referral_id ) VALUES ( "' . mysql_real_escape_string( $gebruikersnaam ) . '", "' . mysql_real_escape_string( $telefoonnummer) . '", "' . mysql_real_escape_string( $wachtwoord ) . '", "' . mysql_real_escape_string( $email ) . '", "1", "' . $code . '", "' . getIPAddress( ) . '", "' . $referral_id . '" )' ) or die( mysql_error( ) );
        
        $subject = $core[ 'conf' ][ 'site' ][ 'naam' ] . ' - Account gegevens';
        $message = '
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
    <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
    <style>
    * {
        outline: none;
        margin: 0;
        padding: 0;
    }
    
    body {
        text-align: center;
        margin-top: 10px;
        font-size: 12px;
        font-family: "Trebuchet MS", Verdana, Arial;
    }
    
    #container {
        text-align: left;
        width: 500px;
        margin: 0 auto;
        border: 1px dotted #969696;
        padding: 10px;
    }
    </style>
    <title>' . $core[ 'conf' ][ 'site' ][ 'naam' ] . ' - Account gegevens</title>
</head>
<body>
<div id="container">
    Beste ' . htmlspecialchars( $gebruikersnaam ) . ',
    Leuk dat je je hebt aangemeld op ' . $core[ 'conf' ][ 'site' ][ 'naam' ] . '!
    Je kan nu gelijk inloggen!<br/><br>
        Veel plezier en succes!<br/><br/>
    -Het team van Hostvoorfree.net!
</div>
</body>
</html>
        ';


        $headers  = 'MIME-Version: 1.0' . "rn";
        $headers .= 'Content-type: text/html; charset=iso-8859-1' . "rn";
        $headers .= 'To: ' . $gebruikersnaam . '<' . $email . '>' . "rn";
        $headers .= 'From: ' . $core[ 'conf' ][ 'site' ][ 'naam' ] . ' <' . $core[ 'conf' ][ 'site' ][ 'email' ] . '>' . "rn";


        mail( $email, $subject, $message, $headers );
        
        $core[ 'tpl' ]->assign( 'succes', 1 );
    
    }
    
} else {
    
    if( isset( $_GET[ 'id' ] ) && is_numeric( $_GET[ 'id' ] ) ) {
        
        $query = mysql_query( 'SELECT gebruikersnaam FROM leden WHERE id = "' . mysql_real_escape_string( $_GET[ 'id' ] ) . '" LIMIT 1' ) or die( mysql_error( ) );
        
        if( mysql_num_rows( $query ) == 1 ) {
            
            $_SESSION[ 'ref_id' ] = $_GET[ 'id' ];
            
            $fetch = mysql_fetch_assoc( $query );


            $core[ 'tpl' ]->assign( 'referral', htmlspecialchars( stripslashes( $fetch[ 'gebruikersnaam' ] ) ) );
            
        }
        
    } elseif( isset( $_SESSION[ 'ref_id' ] ) ) {
    
        $query = mysql_query( 'SELECT gebruikersnaam FROM leden WHERE id = "' . mysql_real_escape_string( $_SESSION[ 'ref_id' ] ) . '" LIMIT 1' ) or die( mysql_error( ) );
        
        if( mysql_num_rows( $query ) == 1 ) {
            
            $fetch = mysql_fetch_assoc( $query );


            $core[ 'tpl' ]->assign( 'referral', htmlspecialchars( stripslashes( $fetch[ 'gebruikersnaam' ] ) ) );
            
        } else {
        
            unset( $_SESSION[ 'ref_id' ] );
        
        }
    
    }
    
    $core[ 'tpl' ]->assign( 'prijzen', $core[ 'conf' ][ 'site' ] );


}


/************************************************************
 *    Assign pagina & titel
************************************************************/
$core[ 'tpl' ]->assign( 'titel', 'Aanmelden' );
$core[ 'tpl' ]->assign( 'pagina', 'pagina/leden/aanmelden.tpl' );


} else {


/************************************************************
 *    Assign pagina
************************************************************/
$core[ 'tpl' ]->assign( 'titel', 'Geen toegang' );
$core[ 'tpl' ]->assign( 'pagina', 'pagina/gasten/geen_toegang.tpl' );


}


/************************************************************
 *    Display layout
************************************************************/
$core[ 'tpl' ]->display( 'layout.tpl' );
?>

Meer tonen

Tim

Citaat van mentimjojo

@brokenTrack, ten eerste word het telefoon nummer later pas opgeslagen in de database.
En ten tweede je telefoon nummer invoeren is niet verplicht. Zie hier onder mijn gehele aanmelden.php:(hoe die nu is en hoe die het dus niet doet;))

Precies wat jij wilt dus.

Malik

PHP

if ( isset($telefoonnummer) && !ctype_digit($telefoonnummer) )
{
    $error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.'; 
}

Stefan.J

Leest niemand andermans reactie? Darsstar had het in het begin al goed met de volgende code:

PHP

if( ! empty( $telefoonnummer ) AND ! ctype_digit( $telefoonnummer ) ) {
    
        $error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.'; 
    
    }

In pseudocode staat hier:

ALS het veld telefoonnummer niet leeg is EN het telefoonnummer niet uit alleen maar cijfers bestaat

FOUT

Dit voldoet perfect aan de eis van TS...

Jasper

EDIT: reactie van killingdevil niet goed gelezen. Die moet wel werken

Tarwewijk

PHP

if(! ctype_digit($telefoonnummer) AND $telefoonnummer != '')
        $error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.';

Doe dit inplaats

PHP

if( preg_match( '/^[0-9-]+$/', $telefoonnummer ) == 0 ) { 
     
        $error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.';

Dit werkt gewoon:)

DevIT

PHP

if( ! empty( $telefoonnummer ) AND ! ctype_digit( $telefoonnummer ) AND strlen( $telefoonnummer ) < 8 ) {
    
        $error .= '- Je telefoonnummer mag alleen uit cijfers bestaan.'; 
    
    }

Hier word gekeken of het de juiste lengte heeft;)

ICTscripters

Dé plek voor IT

Dé plek voor IT

[Beta] Ottomaan: gratis browserspel met een echte, levende stad. Feedback gezocht!

Port Father: beta van een browsergame in Antwerpen 1931, feedback welkom

Maffia Hotel: een browsergame waarin je van lobbygast naar spil van het hotel groeit

Crime Club volledig herschreven vanaf nul

Malavita (nieuwe maffia game, van 0)

Herkent deze Notion-structuurscan echte problemen?

🚀 WE ZOEKEN TESTERS! 🚀

Luchtvaartsoftware: Hoe technologie de moderne luchtvaart verandert

[Verkoop] G4G (Gangsters 4 Gangsters): volledige broncode, nieuw gebouwd in PHP 8.3 | € 30 per domein

[TE KOOP] Malavita.be [Maffiaspel, gebaseerd op omicidio.nl, unieke code]

TK: Kraai.eu

Alleen cijfers?

Registratie

PHP waarschuwt voor twee ernstige bugs die makkelijk uit te buiten lijken

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Gebruikers die dit topic bekijken