[PHP]Problem z wykonaniem warunku if Opcje

[--Joanna--]

Witam forumowiczów :-)

Na stronie mam adresowanie:

[HTML] pobierz, plaintext 
index.php?page=logowanie
index.php?page=dodaj
index.php?page=usr&funkcja=nowy
[HTML] pobierz, plaintext 

Chcę na części tych stron nie wyświetlać części kodu.
Od wczoraj mam problem z kodem

[PHP] pobierz, plaintext 
<?php
if ( isset( $_GET['page'] ) )
{
if( ($_GET['funkcja']=='nowy') or ($_GET['']=='') )
 {
 
 echo "ok ";
 }
 echo "... ";
  }
else {
  echo " no ";
}
 ?>
[PHP] pobierz, plaintext 

Nie wiem jak go poprawić, aby nie reagował na wszystkie warunki mam problem co wstawić dla o

[PHP] pobierz, plaintext 
r ($_GET['']=='')
[PHP] pobierz, plaintext 

by dotyczył adresów

[HTML] pobierz, plaintext 
index.php?page=logowanie
index.php?page=dodaj
[HTML] pobierz, plaintext 

[Daiquiri]

[PHP] pobierz, plaintext 
if (isset($_GET['page']) {
 
   // tutaj wsadzasz wszystko mające reagować na warunek: z ustawionym $_GET['page'] lub więcej
 
   if ($_GET['funkcja']=='nowy') {
        // tutaj wsadzasz wszystko mające reagować na warunek: ustawiony $_GET['page'] oraz $_GET['funkcja'] równa 'nowy'
   }
 
} else {
    // tutaj wsadzasz wszystko mające reagować na warunek: bez $_GET['page]
}
[PHP] pobierz, plaintext 

Nie wiem, szczerze powiedziawszy, jaki warunek masz konkretnie na myśli.

[Dominis]

jeśli Cie dobrze zrozumiałem to:

[PHP] pobierz, plaintext 
if($_GET['page'] == 'logowanie') {
 
//kod ktory wyswietli sie gdy masz w linku logowanie
}
 
if($_GET['page'] == 'dodaj') {
 
//kod ktory wyswietli sie gdy masz w linku dodaj
}
 
if($_GET['page'] == 'usr' && $_GET['nowy'] == 'funkcja') {
 
//kod ktory wyswietli sie gdy masz w linku usr i funkcja
}
[PHP] pobierz, plaintext 

itd...

[--Joanna--]

Dominis:
powinno być chyba

[PHP] pobierz, plaintext 
if($_GET['page'] == 'usr' && $_GET['funkcja'] == 'nowy') {
[PHP] pobierz, plaintext 

zamiast

[PHP] pobierz, plaintext 
#
if($_GET['page'] == 'usr' && $_GET['nowy'] == 'funkcja') {
[PHP] pobierz, plaintext 

czy trzeba tu sprawdzać $_GET['page'] == 'usr' nie wystarczy samo $_GET['funkcja'] == 'nowy'
?

[nospor]

nie wystarczy samo $_GET['funkcja'] == 'nowy'

No jesli uwazasz, że taki link jest prawidłowy:
index.php?funkcja=nowy
to wystarczy.

[--Joanna--]

Ok rozumiem.
A co do mego pytania
Dominis:
powinno być chyba

[PHP] pobierz, plaintext 
 if($_GET['page'] == 'usr' && $_GET['funkcja'] == 'nowy') {
[PHP] pobierz, plaintext 

zamiast:

[PHP] pobierz, plaintext 
if($_GET['page'] == 'usr' && $_GET['nowy'] == 'funkcja') {
[PHP] pobierz, plaintext 

czy dobrze zauważyłam?

[nospor]

Tak, dobrze zauwazylas.

[--Joanna--]

Dział mi

[PHP] pobierz, plaintext 
if($_GET['page'] == 'logowanie' or $_GET['page'] == 'dodaj' or $_GET['page'] == 'AAA' ) {
[PHP] pobierz, plaintext 

teraz zastanawiam się jak to połączyć z dodatkowym warunkiem

[PHP] pobierz, plaintext 
 if($_GET['page'] == 'usr' && $_GET['funkcja'] == 'nowy') {
[PHP] pobierz, plaintext 

czy tak będzie poprawnie:

[PHP] pobierz, plaintext 
if($_GET['page'] == 'logowanie' or $_GET['page'] == 'dodaj' or $_GET['page'] == 'AAA'   or ($_GET['page'] == 'usr' && $_GET['funkcja'] == 'nowy')   ) {
[PHP] pobierz, plaintext 

2 pytanie: czy jeżeli mam kilkanaście warunków OR - to czy nie będzie to mało optymalne czy można to jakoś zoptymalizować?

[nospor]

ad1) A działa? Znaczy że jest poprawnie

[--Joanna--]

Nie sprawdzałam ale wnioskuję że jest OK bo bym opierdziel dostała za błędny kod

A co do mego drugiego pytania wypowie się ktoś?

[nospor]

Nie sprawdzałam

Uuuu.... jeden opierdziel cie minął, ale drugi nie.... Jak to mogłaś nie sprawdzać, przychodzisz na forum bez sprawdzania?! Marsz do swego pokoju młoda damo i masz tam w milczeniu przemyśleć swoje postępowanie..... i z kolacji nici dzisiaj

[--Joanna--]

Sprawdziłam i nawet zmodyfikowałam na własne potrzeby do && dodałam nowe warunki z OR :-)
To jak z tym moim pytaniem o optymalizację tego kodu?

[nospor]

[PHP] pobierz, plaintext 
if(in_array($_GET['page'] ,array('logowanie','dodaj','AAA'))   or ($_GET['page'] == 'usr' && $_GET['funkcja'] == 'nowy')   ) {
[PHP] pobierz, plaintext 

[--Joanna--]

Super. Wygląda zdecydowanie lepiej i dla oka i parsera PHP :-)
Dziękuję.