Errore nell'invio query

[criric]

Ciao, ho cambiato il titolo
sicuro che sia

if ($result->errno) {

e non

if ($mysqli->errno) {

?

 

[blacksbugs]

Certo ho fatto la copia, comunque non funziona..

 

[criric]

quell'istruzione comunque verifica se la query è stata eseguita correttamente ma non se restituisce qualche risultato
modifica il controllo cosi

if ($result->num_rows == 0) {
    printf("Nome utente o password errati: <br/> %s", $mysqli->error);
    exit();
} else {
    session_start();
    $_SESSION['user_id'] = $record['id'];
    echo "Utente registrato coretatmente!";
}

 

[blacksbugs]

Non funziona.. solita pagina bianca..

 

[Longo8]

Se hai pagina bianca c'è qualcosa che non va di fondo.
In entrambi i casi dovrebbe stampare qualcosa a video.

 

[criric]

potrebbe essere il require, di solito sbagliando percorso del file da questo tipo di errore

 

[blacksbugs]

potrebbe essere il require, di solito sbagliando percorso del file da questo tipo di errore

Stessa cartella in localhost

 

[criric]

In effetti guardando meglio il codice hai sbagliato l'esecuzione della query

// preparo ed invio la query
    $query = $mysqli->real_query("SELECT id FROM users WHERE username = '$user' AND password = '$pwd'");
    
// controllo l'esito
    $result = $mysqli->query($query);

modifica cosi

// preparo  la query
    $query ="SELECT id FROM users WHERE username = '$user' AND password = '$pwd'";
    
// la eseguo
    $result = $mysqli->query($query);

 

[blacksbugs]

In effetti guardando meglio il codice hai sbagliato l'esecuzione della query

// preparo ed invio la query
    $query = $mysqli->real_query("SELECT id FROM users WHERE username = '$user' AND password = '$pwd'");
    
// controllo l'esito
    $result = $mysqli->query($query);

modifica cosi

// preparo  la query
    $query ="SELECT id FROM users WHERE username = '$user' AND password = '$pwd'";
    
// la eseguo
    $result = $mysqli->query($query);

Niente caro, anche questa volta restituisce pagina bianca... e io mi chiedo se sto facendo perdere tempo a voi inutilmente con qualcosa di fondo sbagliato.. ma vi assicuro che non è così.. Mistero !!

 

[criric]

inizia a mettere degli echo in tutto il codice per vedere dove si blocca

<?php
echo "inizio";
include_once 'connessione.php';
echo "connessione";
$user = trim($_POST['user']);
$pwd  = trim($_POST['pwd']);
echo "recupero post";

etc etc

in questo modo riuscirai a capire dove si blocca

 

[blacksbugs]

<?php
require_once 'connessione.php';
echo 'uno';
$user = trim($_POST['user']);
$pwd  = trim($_POST['pwd']);
echo '2';
$user = $mysqli -> mysql_real_escape_string($user);
$pwd = $mysqli ->mysql_real_escape_string($pwd);	
$pwd= md5($pwd); 
echo 'tre';
.....

Okk si blocca al punto 2, il terzo non lo stampa già !

 

[criric]

la sintssi corretta è cosi

$user = $mysqli->real_escape_string($user);

 

[blacksbugs]

la sintssi corretta è cosi

$user = $mysqli->real_escape_string($user);

Caspita è vero chisà da dove mi è uscito quel mysql_ ..grazie mille criric ora funziona perfettamente !
Mi rimane solo una domanda.. ora che gli utenti si possono autentificare ci saranno pagine protette e pagine non protette.
Per proteggere queste pagine basta che inserisco queste righe all'inizio della mia pagina, vero?

<?php
require_once 'connessione.php';
session_start();
$_SESSION['user_id'] = $record['id'];
?>

 

[Longo8]

Dovresti mettere un reindirizzamento che avviene quando non c'è una sessione attiva.
Così facendo obblighi l'utente a fare il login prima di andare avanti.

 

[blacksbugs]

Dovresti mettere un reindirizzamento che avviene quando non c'è una sessione attiva.
Così facendo obblighi l'utente a fare il login prima di andare avanti.

if ($result->num_rows == 0) {
    printf("Nome utente o password errati: <br/> %s", $mysqli->error);
    header("Location: /INDEX.php");
    exit();
} else {
    ......
}

Intendi semplicemente questo?

 

[flameseeker]

if ($result->num_rows == 0) {
    printf("Nome utente o password errati: <br/> %s", $mysqli->error);
    header("Location: /INDEX.php");
    exit();
} else {
    ......
}

Intendi semplicemente questo?

Una volta che

$_SESSION['user_id'] = $record['id'];

è stata valorizzata al login non devi più ridefinirla nelle altre pagine, altrimenti rischi di creare una sessione che contiene dati confusi e questo potrebbe rendere instabile la tua applicazione.


Il tuo controllo diventa qualcosa di molto semplice da fare nelle pagine dove vuoi sia richiesta una sessione:

<?php
session_start();

#> se $_SESSION['user_id'] non esiste o è vuoto, rimando l'utente ad index.php
if (empty($_SESSION['user_id'])) {
    header("Location: /INDEX.php");
    exit();
}

#> se arrivo fin qui l'utente è autenticato, procedo con lo script..
require_once 'connessione.php';

 

[blacksbugs]

Allora io ho fatto una pagina chiamata protetta.php e l'ho inserita in un collegamento presente nella home.
Se faccio click su di essa senza loggarmi mi fà il refresh diciamo sulla home, anche se la richiamo da url fa la stessa cosa.. come faccio a vedere la pagina protetta?

 

[criric]

Se fai il login setti la $_SESSION['user_id'] e quindi vedi la pagina altrimenti vieni reindirizzato alla index o all apagina di login dipende cosa hai impostato

 

[blacksbugs]

Allora ricapitolando tutto vorrei che la mia pagina accedi.php rimandasse ad una pagina del sito e non ad un semplice echo su pagina bianca.
Se il login non avviene con successo rimanda alla index con la stampa nome e pwd errati, mentre se è andato a buon fine alla mia pagina protetta.php:
accedi.php

if ($result->num_rows == 0) {
    header('Location:index.php');
    printf("Nome utente o password errati: <br/> %s", $mysqli->error);
    exit();
} else {
    session_start();
    $_SESSION['user_id'] = $record['id'];
    header('Location:protetta.php');
    echo "Utente loggato coretatmente!"; 
}

e questa è la pagina protetta.php:

<html>
 <?php
session_start();

#> se $_SESSION['user_id'] non esiste o è vuoto, rimando l'utente ad index.php
if (empty($_SESSION['user_id'])) {
	header('Location:index.php');
   // echo "qualcosa non quadra!";
    exit();
}
echo 'ciao';
#> se arrivo fin qui l'utente è autenticato, procedo con lo script..
require_once 'connessione.php'; 
echo 'la pagina protetta';
?>
<head>
<title> Pagina protetta </title>
</head>
<body>
<h1> Questa è una pagina protetta </h1>
</body>
</html>

Se decommentavo // echo "qualcosa non quadra!"; e toglievo la riga dell header che portava all'index esce costantemente qualcosa non quadra, come se la sessione fosse sempre vuota..
Ringrazio tutti per l'aiuto che mi state dando, siete davvero gentilissimi.. spero di non chiedere troppo.

 

[flameseeker]

Per natura di funzionamento del protocollo http, gli header vanno inviati sempre prima di un qualsiasi altro output.
Inoltre è buona pratica inizializzare la sessione come prima istruzione dello script.

Di conseguenza ti suggerirei di operare queste modifiche:

<?php
session_start();

#> controllo sessione valida
if (empty($_SESSION['user_id'])) {
    header('Location:index.php');
   // echo "qualcosa non quadra!";
    exit();
}


#> se arrivo fin qui l'utente è autenticato, procedo con lo script..
echo 'ciao';

require_once 'connessione.php'; 
echo 'la pagina protetta';

?><html>
<head>
<title> Pagina protetta </title>
</head>
<body>
<h1> Questa è una pagina protetta </h1>
</body>
</html>