PHP - pomoc w rozszerzeniu opcji.

Strona:
www.Seqa.pl
Kod php:

Cytuj:

---

<?php
$wynik = 0;
if($_POST['q1'] == 2) { $wynik = 1;}
if($_POST['q2'] == 2) { $wynik = $wynik+1;}
if($_POST['q3'] == 1) { $wynik = $wynik+1;}
if($_POST['q4'] == 3) { $wynik = $wynik+1;}
if($_POST['q5'] == 2) { $wynik = $wynik+1;}
if($_POST['q6'] == 3) { $wynik = $wynik+1;}
if($_POST['q7'] == 3) { $wynik = $wynik+1;}
if($_POST['q8'] == 1) { $wynik = $wynik+1;}
if($_POST['q9'] == 1) { $wynik = $wynik+1;}
if($_POST['q10'] == 2) { $wynik = $wynik+1;}
if($_POST['q11'] == 2) { $wynik = $wynik+1;}
if($_POST['q12'] == 1) { $wynik = $wynik+1;}
if($_POST['q13'] == 2) { $wynik = $wynik+1;}
if($_POST['q14'] == 3) { $wynik = $wynik+1;}
if($_POST['q15'] == 1) { $wynik = $wynik+1;}
if($_POST['q16'] == 1) { $wynik = $wynik+1;}
if($_POST['q17'] == 1) { $wynik = $wynik+1;}
if($_POST['q18'] == 2) { $wynik = $wynik+1;}
if($_POST['q19'] == 2) { $wynik = $wynik+1;}
if($_POST['q20'] == 3) { $wynik = $wynik+1;}
if($_POST['q21'] == 2) { $wynik = $wynik+1;}
if($_POST['q22'] == 1) { $wynik = $wynik+1;}
if($_POST['q23'] == 2) { $wynik = $wynik+1;}
if($_POST['q24'] == 3) { $wynik = $wynik+1;}
if($_POST['q25'] == 2) { $wynik = $wynik+1;}
if($_POST['q26'] == 1) { $wynik = $wynik+1;}
if($_POST['q27'] == 2) { $wynik = $wynik+1;}
if($_POST['q28'] == 2) { $wynik = $wynik+1;}
if($_POST['q29'] == 3) { $wynik = $wynik+1;}
if($_POST['q30'] == 2) { $wynik = $wynik+1;}
echo "Twój wynik to: ".$wynik.".<br/ >";
echo "Poprawne odpowiedzi: ".$wynik.".<br />";
$bledne = 30-$wynik;
echo "Błędne odpowiedzi: ".$bledne;
?>

---

Problem polega na tym, że chce przy wyświetleniu wyników dodać klawisz szczegóły, który będzie pokazywać, które odpowiedzi są poprawne a, które błędne.

Przepisany skrypt:

Kod:

---


<?php

$odp = array(2,2,1,3,2,3,3,1,1,2,2,1,2,3,1,1,1,2,2,3,2,1,2,3,2,1,2,2,3,2);
$poprawne = array();
$wynik = 0;
for($i = 1; $i <= 30; $i++)
{

$qz = 'q'.$i;
$k = $i - 1;

if($_POST[$qz] == $odp[$k])
{ $poprawne[] = true;  ++$wynik; } else { $poprawne[] = false; }


}

echo "Twój wynik to: ".$wynik.".<br/ >";
echo "Poprawne odpowiedzi: ".$wynik.".<br />";

if($_GET['szczegoly'] == 1)
{

for($i = 1; $i <= 30; $i++)
{

$k = $i - 1;

if($poprawne[$k] == true)
{
echo 'Zadanie '.$i.' poprawnie.<br/>';

}
else
echo 'Zadanie '.$i.' błędnie.<br/>';

}

}

$bledne = 30-$wynik;
echo "Błędne odpowiedzi: ".$bledne;

---

Zaraz to sprawdzę i z edytuje post.


@Up, chyba nie o to chodziło. Miało to wyglądać tak, że po tym jak nacisniesz na button wyślij to pokazuje Ci wyniki:

Cytuj:

---

Twój wynik to: XXX.
Poprawne odpowiedzi: XXX.
Błędne odpowiedzi: XXX

---

a pod tym klawisz szczegóły, który pokazuje dokładnie w, którym pytaniu był błąd i jaka powinna być poprawna odp.

@up
Wykodzilem mu to tak ze jak dorzuci ?szczegoly=1 to wtedy pokaze mu ktore odpowiedzi byly dobre a ktore zle ( zadanie 1 zle, zadanie 2 dobrze i tak dalej ). A chyba link.php?szczegoly=1 sam umie zrobic?

Ja mam to wrzucić w osobny plik czy podpiąć pod ten?

@Down, to moja pierwsza styczność z php, a miałem od razu robić pracę na zaliczenie praktyk :<

@up
a) Wrzuc w jeden plik zamiast Twojego skryptu ( zrob sobie backup jak chcesz ).
b) W momencie kiedy juz pokazesz wyniki, zrob pod tym link <a href="tenskrypt.php?szczegoly=1">pokaz szczegoly</a>
Wtedy jak wywola sie ponownie, z $_GET['szczegoly'] == 1, pokaze Ci ta liste.

Rozumiem ze sie dopiero uczysz? ;d

https://i.imgur.com/w7DtUp2.png

Po naciśnięciu tego buttona, wyniki pokazują mi się między poprawnymi i błędnymi odpowiedziami, nie da rady tego przenieść na koniec? I czy da radę aby pokazało co się zaznaczyło, a co się powinno zaznaczyć.

@edit. btw. szkoda, że za każdy post nie mogę dodać reputacji, zajebiście mi pomagasz :)

@edit2, okej przerobiłem to aby pokazywało w dobrej kolejności. Jakby ktoś kiedyś potrzebował :

Cytuj:

---

<?php

$odp = array(2,2,1,3,2,3,3,1,1,2,2,1,2,3,1,1,1,2,2,3,2,1, 2,3,2,1,2,2,3,2);
$poprawne = array();
$wynik = 0;
for($i = 1; $i <= 30; $i++)
{

$qz = 'q'.$i;
$k = $i - 1;

if($_POST[$qz] == $odp[$k])
{ $poprawne[] = true; ++$wynik; } else { $poprawne[] = false; }


}

echo "Twój wynik to: ".$wynik.".<br/ >";
echo "Poprawne odpowiedzi: ".$wynik.".<br />";

if($_GET['szczegoly'] == 1)
{

for($i = 1; $i <= 30; $i++)
{

$k = $i - 1;

if($poprawne[$k] == true)
{
echo 'Zadanie '.$i.' poprawnie.<br/>';
$bledne = 30-$wynik;
echo "Błędne odpowiedzi: ".$bledne;

}

else
echo 'Zadanie '.$i.' błędnie.<br/>';

}

}

---

@Edit3, dopiero teraz się szcziałem, że wgl nie wyświetla po mojej edycji ile błędnie, muszę się z tym pobawić.

Kod:

---

if($poprawne[$k] == true)
{
echo 'Zadanie '.$i.' poprawnie.<br/>';
$bledne = 30-$wynik;
echo "Błędne odpowiedzi: ".$bledne;

}

---

To jest zle, powinno byc samo:

Kod:

---

if($poprawne[$k] == true)
{
echo 'Zadanie '.$i.' poprawnie.<br/>';
}

---

Natomiast to:

Kod:

---

$bledne = 30-$wynik;
echo "Błędne odpowiedzi: ".$bledne;

---

musisz wywalic poza kontekst tej klauzuli, no bo czemu ma pokazywac ilosc blednych odpowiedzi tylko jeli ktoras odpowiedz jest dobra? To bez sensu. Postaw to po:

Kod:

---

echo "Twój wynik to: ".$wynik.".<br/ >";
echo "Poprawne odpowiedzi: ".$wynik.".<br />";

---

I bedzie dobrze :)

Cytuj:

---

<?php

$odp = array(2,2,1,3,2,3,3,1,1,2,2,1,2,3,1,1,1,2,2,3,2,1, 2,3,2,1,2,2,3,2);
$poprawne = array();
$wynik = 0;
for($i = 1; $i <= 30; $i++)
{

$qz = 'q'.$i;
$k = $i - 1;

if($_POST[$qz] == $odp[$k])
{ $poprawne[] = true; ++$wynik; } else { $poprawne[] = false; }


}

echo "Twój wynik to: ".$wynik.".<br/ >";
echo "Poprawne odpowiedzi: ".$wynik.".<br />";
$bledne = 30-$wynik;
echo "Błędne odpowiedzi: ".$bledne.".<br />";

if($_GET['szczegoly'] == 1)
{

for($i = 1; $i <= 30; $i++)
{

$k = $i - 1;

if($poprawne[$k] == true)
{
echo 'Zadanie '.$i.' poprawnie.<br/>';

}
else
echo 'Zadanie '.$i.' błędnie.<br/>';

}

}

---

Teraz mam coś takiego, i za cholerę nie mogę załapać z tymi podmianami co napisałeś :<

+ Tutaj mam filmik jak bugują się wyświetlane poprawne i błędne odpowiedzi:

http://www.youtube.com/watch?v=B89UlpMOw2M

podmień zmienną k na i

Cytuj:

---

if($poprawne[$i] == true)

---

@Up, Nie wiem co miało to poprawić ale nic nie zmieniło :<

@Topic, wygląda to jakby szczegóły nie pobierały danych z tego :(

Cytuj:

---

filip955 napisał

podmień zmienną k na i

---

k jest po to aby indeksowalo od 0 a nie od 1, bo arraye sie indeksuja od 0, a on wysyla odpowiedzi od 1.

Kod PHP:

---

<?php $odp = array(2,2,1,3,2,3,3,1,1,2,2,1,2,3,1,1,1,2,2,3,2,1,2,3,2,1,2,2,3,2); $poprawne = array(); $_bledne = array(); $wynik = 0; for($i = 1; $i <= 30; $i++) { $qz = 'q'.$i; $k = $i - 1; if(isset($_POST[$qz]) ? $_POST[$qz] == $odp[$k] : false) { $poprawne[$i] = true; ++$wynik; } else { $poprawne[$i] = false; $_bledne[$k] = isset($_POST[$qz]) ? $_POST[$qz] : false; } } if(isset($_GET['szczegoly']) ? $_GET['szczegoly'] == 1 : false) { for($i = 1; $i <= 30; $i++) { $k = $i - 1; if($poprawne[$i] == true) { echo 'Zadanie '.$i.' poprawnie ('. $odp[$k] .')<br/>'; } else { echo 'Zadanie '.$i.' błędnie. (<strike>'.$_bledne[$k].'</strike> <b>'. $odp[$k] .'</b>)<br/>'; } } } echo "<hr>Twój wynik to: ".$wynik.".<br/ >"; echo "Poprawne odpowiedzi: ".$wynik.".<br />"; $bledne = 30-$wynik; echo "Błędne odpowiedzi: ".$bledne; ?>

---

Troche zmodyfikowalem wasz kod.

@Up, kod boski :D, tylko dalej problem, że nie pokrywa się z tym co było przed rozwinięciem szczegółów. Screen:

https://i.imgur.com/DhS74Wk.png

Refresh, wiem, że nie po 24/48h ale na poniedziałek muszę mieć to zrobione :(

Mógłby ktoś pomóc to ogarnąć? Zostało mi mało czasu, a fajny patent z tymi szczegółami.

Hej.

Z racji Twojej "wyższej" potrzeby, przepisałem dla Ciebie Twój kod. Powinien teraz zwracać to, czego oczekujesz. Pisałem go bez sprawdzania, mogą być błędy składni - mam nadzieję, że jeżeli takie będą, to uda Ci się je poprawić.

Dorzucę parę uwag:

• Zawsze warto "wyciągać" powtarzalne kawałki programu i przekształcenia danych wejściowych w postaci funkcji. Nazwy funkcji powinny dotyczyć Twojej domeny problemu - pisząc np. aplikację dla komisu samochodowego, nie nazywaj funkcji sortującej po cenie samochodu "posortujTablicę", lecz "posortujSamochody".
• Korzystamy tutaj z konstrukcji foreach. Jest to przystosowana wersja pętli for, która operuje na kolekcjach - w tym wypadku tablicy.
• Wartość zwracana funkcji "sprawdzOdpowiedzi" to złożony typ danych. Staraj się zawsze modelować zwracane dane w podobny sposób - słownik (hash) jest dosyć wygodnym sposobem na tworzenie bardziej skomplikowanych typów danych - takich jak tutaj. Zwracamy tak naprawdę cztery odpowiedzi jednocześnie - liczbę błędnych i poprawnych odpowiedzi, oraz odpowiedzi użytkownika i poprawne odpowiedzi.
• Staraj się za wszelką cenę unikać mieszania do PHP elementów składni HTMLa. PHP powinien pracować na Twoich danych. Nie jest stworzony do generowania kodu HTML. W przypadku, gdy musisz wzbogacić stronę o elementy HTML, po prostu zakończ znacznik kodu PHP i zacznij pisać czysty HTML. Jest to czystsze i bardziej eleganckie rozwiązanie.
• Twój "kod HTML", o ile można go tak nazwać nie był poprawnym dokumentem HTML. Stworzyłem dla Ciebie poprawną strukturę.

A oto kod:

Kod PHP:

---

<?php function poprawne($odpowiedz, $poprawna) { return $odpowiedz == $poprawna; } function sprawdzOdpowiedzi($tablicaOdpowiedzi) { /** * Tablica poprawnych odpowiedzi dla pytań. **/ $poprawneOdpowiedzi = Array(2,2,1,3,2,3,3,1,1,2,2,1,2,3,1,1,1,2,2,3,2,1,2,3,2,1,2,2,3,2); /** * Stwórzmy sobie słownik, który będzie nam reprezentował odpowiedź. **/ $wynik = Array( "odpowiedzi" => Array(), "poprawne" => $poprawneOdpowiedzi, "iloscBlednych" => 0, "iloscPoprawnych" => 0, ); foreach($poprawneOdpowiedzi as $indeks => $poprawnaOdpowiedz) { $klucz = "q" . strval($indeks+1); $wynik["odpowiedzi"][] = $_POST[$klucz]; if(poprawne($_POST[$klucz], $poprawnaOdpowiedz)) { $wynik["iloscPoprawnych"]++; } else { $wynik["iloscBlednych"]++; } } return $wynik; } function szczegolyRozwiniete() { return isSet($_GET['szczegoly']) && $_GET['szczegoly'] == 1; } $wynikTestu = sprawdzOdpowiedzi($_POST); ?> <!doctype html> <html> <head> <title>Wyniki Testu</title> <meta charset="UTF-8" /> </head> <body> <section class="results"> <p>Twój wynik to: <?php echo $wynikTestu["iloscPoprawnych"]; ?></p> <p>Liczba poprawnych odpowiedzi: <?php echo $wynikTestu["iloscPoprawnych"]; ?></p> <p>Liczba błędnych odpowiedzi: <?php echo $wynikTestu["iloscBlednych"]; ?></p> </section> <?php if(szczegolyRozwiniete()) { ?> <section class="details"> <ol> <?php foreach($wynikTestu["odpowiedzi"] as $indeks => $odpowiedz) { $wynikowyTekst = ""; $poprawnaOdpowiedz = $wynikTestu["poprawne"][$indeks]; if(poprawne($odpowiedz, $poprawnaOdpowiedz)) $wynikowyTekst = "<strong>Poprawnie.</strong> Odpowiedź: $odpowiedz"; else $wynikowyTekst = "<strong>Błędnie.</strong> Odpowiedź: $odpowiedz, Powinno być: $poprawnaOdpowiedz."; ?> <li><?php echo $wynikowyTekst; ?></li> <?php } ?> </ol> </section> <?php } else { ?> <section class="details"> <p><a href="?szczegoly=1">Pokaż szczegóły</a></p> </section> <?php } ?> </body> </html>

---

No i powodzenia z PHP. Po pewnym czasie stwierdzisz (całkiem zresztą słusznie), że PHP jest bardzo męczącym językiem i istnieją lepsze alternatywy używające wygodniejszych języków programowania, a realizujących podobne funkcje - przetwarzanie żądań Twoich użytkowników. Są to tzw. frameworki sieciowe.

Niemniej jednak, życzę Ci wszystkiego dobrego na "programistycznej stronie życia" :).

Przydatne linki:
http://me.veekun.com/blog/2012/04/09...of-bad-design/
http://stackoverflow.com/questions/1...input-with-php (Trochę o walidacji [sprawdzaniu] danych, które wysyła Ci użytkownik. Ten kod akurat tego nie potrzebuje (za bardzo), aczkolwiek brakuje mu tego i jest to bardzo ważna rzecz.)

Pozdrawiam
Killavus