Moja pierwsza gierka :) Opcje

[Th0e]

Witam. Chciałbym państwu przedstawić moją pierwszą grę. Oto jej kod źdódłowy:

[PHP] pobierz, plaintext 
<?php
$level = file_get_contents('lvl.txt');
$exp = file_get_contents('exp.txt');
$hp = file_get_contents('hp.txt');
$zloto = file_get_contents('zloto.txt');
if ($exp > 99) //po zdobyciu ponad 99 expa daje 2 lvl
{
$level = 2;
file_put_contents("lvl.txt",$level);
}
 
if ($exp > 199) //po zdobyciu ponad 99 expa daje 2 lvl
{
$level = 3;
file_put_contents("lvl.txt",$level);
}
 
if ($exp > 399) //po zdobyciu ponad 99 expa daje 2 lvl
{
$level = 4;
file_put_contents("lvl.txt",$level);
}
 
if ($exp > 699) //po zdobyciu ponad 99 expa daje 2 lvl
{
$level = 5;
file_put_contents("lvl.txt",$level);
echo "maj dik gut kak <br />";
}
?>
 
<h1>walka</h1>
<p>Aby rozpoczac walke musisz wpisac "1" i kliknac "walcz".<p>
<form method="post" />
<input type="text" name="wartosc" />
<input type="submit" value="walcz!" />
 
<?php
$potworek = 10;
 
$otrzymane = $_POST['wartosc'];
 
if($otrzymane == 2) // po wpisaniu 2 zresetuje EXP
$expplik = fopen("exp.txt", "w");
 
if($otrzymane == 1) // po wpisaniu 1 zaatakuje potworka
{
if ($hp >10) //sprawdza czy masz wystarczajaco HP by walczyc
{
$exp = $exp+$potworek;
$zloto += 50;
file_put_contents("exp.txt",$exp); // zapisze dodane exp do pliku z expem
file_put_contents("zloto.txt",$zloto); // zapisze dodane zloto do pliku
}
}
 
if ($hp >10)
{$hp -=10; //odbierze 10hp
file_put_contents("hp.txt",$hp); //zapisze stan HP w pliku
}
else
{
echo "nie masz już zycia";
} 
?>
 
<h1>uzdrowiciel </h1>
<p>Aby sie uzdrowic musisz wpisac ponizej "1" i kliknac "uzdrow". koszt 100zlota</p>
<form method="post" />
<input type="text" name="uzdrowiciel" />
<input type="submit" value="uzdrow" />
 
<?php
$uzdrowiciel=$_POST['uzdrowiciel'];
if ($uzdrowiciel == 1)
{
 
if($zloto > 100)
{$hp = 100;
$zloto -= 100;
file_put_contents("zloto.txt",$zloto);
file_put_contents("hp.txt",$hp);
}
 
}
?>
 
<h2>statystyki postaci</h2>
<?php
echo "poziom $level <br />";  //pokazuje lvl
echo "doswiadczenie $exp <br />"; //pokazuje EXP
echo "zycie $hp <br />"; //pokazuje zycie
echo "zloto $zloto"; //pokazuje zloto
?>
[PHP] pobierz, plaintext 

aby w nią pograć, należy w folderze symulatora serwera stworzyć plik php i wkleić do niego kod. potem utwórz 4 pliki tekstowe o nazwach:
hp --- w nim wpisz 100
level ---- w nim wpisz 1
exp ---- w nim wpisz 0
zloto ---- w nim wpisz 0

Teraz odpal symulator serwera i wpisz w przeglądarkę localhost

Żałosny kicz co nie? kicz, ale mój pierwszy <3

proszę o niezbyt bolesne krytyki, oraz rady i propozycje co dodać, w ten sposób odkryłem że świetnie się szkolę

Ten post edytował Th0e 20.09.2012, 15:21:20

[SHINvain]

Moim zdaniem powinieneś zamienić formularze na bardziej intuicyjny, np.

Kod

<select name="wartosc">
    <option value="1">Walcz</option>
    <option value="2">Zresetuj doświadczenie</option>
</select>

Albo przy uzdrawianiu i tak masz jeden wybór, więc.

Kod

<form method="post">
<input type="hidden" name="uzdrowiciel" value="1" />
<input type="submit" name="submit" value="uzdrow" />
</form>

Albo zostawić samo pole "submit" i wtedy możesz sprawdzić to tak:

[PHP] pobierz, plaintext 
if(isset($_POST['submit']))
[PHP] pobierz, plaintext 

zamiast tak:

[PHP] pobierz, plaintext 
if ($uzdrowiciel == 1)
[PHP] pobierz, plaintext 

Dodatkowo powinieneś przefiltrować otrzymane dane funkcją intval().

[PHP] pobierz, plaintext 
$otrzymane = intval($_POST['wartosc']);
[PHP] pobierz, plaintext 

Wpisywanie zmiennych w stringu nie jest chyba zbyt dobrym pomysłem. Powinieneś zrobić coś takiego.

[PHP] pobierz, plaintext 
echo "poziom " . $level . " <br />";
[PHP] pobierz, plaintext 

I przy <form> walnąłeś gafę. Powinno być coś takiego.

Kod

<form method="post">
...
</form>

[PHP] pobierz, plaintext 
if ($exp > 699) //po zdobyciu ponad 99 expa daje 2 lvl
[PHP] pobierz, plaintext 

To 699 czy 99?

Pokombinowałbym przy tym, aby zapisać wszystkie statystyki w jednym pliku albo bazie danych (dużo prostszy mechanizm).

Ten post edytował SHINvain 20.09.2012, 15:38:15

[Th0e]

@up

Dzięki znam teraz kilka przydatnych sposobów.

Oto aktualny kod:

[PHP] pobierz, plaintext 
<?php
$level = file_get_contents('lvl.txt');
$exp = file_get_contents('exp.txt');
$hp = file_get_contents('hp.txt');
$zloto = file_get_contents('zloto.txt');
if ($exp > 99) //po zdobyciu ponad 99 expa daje 2 lvl
{
$level = 2;
file_put_contents("lvl.txt",$level);
}
 
if ($exp > 199) //po zdobyciu ponad 199 expa daje 3 lvl
{
$level = 3;
file_put_contents("lvl.txt",$level);
}
 
if ($exp > 399) //po zdobyciu ponad 399 expa daje 4 lvl
{
$level = 4;
file_put_contents("lvl.txt",$level);
}
 
if ($exp > 699) //po zdobyciu ponad 699 expa daje 5 lvl
{
$level = 5;
file_put_contents("lvl.txt",$level);
echo "maj dik gut kak <br />";
}
?>
 
<h1>walka</h1>
 
<form method="post">
<select name="wartosc">
    <option value="1">Walcz</option>
    <option value="2">Zresetuj doświadczenie</option>
</select>
	<input type="submit" value="wykonaj">
	</form>
<?php
$potworek = 10;
 
$otrzymane = $_POST['wartosc'];
 
if($otrzymane == 2) // po wpisaniu 2 zresetuje EXP
$expplik = fopen("exp.txt", "w");
 
if($otrzymane == 1) // po wpisaniu 1 zaatakuje potworka
{
if ($hp >10) //sprawdza czy masz wystarczajaco HP by walczyc
{
$exp = $exp+$potworek;
$zloto += 50;
file_put_contents("exp.txt",$exp); // zapisze dodane exp do pliku z expem
file_put_contents("zloto.txt",$zloto); // zapisze dodane zloto do pliku
}
}
 
if ($hp >10)
{$hp -=10; //odbierze 10hp
file_put_contents("hp.txt",$hp); //zapisze stan HP w pliku
}
else
{
echo "nie masz już zycia";
} 
?>
 
<h1>uzdrowiciel </h1>
 
<form method="post">
<input type="hidden" name="uzdrowiciel" value="1" />
<input type="submit" name="submit" value="uzdrow" />
</form>
<?php
$uzdrowiciel=$_POST['uzdrowiciel'];
if ($uzdrowiciel == 1)
{
 
if($zloto > 100)
{$hp = 100;
$zloto -= 100;
file_put_contents("zloto.txt",$zloto);
file_put_contents("hp.txt",$hp);
}
 
}
?>
 
<h2>statystyki postaci</h2>
<?php
echo "poziom ". $level . "<br />";  //pokazuje lvl
echo "doswiadczenie" .$exp. "<br />"; //pokazuje EXP
echo "zycie ". $hp ." <br />"; //pokazuje zycie
echo "zloto" . $zloto; //pokazuje zloto
 
?>
[PHP] pobierz, plaintext 

ma ktoś jeszcze jakieś propozycje? co dodać, zmienić itp.

[red.orel]

Kod się wykonuje? Jeśli tak to jest poprawny. Jakościowo słabo. Pisać może każdy, jednak mało kto robi to sensownie. Stwórz przynajmniej schemat blokowy na kartce papieru.

[materkamil]

[PHP] pobierz, plaintext 
$uzdrowiciel=$_POST['uzdrowiciel'];
[PHP] pobierz, plaintext 

1. Nie cierpię tych tutoriali o php z uw-team.org -> własnie one wprowadzają w błąd i wymyślają te przypisywania zmiennych. Założę się że to je właśnie oglądałeś.

2. Korzystaj z bazy danych a nie z plików txt

[PHP] pobierz, plaintext 
$otrzymane = $_POST['wartosc'];
[PHP] pobierz, plaintext 

3. A co gdy nie istnieje w poście wartosc? Wywala błąd undefined index dlatego dodaj tam ifa z isset.

4. Nie wprowadzaj tylu zmiennych!

5. Zamiast " pisz ' - to lepiej wygląda.

6.

Nie: if()
{

tylko

if() {

Racja, trzeba się zabrać od nowa za ten kod.

Kiedyś pisałem coś podobnego, teraz sam nie mogę w to uwierzyć, jak można pisać taki kod

[peter13135]

5. Zamiast " pisz ' - to lepiej wygląda.

6.

Nie: if()
{

tylko

if() {

5. Mi się akurat bardziej podoba cudzysłów Stosowanie apostrofu zamiast cudzysłowiu jest lepsze, ale nie ze względów estetycznych
6. A to niby czemu ? Są "dwie szkoły" stawiania klamerek, trudno rozsądzić która lepsza, więc niech sobie pisze tak jak chce.

[materkamil]

5. Tak, wiem, chodzi tutaj o setne sekundy optymalności. Ale już się nie zajmujmy tak drobnym czasem - tłumacząc początkującemu mówię prosto
6. Zdziwiony jestem twoją wypowiedzią. W przypadku "dwóch szkół" sięgnijmy do szkoły główne czyli do strony php.net o warunkach:
http://php.net/manual/en/control-structures.if.php

W przykładach pięknie pokazane jest że najbardziej prawidłową konstrukcją, używaną przez twórców php jest:

if() {
}

A konstrukcja:

if()
{
}

jest jedynie wymysłem programistów którzy piszą w PHP

[Szymciosek]

Ma być czytelnie i przejrzyście żebyś po x miesiącach zajrzał i widział wszystko i wiedział co gdzie jest.

[materkamil]

Podsumowywując:
1. Piszesz kod od początku
2. Korzystasz z bazy danych, i musisz koniecznie użyć PDO
3. Rezygnujesz z tego:

[PHP] pobierz, plaintext 
$exp = $exp+$potworek;
file_put_contents("exp.txt",$exp); 
[PHP] pobierz, plaintext 

zapisując to tak:

[PHP] pobierz, plaintext 
file_put_contents("exp.txt",$exp+$potworek); 
[PHP] pobierz, plaintext 

4. " -> '

I to tyle w tym temacie (na razie)

[pamil]

if() {
}

Jak już powołujesz się na link, to przykład powinien wyglądać tak:

[PHP] pobierz, plaintext 
/* po słowie kluczowym spacja */
if ($condition === true && evaluate($sth) >= 2) {
 
}
[PHP] pobierz, plaintext 

Polecam zapoznać się z dokumentami PSR-*.

A tak, jak już się czepiamy standardów formatowania kodu - stawiaj spację po słowach kluczowych, przecinkach oddzielających argumenty. Stawiaj spację przed i po operatorach wszelakich (przypisania, porównania, etc.).

Poza tym DRY, w fragmencie z zmienianiem poziomu doświadczenai zawsze wykonujesz file_put_contents().

Ten post edytował pamil 20.09.2012, 18:59:23

[peter13135]

5. Tak, wiem, chodzi tutaj o setne sekundy optymalności. Ale już się nie zajmujmy tak drobnym czasem - tłumacząc początkującemu mówię prosto
6. Zdziwiony jestem twoją wypowiedzią. W przypadku "dwóch szkół" sięgnijmy do szkoły główne czyli do strony php.net o warunkach:
http://php.net/manual/en/control-structures.if.php

W przykładach pięknie pokazane jest że najbardziej prawidłową konstrukcją, używaną przez twórców php jest:

if() {
}

A konstrukcja:

if()
{
}

jest jedynie wymysłem programistów którzy piszą w PHP

5. Ale akurat czytelność tu nie gra żadnej roli, apostrof wcale nie jest czytelniejszy od cudzysłowiu. Moim zdaniem cudzysłów może być bardziej czytelny i intuicyjny ze względu na to, że stringu w c,c++, java i pewnie wielu innych językach stringi są właśnie w cudzysłowach, a nie w apostrofach.
Uważasz, że gdybyś powiedział początkującemu, że zapis w apostrofie jest optymalniejszy to by było dla autora trudne do zrozumienia i dla tego trzema mu wciskać, że to jest brzydki zapis ?
6. Nie bardzo rozumiem o co chodzi z tą "szkołą główną php.pl". Klamerki nie są tylko własnością języka php, tylko ogólnie przyjętym standardem w językach, który chyba zapoczątkował język C (mogę się mylić). Poza tym, zarówno w dokumentacji php (ogólnie) jak i w tym konkretnym artkule dotyczącym ifów, który przytoczyłeś, oraz kodach pisanych w innych językach, w których blok kodu jest w klamrach są stosowane obydwie wersje formatowania. Czy to formatowanie, gdzie klamra otiwerająca jest w nowej linii jest wymysłem php tego nie wiem i nijak nie ma się do rzeczy.
Obie wersje formatowania mają swoich zwolenników, myślę, że ilościowo mniej więcej dzielą się po równo, a więc obie wersje są dopuszczalne, bo nie utarł się jeden schemat, który dla wszystkichjest czytelny. Ty wolisz taką wersję i masz na to swoje argumenty, ja wolę inną wersję i też mam na to swoje argumenty. To sprawa gustu, a nie poprawności kodu.

Tak samo są samochody z automatyczną skrzynią biegów jak i z manualną. Są tacy, któzy lubią sobie powajchować, są też tacy, którzy wolą trzymać kierownicę dwoma rękoma. Nie sądzisz, że śmiesznie by było, jakby kierowca lubujący się w automatach krytykował samochody z ręczną skrzynią biegów, albo ich kierowców ?

Również się zgadzam, że ma być czytelnie i przejrzyście. A dla orientacyjnie połowy programistów, czytelnie jest wtedy, gdy klamra otwierająca jest w nowej linijce.

PS. nie sądziłem, że znajdę Twój nick na tym forum w innym temacie niż w HP->HUMOR

Ten post edytował peter13135 20.09.2012, 19:13:03

[materkamil]

PS. nie sądziłem, że znajdę Twój nick na tym forum w innym temacie niż w HP->HUMOR

?

Jeśli chodzi o ify, jest to prawidłowe gdy piszemy sami skrypt. Jednak gdy tworzymy skrypt w grupie nie pasuje używać różnych stylów i bardziej pasuje używać jednego, najpopularniejszego if() {
}

Przykład: kod wordpressa, joomli, itp.

Jednak, jak kto woli. I tutaj zgadzam się z wami

Ten post edytował materkamil 20.09.2012, 19:22:12

[Szymciosek]

Nigdy nie wiesz do jakiej grupy trafisz więc raczej ciężko ocenić jak lepiej a jak gorzej, taki zapis jest prosty do ewentualnie przestawienia się i pisania inaczej więc raczej nie robi to problemu gdybyś pisał:
if () {
}

a trafił na grupę, która pisze
if ()
{
}

[Adi32]

Z początku wydawało mi się, że ktoś przeniesie do ocen ale z biegiem czasu temat zaczął bardziej pasować do hydeparku.
Co do if'ów to nie spotkałem się z nikim kto stosowałby 2 linijkowe rozwiązanie, poza jakimiś jednoosobowymi kursami itp.
Za dziwne uważam, że komuś może poprawić czytelność przeniesienie '{' fo nowej linij, sądzę, że to kwestia przyzwyczajenia oczywiście nie chce nikogo obrazić.
Dodatkowo kod jest krótszy choć to słaby argument.

[viking]

Najlepiej zastosować się do tych https://github.com/php-fig/fig-standards/bl...-style-guide.md wytycznych i z głowy cała dyskusja.

[Adi32]

Nowa linia po nagłówkach funkcji i metod? Moim zdaniem taka sama sprawa jak z if'ami...

[redeemer]

Mała dygresja z mojej strony: kiedyś, dobrych parę lat temu jak pisałem w C nie wyobrażałem sobie pisać inaczej niż { w nowej linii. Teraz jest zupełnie na odwrót @Adi32 kod jest krótszy, ale w takim sensie, że więcej go widać na ekranie i jest to jeden IMHO z ważniejszych argumentów za tym stylem. Podoba mi się rozwiązanie z Pythona, gdzie całkowicie zrezygnowano z instrukcji określających początek i koniec bloku, a tą rolę pełnią zwyczajnie "wcięcia" (które i tak robimy, a przynajmniej powinniśmy). Nie programowałem jednak w tym języku i nie wiem na ile jest to wygodne i czy nie wiąże się z jakimiś uciążnieniami (taby? spacje? ile spacji? szybkie przenoszenie na koniec/początek bloku w IDE?)

[viking]

Dziwne to trochę (osobiście też wolę w jednej linii) ale zarówno Zend jak i Symfony a pewnie jeszcze masa innych projektów się do tego stosuje, łatwo znaleźć ustawienia dla edytorów więc cóż.. jak standard to standard.

[Th0e]

ludzie co to za dyskusja.
piszę { od nowej linii, bo dzięki temu kod zdaje mi się być czytelniejszy, z resztą jeśli ktoś się wczytał to zauważył że ogólnie lubię zamieszczać jak najmniej danych w linii
Skoro to taka różnica to od dziś będę pisał if(){
załatwione
Wiem że mogłem to zrobić z bazami, ale jeszcze nie miałem styczności z MYSQL, poza tym to było robione tak 4fun by utrwalić wiedzę po przebrnięciu jakimś cudem przez 5 rozdziałów kursu: http://kursphp.com

a co do projektu to jeszcze coś wstawię, na razie nie mam pojęcia co

[Adi32]

Też zaczynałem od pisania takich rzeczy.
Zapoznaj się z MySQL'em - podstawy pracy z bazą danych są prostsze niż się to wcześniej wydaje

I chyba trochę przesadzasz w rzucaniu pomagajkami.

Ten post edytował Adi32 20.09.2012, 20:18:55