Forum PHP.pl

Kod:

[PHP] pobierz, plaintext 
if($pyt1 == 2) { $punkty++; }
if($pyt2 == 4) { $punkty++; }
if($pyt3 == 1) { $punkty++; }
if($pyt4 == 4) { $punkty++; }
if($pyt5 == 2) { $punkty++; }
if($pyt6 == 3) { $punkty++; }
if($pyt7 == 2) { $punkty++; }
if($pyt8 == 4) { $punkty++; }
if($pyt9 == 3) { $punkty++; }
if($pyt10 == 4) { $punkty++; }
if($pyt11 == 1) { $punkty++; }
if($pyt12 == 5) { $punkty++; }
if($pyt13 == 5) { $punkty++; }
if($pyt14 == 4) { $punkty++; }
if($pyt15 == 2) { $punkty++; }
if($pyt16 == 4) { $punkty++; }
if($pyt17 == 4) { $punkty++; }
if($pyt18 == 3) { $punkty++; }
if($pyt19 == 4) { $punkty++; }
if($pyt20 == 1) { $punkty++; }
[PHP] pobierz, plaintext 

Czy można zapisać ten kod inaczej, krócej, niż używając 20 if'ów?

do usunięcia

Ten post edytował Turson 15.01.2014, 14:59:43

[PHP] pobierz, plaintext 
<?php
$answers = array(2, 4, 1, 4, 2, 3, 2, 4, 3, 4, 1, 5, 5, 4, 2, 4, 4, 3, 4, 1);
for($i=0; $i<20;$i++){
	if(${"pyt" . $i+1} == $answers[$i]){
		$punkty++;
	}
}
 
[PHP] pobierz, plaintext 

Moze nie działac, bo z palca.

Ten post edytował werdan 15.01.2014, 15:22:25

@werdan
Trzeba było mu tego nie pisać bo wklei zadziała i uzna że ma dobrze napisany kod. Crozin ma 100% racji autor powinien użyć tablicy nie faktu że php jest językiem dynamicznie wykonywanym i używać "magi"

@Topic

[PHP] pobierz, plaintext 
<?php
 
$correctAnswer = array(2, 4, 1, 4, 2, 3, 2, 4, 3, 4, 1, 5, 5, 4, 2, 4, 4, 3, 4, 1);
$userAnswer = array(2, 4, 1, 4, 2, 3, 2, 4, 3, 4, 1, 5, 5, 4, 2, 4, 4, 3, 4, 1);
 
 
/**
 * 	Zwraca ilość takich samych pól w tabelach (bierze pod uwagę kolejność indexów)
 * @param Array $corrctArray Tabela przykładowa (do niej porównujemy)
 * @param Array $userArray Tabela od użytkownika
 */
function getPointFromAnswer(Array $correctArray, Array $userArray)
{
	$countCorrectArray = count($correctArray);
 
	if( $countCorrectArray != count($userArray) )
		throw new \InvalidArgumentException('Tablice posiadają różne elementy');
 
	$pkt = 0;
	for($i = 0; $i < $countCorrectArray; $i++)
		if( $correctArray[$i] === $userArray[$i] )
			$pkt++;
 
	return $pkt;
}
 
var_dump( getPointFromAnswer($correctAnswer, $userAnswer) );
[PHP] pobierz, plaintext 

W kodzie dałem "===" ponieważ chciałem być pewny że mi jakimś cudem int'a z tablicą nie sprawdzi.

Ten post edytował YourFrog 15.01.2014, 19:25:16

albo tak:
wyliczasz część wspólną 2 tablic, za pomocą funkcji array_intersect_assoc, a później patrzysz ile elementów ma częśc wspólna. I tyle.

[PHP] pobierz, plaintext 
 
// 20 elementow
$correctAnswer = array(2, 4, 1, 4, 2, 3, 2, 4, 3, 4, 1, 5, 5, 4, 2, 4, 4, 3, 4, 1); 
$userAnswer = array(2, 4, 1, 4, 2, 3, 10000/*bledna odp*/, 4, 2039/*bledna odp*/, 4, 1, 5, 5, 4, 2, 4, 4, 3, 4, 1);
 
$points = count(array_intersect_assoc($userAnswer, $correctAnswer));
echo "Masz $points poprawnych odpowiedzi"; 
[PHP] pobierz, plaintext