Forum PHP.pl

Witam,

nie mogę zlokalizować błędu mojego skryptu, wyskakuje błąd:



Problem w tym, że linia 160 jest poprawna w 100% wg. mojej wiedzy:

[PHP] pobierz, plaintext 
<?php
foreach( $vat as $klucz => $wartosc )
?>
[PHP] pobierz, plaintext 

kolejnym szukanym błędem była zła nazwa tablicy - niestety - tablica $vat istnieje. Gdzie leży błąd?

niestety komunikat glosi jasno, ze $vat nie jest tablicą.

daj tak:

[PHP] pobierz, plaintext 
<?php
print_r($vat);
?>
[PHP] pobierz, plaintext 

by sie dowiedziec co zawiera $vat

Faktycznie zmienna jest pusta, ale dlaczego?

Kod wygląda następująco:

[PHP] pobierz, plaintext 
<?php
		$result = sql("SELECT * FROM ". $conf['tbl_skladki'] .", ". $conf['tbl_list'] ." WHERE ". $conf['tbl_skladki'] .".s_id_l=". $conf['tbl_list'] .".l_id AND ". $conf['tbl_list'] .".l_id='". $glowne['f_id_l'] ."' ORDER BY ". $conf['tbl_skladki'] .".s_time ASC");
		while($row = mysql_fetch_array($result)){
			if($row['s_typ']=='pzl') {$typ = "Składka do PZŁ";}elseif($row['s_typ']=='kolo') {$typ = "Składka do koła";}
			$vat['0']['netto'] = $vat['0']['netto'] + $row['s_kwota'];
			$vat['0']['bruto'] = $vat['0']['netto'];
				$body .= $body_f;
				$body = str_replace("<<lp>>", $lp, $body); 
				$body = str_replace("<<nazwa>>", $typ . ' ('. $row['s_okres'] . ')', $body);
				$body = str_replace("<<ilosc>>", '1', $body);
				$body = str_replace("<<jm>>", 'szt.', $body);
				$body = str_replace("<<netto>>", number_format(str_replace(",",".",$row['s_kwota']), 2, '.', ' '), $body);
				$body = str_replace("<<vatp>>", '0 ', $body);
				$body = str_replace("<<vat>>", '0.00', $body);
				$body = str_replace("<<brutto>>", number_format(str_replace(",",".",$row['s_kwota']), 2, '.', ' '), $body);
				$lp++;
			}
 
foreach( $vat as $klucz => $wartosc ) {
   $vat_b .= str_replace("<<stawka>>", $wartosc, $vat_sum_body);
		foreach ($vat[$klucz] as $klucz2 => $wartosc2) {
		   $vat_b = str_replace("<<". $klucz2 .">>", $wartosc2, $vat_sum_body);
		}  
}
?>
[PHP] pobierz, plaintext 

Hmm..
Może daj

[PHP] pobierz, plaintext 
 <?php
$vat = array();
?>
[PHP] pobierz, plaintext 

Na początku

Wszystko pięknie - błędu brak, ale tablica jest pusta i nie wykonują się zadania które są w pętli.

Hmm.. masz stworzoną funkcję "sql"? Jeżeli nie, to zamiast tego powinno być "mysql_query"...

Tak, oczywiście - jest stworzona.

Skoro petla while nie dziala to pewnie zapytanie nie zwraca ci zadnych rekordow. Wypisz sobie to zapytanie:

[PHP] pobierz, plaintext 
<?php
$query = "SELECT * FROM ". $conf['tbl_skladki'] .", ". $conf['tbl_list'] ." WHERE ". $conf['tbl_skladki'] .".s_id_l=". $conf['tbl_list'] .".l_id AND ". $conf['tbl_list'] .".l_id='". $glowne['f_id_l'] ."' ORDER BY ". $conf['tbl_skladki'] .".s_time ASC";
echo $query;
?>
[PHP] pobierz, plaintext 

i zapusc je na bazie (np. phpMyAdminem) i sprawdz czy ci cos zwroci.

Zwraca.

skoro to zapytanie zwraca ci rekordy w PMA, nie ma bata musi i tu.
daj tak i spradźmy czy oby ci do petli w chodzi:

[PHP] pobierz, plaintext 
<?php
 
while($row = mysql_fetch_array($result)){
print_r($row);
///...
}
?>
[PHP] pobierz, plaintext 

jesli ci nie wyswietli nic, to jednak masz cos nie tak. jesli ci wyswietli, to tę tablice źle budujesz. A ze źle budujesz to widać od razu

[PHP] pobierz, plaintext 
<?php
 
$vat['0']['netto'] = $vat['0']['netto'] + $row['s_kwota'];
			$vat['0']['bruto'] = $vat['0']['netto'];
 
?>
[PHP] pobierz, plaintext 

czy jestes pewien ze tak to ma wyglądać? zawsze przypisujesz do indexu '0'. Niezależnie od iteracji. powinienes zmieniac index. na dole zwiekszasz $lp++; .to powinno byc indexem. oczywiscie przed petlą opwinno byc $lp=0;
nastepna rzecz. do $vat['0']['netto'] przypisujesz $vat['0']['netto'] powiekszone o cos tam. a przeciez za pierwszym razem $vat['0']['netto'] nic nie zawiera
moze napisz co chciales osiągnąć, bo to co tu masz to ... sam nie wiem co tu masz

Zatem tak - $vat['0']['netto'] - to jest wartość netto produktów o stawce VATowskiej 0%, jak by było $vat['3']['netto'] to wartość netto dla produktów ze stawką 3% itd...


Generalnie ja już głupieję - w końcu chyba się skończy na tym, że przepiszę to od nowa