Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP]Czy ten kod jest estetyczny
crazy_programmer
post
Post #1





Grupa: Zarejestrowani
Postów: 17
Pomógł: 0
Dołączył: 8.01.2011

Ostrzeżenie: (0%)
-----

Witam wszystkich. Naukę PHP musiałem porzucić ze względu na maturę i od tego tygodnia, a konkretnie poniedziałku zacząłem znowu. PHP uczę się z książki "PHP i MySQL Vademecum Profesjonalsity Wydanie Trzecie". Chciałbymbym abyście wyrazili swoją opinię czy ten kod jest estetyczny. Pod tym względem dostaje świra.

[PHP] pobierz, plaintext 
  1. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
  2. "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  3. 	<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="pl" lang="pl">
  4. 		<head>
  5. 			<title>Home Page</title>
  6. 			<meta http-equiv="Content-Type" content="application/xhtml+xml; charset=utf-8" />
  7. 			<meta http-equiv="Content-Style-Type" content="text/css" />
  8. 			<link rel="stylesheet"  href="style.css" type="text/css" />
  9.  
  10. 			<meta name="Author" content="crazy_programmer" />
  11. 			<meta name="Generator" content="Notepad++" />
  12. 			<meta name="Description" content="Strona testowa" />
  13. 		</head>
  14. 		<body>
  15.  
  16. 		<?php
  17. 			//Utwórz krótkie nazwy zmiennych
  18. 			$iloscopon = $_POST['opony'];
  19. 			$iloscoleju = $_POST['olej'];
  20. 			$iloscswiec = $_POST['swiece'];
  21. 			$sklep_wiedza = $_POST['opinia'];
  22.  
  23. 			//Utworzenie zmiennych potrzebnych do oblicznenia kosztów
  24. 			$ilosc = 0;
  25. 			$wartosc = 0.00;
  26. 			$wartosc_vat = 0.00;
  27. 			$wartosc_vat_znizka = 0.00;
  28. 			$vat = 0.22;
  29. 			$suma_znizki = 0.00;
  30.  
  31. 			//Utworzenie stałych, które przechowują ceny
  32. 			define("CENAOPON", 200);
  33. 			define("CENAOLEJU", 130);
  34. 			define("CENASWIEC", 100);
  35.  
  36. 			//Zniżki na cene opon
  37. 			if($iloscopon < 10)
  38. 				$znizka = 0;
  39. 			elseif($iloscopon >= 10 && $iloscopon <= 49)
  40. 				$znizka = 0.05;
  41. 			elseif($iloscopon >= 49 && $iloscopon <= 99)
  42. 				$znizka = 0.10;
  43. 			elseif($iloscopon >= 100)
  44. 				$znizka = 0.15;
  45.  
  46. 			//Obliczenie ilości przedmiotów
  47. 			$ilosc = $iloscopon + $iloscoleju + $iloscswiec;
  48.  
  49. 			//Obliczenie wartości przedmiotów bez VAT
  50. 			$wartosc = $iloscopon * CENAOPON
  51. 					 + $iloscoleju * CENAOLEJU
  52. 					 + $iloscswiec * CENASWIEC;
  53.  
  54. 			//Obliczenie wartości przedmiotów z VAT
  55. 			$wartosc_vat = $wartosc * (1 + $vat);
  56.  
  57. 			//Obliczanie wartości przedmiotó ze zniżką
  58. 			$wartosc_vat_znizka = $wartosc * (1 + $vat) - ($wartosc * $znizka);
  59.  
  60. 			//Obliczenie sumy znizki
  61. 			$suma_znizki = $wartosc * $znizka;
  62.  
  63.  
  64.  
  65.  
  66.  
  67.  
  68. 			if($ilosc < 1)
  69. 			{
  70. 				echo "<h1>Na poprzedniej stronie nie zostało złożone żadne zamówienie</h1>";
  71. 			}
  72. 			else
  73. 			{
  74. 				if($ilosc > 0)
  75. 					{
  76. 						echo "<h1>Części samochodowe Janka</h1><br />
  77. 							  <h2>Wyniki zamówienie</h2>";
  78. 						echo "<p>Zamówienie przyjęte o ";
  79. 						echo date('H:i, jS F')."</p>";
  80. 						echo "<p>Zamówione produkty:<br />";
  81. 					}
  82. 				if($iloscopon > 0)
  83. 					echo $iloscopon.' opon<br />';
  84. 				if($iloscoleju > 0)
  85. 					echo $iloscoleju.' oleju<br />';
  86. 				if($iloscswiec > 0)
  87. 					echo $iloscswiec.' świec zapłonowych</p>';
  88. 				if($ilosc > 0)
  89. 					{
  90. 						echo '<p>Zamówionych części: '.$ilosc.'</p>';
  91. 						echo '<p>Cena Netto: '.number_format($wartosc, 2).' PLN</p>';
  92. 						echo '<p>Cena Brutto: '.number_format($wartosc_vat, 2).' PLN</p>';
  93. 					}
  94. 				if($iloscopon >= 10)
  95. 					{
  96. 						echo '<p>Cenat Brutto + zniżka: '.number_format($wartosc_vat_znizka, 2).' PLN</p>';
  97. 						echo '<p>Zniżka: '.number_format($suma_znizki, 2).' PLN</p>';
  98. 					}
  99. 			}
  100.  
  101. 			//Zapytanie klientów o źródło informacji na temat sklepu
  102. 			switch($sklep_wiedza  && $ilosc > 0)
  103. 			{
  104. 				case 'a' :
  105. 					echo '<p>Stały Klient</p>';
  106. 					break;
  107. 				case 'b' :
  108. 					echo '<p>Reklama Telewizyjna</p>';
  109. 					break;
  110. 				case 'c' :
  111. 					echo '<p>Książka Telefoniczna</p>';
  112. 					break;
  113. 				case 'd' :
  114. 					echo '<p>Znajomy</p>';
  115. 					break;
  116. 			}
  117. 			?>
  118.  
  119. 		</body>
  120. 	</html>
[PHP] pobierz, plaintext


Ten post edytował crazy_programmer 29.06.2011, 19:09:18
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
peter13135
post
Post #2





Grupa: Zarejestrowani
Postów: 1 447
Pomógł: 191
Dołączył: 26.03.2008

Ostrzeżenie: (0%)
-----

1. Wcięcia trochę przy duże (chyba, że takie lubisz) moim zdaniem 2 spacje wystarczy. Ale dobrze, że są
2. Komentarze, jak najbardziej dodają przejrzystości
3. Poczytaj o MVC, php nie powinno się mieszać z html. To znacznie "uprzejrzyści" twój kod.
4. Żeby kod było łatwo analizować, powinno się go grupować. Mam na myśli tworzenie własnych funkcji. Później tworzenie klas.

Ogólnie, przy małych skryptach (do 200 linii powiedzmy) taki kod będzie wystarczająco dobry. Przy dużych projektach, patrz punkt 3 i 4. Bez tego trudno będzie się połapać.

Ten post edytował peter13135 29.06.2011, 18:39:08
Go to the top of the page
+Quote Post

Posty w temacie
- crazy_programmer   [PHP]Czy ten kod jest estetyczny   29.06.2011, 18:26:34
- - peter13135   1. Wcięcia trochę przy duże (chyba, że takie lubis...   29.06.2011, 18:33:20
- - sebap123   Jak na mój gust nawet bardzo bym powiedział. Fajny...   29.06.2011, 18:36:52
- - crazy_programmer   Dopiero zaczynam zabawę z PHP Jedyne co mnie w te...   29.06.2011, 18:41:23
- - peter13135   Cytat(sebap123 @ 29.06.2011, 19:36:52...   29.06.2011, 18:41:34
- - crazy_programmer   Mam do was jeszcze jedno pytanko. Tam gdzie użyłem...   29.06.2011, 19:02:45
- - Daimos   Dobrym nawykiem jest używanie klamerek {}, nie kor...   29.06.2011, 19:18:02
- - crazy_programmer   A mógłbyś mi pokazać użycie instrukcji "switc...   29.06.2011, 19:26:21
- - zegarek84   http://www.php.net/manual/en/control-struc...witch...   29.06.2011, 19:36:05
- - crazy_programmer   Dzięki wielkie   29.06.2011, 19:38:00

« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 12.10.2025 - 07:55
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn