eigene funktionen

[|Coding]

hi,

da ich das nicht nicht in diesem faden "Php Postingscript" diskutieren wollte, mache ich es mal hier :)

ok, meine meinung zu eigenen funktionen:

eigene funktionen, sollten immer mit der übergabe von rohdaten klar kommen.

beispiel:

PHP-Code:

<?PHP
function colored_text($text){
    //datentest
    if(!is_string($text)) return FALSE;

    //string vorformatieren
    $text = htmlentities(trim($text));
    
    //text einfaerben
    for($i = 0; $i < (strlen($text)); $i++){
        $letter = substr($text, $i, 1);
        if($letter != ' ' && $letter != "r" && $letter != "n" && $letter != "rn"){
            $colored_text .= '<span style="color: ';
            if($i % 2){
                $colored_text .= '#ff0000';
            }else{
                $colored_text .= '#0000ff';
            }
            $colored_text .= '">'.$letter.'</span>';
        }else{
            $colored_text .= $letter;
        }
    }
    return nl2br($colored_text);
}

$text = ' bla bla
blub foo';

echo ($text = colored_text($text)) ? $text : 'FALSE';
?>

hier muss also nicht darauf geachtet werden, das die daten so 'string foo' der funktion übergeben werden. die daten können auch so ' string foo' oder
' string foo
'
der funktion übergeben werden, völlig gleich. das liegt daran, das die funktion die datenverarbeitung komplett übernimmt.

warum bin ich der meinung, das eigene funktionen immer - soweit möglich - alle datenverarbeitungsschritte übernehmen sollen?

ganz einfach, so können user die den code zu gesicht bekommen, ganz einfach die funktion kopieren und in ihre anwendungen einbauen, ohne den code durchsuchen zu müssen, was für vorformatierungen an den daten vorgenommen werden müssen, damit die funktion diese verspeisen kann. auch wenn das vielleicht heißen könnte, das die funktion etwas umfangreicher ist als nötig. scheiß der hund drauf. das bringt doch gerade einsteigern was.

[xabbuh]

Grundsätzlich gebe ich dir da Recht.

Aber:

Zitat von |Coding

das bringt doch gerade einsteigern was.

Wenn deine Funktionen anfängerfreundlich sein sollen, solltest du aber auch so konsequent sein und auf solche Zeilen verzichten, auch wenn es dann ein wenig länger wird :-):

PHP-Code:

$colored_text .= $letter != ' ' && $letter != "r" && $letter != "n" && $letter != "rn" ? '<span style="color: '.($i % 2 ? '#ff0000' : '#0000ff').'">'.$letter.'</span>' : $letter; 


[|Coding]

Zitat von xabbuh

Grundsätzlich gebe ich dir da Recht.

Aber:


Wenn deine Funktionen anfängerfreundlich sein sollen, solltest du aber auch so konsequent sein und auf solche Zeilen verzichten, auch wenn es dann ein wenig länger wird :-):

PHP-Code:

$colored_text .= $letter != ' ' && $letter != "r" && $letter != "n" && $letter != "rn" ? '<span style="color: '.($i % 2 ? '#ff0000' : '#0000ff').'">'.$letter.'</span>' : $letter; 


jo, in scripten welche ich public mache, bin ich ja auch so konsequent. hier war ich es jetzt nicht, weil ersten gepennt und zweitens ist es nur ein beispiel :-)

ob wohl, manchmal finde ich das echt mehr als nur angenehm...
z.b.: print $a == $b ? 'gleich' : 'ungleich';

stimmt aber schon, für blutige anfänger sicher nicht geeignet.

[xabbuh]

Zitat von |Coding

ob wohl, manchmal finde ich das echt mehr als nur angenehm...
z.b.: print $a == $b ? 'gleich' : 'ungleich';

So geht es ja sogar, aber diese verschachtelte Variante von dir in deinem anderen Beispiel... ;-)
Dagegen kann man ein so einfach Beispiel, meiner Meinung nach, ruhig drinlassen. Als Erklärung kann ja ruhig zum entsprechenden Kapitel auf php.net verlinkt werden.

[|Coding]

jau, hädde man. ich habe das beispiel mal eben umgeschieben, so müsste es auch ein "dau" verstehen :-)

[xabbuh]

Zitat von |Coding

jau, hädde man. ich habe das beispiel mal eben umgeschieben, so müsste es auch ein "dau" verstehen :-)

Das sieht doch schon viel besser aus. :-)