Optymalizacja wyciągania ciągu z ciągu url Opcje

[TheBlackJJoker]

Cześć. Aktualnie robię stronkę, która pobiera dane z API w formie JSON'a.
Użytkownik podaje swój nick, lub link do swojego profilu, który może być w dwóch formach, tj. http://steamcommunity.com/id/<nick lub id>/ i http://steamcommunity.com/profiles/<nick lub id>/ .
Jeżeli podaje nick, to kod leci dalej bez zmian. Jeżeli wykryje, że ciąg znaków składa się z url to wtedy ma wyjąć nick lub id profilu.
Kod działa, ale czasem lubi świrować, tj. api zwraca kod 400 (bad request), a gdy nick się nie zgadza powinien zwracać kod 404. Więc wydaje mi się, że coś tutaj jest źle.
Proszę o pomoc, może ogarnięcie tego od nowa.

Pozdrawiam.

[PHP] pobierz, plaintext 
if(isset($_GET['n']) && isset($_GET['p'])){
            if($_GET['p'] == 1){
        if(strpos($_GET['n'], "steamcommunity.com/id/") === false){
 
        }
        else{
            $b = str_replace("/", "", $_GET['n']);
        $c = explode("steamcommunity.comid",$b);
            return header("Location: player.php?n=".$c[1]."&p=".$_GET['p']);
 
        }
 
            if(strpos($_GET['n'], "steamcommunity.com/profiles/") === false){
 
        }
        else{
            $e = str_replace("/", "", $_GET['n']);
        $f = explode("steamcommunity.comprofiles",$e);
            return header("Location: player.php?n=".$f[1]."&p=".$_GET['p']);
 
        }
            }
}
[PHP] pobierz, plaintext 

[Tomplus]

A nie możesz użyć wyrażeń regularnych?

Np.

[PHP] pobierz, plaintext 
$url = 'http://steamcommunity.com/profiles/76561198121038823/';  	
preg_match('/\/(id|profiles)\/([\w\d\-\_]+)\//', $url, $wynik);  	
echo "
 ID z `{$wynik[1]}` to: {$wynik[2]}
";
[PHP] pobierz, plaintext 

Potem taki wynik przetwarzasz jak chcesz. Z tego co sprawdziłem, to są dwa rodzaje URLi do profilu:

- /id/ gdzie jest numer lub nazwa
- /profiles/ gdzie jest tylko numer ID użytkownika

[TheBlackJJoker]

Dzięki za pomoc. Usprawniłem to co napisałeś. Zrobione. Dzięki.

[PHP] pobierz, plaintext 
if(strstr($_GET['n'], "steamcommunity.com")!==False){
$wynik = preg_split("/(id|profiles)/", $_GET['n']);
$wynik[1] = str_replace("/", "", $wynik[1]);
}
[PHP] pobierz, plaintext 

Ten post edytował TheBlackJJoker 27.09.2017, 12:05:29