Witam,
niedawno nie miałem dostępu do internetu i postanowiłem zajrzeć do jakiejś książeczki o PHP którą u siebie znalazłem. Skorzystałem z PSPada i serwera Apache który postawiłem u mnie aby sprawdzić czy kod który napisałem działa.

Na początku miało to być liczenie potęg liczby 2 (od 2^1 do 2^10), w tracie robienia pojąłem nieco o co chodzi i nieco rozbudowałem skrypt i teraz wygląda on tak:

[PHP] pobierz, plaintext 
<?php
// Sprawdzanie danych z GET
if (isset($_GET['liczba'])) $liczba=trim($_GET['liczba']);
if (isset($_GET['potegi'])) $potegi=trim($_GET['potegi']);
?>
 
 
 
<form action="index.php" method="GET">
Liczba podnoszona do potęgi: <input type="text" name="liczba" value="<?php echo $liczba; ?>" />
<br><br>
Do której potęgi: <input type="text" name="potegi" value="<?php echo $potegi; ?>" />
<input type="submit" value="OK" />
 
<?php
 
// Zapamiętanie liczby podanej na starcie
 
  $x = $liczba;
  if($liczba<>0)
  {
	echo('<br><br><b>Kolejne potęgi liczby '.$liczba.'</b><br><br>');
	for($potega=0;$potega<=$potegi;$potega++)
	{
 
	// Liczenie wartoci potęgi
 
	  for($y=1;$y<$potega;$y++)
	  {
	  $liczba = $liczba * $x;
	  }
	  echo($x.'^'.$potega.' = '.$liczba);
	  echo('<br>');
 
	  // Wczytanie podstawowej liczby
 
	  $liczba = $x;
	}
  }
  else
  {
  echo('Niewłaciwa liczba!');
  }
?>
[PHP] pobierz, plaintext 

Chciałbym się zapytać czy jest on poprawny i czy działa w sposób optymalny - to mój pierwszy skrypt i chciałbym aby był jak najbardziej dopracowany a IMO działa on dobrze, z związku z tym mam pytanka.

1. Liczby po pobraniu z formularza są zapisywane jako dane typu STRING - przez warunek if($liczba<>0) nie da się wyliczać dla 0 ale także dla znaków typu: "A", "B", "C", "D" itp. co jest mi na rękę. Czy to można uznać to za rozwiązanie poprawne czy lepeij dokonać konwersji na typ integer poprzez intval($liczba)?

2. Czy w skrypcie dobrze użyłem apostrofy i cudzysłowy? Wiem że różnica jest kolosalna i ważna dla optymalizacji skryptu.

Hej,

Skrypt wydaje sie ok, apostrofy tez, aczkolwiek moze warto zapoznac sie z manualem juz teraz

pow() zrobi to co chciales napewno szybciej, pozostaje jedynie dopisac otoczke

Co do filtrowania parametrow, to zalezy od specyfikacji i zalozen projektu nad ktorym pracujesz. Jesli zalozenie jest takie (woje, klienta etc) ze ma to byc konwertowane i wyliczane to konwertowac, ja ma byc zgloszony blad to zglaszac

I pomysl sobie co bedzi ejesli wprowadze wykladnik mniejszy od zera ? jak wiadomo podstawa powinna byc spotegowana a wynik przedstawiony jako jej odwrotnosc (<- to chyba dobry keyword prawda ?) ogolnie 1/n gdzie n to podstawa do potegi.

Hehehe a ja jak głupi ciągle zastanawiałem się dlaczego nie działa kombinacja $liczba^$potega

No cóż... na błędach człowiek się uczy

EDIT:
Zmodyfikowałem kod i teraz wygląda to tak, liczy i dla potęg dodatnich i dla ujemnych choć w tym drugim wypadku w formie np. 4^-1 = 0.25

[PHP] pobierz, plaintext 
<?php
// Sprawdzanie danych z GET
if (isset($_GET['liczba'])) $liczba=trim($_GET['liczba']);
if (isset($_GET['potegi'])) $potegi=trim($_GET['potegi']);
?>
 
<form action="potegi2.php" method="GET">
Liczba podnoszona do potęgi: <input type="text" name="liczba" value="<?php echo $liczba; ?>" />
<br><br>
Do której potęgi: <input type="text" name="potegi" value="<?php echo $potegi; ?>" />
<input type="submit" value="OK" />
 
<?php
 
// Zapamiętanie liczby podanej na starcie
 
  if($liczba<>0)
  {
	echo('<br><br><b>Kolejne potęgi liczby '.$liczba.'</b><br><br>');
 
	// Dla potęg mniejszych niż 0
 
	if($potegi<0)
	{
	  for($potega=0;$potega<=-$potegi;$potega++)
	  {  
		for($y=0;$y<=$potega;$y++) $wynik=pow($liczba,-$potega);
		echo($liczba.'^'.-$potega.' = '.$wynik);
		echo('<br>');	
	  }
	}
 
	// Dla potęg większych niż 0
 
	else
	{
	  for($potega=0;$potega<=$potegi;$potega++)
	  {  
		for($y=0;$y<=$potega;$y++) $wynik=pow($liczba,$potega);
		echo($liczba.'^'.$potega.' = '.$wynik);
		echo('<br>');	
	  }
	}
  }
  else
  {
  echo('Niewłaciwa liczba!');
  }
?>
[PHP] pobierz, plaintext 

Jeszcze zmienię bo chyba jednak lepeij będzie gdy będzie pokazywał 1/n albo zrobię aby pokazywał obie formy (jeszcze nie jestem pewien). Co do komunikatu - nad tym też jeszcze popracuję.

Zmodyfikowałem dzisiaj kod aby usunąć niepotrzebne 2 pętle które były w poprzedniej wersji i dołożyłem pokazywanie minusowych potęg jako 1/n. Zmieniłem także zapis na nieco krótszy ponieważ sądzę że przy istniejących pętlach nie musiałem używać kilka razy funkcji echo(), postanowiłem połączyć napisy.

[PHP] pobierz, plaintext 
<?php
// Sprawdzanie danych z GET
if (isset($_GET['liczba'])) $liczba=trim($_GET['liczba']);
if (isset($_GET['potegi'])) $potegi=trim($_GET['potegi']);
?>
 
<form action="potegi2.php" method="GET">
Liczba podnoszona do potęgi: <input type="text" name="liczba" value="<?php echo $liczba; ?>" />
<br><br>
Do której potęgi: <input type="text" name="potegi" value="<?php echo $potegi; ?>" />
<input type="submit" value="OK" />
 
<?php
 
// Zapamiętanie liczby podanej na starcie
 
  if($liczba<>0)
  {
	echo('<br><br><b>Kolejne potęgi liczby '.$liczba.'</b><br><br>');
 
	// Dla potęg mniejszych niż 0
 
	if($potegi<0)
	{
	  for($potega=0;$potega<=-$potegi;$potega++) echo($liczba.'^'.-$potega.' = '.pow($liczba,-$potega).' = 1/'.pow($liczba,$potega).'<br>');
	}
	// Dla potęg większych niż 0 
	else
	{
	  for($potega=0;$potega<=$potegi;$potega++) echo($liczba.'^'.$potega.' = '.pow($liczba,$potega).'<br>');
	}
  }
  else
  {
  echo('Niewłaciwa liczba!');
  }
?>
[PHP] pobierz, plaintext 

Zadowolenie z tego że to napisałem i że działa jest niesamowite

No to teraz zrob sobie z tego funckje. DLa treningu. I jeszcze tylko przyczepilbym sie do nazw zmiennych. Sprobuj tak ponazywac zmienne zeby od razu bylo wiadomo co one przechowuja, czyli jak np zamiast $poteki mozna by zmienic na $iWykladnik, a $liczba na $iPodstawa mysle ze to bardziej rpzejrzyste. I jak widac jestem zwolennikiem notacji wegierskiej, co za tym idzie nazwy zmiennych zaczynaja sie jedna literka okreslajaca jej tym u mnie:
i - integer,
s- string,
o - object,
m - mixed
a - array,
r - resource

A pozniej to juz NormalnieCamelCaseZebyLadnieByloWidacCoIJak.

Pozdrawiam.

Nie wiem czy o to dokładnie chodziło ale w funkcjach i działając wygląda u mnie teraz tak:

[PHP] pobierz, plaintext 
<?php
// Sprawdzanie danych z GET
if (isset($_GET['iPodstawa'])) $iPodstawa=trim($_GET['iPodstawa']);
if (isset($_GET['iWykladnik'])) $iWykladnik=trim($_GET['iWykladnik']);
?>
 
<form action="<?php echo($_SERVER["PHP_SELF"]); ?>" method="GET">
Podstawa potęgi: <input type="text" name="iPodstawa" value="<?php echo $iPodstawa; ?>" />
<br><br>
Do jakiego wykładnika: <input type="text" name="iWykladnik" value="<?php echo $iWykladnik; ?>" />
<input type="submit" value="OK" />
 
<?php
  function potengi_dodatnie()
  {
	global $iPodstawa;
	global $iPrzebieg;
	echo($iPodstawa.'^'.$iPrzebieg.' = '.pow($iPodstawa,$iPrzebieg).'<br>');
  } 
 
  function potengi_ujemne()
  {
	global $iPodstawa;
	global $iPrzebieg;
	echo($iPodstawa.'^'.-$iPrzebieg.' = '.pow($iPodstawa,-$iPrzebieg).' = 1/'.pow($iPodstawa,$iPrzebieg).'<br>');
  }
   
  if($iPodstawa<>0)
  {
	echo('<br><br><b>Kolejne potęgi liczby '.$iPodstawa.'</b><br><br>');
	if($iWykladnik<0) for($iPrzebieg=0;$iPrzebieg<=-$iWykladnik;$iPrzebieg++) potengi_ujemne();
	else for($iPrzebieg=0;$iPrzebieg<=$iWykladnik;$iPrzebieg++) potengi_dodatnie();
  }	
  else echo('Niewłaciwa podstawa!');  
?>
[PHP] pobierz, plaintext