Moja pierwsza gierka :) |
Moja pierwsza gierka :) |
20.09.2012, 15:20:00
Post
#1
|
|
Grupa: Zarejestrowani Postów: 122 Pomógł: 0 Dołączył: 16.09.2012 Ostrzeżenie: (0%) |
Witam. Chciałbym państwu przedstawić moją pierwszą grę. Oto jej kod źdódłowy:
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 |
|
|
20.09.2012, 15:37:14
Post
#2
|
|
Grupa: Zarejestrowani Postów: 0 Pomógł: 1 Dołączył: 19.09.2012 Ostrzeżenie: (0%) |
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: zamiast tak:
Dodatkowo powinieneś przefiltrować otrzymane dane funkcją intval(). Wpisywanie zmiennych w stringu nie jest chyba zbyt dobrym pomysłem. Powinieneś zrobić coś takiego. I przy <form> walnąłeś gafę. Powinno być coś takiego. Kod <form method="post"> ... </form>
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 |
|
|
20.09.2012, 16:09:40
Post
#3
|
|
Grupa: Zarejestrowani Postów: 122 Pomógł: 0 Dołączył: 16.09.2012 Ostrzeżenie: (0%) |
@up
Dzięki znam teraz kilka przydatnych sposobów. Oto aktualny kod:
ma ktoś jeszcze jakieś propozycje? co dodać, zmienić itp. |
|
|
20.09.2012, 17:37:30
Post
#4
|
|
Grupa: Zarejestrowani Postów: 42 Pomógł: 9 Dołączył: 3.02.2012 Ostrzeżenie: (0%) |
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.
|
|
|
20.09.2012, 18:18:58
Post
#5
|
|
Grupa: Zarejestrowani Postów: 188 Pomógł: 0 Dołączył: 24.04.2012 Ostrzeżenie: (0%) |
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
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 -------------------- Strona o bezpieczeństwie komputerowym
http://www.materdefense.hostzi.com |
|
|
20.09.2012, 18:24:32
Post
#6
|
|
Grupa: Zarejestrowani Postów: 1 447 Pomógł: 191 Dołączył: 26.03.2008 Ostrzeżenie: (0%) |
Cytat 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. -------------------- :)
|
|
|
20.09.2012, 18:34:16
Post
#7
|
|
Grupa: Zarejestrowani Postów: 188 Pomógł: 0 Dołączył: 24.04.2012 Ostrzeżenie: (0%) |
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 -------------------- Strona o bezpieczeństwie komputerowym
http://www.materdefense.hostzi.com |
|
|
20.09.2012, 18:35:31
Post
#8
|
|
Grupa: Zarejestrowani Postów: 1 168 Pomógł: 126 Dołączył: 5.02.2010 Skąd: Świdnica Ostrzeżenie: (0%) |
Ma być czytelnie i przejrzyście żebyś po x miesiącach zajrzał i widział wszystko i wiedział co gdzie jest.
|
|
|
20.09.2012, 18:48:33
Post
#9
|
|
Grupa: Zarejestrowani Postów: 188 Pomógł: 0 Dołączył: 24.04.2012 Ostrzeżenie: (0%) |
Podsumowywując:
1. Piszesz kod od początku 2. Korzystasz z bazy danych, i musisz koniecznie użyć PDO 3. Rezygnujesz z tego:
zapisując to tak:
4. " -> ' I to tyle w tym temacie (na razie) -------------------- Strona o bezpieczeństwie komputerowym
http://www.materdefense.hostzi.com |
|
|
20.09.2012, 18:54:31
Post
#10
|
|
Grupa: Zarejestrowani Postów: 97 Pomógł: 15 Dołączył: 12.08.2012 Skąd: Zabrze Ostrzeżenie: (10%) |
if() { } Jak już powołujesz się na link, to przykład powinien wyglądać tak:
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 |
|
|
20.09.2012, 19:10:45
Post
#11
|
|
Grupa: Zarejestrowani Postów: 1 447 Pomógł: 191 Dołączył: 26.03.2008 Ostrzeżenie: (0%) |
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 -------------------- :)
|
|
|
20.09.2012, 19:21:53
Post
#12
|
|
Grupa: Zarejestrowani Postów: 188 Pomógł: 0 Dołączył: 24.04.2012 Ostrzeżenie: (0%) |
Cytat 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 -------------------- Strona o bezpieczeństwie komputerowym
http://www.materdefense.hostzi.com |
|
|
20.09.2012, 19:25:29
Post
#13
|
|
Grupa: Zarejestrowani Postów: 1 168 Pomógł: 126 Dołączył: 5.02.2010 Skąd: Świdnica Ostrzeżenie: (0%) |
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 () { } |
|
|
20.09.2012, 19:25:39
Post
#14
|
|
Grupa: Zarejestrowani Postów: 348 Pomógł: 26 Dołączył: 8.10.2008 Skąd: Lublin Ostrzeżenie: (0%) |
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. -------------------- Wolałem języki z rodziny C ale poszedłem na łatwizne...
|
|
|
20.09.2012, 19:32:01
Post
#15
|
|
Grupa: Zarejestrowani Postów: 6 366 Pomógł: 1115 Dołączył: 30.08.2006 Ostrzeżenie: (0%) |
Najlepiej zastosować się do tych https://github.com/php-fig/fig-standards/bl...-style-guide.md wytycznych i z głowy cała dyskusja.
-------------------- |
|
|
20.09.2012, 19:36:14
Post
#16
|
|
Grupa: Zarejestrowani Postów: 348 Pomógł: 26 Dołączył: 8.10.2008 Skąd: Lublin Ostrzeżenie: (0%) |
Nowa linia po nagłówkach funkcji i metod? Moim zdaniem taka sama sprawa jak z if'ami...
-------------------- Wolałem języki z rodziny C ale poszedłem na łatwizne...
|
|
|
20.09.2012, 19:46:24
Post
#17
|
|
Grupa: Zarejestrowani Postów: 915 Pomógł: 210 Dołączył: 8.09.2009 Skąd: Tomaszów Lubelski/Wrocław Ostrzeżenie: (0%) |
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?)
-------------------- |
|
|
20.09.2012, 19:47:51
Post
#18
|
|
Grupa: Zarejestrowani Postów: 6 366 Pomógł: 1115 Dołączył: 30.08.2006 Ostrzeżenie: (0%) |
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.
-------------------- |
|
|
20.09.2012, 20:08:32
Post
#19
|
|
Grupa: Zarejestrowani Postów: 122 Pomógł: 0 Dołączył: 16.09.2012 Ostrzeżenie: (0%) |
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 |
|
|
20.09.2012, 20:17:59
Post
#20
|
|
Grupa: Zarejestrowani Postów: 348 Pomógł: 26 Dołączył: 8.10.2008 Skąd: Lublin Ostrzeżenie: (0%) |
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 -------------------- Wolałem języki z rodziny C ale poszedłem na łatwizne...
|
|
|
Wersja Lo-Fi | Aktualny czas: 8.06.2024 - 07:21 |