Avatar billede testwatest Nybegynder
25. februar 2007 - 12:22 Der er 3 kommentarer

'god kodeskik'

Herunder har jeg en funktion der virker fint, men over min funktion har jeg i sinde at fortælle hvilke parametre der er, og hvad funktionen returnerer. Men i nedenstående funktion har jeg jo et lille problem. Paramteren kan både være array eller string, og det samme kan returneringen. Er der nogle der har forslag til hvad man så skal skrive? Og hvad synes i min variabel skal hedde istedet for $string når det jo ikke nødvendigvis er af typen string?

# parameter: string $string
# returnerer: string
function escape($string) {
    if(!get_magic_quotes_gpc()) {
        if(is_array($string)) {
            foreach($string as $key => $value) {
                $string[$key] = addslashes($value);
            }
        }
        else {
            $string = addslashes($string);
        }
    }
   
    return $string;
}
Avatar billede kalp Novice
25. februar 2007 - 12:24 #1
god skik er ikke at kommentere det åbentlyse, men blot kommentere det avancerede... det som andre ikke bare kan sætte sig ned og forstå uden den store sammenhæng eller blot noget avanceret kode.

Der er dog mange holdning på området.. det var bare min:)
Avatar billede testwatest Nybegynder
25. februar 2007 - 12:41 #2
det er jo sådanset også underordnet. folk kommenterer jo hvad de mener skal kommenteres, og hvad de  mener ikke er indlysende.
Avatar billede radion Nybegynder
25. februar 2007 - 12:50 #3
jeg ville umiddelbart fortælle hvad den gør, og undervejs i koden hvis der er nogle parametre man skal være ekstra opmærksomme på.. hvad den returnerer og hvilke parametre den modtager er imho ikke vigtige, og vil komme ind under beskrivelsen
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
Vi tilbyder markedets bedste kurser inden for webudvikling

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester