🚀 go-pugleaf

W dniu niedziela, 12 lipca 2015 21:02:15 UTC+2 użytkownik Borys Pogoreło napisał:
> Dnia Sun, 12 Jul 2015 09:27:19 -0700 (PDT), Dariusz Budyta napisał(a):
> 
> > Suma na fakturze liczy dobrze jak jest jedna stawka VAT, jak jest ich
> > więcej liczy mi z taką stawką jaka jest na pierwszej pozycji faktury,
> > jestem jaszcze trochę cienki ale staram się doczytać - tyle że skończyły
> > mi się pomysły?
> 
> Zacznijmy od tego, że ten przykład to jest jakaś masakra... Wymieszany kod
> z prezentacją, nieczytelny układ i nazwy zmiennych. Gdybyś to sprowadził do
> czytelniejszej postaci, to znacznie łatwiej by Ci było znaleźć błąd.
> 
> A tych błędów tu jest sporo:
> 
> > 	$ret = $db->Query('select N_Kier from import_faktury where nf=\''.$nf.'\'');
> 
> Książkowy przykład SQL injection. To akurat "tylko" select, ale w żaden
> sposób nie zabezpieczasz zapytań przed groźną zawartością.
> 
> >       $nf = $_GET['nf'];
> > 	$n_kier = $ret['N_Kier'];
> > 	$ret;
> 
> To jest przykład jak nie nazywać zmiennych. Jak ktoś ma pomóc, jeśli widzi
> jakieś $n_kier? Co to jest n_kier? Co oznacza $nf? Żeby chociaż tam były
> jakieś komentarze. Czy dobrze zgaduję, że n_kier to faktura wystawiana od
> cen netto, a nf to numer faktury?
> 
> > 		$ret = $db->Query('select *,
> > 			(select round(sum(netto)*(100+svat))/100 from import_pozycje where nf=import_faktury.nf) as suma,
> 
> Trzymasz kwoty jako float w bazie, że musisz kombinować z zaokrągleniami?
> Od tego masz typ "decimal". Prędzej czy później zniknie Ci jakiś grosz i
> będzie problem.
> 
> > 		$md_nazwa = 'Zrzeszenie Transportu Prywatnego m.st. Warszawy';
> 
> Nie wiem czy klient będzie zadowolony z takiej publikacji...
> 
> > 		$_vat = $r['svat'];
> > 		$vat = (float)$_vat;
> > 			$_vat = $_vat.' %';
> 
> Kolejny dobry przykład jak nie nazywać zmiennych. Jak osoba z zewnątrz ma
> zgadnąć, co autor miał na myśli i czym się różnią te VAT-y?
> 
> I gdzie jest tabelka z sumą kwot VAT w rozbiciu na stawki? Bez tego
> wystawiasz fakturę niezgodnie z ustawą.
> 
> -- 
> Borys Pogoreło
> borys(#)leszno,edu,pl

Co powinienem podać - żeby dało się to uratować?