Witam!

Z braku laku i tego że nie miałem neta przez parę minut napisałem taką oto klasę do tworzenia pliku konfiguracyjnego:

[PHP] pobierz, plaintext 
<?php class config_file{
 
		var $file; //uchwyt fopen
		var $var_name; //nazwa zmiennej tablicowej
		var $index_name; //nazwa indeksu tablicy
		var $num_values; //ilość zmiennych
		var $content; //to co ostatecznie zostanie zapisane do pliku
 
		public function __construct($filename='config.php', $var_name='$config')
		{
			$this->file=fopen($filename, 'w');
			$this->var_name=$var_name;
		}
 
		public function add_value($index_name, $value)
		{
 
			if(!$this->num_values)
			{
				$this->content='<?php'."\n".$this->var_name.'=array('."\n".'''.$index_name.''=>''.$value.'''; 
			}else{
				$this->content.=','."\n".'''.$index_name.''=>''.$value.''';
			}
 
			++$this->num_values;
		}
 
		public function excute()
		{
			$this->content.="\n".');'."\n".'?>';
 
			$result=fputs($this->file,$this->content);
			fclose($this->file);
 
			return $result;
		}
	}?>
[PHP] pobierz, plaintext 

Dość prosta klasa... Myślę że zrozumiała bez komentarzy.
Co o tym sądzicie? Coś by tu dodać?

A odczyt konfiguracji? i czemu uzywasz "var" jesli korzystasz z php5?

Dlaczego od razu robisz string z przekazanej wartości? Nie lepiej zbierać to w tablicy i dopiero przed samym zapisem zrobić z z niej string?

Pozdrawiam.

Uwagi poprzedników + moja: plik otwierasz w konstruktorze, a zamykasz w metodzie excute (execute?). Co będzie w przypadku jeśli nie wywołam tej metody albo gdy do momentu jej wywołania nastąpi błąd? Słabe to: mimo, że krótkie i proste to jednak źle przemyślane.

Tak zrobiłem ponieważ pomyślałem sobie że potem mogę rozbudować add_value:

[PHP] pobierz, plaintext 
<?php
public function add_value($index_name, $value)
{
if(is_array($index_name))
{
$count=count($index_name)
 
for($i=0; $i<$count; $i++)
{
$this->add_value($index_name[$i], $value[$i])
}
 
}else{
			if(!$this->num_values)
			{
				$this->content='<?php'."\n".$this->var_name.'=array('."\n".'''.$index_name.''=>''.$value.'''; 
			}else{
				$this->content.=','."\n".'''.$index_name.''=>''.$value.''';
			}
 
			++$this->num_values;
}
 
}
?>
[PHP] pobierz, plaintext 

Nie robiąc stringa i w przypadku gdy przekazuje wartości poprzez tablice np.
add_value(array('index1','index2'), array('wartosc1','wartosc2'));
To z jednej zmiennej tablicowej "przelewam" wartości do drugiej... a jakoś nie lubie przelewać z wiadra do wiadra.

Include... zapomniałem... var? za dużo javascriptu...

@phpion
No jak nie wywołasz execute To tak jakbyś zapomniał spuścić wody... nic się nie stanie ale będziesz musiał wrócić i pociągnąć za spłuczkę. Co do tego błędu... wiem że słabe. Jakby było mocne to bym tego tutaj nie dawał, ale chce żeby było jednocześnie też proste... więc czekam na ew. propozycje...

To można zrobić w jednej funkcji, bez mieszania klas do tego. Jak już ktoś wspomniał, będzie lepiej jeśli metoda add_value będzie dodawała element do tablicy, a dopiero później będziesz generował kod.

Czemu w zmiennej var_name przekazujesz zmienna w formiei ($var) a nie tylko nazwe (var) ?
Skoro używasz php5 to po co fopen ? Nie lepiej użyć file_get_contents i file_put_contents ?

Nic specjalnego, kilka minut roboty a i tak można by to lepiej zrobić.

Ocena: 1/10

Przekazuje w formie $var ponieważ wiem wtedy że ma to być zmienna... Bo możne przecież zrobić plik z configiem ale ze stałymi....

Ok. dzięki all za rady...

var w php5 jest tylko dla celow kompatybilnych wstecz. nie nalezy go uzywac.
do oznaczenia stalej w php5 sluzy co innego.

No to nie mozesz zrobic tak

[PHP] pobierz, plaintext 
<?php
function funkcja ( $var = 'config' ) 
{
$this->var = $var ;
}
?>
[PHP] pobierz, plaintext 

a później

[PHP] pobierz, plaintext 
<?php
$content .= '$' . $this->var . .....
?>
[PHP] pobierz, plaintext 

?