Witam. Napisałem na własne potrzeby funkcje do edycji plików ini. Chciałbym abyście ocenili i powiedzieli co jest źle, co poprawić etc.

[PHP] pobierz, plaintext 
<?php
 
function edit_ini_file( $file, $values, $section = false ) {
        $str = "";
        $fo  = fopen( $file, "r" );
        while ( true ) {
                if ( feof( $fo ) ) { //dopisanie nowych wartości na końcu pliku
                        if ( $section === true ) {
                                foreach ( $values[ $currentSection ] as $name => $value ) { // dopisanie do ostatniej sekcji
                                        $str .= "{$name} = {$value}\n";
                                        unset( $values[ $currentSection ][ $name ] );
                                }
                                unset( $values[ $currentSection ] );
                                foreach ( $values as $sectionN => $sectionV ) { // dopisanie nowych sekcji i wartosci
                                        $str .= "[{$sectionN}]\n";
                                        foreach ( $sectionV as $name => $value ) {
                                                $str .= "{$name} = {$value}\n";
                                        }
                                }
                        }
                        else{
                                foreach($values as $name => $value){ //dopisanie nowych wartosci bez sekcji
                                        $str .= "{$name} = {$value}\n";
                                }
                        }
                        break;
                }
 
                $line = fgets( $fo );
 
                if ( $line{0} == ";"){ //przepisanie komentarzy
                        $str .= $line."\n";
                        continue;
                }
                if(trim($line) == "" || $line == "\n")                        continue;
                if ( $section === true && preg_match( "/\[(.*?)\]/", $line, $sectionInfo ) ) {
                        if ( isSet( $currentSection ) && isset( $values[ $currentSection ] ) ) {
                                foreach ( $values[ $currentSection ] as $name => $value ) { //dodanie pozostalych wartosci do sekcji
                                        $str .= "{$name} = {$value}\n";
                                        unset( $values[ $currentSection ][ $name ] );
                                }
                                unset( $values[ $currentSection ] );
                        }
                        $currentSection = $sectionInfo[ 1 ];
                        $str .= $sectionInfo[ 0 ]."\n";
                        continue;
                }
                else if ( $section === true ) {
                        list($name, $value) = explode( "=", $line );
                        $name = trim($name);
                        if ( isSet( $values[ $currentSection ][ $name ] ) ) { //zmiana wartosci
                                $str .= "{$name} = {$values[ $currentSection ][ $name ]}\n";
                                unset( $values[ $currentSection ][ $name ] );
                                continue;
                        }
                        else {
                                $str .= $line."\n";
                                continue;
                        }
                }
                else{
                        list($name, $value) = explode( "=", $line );
                        $name = trim($name);
                        if ( isSet( $values[ $name ] ) ) { //zmiana wartosci
                                $str .= "{$name} = {$values[ $name ]}\n";
                                unset( $values[ $name ] );
                                continue;
                        }
                        else {
                                $str .= $line."\n";
                                continue;
                        }
                }
        }
        $fw = fopen($file, "w+");
        fwrite($fw, $str);
}
//przykładowe użycie
edit_ini_file( "file.ini", array( "istniejaca_sekcja" => array( "klucz" => "wartosc", "nowy-klucz" => "wartosc" ), "nowa-sekcja" => array( "nowy-klucz" => "wartosc" ) ), true );
edit_ini_file( "file2.ini", array( "klucz" => "wartosc", "nowy-klucz" => "wartosc" ), false );
?>
[PHP] pobierz, plaintext 

Pozdrawiam

Zamknij skrypt w klase. Zrób w niej metody prywatene typu 'dopisanie nowych wartości na końcu pliku', 'przepisanie komentarzy', 'zmiana wartosci'
Klasa może być statyczna lub nie zaleznie od potrzeb. Zamknięcie w klase i ekstrakcja metod do tej klasy z tego skryptu zapewni ci lepszą możliwość pielegnacji, refaktoringu, elestyczność, czytelność itp
I jeśli nie potrzebujesz specjalistycznych funkcjonalności i nie robisz tego w celach edukacyjnych to możesz skorzystać z gotowych rozwiązań do zarządzania plikami ini Config_Lite
pissss joł

Nie wiem czy jest sens zamknięcia tej funkcji w klasę. Gdyby to było kilka funkcji to jeszcze rozumiem, no chyba że się mylę i lepiej rozbić tą funkcje na kilka mniejszych. Robię to w celach zarazem edukacyjnych oraz na potrzeby projektu

http://framework.zend.com/manual/1.12/en/z...troduction.html

Poczytaj, popatrz, wyciągnij wnioski.

Czyli jednak zamknąć to w klasę. Jak znajdę chwile to przerobie

Poczytaj też o funkcji parse_ini_file.