Error class speciaal voor jou

Hallo,

Ik heb een error class gemaakt. Een beetje zinloos, maar toch

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/class"]class[/URL] error
[COLOR= #007700]{[/COLOR]
    [URL="http://www.php.net/public"]public[/URL] [COLOR= #0000BB]$errors[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/array"]array[/URL][COLOR= #007700]();[/COLOR]
    [URL="http://www.php.net/public"]public[/URL] [COLOR= #0000BB]$count[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]0[/COLOR][COLOR= #007700];[/COLOR]
    [URL="http://www.php.net/public"]public[/URL] [COLOR= #007700]function[/COLOR] add[COLOR= #007700]([/COLOR][COLOR= #0000BB]$problem[/COLOR][COLOR= #007700])[/COLOR] [COLOR= #FF8000]// function for adding an error[/COLOR]
    [COLOR= #007700]{[/COLOR]
        [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/empty"]empty[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$problem[/COLOR][COLOR= #007700]))[/COLOR] [COLOR= #FF8000]// check if $error is not empty[/COLOR]
        [COLOR= #007700]{[/COLOR]
            [COLOR= #FF8000]// do nothing[/COLOR]
        [COLOR= #007700]}[/COLOR]
        [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/strlen"]strlen[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$problem[/COLOR][COLOR= #007700]) <[/COLOR] [COLOR= #0000BB]5[/COLOR][COLOR= #007700])[/COLOR] [COLOR= #FF8000]// if the error is to short[/COLOR]
        [COLOR= #007700]{[/COLOR]
            [COLOR= #FF8000]// do nothing[/COLOR]
        [COLOR= #007700]}[/COLOR]
        [URL="http://www.php.net/else"]else[/URL]
        [COLOR= #007700]{[/COLOR]
            [COLOR= #0000BB]$this[/COLOR][COLOR= #007700]->[/COLOR]errors[COLOR= #007700][[/COLOR][COLOR= #0000BB]$this[/COLOR][COLOR= #007700]->[/COLOR][URL="http://www.php.net/count"]count[/URL][COLOR= #007700]] =[/COLOR] [COLOR= #0000BB]$problem[/COLOR][COLOR= #007700];[/COLOR]
            [COLOR= #0000BB]$this[/COLOR][COLOR= #007700]->[/COLOR][URL="http://www.php.net/count"]count[/URL][COLOR= #007700]++;
        }
    }[/COLOR]
    [URL="http://www.php.net/public"]public[/URL] [COLOR= #007700]function[/COLOR] countError[COLOR= #007700]()[/COLOR] [COLOR= #FF8000]// returns the number of errors[/COLOR]
    [COLOR= #007700]{[/COLOR]
        [URL="http://www.php.net/return"]return[/URL] [COLOR= #0000BB]$this[/COLOR][COLOR= #007700]->[/COLOR][URL="http://www.php.net/count"]count[/URL][COLOR= #007700];
    }[/COLOR]
    [URL="http://www.php.net/public"]public[/URL] [COLOR= #007700]function[/COLOR] showError[COLOR= #007700]()[/COLOR] [COLOR= #FF8000]// shows the errors[/COLOR]
    [COLOR= #007700]{[/COLOR]
        [COLOR= #0000BB]$returnError[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000][/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/foreach"]foreach[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$this[/COLOR][COLOR= #007700]->[/COLOR]errors [COLOR= #007700]as[/COLOR] [COLOR= #0000BB]$error[/COLOR][COLOR= #007700])
        {[/COLOR]
            [COLOR= #0000BB]$returnError[/COLOR] [COLOR= #007700].=[/COLOR] [COLOR= #DD0000]<li>[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$error[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]</li>[/COLOR][COLOR= #007700];
        }[/COLOR]
        [URL="http://www.php.net/return"]return[/URL] [COLOR= #0000BB]$returnError[/COLOR][COLOR= #007700];
    }
}[/COLOR]
[COLOR= #0000BB]$error[/COLOR] [COLOR= #007700]= new[/COLOR] Error[COLOR= #007700]();[/COLOR]
[COLOR= #0000BB]$error[/COLOR][COLOR= #007700]->[/COLOR]add[COLOR= #007700]([/COLOR][COLOR= #DD0000]There is something wrong![/COLOR][COLOR= #007700]);[/COLOR] [COLOR= #FF8000]// add an error[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$error[/COLOR][COLOR= #007700]->[/COLOR]countError[COLOR= #007700]() !=[/COLOR] [COLOR= #0000BB]0[/COLOR][COLOR= #007700])[/COLOR] [COLOR= #FF8000]// if there is an error[/COLOR]
[COLOR= #007700]{[/COLOR]
    [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]The following errors occurred:[/COLOR][COLOR= #007700];[/COLOR]
    [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #0000BB]$error[/COLOR][COLOR= #007700]->[/COLOR]showError[COLOR= #007700]();[/COLOR] [COLOR= #FF8000]// show error[/COLOR]
[COLOR= #007700]}[/COLOR]
?>

Commentaar is altijd welkom.

Groetjes, Maarten

Je hebt wel

... maar moet je ook niet ergens

openen?

Op lijn 8 maak je $errors aan, maar die gebruik je verder niet. Waarom?

Ik zou niet "$returnError = The following errors occurred:;" doen. Als iemand een andere tekst wil dan moet dat in de class aangepast worden, en dat wil je niet. Je wilt eigenlijk alleen maar logica in je class.

Rik schreef:

Op lijn 8 maak je $errors aan, maar die gebruik je verder niet. Waarom?

Ik zou niet "$returnError = The following errors occurred:;" doen. Als iemand een andere tekst wil dan moet dat in de class aangepast worden, en dat wil je niet. Je wilt eigenlijk alleen maar logica in je class.

Klopt, vergeten weg te halen. Dat had ik toegevoegd maar vergeten weg te halen. Je hebt gelijk.

Ik pas het aan

Ziet er verder wel goed uit volgens mij.