[OOP] Klasa do parsowania/edytowania plików INI, Problem z zwracaniem pliku INI (XML działa...) Opcje

[Babcia@Stefa]

Witam, napisałem klasę do parsowania/edytowania plików INI.

[PHP] pobierz, plaintext 
<?php
class SimpleINIElement
{
	protected $DataArray, $Mode, $Content, $TempVar, $CurrentData;
 
	public function __construct($_FILE='')
	{
		if(is_file($_FILE))
		{
			$this->Mode = 'Edit';
			$this->Content = file($_FILE);
			$this->ReadFromFile();
		} else
			$this->Mode = 'Create';
	}
 
	protected function ReadFromFile()
	{
		foreach ($this->Content as $line_number => $line)
		{
			// category
			if(eregi('[([0-9a-zA-Z]{0,100})]', $line)) //check valid
			{
				// gets an array
				preg_match_all("|[(.*.)]|U", $line, $this->TempVar['section']['pattern'], PREG_PATTERN_ORDER);
				$this->TempVar['section']['line'] = $line_number; // set temporary
				$this->DataArray[$this->TempVar['section']['pattern'][1][0]] = array(); // creating new category
				$this->CurrentData['section'] = $this->TempVar['section']['pattern'][1][0]; // sets a current category for new variables
			} elseif(preg_match('#.*([A-z0-9]+).*=.*([A-z0-9]+)*#', $line, $data)) { // add new variables (comments and variables)
				$_value = explode('=', $data[0]);
				if(substr($_value[0], 0, 1) == ';')
				{
					$var_key = substr($_value[0], 1, strlen($_value[0]));
					$this->DataArray[$this->CurrentData['section']]['vars'][substr($var_key, 0, strlen($var_key)-1)] = array('type' => 'comment', 'value' => substr($_value[1], 1, strlen($_value[1])));
				} else {
					$var_key = $_value[0];
					$this->DataArray[$this->CurrentData['section']]['vars'][substr($var_key, 0, strlen($var_key)-1)] = array('type' => 'variable', 'value' => substr($_value[1], 1, strlen($_value[1])));
				}
			}
		}
	}
 
	public function DocumentWrite($_STATUS='')
	{
		if($_STATUS == 'XML')
			return $this->WriteToXML();
		elseif($_STATUS == 'INI')
			return $this->WriteToINI();
		else
			return $this->DataArray;
 
	}
 
	public function ThrowSection($_NAME='', $_COMMENT='')
	{
		if(!is_array($this->DataArray[$_NAME]) AND $_NAME)
		{
			if($_COMMENT)
				$_COMMENT = 'y';
			else
				$_COMMENT = 'n';
			$this->DataArray[$_NAME] = array('vars' => array(), 'blocked' => $_COMMENT);
		} else
			throw new Exception('SimpleINIElement::ThrowVariable error: Category already exists or is argument is
 empty.');
 
	}
 
	public function ThrowVariable($_NAME='', $_VALUE='', $_CAT='', $_COMMENT='')
	{
		if(!$_NAME)
			throw new Exception('SimpleINIElement::ThrowVariable error: The first param. is empty, its a variable
 name!');
 
		if(!$_VALUE)
			throw new Exception('SimpleINIElement::ThrowVariable error: The 2nd param. is empty, its a variable v
alue!');
 
		if(!$_CAT)
			throw new Exception('SimpleINIElement::ThrowVariable error: The 3rd param. is empty, its a variable c
ategory!');
 
		if(!is_array($this->DataArray[$_CAT]))
			throw new Exception('SimpleINIElement::ThrowVariable error: Variable category does not exists.');
 
		if($_COMMENT == '')
			$_temp_type = 'variable';
		else
			$_temp_type = 'comment';
 
		$this->DataArray[$_CAT]['vars'][$_NAME] = array('value' => $_VALUE, 'type' => $_temp_type);
	}
 
	protected function WriteToXML()
	{
		$xml = '<?xml version="1.0"?>';
		foreach ($this->DataArray as $key => $value)
		{
			$xml .= '<' .$key. '>';
 
			if(count($value['vars']) > 0)
			{
				foreach ($value['vars'] as $kkey => $vvalue)
				{
					$xml .= '<variable name="' .$kkey. '" value="' .$vvalue['value']. '" type="' .$vvalue['type']. '"/>';
				}
			}
			$xml .= '</' .$key. '>';
		}
		return $xml;
	}
 
	protected function WriteToINI()
	{
		$output = '';
		foreach ($this->DataArray as $key => $value)
		{
			$output .= "[".$key."]\n";
			if(count($value['vars'] > 0))
			{
				foreach ($value['vars'] as $kkey => $vvalue)
				{
					if($vvalue['type'] == 'comment')
						$output .= ';';
					$output .= $kkey." = ".$vvalue['value']."\n";
				}
			}
			echo '<br/>I: ' .$i. '<br/>';
			return $output;
		}
	}
}
$ini = new SimpleINIElement();
$ini->ThrowSection('section1');
$ini->ThrowSection('section2');
$ini->ThrowSection('section3');
$ini->ThrowSection('section4');
$ini->ThrowVariable('TestVariable', 'Test variable value :)', 'section1', true);
$ini->ThrowVariable('TestVariable', 'Test variable value :)', 'section2', true);
print_r($ini->DocumentWrite('INI'));
?>
[PHP] pobierz, plaintext 

Problem tkwi w funkcji WriteToINI() gdyż zapisuje ona tylko jedną kategorię zmiennych, a funkcja WriteToXML() zwraca poprawny wynik w XML...

Nie wiem co jest przyczyną... może to mały błąd, literówka?

Dziękuję, Babcia@Stefa

[-=Peter=-]

jest tak:

[PHP] pobierz, plaintext 
<?php
protected function WriteToINI()
	{
		$output = '';
		foreach ($this->DataArray as $key => $value)
		{
			$output .= "[".$key."]\n";
			if(count($value['vars'] > 0))
			{
				foreach ($value['vars'] as $kkey => $vvalue)
				{
					if($vvalue['type'] == 'comment')
						$output .= ';';
					$output .= $kkey." = ".$vvalue['value']."\n";
				}
			}
			echo '<br/>I: ' .$i. '<br/>';
			return $output;//przerywa pętle foreach
		}
	}
?>
[PHP] pobierz, plaintext 

a powinno być:

[PHP] pobierz, plaintext 
<?php
protected function WriteToINI()
	{
		$output = '';
		foreach ($this->DataArray as $key => $value)
		{
			$output .= "[".$key."]\n";
			if(count($value['vars'] > 0))
			{
				foreach ($value['vars'] as $kkey => $vvalue)
				{
					if($vvalue['type'] == 'comment')
						$output .= ';';
					$output .= $kkey." = ".$vvalue['value']."\n";
				}
			}
			echo '<br/>I: ' .$i. '<br/>';
		}
		return $output;
	}
?>
[PHP] pobierz, plaintext 

Pozatym po co odkrywasz drugi raz koło? Na co Ci metoda ReadFormFile? Przecież jest wbudowana funkcja parse_ini_file()...

Ten post edytował -=Peter=- 23.03.2008, 16:17:44

[Babcia@Stefa]

Pozatym po co odkrywasz drugi raz koło? Na co Ci metoda ReadFormFile? Przecież jest wbudowana funkcja parse_ini_file()...

1. To klasa która od zera czyta pliki INI i je przetwarza.

Pozatym po co odkrywasz drugi raz koło?

TO MÓJ 200 POST !

Dziękuję, Babcia@Stefa

Ten post edytował Babcia@Stefa 23.03.2008, 16:22:16

[-=Peter=-]

Oj, chodzi mi o to że metoda ReadFromFile mogła by wyglądać tak:

[PHP] pobierz, plaintext 
<?php
public function ReadFromFile($fileName){
   $this->DataArray = parse_ini_file($fileName, true);
}
 
//lub
public function ReadFromFile($fileName){
   $this->DataArray = parse_ini_file($fileName);
}
 
//lub jakis warunek (wyrażenia regularne), sprawdzające które wywołanie wykorzystać (takie coś jak masz w swojej metodzie)
?>
[PHP] pobierz, plaintext 

[Babcia@Stefa]

Heh, a to wykorzystywanie gotowej funkcji, ja chciałem mieć własną

@edit
Do konfiguracji w formacie LUA jest jakaś wbudowana klasa? (jeśli nie to napiszę takową )

Dziękuję, Babcia@Stefa

Ten post edytował Babcia@Stefa 23.03.2008, 16:29:12

[-=Peter=-]

Hmm a dlaczego uparłeś się aby tej funkcji nie wykorzystać? Nie pojmuję tego... No chyba, że wolisz sobie utrudniać życie, jezdzić z Krakowa do Warszawy przez Gdańsk

Pozatym ta Twoja funkcja jest dużo mniej wydajna od tej, którą ja podałem, bo wyrażenie regularne w pętli raczej nie jest zbyt ekonomiczne.

[Babcia@Stefa]

Tak.. ale przebudowa teraz klasy będzie bardzo kosztowna...

Dziękuję, Babcia@Stefa

[wlamywacz]

Tak.. ale przebudowa teraz klasy będzie bardzo kosztowna...

Wlasnie w OOP chodzi o to aby tak nie bylo...

P.S. Nie mam polskich znakow

[Sedziwoj]

Wlasnie w OOP chodzi o to aby tak nie bylo...

Dokładnie, więc jak mówisz że taka zamiana wymagają dużo zmian, znaczy że masz źle napisane. Bo wymagana jest hermetyzacja, wtedy pewne fragmenty podmieniasz, ale nic oprócz tego nie musisz robić, całość już powinna z tego korzystać nieświadoma zmian.

[likemandrake]

Wiemy wszyscy, że OOP jest zwykle wolniejsze od programowania strukturalnego, czemu więc nie użyjesz funkcji parse_ini_file (o której mowa powyżej) dla przyspieszenia wydajności, tylko "uparcie" spowalniasz swój skrypt?

[Sedziwoj]

Wiemy wszyscy, że OOP jest zwykle wolniejsze od programowania strukturalnego, czemu więc nie użyjesz funkcji parse_ini_file (o której mowa powyżej) dla przyspieszenia wydajności, tylko "uparcie" spowalniasz swój skrypt?

Użycie parse_ini_file() lub nie jest niezależne od stylu programowania, bo równie dobrze implementacje można napisać strukturalnie. Ogólnie nie ma sensu pisać coś co już jest napisane i to lepiej.

[wlamywacz]

Wiesz to minimalne zwolnienie nadrabia sie przejrzystoscią i latwoscia w modyfikowaniu

[Cysiaczek]

Hm? Zwolnienie? No bez jaj, to są milionowe części sekundy, to nie spowalnia. Nie żyjemy w czasach procesorów 286. Będzie ta funkcja wywoływana 10000000000 razy dziennie? Śmiem wątpić. Sam kiedyś myślałem, że trzeba każdą rzeczy napisać tak, aby była maksymalnie szybka, ale to ślepa uliczka, bo nie ma nic za darmo i spada przejrzystość kodu.

Pozdrawiam.

[wlamywacz]

Cysiaczek moj post mial sie odniesc do wypowiedzi likemandrake

[likemandrake]

Ogólnie nie ma sensu pisać coś co już jest napisane i to lepiej.

W PHP zwykle jest tak (nie zawsze), że jeśli coś jest zaimplementowane natywnie, czy to przez moduł do parsera PHP (a więc jest napisane dla języka PHP w innym języku niż PHP - C, C++, itp.) działa po prostu szybciej, więc pod tym względem nie ma sensu pisać własnych implementacji, po za tym, że kod działa wolniej, to jest bardziej obszerny. Taki głupi przykład: pisanie wyrażenia regularnego do sprawdzania, czy ciąg tekstowy zawiera tylko znaki alfabetu łacińskiego i puszczenie przez powiedzmy funkcję preg_match jest znacznie wolniejsze niż użycie funkcji ctype_alfanum, która robi to samo, lecz znacznie wydajniej. Z wyrażeniami regularnymi jest troche inna bajka, ale tu chciałem tylko pokazać jaki skok wydajności można uzyskać.

Wiesz to minimalne zwolnienie nadrabia sie przejrzystoscią i latwoscia w modyfikowaniu

Zgadzam się z Tobą. Przede wszystkim kod staje się przenośny, co jest bardzo ważne w zastosowaniach biznesowych i nie tylko. To minimalne zwolnienie może mieć duży wpływ na wydajność w baaardzo rozbudowanej aplikacji, ale coś za coś.

@Cysiaczek

Bardziej chodziło mi o optymalizację jak największą jaką się da, lecz żeby nie przesadzać z czasem poświęconym na ten proceder. Mam tu na myśli wiedzę o danym problemie i jak sobie z nim poradzić, a nie potem spędzać dziesiątki godzin nad szukaniem problemu "Czemu to tak wolno działa?".

Podsumowując, potrzebne przede wszystkim doświadczenie, znajomość funkcjonalności i wydajności funkcji wbudowanych, wiedza na temat jakiegoś problemu, czy lepiej użyć funkcji A, czy funkcji B.

Ameryki nie odkryłem, chciałem troszkę (za bardzo) rozwinąć własną myśl nad moją poprzednią wypowiedzią.

Pozdrawiam

[Sedziwoj]

W PHP zwykle jest tak (nie zawsze), że jeśli coś jest zaimplementowane natywnie, czy to przez moduł do parsera PHP (a więc jest napisane dla języka PHP w innym języku niż PHP - C, C++, itp.) działa po prostu szybciej, więc pod tym względem nie ma sensu pisać własnych implementacji, po za tym, że kod działa wolniej, to jest bardziej obszerny. Taki głupi przykład: pisanie wyrażenia regularnego do sprawdzania, czy ciąg tekstowy zawiera tylko znaki alfabetu łacińskiego i puszczenie przez powiedzmy funkcję preg_match jest znacznie wolniejsze niż użycie funkcji ctype_alfanum, która robi to samo, lecz znacznie wydajniej. Z wyrażeniami regularnymi jest troche inna bajka, ale tu chciałem tylko pokazać jaki skok wydajności można uzyskać.

Z preg_match() trzeba uważać, bo nie wiem czy wiesz, że rozbicie przez nią jest szybsze od expolode()... Ale jak napiszesz coś co ręcznie, znak po znaku, sprawdza... lepiej nie mówić. Wydajności nie ma, do tego napomknięta znajomość problemów, przy wyszukiwaniu wzorca są dobre algorytmy, ale ich implementacja nie należy do przyjemnych, więc użyjmy tego co jest. Mimo wszystko trzeba wiedzieć jak to działa, bo czasem lepiej jest pominąć i użyć czegoś do specyficznego przypadku... ale to już jest wyższa szkoła jazdy.

Ogólnie jak coś jest w bibliotece należy z tego korzystać. A naukę poświęcić na pisane czegoś innego (chyba że ktoś chce wchodzić w problemy algorytmiki itp. ale raczej wątpię, do tego nie ten język)