PHP warunek if dla wiecej niz jedej wartosci

Question

Witam prosze czy moglby mi ktos wskazac blad w tym kodzie?

<?php 
 if(($_SERVER["REMOTE_ADDR"]) != ("adres ip") || ("adres ip") || ("adres ip")){
	 echo "<script type='text/javascript'>
         window.location.href = 'https://www.google.com';</script> ";
        exit;
 }

 

 

Answer 1

Wrzuc adresy ip do tablicy i funkcja in_array.. jak bedzie 20 adresow to jaki bedziesz mial fajny warunek ;) po co komplikowac.

Comments

W sumie prosciej ,ale ja potrzebuje tam tylko 4 ip od kolegow

---

Ale warunek będzie wyglądał czytelniej. Nie rob spaghetti.

1. Adresy do tablicy

2. if ( ! in_array($_SERVER[...]) )

3 Przekierowujesz.

---

Możesz mieć nawet dwa adresy, liczy się rozszerzalność...

Answer 2

Źle napisany jest warunek. Jeżeli chcesz przekierowywać komputery z innym IP nic podane przez Ciebie - mogłoby to wyglądać na przykład tak: 

 if(!(
   $_SERVER["REMOTE_ADDR"]) == ("adres ip 1") ||
   $_SERVER["REMOTE_ADDR"]) == ("adres ip 2") ||
   $_SERVER["REMOTE_ADDR"]) == ("adres ip 3")
)){
 // przekierowanie
}

Następny błąd jest w funkcji echo 

"<script type='text/javascript'>window.location.href = '<a href=" <- w tym miejscu dla PHP kończy się to co ma wyświetlić, a resztę próbuje interpretować jako swoją składnie. Można by to poprawić bawiąc się cudzysłowami, jednak znacznie ładniejsze i wydajniejsze będzie przekierowanie wysłane w nagłówku z poziomu PHPa - po prostu

header("Location: http://www.jakas-domena.com/"); 

Edit: Co to forum robi z wyswietlaniem linków w kodzie?? wszystko w <script> chyba jest okej.

Comments

Dobra dziala, ale moje przekierowanie tez jest ok ;* , a ty nie domknoles nawiasow w swoim skrypcie ale jset rep+

---

Niby jest, jednak nie wychodziłbym z założenia, że wszyscy którzy będą próbować dostać się na Twoją stronę będą też mieć włączony JavaScript.

---

@ Argeento : Headera tak się nie robi jak to napisałeś wyżej.

Tak to się robi:

header("Location: http ://jakasdomena.com");
exit;

(forum samo dokleja tutaj linka i w kodzie wyszły Ci krzaki, dałem spację po "p", żeby nie parsowało jako linka)

Answer 3

Znacznik html gryzie się z tym z pehapa.

Comments

Sorki zle przekleilem

znaczniki sa podomykane