Problema con echo in un if che richiama funzione

[cosmoweb]

Risolto!

codice nella pagina locale.php

if (prossime_serate($nome_up))

funzione

function prossime_serate($locale)
{

$data=DATE('c');

$data2=explode("T", $data);

$query = ("SELECT id,titolo FROM eventi WHERE locale='$locale' AND data_evento>='$data2[0]' ORDER BY data_evento ASC");

$ris = mysql_query($query);

echo "Prossime Serate: 
";

while ($row = mysql_fetch_assoc($ris))
	{
	echo "<a class=\"prossimi_eventi\" href=\"http://www.miosito.it/eventi/evento-{$row[id]}.php\">{$row[titolo]}</a>

";
	}

}

[sergioDario]

Originariamente inviato da RoTeam
L'idea dell'echo riutilizzabile andrebbe gestita con un altra variabile a parer mio

Interessante, è una cosa che può tornare sempre utile.
E' il bello di php, specialmente con le classi può essere molto utile.
In qualche caso ho usato una variabile in più così come hai fatto tu, con 1 e 0 oppure true false per attivare o disattivare parti di codice.

Ciaoooo.

[cosmoweb]

Il codice l'ho scritto a volo penso che va più che bene!

Pareri sull'ottimizzazione del codice sono sempre ben acceti ... con il php non si smette mai di imparare!

[RoTeam]

Originariamente inviato da cosmoweb
Risolto!

codice nella pagina locale.php

if (prossime_serate($nome_up))

funzione

function prossime_serate($locale)
{

$data=DATE('c');

$data2=explode("T", $data);

$query = ("SELECT id,titolo FROM eventi WHERE locale='$locale' AND data_evento>='$data2[0]' ORDER BY data_evento ASC");

$ris = mysql_query($query);

echo "Prossime Serate: 
";

while ($row = mysql_fetch_assoc($ris))
	{
	echo "<a class=\"prossimi_eventi\" href=\"http://www.miosito.it/eventi/evento-{$row[id]}.php\">{$row[titolo]}</a>

";
	}

}

Ciao a $query non sono necessarie le parentesi tonde visto che lo includi in mysql_query(), poi a questo punto puoi anche semplicemente richiamare la funzione senza l'if

Interessante, è una cosa che può tornare sempre utile.
E' il bello di php, specialmente con le classi può essere molto utile.
In qualche caso ho usato una variabile in più così come hai fatto tu, con 1 e 0 oppure true false per attivare o disattivare parti di codice.

Ciaoooo.

Infatti il bello del php è che ci possono essere piu soluzioni

[sergioDario]

Originariamente inviato da cosmoweb

if (prossime_serate($nome_up))

Non so, non mi convince, in questo modo se la query fallisse come gestiresti l'errore?
Forse è meglio seguire il consiglio di RoTeam

function prossime_serate($locale,$s)
{

$data=DATE('c');

$data2=explode("T", $data);

$query = ("SELECT id,titolo FROM eventi WHERE locale='$locale' AND data_evento>='$data2[0]' ORDER BY data_evento ASC");
$ris = mysql_query($query);
if($ris)
{
    if(!$s)
       {
        return true;
        }
        else
        {
          echo "Prossime Serate: 
";
          while ($row = mysql_fetch_assoc($ris))
         {
         echo "<a class=\"prossimi_eventi\" href=\"http://www.miosito.it/eventi/evento-{$row[id]}.php\">{$row[titolo]}</a>

";
         }
        }
  }
     else
{
return false;
} 

if (prossime_serate($nome_up))

if (prossime_serate($nome_up,0))
   {
    prossime_serate($nome_up,1)
    }
    else
    {
     echo "Non ci sono serate!";
     } 

da verificare comunque.

Ciao.

[cosmoweb]

Errore nella funzione: Parse error: syntax error, unexpected $end

function prossime_serate2($locale,$s)
{

$data=DATE('c');

$data2=explode("T", $data);

$query = ("SELECT id,titolo FROM eventi WHERE locale='$locale' AND data_evento>='$data2[0]' ORDER BY data_evento ASC");
$ris = mysql_query($query);
if($ris)
{
    if(!$s)
       {
        return true;
        }
        else
        {
          echo "Prossime Serate: 
";
          while ($row = mysql_fetch_assoc($ris))
         {
         echo "<a class=\"prossimi_eventi\" href=\"http://www.miosito.it/eventi/evento-{$row[id]}.php\">{$row[titolo]}</a>

";
         }
        }
  }
     else
{
return false;
}

[RoTeam]

Originariamente inviato da cosmoweb
Errore nella funzione: Parse error: syntax error, unexpected $end

function prossime_serate2($locale,$s)
{

$data=DATE('c');

$data2=explode("T", $data);

$query = ("SELECT id,titolo FROM eventi WHERE locale='$locale' AND data_evento>='$data2[0]' ORDER BY data_evento ASC");
$ris = mysql_query($query);
if($ris)
{
    if(!$s)
       {
        return true;
        }
        else
        {
          echo "Prossime Serate: 
";
          while ($row = mysql_fetch_assoc($ris))
         {
         echo "<a class=\"prossimi_eventi\" href=\"http://www.miosito.it/eventi/evento-{$row[id]}.php\">{$row[titolo]}</a>

";
         }
        }
  }
     else
{
return false;
}

Aggiungi una graffa alla fine e chiudi la funzione

[cosmoweb]

mi da l'echo della funzione anche quando la query fallisce...

[RoTeam]

In che senso fallisce? comunque prova cosi:

function prossime_serate2($locale,$s)
{

$data=DATE('c');

$data2=explode("T", $data);

$query = ("SELECT id,titolo FROM eventi WHERE locale='$locale' AND data_evento>='$data2[0]' ORDER BY data_evento ASC");
$ris = mysql_query($query) or die(mysql_error());
if(mysql_num_rows($ris)>0)
{
    if(!$s)
       {
        return true;
        }
        else
        {
          echo "Prossime Serate: 
";
          while ($row = mysql_fetch_assoc($ris))
         {
         echo "<a class=\"prossimi_eventi\" href=\"http://www.miosito.it/eventi/evento-{$row[id]}.php\">{$row[titolo]}</a>

";
         }
        }
  }
     else
  {
return false;
  }
} 

[cosmoweb]

Perfetto! Adesso funziona egregiamente, grazie!