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%)
-----

Cytat(sebap123 @ 29.06.2011, 19:36:52 ) *
Jak na mój gust nawet bardzo bym powiedział. Fajny, czysty i przejrzysty kod.

Widze, że kolega mnie ubiegł. A co do funkcji i klas to się zgodzę, ale chodziło chyba tylko o ten fragment a nie o regułę. Na pewno lepiej jest pisać obiektowo ale według mnie, przy tak krótkim kodzie to się trochę mija z celem.


Czyżbym inaczej, niż Ty ? (IMG:style_emoticons/default/tongue.gif)

Napisałem przecież, że przy małych projektach, tak jak on pisze jest dobrze. A przy większych, należało by zastosować MVC i obiektowość
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: 13.10.2025 - 17:34
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn