Forum PHP.pl

Ten kot strasznie brzydki, a Ty ciągle na niego głosujesz
EDIT: Słabo Ci idzie, przeciwnik kota ciągle na plusie



Nie mam pojęcia o co Ci chodzi. To jest prosty skrypt. Fakt, że tam zapomniałem objąć to w div'a i rozlatuje się na lewo. Ale takie info jest wystarczające przy takim prostym skrypcie. Gdybym chciał naprawdę rzucić tutaj dobry skrypt to rozbudowałbym go dużo bardziej, ranking najlepiej ocenianych, zabezpieczenie przed powtórzeniem tych samych obrazków, itp. Faktem jest, że tam mogłem dać jeszcze numer błędu, a nie prosty tekst, ale to mi zupełnie wystarczyło.
Niemniej jednak uwagi są pożądane i jeżeli mogę prosić to rozwiń wypowiedź.

Jeśli koniecznie muszę coś poprawić to piszcie tutaj, albo tam w opiniach. Jutro wszystko zrobię, bo dziś mi się już nie chcę.

Ten post edytował zdemotywowany 17.01.2012, 17:17:27

Był na minusie i to dużo ;d



Głosowanie tylko raz na obrazek.

Był, ale go ogarnąłem, dostał moc wujka Google'a i miał na plusie 99995

Ja nigdzie nie napisałem, że chcę zbudować pałac. Po prostu uważam, że robiąc prosty skrypt lub może inaczej - taki, który nie sprawdzi moich umiejętności nie ma po co wkładać w niego wszystkich możliwości. Nie mam zamiaru rozbudowywać tego projektu. Poza tym jak nie ma znacznika html5? Widocznie nie wiesz jak wyglądają takie znaczniki. Jest chociażby meta. Chciałem zrobić skrypt demotywatory, który byłby rozbudowany dużo bardziej czy chciałbym czy nie. Został rzucony pomysł, postanowiłem go zrobić, ale to nie znaczy, że z kartki papieru będę robił komputer.

[PHP] pobierz, plaintext 
@ $con = mysql_connect("localhost", "root", "root") or die('Nie można połączyć się z bazą danych!');
[PHP] pobierz, plaintext 

Rzuciłeś się na coś, co już zostało wytknięte, napisałem, że to poprawię. I tak też zrobię. Aha, jeszcze co do dziwienia się. Nie wiem, wydaje mi się oczywiste, że robiąc prosty skrypt nie będę do niego robił zaplecza admina, może jeszcze powinienem zrobić panel logowania/rejestracji, bo nie jest potrzebny, ale może być.

Nie mówię, że jest super i projekt jest świetny. Zdaję sobie z tego sprawę, że jeszcze wiele przede mną. Nigdy nie robiłem czegoś takiego. Tzn. robiłem to co było w książce... Problem w tym, że właśnie w książce był utworzony projekt, były podpowiedzi co należy robić, a czego nie.

Tak czy inaczej krytykę uważam za dobrą i przydatną i na pewno dużo mi to podpowiedziało. W przyszłości będę starał się omijać takie błędy, ale nadal uważam, że mysql_connect nie musiało być bardziej rozbudowane. Tutaj użytkownik nie potrzebuje wyłapywać numeru i treści błędu. W ogóle wystarczy mu informacja, że nie działa.

Ten post edytował zdemotywowany 17.01.2012, 19:26:07

Wiesz, rzeczywiście masz tam znacznik meta w standardzie HTML5, ale to chyba nie jest sprawdzenie Twoich umiejętności. Nie twierdzę że znam HTML5, ale czytałem o nim, również dokumentację, zrobiłem prosty szablon, ale użyłem zdecydowanie więcej tego HTML5.
Nie czepiam się tej @, tylko tego że nie rozumiałeś czemu ona jest zbędna. Poza tym jak patrzyłem na Twój skrypt to po wczytaniu obrazka i naciśnieciu przycisku nie miałem żadnego info czy zauploadowało czy nie, próbowałem wczytać plik txt, żadnej odpowiedzi, dałem na opinie, pozostawiłem puste pole i nic, a to przecież długo nie zajmuje sprawdzenie czy textarea jest puste. Więc nie odbieraj moich wypowiedzi jako jakiś atak na Ciebie albo surową krytykę, tylko nie za bardzo widzę sens publikowania takich skryptów które są zrobione może w 60%, ważne jednak że Ty widzisz, ja nie muszę, prosiłeś o opinię więc napisałem. Mam nadzieję że do tego klasobook'a się bardziej przyłożysz, powodzenia!

Widzisz, ja jak coś staram się robić, to staram się to robić zawsze na takim samym poziomie, unikając elementów których nie powinno się stosować, a powstały jedynie w imię wygody domorosłych programistów.



Bo widzisz, zrobiłeś stronkę, dziobałeś sobie proceduralny kod, poświęciłeś na to 3 godziny i zrobiłeś w sumie pseudo facemash, który z facemashem ma wspólnego tylko wygląd. Tam nie są wyświetlane losowe fotki (też losowe, ale z pominięciem tych, na której już głosowałeś), a u ciebie jest to zwykły rand. Taką stronkę to sobie w kilkanaście minut można wyklikać w jakimś frameworku. Jest mnóstwo tutoriali jak zrobić coś w określonym czasie. Chociażby w Yii w godzinę wystukać blog, który IMO będzie lepiej wyglądał niż te kilka sztywnych podstron przez ciebie zrobionych.

Jak ja chciałem się sprawdzić, to napisałem swojego frameworka, jednocześnie rozkładając na czynniki pierwsze kilka najpopularniejszych. Jeżeli czujesz się dobrze w kodzie proceduralnym, czas pomyśleć o kodzie obiektowym.

Racja, teraz zauważyłem, że nie ma info na temat wgrania pliku czy opinii. Sam tego błędu nie wyłapałem, a i nikt inny też widocznie nie zauważył (lub zauważył, a nie napisał). Tego nie widziałem i oczywiście to poprawię.

@by_ikar
Ja naprawdę nie widziałem nigdy oryginalnego facemasha, wzorowałem się na stronce, która została podana w linku. Bawię się w php naprawdę nie długo i jeśli dla Ciebie pisanie framework'a było czymś dobrym na sprawdzenie to gratulacje, ja myślę, że jeszcze zbyt dużo mi brakuje żeby brać się za coś takiego. Może i ja kiedyś się za coś takiego wezmę, ale każdy zaczynał od zera. Ja uczyłem się na książce sprzed 5 lat i w3schools.

Jutro poprawię i mam nadzieję, że efekty bardziej Wam się spodobają.

Reasumując, na jutro, obsługa błędów, losowanie (tutaj nie wiem, czy ma się powtarzać najczęściej co np. 2 rundy czy w ogóle nie ma już się powtórzyć?), informacja o dodanym obrazku, wysłaniu lub nie opinii, coś jeszcze?

Co do wgrywania zdjęć użyj jakiegoś np. flashowego uploadera, ew. java - jeżeli to nie jest dostępne to dopiero wyświetl formularz w html, z odpowiednią informacją, że znacznie lepiej mieć np. flasha - tak robi np. fotka z tego co pamiętam.

Informacje o:
- Dodaniu opinii
- Dodaniu głosu
- Wgraniu pliku

Dodaj rejestracje i logowanie, tak aby można powiązać głos z użytkownikiem, tak aby zdjęcia się nie powtarzały.

Dorób też ranking zdjęć (najwyżej oceniane, najczęściej oceniane)

Samo głosowanie załatwiłbym przez ajax'a - możesz się troszkę sprawdzić z użyciem bibliotek takich jak jQuery i przy okazji troszkę JS.

Co do HTML5 - mało to ma wspólnego z HTML5, sam układ strony zrobiłeś na div'ach, co jest niedopuszczalne, gdyż są do tego odpowiednie znaczniki takie jak:
header, footer, section, nav etc...

Wydaje mi się że sama sesja z dużym czasem żywotności jako przykład powinna wystarczyć

@up
A mi się tak nie wydaje

Przepraszam, że nie wrzuciłem wczoraj mimo, iż mówiłem, że to zrobię. Nie wiem też czy dziś dam radę, ponieważ mam naciągnięty mięsień międzyżebrowy i ciężko się oddycha. A przez to komfort pracy, a w ogóle komfort wszystkiego spada do zera.

Życzymy powrotu do zdrowia