Czesc, witajcie.

Jestem w trakcie pisania klasy do laczenia sie z baza danych, a dokladniej tworzeniem zapytan no i mam taki kawalek kodu, ktory w moim mniemaniu nie wyglada najlepiej, cos tu nie gra, tak mi sie wydaje.

[PHP] pobierz, plaintext 
				foreach($arConditions as $tColumnName => $mixValue) {
					if (isset($arParenthesisOpenPos)) 
						foreach ($arParenthesisOpenPos as $iPosition)
							$t .= ($iPosition == $iCursorPos+1 ? '(' : null);
					$t .= $this->_formatMultiSeparatedValue($cns, $mixValue, $tColumnName, $bHidden);
					if (isset($arParenthesisClosePos)) 
						foreach ($arParenthesisClosePos as $iPosition)
							$t .= ($iPosition == $iCursorPos+1 ? ')' : null);
 
					$t .= ' ' . $arSeparators[$iCursorPos++] . ' ';	
				}
[PHP] pobierz, plaintext 

i tak sie zastanawiam jak to skrocic, myslalem zeby ten foreach w metode wsadzic, ale to z drugiej strony wydaje mi sie glupie, zeby tworzyc metode na 2 liniowy kod, czyz nie?
takze.. pomocy?

Pozdrawiam

Jeżeli wykorzystujesz do dużo razy, to zawsze osobna metoda będzie lepsza, bo w razie ewentualnej zmiany wystarczy zedytować funkcję a nie ileś tam linijkek.
Poza tym, to

[PHP] pobierz, plaintext 
if (isset($arParenthesisClosePos)) 
foreach ($arParenthesisOpenPos as $iPosition)
[PHP] pobierz, plaintext 

w przypadku pustej tablicy wypluje błąd typu Invalid parameter/argument supplied for foreach. Powinieneś jeszcze dodać czy można ją policzyć if(count())

ajajaj, liczylem na jakis skrot typu

[PHP] pobierz, plaintext 
return ($x ? true : false);
[PHP] pobierz, plaintext 

No, ale nic, dzieki za pomoc anyway, a co do pustej tablicy, jest sprawdzana troszke wczesniej, wiec tutaj juz nie potrzeba.

pozdrawiam.

Turson jezeli dasz do foreach pustą tablicę to nic się nie stanie.

owszem, pusta tablica przejdzie bez glosu, chyba ze wartosc bedzie np NULL to wtedy wywali Notice.

Mnie ostatnio blue screen wyskoczył

To ty dalej na M$ pracujesz ? Wstydź się

Haha, nie śmiałbym.