[HaBo]

skymuss · 21.06.08, 15:46

---------------------------------

**Elderan** · 21.06.08, 15:48

Hallo,

Zitat:

Original von skymuss
Frage: Wie kann ich diese PHP-Klasse noch verbessern
oder optimieren ??
thx
skymuss

Zeilen einrücken, kommentare.

Mal im ernst: Dein Code kann doch keine Sau lesen.

Anzeige

- Anzeige -

easteregg · 21.06.08, 16:31

ich war mal so frei

PHP-Code:

  <?php
class cookie {

    // cookie_get
    function cookie_get($f) {
    
        if(is_array($f)) {
            $o = new cookie;
            foreach($f as $v) {
                $x[] = $o->cookie_get($v);
            }
        
        } else {
            
            if(!empty($f) && (is_strig($f) || is_numeric($f))) {
                    if (isset($_COOKIE[$f]) && !empty($_COOKIE[$f])) {
                        $c = $_COOKIE[$f];
                        $v = nl2br($c);
                        $x = htmlspecialchars($v);
                    } else {
                        $x = false;
                    }
            } else {
                $x = false;
            }
        
            return $x;
        }
    }

    // cookie_set
    function cookie_set($v, $n, $t) {

        if(is_array($v) && is_array($n) ) {
            $o = new cookie;
            $h = count($v);
            $k = count($n);
            
            if($h == $k) {
                for($i = 0; $i < $k; $i++) {
                    $o->cookie_set($v, $n, $t);
                }
                $x = true;
            } else {
                $x = false;
            }
        } else {
            if (is_numeric($t) && (is_string($n) || is_numeric($n)) && (is_string($v) || is_numeric($v))) {
                
                if(setcookie($n, $v, time() + $t)) {
                    $x = true;
                } else {
                    $x = false;
                }
            } else {
                $x = false;
            }
            
            return $x;
        }
    }

    // cookie_del
    function cookie_del($f) {

        if(is_array($f)) {
            $o = new cookie;
            
            foreach($f as $v) {
                $o->cookie_delete($v);
            }
            $x = true;
        } else {
            
            if(!empty($f) && (is_strig($f) || is_numeric($f))) {
                
                if (isset($_COOKIE[$f]) && !empty($_COOKIE[$f])) {
    
                    if(setcookie($f, '1', '0')) {
                        $x = true;
                    } else {
                        $x = false;
                    }
                } else {
                    $x = false;
                }
            } else {
                $x = false;
            }
            return $x;
        }
    }
}
?>

und anstatt dieser ewigen ifs würde ic h bei der variablenzuweisung aufjedenfall diese kurzform verwenden!

PHP-Code:

  $var = (is_true) ? wenn ja : sonst das!;

und sowas hier:

PHP-Code:

                      if(setcookie($f, '1', '0')) {
                        $x = true;
                    } else {
                        $x = false;
                    }

die function setcookie gibt ja ohnehin true oder false zurück, wieso dann die if schleife?
setz doch einfach

PHP-Code:

  $x = setcookie($f,'1','0');

und fertig

skymuss · 21.06.08, 19:55

---------------------------------

Anzeige

- Anzeige -

21.06.08, 15:46	#1 (permalink)
skymuss Registriert seit: 12.06.08 Likes: 0	PHP class Anzeige --------------------------------- Geändert von skymuss (04.06.10 um 09:43 Uhr)

21.06.08, 16:31	#3 (permalink)
easteregg Member of Honour Registriert seit: 14.09.07 Likes: 62	ich war mal so frei PHP-Code: <?php class cookie { // cookie_get function cookie_get($f) { if(is_array($f)) { $o = new cookie; foreach($f as $v) { $x[] = $o->cookie_get($v); } } else { if(!empty($f) && (is_strig($f) \|\| is_numeric($f))) { if (isset($_COOKIE[$f]) && !empty($_COOKIE[$f])) { $c = $_COOKIE[$f]; $v = nl2br($c); $x = htmlspecialchars($v); } else { $x = false; } } else { $x = false; } return $x; } } // cookie_set function cookie_set($v, $n, $t) { if(is_array($v) && is_array($n) ) { $o = new cookie; $h = count($v); $k = count($n); if($h == $k) { for($i = 0; $i < $k; $i++) { $o->cookie_set($v, $n, $t); } $x = true; } else { $x = false; } } else { if (is_numeric($t) && (is_string($n) \|\| is_numeric($n)) && (is_string($v) \|\| is_numeric($v))) { if(setcookie($n, $v, time() + $t)) { $x = true; } else { $x = false; } } else { $x = false; } return $x; } } // cookie_del function cookie_del($f) { if(is_array($f)) { $o = new cookie; foreach($f as $v) { $o->cookie_delete($v); } $x = true; } else { if(!empty($f) && (is_strig($f) \|\| is_numeric($f))) { if (isset($_COOKIE[$f]) && !empty($_COOKIE[$f])) { if(setcookie($f, '1', '0')) { $x = true; } else { $x = false; } } else { $x = false; } } else { $x = false; } return $x; } } } ?> und anstatt dieser ewigen ifs würde ic h bei der variablenzuweisung aufjedenfall diese kurzform verwenden! PHP-Code: `$var = (is_true) ? wenn ja : sonst das!;` und sowas hier: PHP-Code: `if(setcookie($f, '1', '0')) { $x = true; } else { $x = false; }` die function setcookie gibt ja ohnehin true oder false zurück, wieso dann die if schleife? setz doch einfach PHP-Code: `$x = setcookie($f,'1','0');` und fertig __________________ » Flattr mich! - Wenn dir mein Beitrag geholfen hat! « <\| 2 AMD Opterons 2384@ 8x3,2ghz \| Tyan S2915 \| 10GB \| 2x 8800GT \| 8400GS \| Dell 3008WFP + 2x2007FP \|>

21.06.08, 19:55	#4 (permalink)
skymuss Themenstarter Registriert seit: 12.06.08 Likes: 0	--------------------------------- Geändert von skymuss (04.06.10 um 09:43 Uhr)

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Ansicht
Linear-Darstellung Zur Hybrid-Darstellung wechseln Zur Baum-Darstellung wechseln

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
.class in .txt (Java)	mP765	Code Kitchen	4	01.05.07 21:47
C++ "komplizierte" class...	NULL!=NULL	Code Kitchen	1	06.11.06 22:33
class	CPU8080	(Web-) Design und webbasierte Sprachen	3	29.05.06 14:12
Microsoft Class Server	Letismoja	(In)security allgemein	6	17.02.04 20:37
[c++] class	soox	Code Kitchen	8	13.09.02 10:16

[HaBo]

PHP class