Strona: CorelisLayout zakodowana (hamachi)

[gryzon95]

Witajcie. 

 

Jakiś czas temu zacząłem naukę z php i ogólnie kodowaniem stron www. Postanowiłem więc zakodować jakąś gotową grafikę strony www i oto mój pierwszy projekt. Nie bardzo wiedziałem czy to dobre miejsce na publikowanie mojej pierwszej pracy, ale zależy mi głównie na szczerej opinii a z tego co widziałem taką tu można otrzymać

 

Dlatego też wrzucam tą stronę tutaj i czekam na komentarze, porady wskazówki itp. itd. 

A być może w jakiś sposób przyda się komuś ta strona.

 

 

Download: 

http://www87.zippyshare.com/v/oZ1HFJOk/file.html

Skan:

https://www.virustotal.com/pl/file/be3dd7ac49ea8038191d6ff88ad43f4d6aa80ce7f9c4a5d7d9a1b775d03209d1/analysis/1451245844/ 

Jeśli o czymś zapomniałem to dajcie znać poprawię/dodam.

 

 

[SzeяyF]

Zawsze od czegoś trzeba zacząć zostawiam like

[SzeяyF]

ps wrzuciłem ją zrobiłem config i na stronie wyskakuje Error: mysqli_connect_error

[Rotcher]

Jakoś dziwnie napisany ten skrypt jak dla mnie i w każdym pliku ten config.php no cóż każdy zaczyna od czegoś lecz myślę jak na początek to i tak bardzo ładnie

Mam tylko pytanie czy wzorowałeś się na jakiś skryptach innych stron?

[Marco Bizzarri]

Jest kilka błędów, jednak strona jak na pierwszy raz napisana bardzo dobrze.

$strona = htmlentities($_GET['strona'],ENT_QUOTES, "UTF-8");

W takim wypadku lepiej zastosować basename niż htmlentities ze względów bezpieczeństwa.

$email = $_POST['email'];
$tytul = 'Przypomnienie hasła na serwerze Nazwa Serwera ';
$data = date('Y-m-d H:i:s');
$naglowek = "Od: Nazwa Serwera ; Temat: Zmiana Adresu E-mail ; Data wyslania: ".$data. " ; \n  ";
$naglowek .= "MIME-Version: 1.0\n";
$naglowek .= "Content-Type: text/html;\n";
$naglowek .= "\tcharset=\"UTF-8\"\n";

$polaczenie->select_db("account");
$dane=$polaczenie->query(sprintf("SELECT * FROM account where email='%s'",
														mysqli_real_escape_string($polaczenie,$email)));

Nie trzeba się koniecznie tak bawić, wystarczy wrzucić $email w mysql_real_escape_string i możesz bezpiecznie wkleić to do zapytania. Oczywiście to tylko uwaga ze względu na mój minimalizm, jak to zrobisz to już zależy od Ciebie, kawałek kodu który wkleiłem też jest dobry.

elseif (preg_match('/^[a-zA-ZąćęłńóśźżĄĆĘŁŃÓŚŹŻ0-9\-\_\.]+\@[a-zA-ZąćęłńóśźżĄĆĘŁŃÓŚŹŻ0-9\-\_\.]+\.[a-z]{2,4}$/',$email) == false)

Moim zdaniem lepiej skorzystać z gotowej funkcji:

elseif (!filter_var($email, FILTER_VALIDATE_EMAIL))

! tyczy się tego samego co false

 

Ogólnie jeśli miałbym podsumować cały kod to pierwszą rzeczą jaką bym zmienił to biblioteka obsługująca funkcje bazy danych. Preferowałbym PDO zamiast mysql/mysqli, bo pozwala na wklepanie danych do bazy, a nie do zapytania. Dobrą sprawą jest też trzymanie sesji w bazie danych dla samej wygody jak i load-balancingu.

 

Nie programujesz przypadkiem w c++?

[gryzon95]

Nie wzorowałem się na skryptach innych stron, po prostu tak jakoś sobie to po układałem i klepałem. :p 

 

Serdecznie dziękuję za uwagi i wskazówki, na pewno jest wykorzystam Tą stronę tworzyłem od jakiegoś czasu, na chwilę też przerywałem jej tworzenie a w tym czasie czytałem, oglądałem różne filmiki itp. dotyczące php, więc wpadały mi nowe pomysły jak mógłbym pisać "lepiej", ale te pomysły spróbuje wykorzystać w następnym projekcie Tą stronę chciałem już dokończyć tak jak zacząłem. Na pewno postaram się bardziej po układać cały kod i użyć więcej funkcji, ponieważ w kilku miejscach użyłem praktycznie tego samego kodu :p

 

Marco Bizzarri naukę z programowaniem zaczynałem od c++, jeszcze się w nim bawię, ponieważ mam go w planie na studiach :p. A skąd to pytanie ?

 

Szczerze mówiąc nie spodziewałem się takich opinii, ale czekam na dalsze

[Marco Bizzarri]

Nie wzorowałem się na skryptach innych stron, po prostu tak jakoś sobie to po układałem i klepałem. :p 

 

Serdecznie dziękuję za uwagi i wskazówki, na pewno jest wykorzystam Tą stronę tworzyłem od jakiegoś czasu, na chwilę też przerywałem jej tworzenie a w tym czasie czytałem, oglądałem różne filmiki itp. dotyczące php, więc wpadały mi nowe pomysły jak mógłbym pisać "lepiej", ale te pomysły spróbuje wykorzystać w następnym projekcie Tą stronę chciałem już dokończyć tak jak zacząłem. Na pewno postaram się bardziej po układać cały kod i użyć więcej funkcji, ponieważ w kilku miejscach użyłem praktycznie tego samego kodu :p

 

Marco Bizzarri naukę z programowaniem zaczynałem od c++, jeszcze się w nim bawię, ponieważ mam go w planie na studiach :p. A skąd to pytanie ?

 

Szczerze mówiąc nie spodziewałem się takich opinii, ale czekam na dalsze

 

Zapytałem z ciekawości, bo widać po wklepywaniu stringów, że w c++ coś robisz.

[Eruris.eu]

Dokładnie tak jak kolega wyżej od czegoś trzeba zacząć.

Strona jak na pierwszy raz dobrze wykonana powodzenia z w dalszej pracy.

[Sheila^^]

Ostatnio sam kodowałem ten layout i wyszło mi to tak:

 

http://screenshot.sh/m8TvP5h8Ldj8K

 

http://screenshot.sh/mK81b19XqShoi

[Marco Bizzarri]

 

Ostatnio sam kodowałem ten layout i wyszło mi to tak:

http://screenshot.sh/m8TvP5h8Ldj8K

http://screenshot.sh/mK81b19XqShoi

 

> Kodowałem

> Podmieniłem grafiki w cssie BombWorka

 

Wybierz jedno i nie okłamuj ludzi przyjacielu.

[SequencexD]

Taki już weles aka koder

[Ceesee]

Jest kilka błędów, jednak strona jak na pierwszy raz napisana bardzo dobrze.

$strona = htmlentities($_GET['strona'],ENT_QUOTES, "UTF-8");

W takim wypadku lepiej zastosować basename niż htmlentities ze względów bezpieczeństwa.

$email = $_POST['email'];
$tytul = 'Przypomnienie hasła na serwerze Nazwa Serwera ';
$data = date('Y-m-d H:i:s');
$naglowek = "Od: Nazwa Serwera ; Temat: Zmiana Adresu E-mail ; Data wyslania: ".$data. " ; \n  ";
$naglowek .= "MIME-Version: 1.0\n";
$naglowek .= "Content-Type: text/html;\n";
$naglowek .= "\tcharset=\"UTF-8\"\n";

$polaczenie->select_db("account");
$dane=$polaczenie->query(sprintf("SELECT * FROM account where email='%s'",
														mysqli_real_escape_string($polaczenie,$email)));

Nie trzeba się koniecznie tak bawić, wystarczy wrzucić $email w mysql_real_escape_string i możesz bezpiecznie wkleić to do zapytania. Oczywiście to tylko uwaga ze względu na mój minimalizm, jak to zrobisz to już zależy od Ciebie, kawałek kodu który wkleiłem też jest dobry.

elseif (preg_match('/^[a-zA-ZąćęłńóśźżĄĆĘŁŃÓŚŹŻ0-9\-\_\.]+\@[a-zA-ZąćęłńóśźżĄĆĘŁŃÓŚŹŻ0-9\-\_\.]+\.[a-z]{2,4}$/',$email) == false)

Moim zdaniem lepiej skorzystać z gotowej funkcji:

elseif (!filter_var($email, FILTER_VALIDATE_EMAIL))

! tyczy się tego samego co false

 

Ogólnie jeśli miałbym podsumować cały kod to pierwszą rzeczą jaką bym zmienił to biblioteka obsługująca funkcje bazy danych. Preferowałbym PDO zamiast mysql/mysqli, bo pozwala na wklepanie danych do bazy, a nie do zapytania. Dobrą sprawą jest też trzymanie sesji w bazie danych dla samej wygody jak i load-balancingu.

 

Nie programujesz przypadkiem w c++?

 

"Preferowałbym PDO zamiast mysql/mysqli, bo pozwala na wklepanie danych do bazy, a nie do zapytania."

 

???????????????????????????

czy ja o czyms nie wiem

 

nie korzystaj z sprintfa

ciekawa hybryda korzystania z mysqli objektowo i strukturalnie, mocne!! wali tutaj miroslawem zelentem XDD

 

skoro już stworzyłeś objekt to na nim operuj, a nie swirujesz na zmiane

[lazarus.1988]

Z ciekawości obejrzę to "cudo"

[Marco Bizzarri]

"Preferowałbym PDO zamiast mysql/mysqli, bo pozwala na wklepanie danych do bazy, a nie do zapytania."

 

???????????????????????????

czy ja o czyms nie wiem

 

nie korzystaj z sprintfa

ciekawa hybryda korzystania z mysqli objektowo i strukturalnie, mocne!! wali tutaj miroslawem zelentem XDD

 

skoro już stworzyłeś objekt to na nim operuj, a nie swirujesz na zmiane

 

Tak, jeśli uważasz inaczej to znaczy, że nie zapoznałeś się z PDO i nie wiesz do czego służy preparowanie stejtmentów.

[Andryw]

Tak, jeśli uważasz inaczej to znaczy, że nie zapoznałeś się z PDO i nie wiesz do czego służy preparowanie stejtmentów.

 

On pewnie stosuje prepare a nie binduje danych

[Lajkkk]

On pewnie stosuje prepare a nie binduje danych

Ważne że robi coś swojego a nie jak 90% jakże uzdolnionych developerów naszego zajebistego forum.

[JarajTo ♋]

Ważne że robi coś swojego a nie jak 90% jakże uzdolnionych developerów naszego zajebistego forum.

Żeby pisać o innych, trzeba jeszcze pomyśleć o sobie

 

Co do tematu się nie wypowiem, ale kod nie najgorszy jak na first tajm ale ja tam nie znam się 

­

[Marco Bizzarri]

Ważne że robi coś swojego a nie jak 90% jakże uzdolnionych developerów naszego zajebistego forum.

 

Proszę Cię, z takim tekstem to możesz wylecieć do 14 letnich pseudo-techników pracujących nad Metinem, a nie do kogoś kto się uważa za Web Developera, jeśli w tym biznesie jesteś słaby to odpadasz i nikt na żadne chęci nie będzie patrzył, nie masz chęci - nie zarabiasz.