leggibilità del codice, ritorno echo error multipli, ritorno echo error errato

[gabar-el]

ciao. ho implementato questo script per verificare l'inserimento del campo in questione:

if(!empty($_POST['crediti']))
                {
                    if(ereg("[0-9]{1,2}", $_POST['crediti']) && $_POST['crediti'])
                    {
                        $crediti = $_POST['crediti'];
                    }
                    else
                    {
                        echo "Crediti non validi!";
                    }
                }
                else
                {
                    echo "Il campo crediti è vuoto! 
";
                } 

mi chidevo 2 cose:
il codice è leggibile così? io lo trovo leggibile, ma vedo che molti scrivono in altro modo, non vorrei prendere brutte abitudini....
ho altri campi similari, ma se tutti sono vuoti si vedrà l'avviso solo del primo che verrà letto, come si può elencare tutti i campi vuoti subito? è solo una curiosità, magari non ne vale la pena
se inserisco lo zero in questo campo il messaggio di errore è che il campo è vuoto e non che il campo non è valido, perchè?
grazie

[weppos]

$crediti = $_POST['crediti'] ? trim($_POST['crediti']) : null;

if ($crediti) {
    if (!ereg("[0-9]{1,2}", $crediti)) {
        echo "Crediti non validi!";
        $crediti = null;
    }
} else {
    echo "Il campo crediti è vuoto! 
";
} 

[gabar-el]

il tuo codice mi pare veramente fico e un pò lo intuisco (intuisco anche che risparmi un bel pò di righe), ma nn lo capisco....

[weppos]

Originariamente inviato da gabar-el
il tuo codice mi pare veramente fico e un pò lo intuisco (intuisco anche che risparmi un bel pò di righe), ma nn lo capisco....

Come non lo capisci... è il tuo riscritto.
Quale riga non comprendi?

[gabar-el]

grazie della risposta. ok, più o meno ho capito il codice, l'operatore ternario mi mandava in difficoltà....
l'ho provato e funzia bene, ma perchè se inserisco lo 0 mi dice che il campo è vuoto anzichè errato?
grazie

[gabar-el]

ok, come non detto, mi sono appena accorto di un errore, funziona tutto come deve, grazie!!!

[gabar-el]

ho un campo simile per delle quote, devo implementare un str_replace, come posso farlo in maniera breve? posso aggiungerlo cioè da qualche parte? altrimenti nn saprei proprio....

[weppos]

str_replace è str_replace.
Più breve di così non puoi.

[gabar-el]

no va beh, intendevo dove dovrei metterlo per non generare errore di sintassi, non sono molto pratico....

[weppos]

Originariamente inviato da gabar-el
no va beh, intendevo dove dovrei metterlo per non generare errore di sintassi, non sono molto pratico....

Direi anche pigro... pensaci, prova e se poi da errore vediamo perché.