Jeden z pierwszych projektów PHP

pytanie zadane 29 czerwca 2020 w Nasze projekty przez _Pita_ Stary wyjadacz (11,410 p.)
edycja 30 czerwca 2020 przez _Pita_

Cześć, zrobiłem taką stronę w PHP: http://webcodinglearn.cba.pl/Index.php

Co myślicie?

Jest to jeden z pierwszych moich projektów w tym języku, więc proszę o wyrozumiałość

P.S

Reklamy wynikają z darmowego hostingu

P.P.S Mam problemy z gitem a nie ma tego tak dużo więc daję kod PHP

<?php 
ini_set('display_errors','off');
require('pliki/simple_html_dom.php');
$nick = $_POST['nick_name'];
$nick = str_replace(" ", "+", $nick);
$link = "https://forum.pasja-informatyki.pl/user/".$nick;

$html = file_get_html($link);
$tester = substr($html,1,1);

if ($tester === "!") {
	echo "Niestety, takie konto już istnieje";
}
else{
	echo "Takie konto jeszcze nie powstało! Możesz je założyć <a href='https://forum.pasja-informatyki.pl/rejestracja?to='> tutaj</a>!";
}

 ?>

komentarz 29 czerwca 2020 przez Landu Stary wyjadacz (11,880 p.)

Na przyszłość polecam korzystać z hostingu https://www.ct8.pl/

3	komentarz 29 czerwca 2020 przez Ehlert Ekspert (212,670 p.) Jak przyszłość to tylko z vpsem.

komentarz 29 czerwca 2020 przez Wiciorny Ekspert (269,710 p.)

@_Pita_, zawiodłem się wchodząc w link :(

komentarz 30 czerwca 2020 przez Landu Stary wyjadacz (11,880 p.)

VPS za darmo nie dostanie, a mi chodziło o to, że nie warto korzystać z cba.

komentarz 30 czerwca 2020 przez _Pita_ Stary wyjadacz (11,410 p.)

@Wiciorny, Szkoda, ale nie wiem czego spodziewałeś się z wiadomością, że to jeden z pierwszych moich projektów? (Dokładnie 3 jak dobrze pamiętam)

3 odpowiedzi

odpowiedź 29 czerwca 2020 przez HaKIM Szeryf (87,590 p.)

odpowiedź 30 czerwca 2020 przez Comandeer Guru (600,810 p.)

Po co Ci Simple HTML DOM? Równie dobrze możesz zastosować curl-a/file_get_contents. Poza tym: pozwólmy temu skryptowi umrzeć w końcu.
Spacje w URL-ach to nie są plusy. Poza tym mogą się pojawić też inne znaki, które trzeba zakodować. O wiele lepiej wykorzystać urlencode.
Zamiast sprawdzać losowy znak z kodu HTML, o wiele lepiej jest po prostu sprawdzić, jaki status HTTP zwraca pobierana strona.

Ja bym to widział tak:

<?php
$nick = wezSkadsNick();
$nick = urlencode( $nick );
$link = 'https://forum.pasja-informatyki.pl/user/' . $nick;
$ch = curl_init();

curl_setopt_array( $ch, [
    CURLOPT_URL => $link,
    CURLOPT_RETURNTRANSFER => true,
    CURLOPT_NOBODY => true
] );
curl_exec( $ch );

$status = curl_getinfo( $ch, CURLINFO_RESPONSE_CODE );

curl_close( $ch );

if ( $status === 200 ) {
    echo 'Niestety, takie konto już istnieje';
} else {
    echo 'Takie konto jeszcze nie powstało! Możesz je założyć <a href="https://forum.pasja-informatyki.pl/rejestracja?to="> tutaj</a>!';
}
?>

komentarz 30 czerwca 2020 przez _Pita_ Stary wyjadacz (11,410 p.)

Do 1. nie mam wiedzy a z 3. muszę sobie potestować ale co do 2. to masz rację, nie zawsze ale tutaj tak, a to jest dedykowane na to forum ;)

komentarz 30 czerwca 2020 przez Comandeer Guru (600,810 p.)

Nie, spacje w URL-ach nigdy nie są plusami. W URL-ach stosuje się percent encoding i spacja jest kodowana jako %20. spacja jako plus z kolei jest rezultatem wysyłki formularza – https://stackoverflow.com/a/2678602

Co nie zmienia faktu, że w nicku mogą wystąpić inne znaki, które wypada zakodować.

odpowiedź 29 czerwca 2020 przez CubeStorm Pasjonat (15,020 p.)

PHP jako język backend'owy nie jest dostępny dla nas jako widzów, także trudno jest nam ocenić coś czego nie widzimy. Nawet jeśli kod działa poprawnie to nie możemy ocenić czy został napisany w sposób czytelny, przejrzysty i najkrótszy.