Funzione e passaggio parametri $_POST $_SESSION

[pmi]

ho il seguente script e vorrei trasformarlo in funzione, ma non so come passare e leggere correttamente i parametri:

contatore_annunci();

//contatore_annunci
function contatore_annunci($HTTP_SESSION_VARS[id_membro], $_POST[an_oggetto]) {
// *** contatore annunci

if ($_POST['an_oggetto'] == "vendo") {
mysql_select_db($database_connetti, $connetti);
$source = "select n_annunci from membri where id = " . $HTTP_SESSION_VARS["id_membro"] . "";
$rsId = mysql_query($source, $connetti) or die(mysql_error());
$row_rsId = mysql_fetch_assoc($rsId);
$next = $row_rsId["n_annunci"] + 1;
$upd = "update membri set n_annunci = " . $next . ", n_annunci_disp=n_annunci_disp-1 where id = " . $HTTP_SESSION_VARS["id_membro"] . "";
$rsId = mysql_query($upd, $connetti) or die(mysql_error());
$rsId = null;
}

if ($_POST['an_oggetto'] != "vendo") {
mysql_select_db($database_connetti, $connetti);
$source = "select n_annunci from membri where id = " . $HTTP_SESSION_VARS["id_membro"] . "";
$rsId = mysql_query($source, $connetti) or die(mysql_error());
$row_rsId = mysql_fetch_assoc($rsId);
$next = $row_rsId["n_annunci"] + 1;
$upd = "update membri set n_annunci = " . $next . " where id = " . $HTTP_SESSION_VARS["id_membro"] . "";
$rsId = mysql_query($upd, $connetti) or die(mysql_error());
$rsId = null;
}
}

[boomboom69]

scusa, ma non fai la stessa cosa in entrambi i casi?

[pmi]

in che senso fai la stessa cosa in entrambi i casi?

[pmi]

se modifico l'intestazione della funzione in questo modo:

function contatore_annunci() {
// *** contatore annunci


ricevo questo errore:

Warning: mysql_select_db(): supplied argument is not a valid MySQL-Link resource in /Users/asdf/Sites/asdf/annunci/inserisci.php on line 80

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in /Users/asdf/Sites/asdf/annunci/inserisci.php on line 82

[boomboom69]

scusa ma avevo visto male le due query

da il warning perché all'interno della funzione non ha il valore di $database_connetti e di $connetti
dovresti dichiararli global all'interno della funzione

//contatore_annunci 
function contatore_annunci($HTTP_SESSION_VARS[id_membro], $_POST[an_oggetto]) { 
// *** contatore annunci 
   global $database_connetti, $connetti;
   if ($_POST['an_oggetto'] == "vendo") { 
   ....

per quanto riguarda la funzione, io ti consiglio di mantenere fuori la connessione al database (in modo da poterla usare anche per altre eventuali query) e farla più o meno in questo modo

function contatore_annunci($id, $oggetto = '') {
      global $connetti;      
      $sql = "UPDATE membri SET n_annunci=n_annunci+1".($oggetto == 'vendo' ? ", n_annunci_disp=n_annunci_disp-1" : "")." WHERE id ='$id'";
      $result = mysql_query($sql, $connetti) or die(mysql_error());
      echo "Sono stati effettuati ".mysql_affected_rows()." aggiornamenti
";
      $result = mysql_query("SELECT n_annunci FROM membri WHERE id='$id'", $connetti);
      list($next) = mysql_fetch_row($result);

      return $next;
}

// connessione db
// selezione db
$next = contatore_annunci($_SESSION['id'], $_POST['an_oggetto']);
// chiusura connessione

[pmi]

Grazie ora funziona correttamente!