[php]impostare un if

[13manuel84]

Ciao a tutti! sto sclerando per impostare un if!! :robadamatti:

dunque, ho uno script che realizza un calendario con la possibilità di navigare tra i vari mesi ed i vari anni; in fondo a questo script c'è la scritta 'Visualizza il mese corrente' solo che questa scritta deve vedersi solo se, giustamente, si sta visualizzando un mese che non sia quello corrente.

Io ho impostato un if come il seguente:

$flag = (!isset($_GET['month']) || $_GET['month'] <> date("n") || $_GET['year'] <> date("Y")) ? 0 : 1; 

$flag avrà valore 1 se devo scrivere 'Visualizza il mese corrente' viceversa 0.
Eppure a me il ragionamento sembra corretto, con la prima parte controllo che il mese non sia settato, oppure (con la seconda parte) che il mese sia diverso da quello corrente, oppure (con la terza) che l'anno sia diverso da quello corrente. Se anche una sola delle tre clausole è vera $flag dovrebbe essere settata a 0...giusto?
Il fatto di confrontare con date("n") o date("Y") è corretto, perché il mese avrà sempre il formato 1-12 senza eventuali zeri, mentre l'anno avrà sempre il formato con 4 caratteri.

Sapete dirmi dove sbaglio?Grazie mille a tutti.

[Tymba]

$flag = (
!isset($_GET['month']) ||
$_GET['month'] == date("n") ||
(isset($_GET['year']) && $_GET['year'] == date("Y"))
) ? 0 : 1;

ciao

[progalba]

ma non puo fare semplicemente

$mese = date("m");
$anno = date("Y");

if (($_GET['month']== $mese) && ($_GET['year'] == $anno)){
echo "vai a li";
}
else{
echo "";}

[Tymba]

certo ed è decisamente più semplice, lineare e facile da leggere a distanza di tempo ma...

a volte uso quella sintassi ma MOLTO raramente proprio per via della difficoltà di lettura.

[progalba]

ma dici a me? ma dov'È la difficoltà?

[Tymba]

di per se non è difficile ma rende il codice più complesso nella lettura e nella manutenzione.

scrivere codice vuol dire scegliere di intraprendere una strada piuttosto che un'altra. ognuno sceglie tra quelle che conosce quella che reputa più semplice o funzionale

[13manuel84]

beh, grazie...ma non va bene nessuno dei due metodi

In particolare progalba non posso fare come dici, il valore del mese e dell'anno vengono presi dalla querystring e possono o essere settati entambi o non essere settati entramni o essere settato solo uno e devo racchiudere tutti questi controlli nello stesso if, inoltre devo controllare che se sono settati tutti e due devono essere diversi dalla data corrente.

Spero di essere stato un pò più chiaro. Se fosse stato così semplice l'avrei risolto pure io penso...magari però è ancora più semplice e mi sto perdendo in un bicchiere d'acqua

ciao!

[13manuel84]

giusto per dovere di cronaca ho risolto così:

$flag = (isset($_GET['month']) && ($_GET['month'] <> date("n") || $_GET['year'] <> date("Y"))) ? 1 : 0; 

Grazie comunque