Forum PHP.pl

Pytanie... Po kiego grzyba Ci tyle IFów, które w zależności od danych mogą prowadzić do nieoczekiwanych wyników? Zwróć uwagę na zależności! Mając określone dane możesz wyliczyć tylko określone inne. Sprowadź więc całość do pewnych zależności, sprawdzając co występuje a co nie i zagnieżdżając owe IFy zamiast wypuszczać obok siebie. Zauważ, że Twój kod wielokrotnie liczy to samo, a na dodatek nie sprawdzasz co się dzieje gdy ktoś wprowadzi dane w celu wywalenia całości Jak? Załóżmy, że ktoś poda $u, $i, $g, to $r będzie kilka razy obliczane (linie 3, 15, 24, 31). A co gdy poda wszystkie 4? Zacznij od najmniej zależnych, czyli $g oraz $r.

[PHP] pobierz, plaintext 
function leitwert($u, $i, $r, $g) {
  if ($r) {
    $g1 = 1 / $r;
    if ($g && $g != $g1) {
      throw Exception('G is malformed!');
    } else {
      $g = $g1;
    }
  } elseif ($g) {
    $r1 = 1 / $g;
    if ($r && $r != $r1) {
      throw Exception('R is malformed!');
    } else {
      $r = $r1;
    }
  }
  if ($i) {
    if ($u) {
      $r1 = $i / $u;
      if ($r && $r != $r1) {
        throw Exception('R is malformed!');
      } else {
        $r = $r1;
      }
    } elseif ($r) {
      $u1 = $i / $r;
      if ($u && $u != $u1) {
        throw Exception('U is malformed!');
      } else {
        $u = $u1;
      }
    }
  } elseif ($u) {
    if ($i) {
      $r1 = $i / $u;
      if ($r && $r != $r1) {
        throw Exception('R is malformed!');
      } else {
        $r = $r1;
      }
    } elseif ($r) {
      $i1 = $u * $r;
      if ($i && $i != $i1) {
        throw Exception('I is malformed!');
      } else {
        $i = $i1;
      }
    }
  } elseif ($r) {
    if ($i) {
      $u1 = $i / $r;
      if ($u && $u != $u1) {
        throw Exception('U is malformed!');
      } else {
        $u = $u1;
      }
    } elseif ($u) {
      $i1 = $u * $r;
      if ($i && $i != $i1) {
        throw Exception('I is malformed!');
      } else {
        $i = $i1;
      }
    }
  }
  if (!$g) {
    $g = 1 / $r;
  }
  if ($i && $u && $r && $g) {
    return array('i' => $i, 'u' => $u, 'r' => $r, 'g' => $g);
  } else {
    throw Exception('Not enough data!');
  }
}
[PHP] pobierz, plaintext 

Czemu tak? Zaczniemy od sprawdzenia czy $g albo $r były podane. Liczenie ma sens w przypadku gdy tylko jedna z nich istnieje. Jeśli są obie lub brak obu - jest to bezcelowe A co gdy ktoś poda obie z powietrza? Przykładowo $r = 10, $g = 50... Skrypt powinien od razu zaprotestować. Na tym etapie może się też okazać, że wciąż nie mamy potrzebnego nam do dalszych obliczeń $r. Może tak być w sytuacji, gdy nie ma ani $r, ani $g potrzebnego do prostego jego wyliczenia.

Teraz "trójkącik" Klasyczny $i = $u * $r, gdzie jedna jest zależna od dwóch pozostałych. Tu można prościutko polecieć zagnieżdżeniem. Sprawdzamy która z trzech wartości występuje i jeśli któraś jest, sprawdzamy czy są pozostałe. Jeśli jakiejś brak - obliczamy brakującą.

Na końcu może się okazać, że skończyliśmy tylko z $u, $i, $r dla wariantu gdy podano do funkcji $u oraz $i, więc w tym wypadku trzeba obliczyć brakujące $g;

I to by było prawie tyle... Ale gdyby ktoś podał tylko jeden parametr do funkcji? Zadanie byłoby niepoliczalne, gdyż za mało danych do policzenia wszystkich. Stąd ostatnie sprawdzenie czy wszystkie dane są niezerowe, które może rzucić wyjątek.

Wydaje się o wiele dłuższe? Pozornie. Ty masz zachodzące po kolei 9 ifów, które zawsze się wykonają, niezależnie co byś nie robił. Jak zwróciłem uwagę, niektóre wielokrotnie mogą liczyć to samo. Ja sprawdzam, które obliczenia są faktycznie niezbędne i tylko je wykonuję. Ograniczam się jedynie do niezbędnych, czyli brakujących obliczeń. Co istotne... Nie poprawiam błędów użytkownika. Jeśli wpisze głupoty w stylu: $i = 5, $u = 10, $r = 300, to Twój kod poleci i wypluje dane pozornie prawidłowe... Wiesz dlaczego? Właśnie dlatego, że się nadpisują. Ty tego nie rozumiesz, ale kod skopiowałeś skądś, gdzie taka kolejność została celowo zastosowana. Dane błędne są bowiem nadpisywane prawidłowymi. Problem z tym, że jest to bez Twojej wiedzy i nie masz kontroli nad tym, co ciąg obliczeń uzna za dane prawidłowe. Zależy to bowiem od tego, który IF wykona się pierwszy i nadpisze jaką wartość. Ja w takiej sytuacji sprawdzam, czy ewentualna wyliczona jest zgodna z podaną w parametrze. Jeśli coś jest nie tak - rzucam wyjątek zamiast liczyć głupoty.

I pamiętaj, że PHP zaokrągla! Do tego więc co mam najlepiej podejść z uwzględnieniem tego. Poczytaj o błędach zaokrągleń i jak ich uniknąć Kod myślę, że kumaci zrozumieją i nieco poprawią by działał mimo owych problemów

Odpal sobie to i testuj. Może ułatwi Ci to pracę z kodem. Ilość IF'ów minimalnie zmniejszyłem ;-). Sprawa jest o tyle ciężka, że tutaj mamy przekazane parametry, które mogą, ale nie muszą być zmieniane w trakcie wykonania funkcji dlatego do funkcji getI, getU itd. trzeba przekazać wszystkie parametry. Jest na to sposób - zastosować klasę i zmienne klasy. Powodzenia.

[PHP] pobierz, plaintext 
testLeitwerg();
 
function testSingle( $u, $r, $i, $g ) {
	$old = old_leitwert( $u, $r, $i, $g);
	$new = leitwert( $u, $r, $i, $g);
	$res = array_diff($old, $new);
	if( !empty( $res ) ){
		echo "ERR;";
 
	} else {
		echo "OK;";
	}
	echo "  params: $u, $r, $i, $g <br />"; 
	var_dump( $old );
	var_dump( $new );
 
}
 
function testLeitwerg(){
	/* testSingle( 0, 2, 2, 2 );
	testSingle( 2, 0, 2, 2 );
	testSingle( 2, 2, 0, 2 );
	testSingle( 2, 2, 2, 0 ); */
 
	testSingle( 15, 6, 2, 2 );
 	testSingle( 666, 8, 2, 2 );
	testSingle( 54, 2, 3, 6 );
	testSingle( 54544, 1, 2, 23 ); 
}
 
function old_leitwert($u, $r, $i, $g) {
	# 1
	if ( $u AND $i ) {
		$r = $u/$i;
		$g = 1/$r;
	}
 
	 # 2
	 if ( $i AND $r ) {
		$u = $r*$i;
		$g = 1/$r;
	}
 
	 # 3
	if ( $u AND $r ) {
		$i = $u/$r;
		$g = 1/$r;
	}
 
	# 4
	if ( $g AND $i ) {
		$r = 1/$g;
		$u = $r*$i;
	}
 
	#5
	if ( $u AND $r ) {
		$g = 1/$r;
		$i = $u/$r;
	}
 
	# 6
	if ( $u AND $g ) {
		$i = $u*$g;
		$r = $u/$i;
	}
 
	# 7
	if ( $i AND $r ) {
		$u = $i*$r;
		$g = 1/$r;
	}
 
	# 8
	if ( $g ) {
		$r = 1/$g;
	}
 
	# 9
	if ( $r ) {
		$g = 1/$r;
	}  
 
	return array('r' => $r, 'u' => $u, 'i' => $i, 'g' => $g );
}
 
function leitwert($u, $r, $i, $g) {
	echo 'NEW: <br />';	
	# 1
	$r = getR( $u, $r, $i, $g );
	$g = getG( $u, $r, $i, $g );
 
        # 2
	$u = getU( $u, $r, $i, $g );
	$g = getG( $u, $r, $i, $g );
 
	# 3
	$i = getI( $u, $r, $i, $g) ;
	$g = getG( $u, $r, $i, $g );
 
	# 4
	$r = getR( $u, $r, $i, $g );
	$u = getU( $u, $r, $i, $g );
 
	# 5
	$g = getG( $u, $r, $i, $g );
	$i = getI( $u, $r, $i, $g );
 
	# 6
	$i = getI( $u, $r, $i, $g );
	$r = getR( $u, $r, $i, $g );
 
	# 7
	$u = getU( $u, $r, $i, $g );
	$g = getG( $u, $r, $i, $g );
 
	# 8
	$r = getR( $u, $r, $i, $g );
 
	# 9
	$g = getG( $u, $r, $i, $g ); 
 
	return array('r' => $r, 'u' => $u, 'i' => $i, 'g' => $g );
}
 
function getR( &$u, &$r, &$i, &$g ){
	if ( $g OR ($g AND $i) ) {
		$r = 1/$g;
	}
	if ( ($u AND $g) OR ($u AND $i) ) {
		$r = $u/$i;
	} 
	return $r;
}
function getG( &$u, &$r, &$i, &$g ){
	if ( $r OR ($r AND $u) OR ($r AND $i) OR ($g AND $i) OR ($u AND $i) ) {
		$g = 1/$r;
	} 
	return $g;
}
function getU( &$u, &$r, &$i, &$g ){
	if ( $i AND ($g OR $r) ) {
		$u = $i*$r;
	}
	return $u;
}
function getI(  &$u, &$r, &$i, &$g ){
	if ( $u AND $g ) {
		$i = $u*$g;
	}
	if ( $u AND $r ) {
		$i = $u/$r;
	}
	return $i;
}
[PHP] pobierz, plaintext 

Ten post edytował LSM 26.03.2013, 10:16:12