Ho un problema alla mia chat e cerco qualcuno che mi aiuti a risolverlo

massimux

io ti consiglierei di passare all'utilizzo di un Database.
Se non hai esperienze ci sono molte chat open-source (ajatizzate) che potrebbero fare al caso tuo!

ayrton2001

@massimux said:

io ti consiglierei di passare all'utilizzo di un Database.
Se non hai esperienze ci sono molte chat open-source (ajatizzate) che potrebbero fare al caso tuo!

Quoto

giulio3x

si ho capito raga...lo so che è facile dire così e cambiare ma io vorrei tenermi la mia che è molto bella e funzionale alle mie esigenze mi basta solo trovare il sistema di mettere a posto sto problema mi aiutate?:bho:

bruno61

Ciao,
puoi provare con flock
Ciao
Bruno

massimux

allora
modifica lo script in modo da aprire il file e verificare la corretta apertura
poi testi che il messaggio sia stato scritto e puoi farla funzionare

mi dispiace ma per i fogli di testo devi perforza procedere con varie operazioni...

Potrei chiederti su quale hosting sei?

ayrton2001

il file di testo viene sovrascritto oppure i nuovi messaggi vengono "aggiunti"?
Ossia, usi fopen([file], "w") oppure fopen([file], "a")?

massimux

se è una chat vengono aggiunti

giulio3x

@Bruno61 said:

Ciao,
puoi provare con flock
Ciao
Bruno

ciao bruno ahimè quella funzione l'ho già provata ma non dà nessun risultato, io vi posso postare la pagina insert.php che inserisce i messaggi, magari ditemi cosa potrei fare...

[PHP]<html>
<head>
<title>Inserimento del messaggio</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<?
include("config.php");
$nick=$_GET['nick'];
$password=$_GET['password'];
$messaggio=$_GET['messaggio'];
$messaggio=stripslashes($messaggio);
$messaggio=str_replace("\n"," ",$messaggio);
$commento="[".date("H:i - d/m/Y")."] ".$nick."|".$messaggio."\n";
if ( !empty($_GET['faccina']) ) $commento = $_GET['faccina']." ".$commento ;
$url="chat.php"."?nick=".$nick."&password=".$password;
if ( !empty($_GET['faccina']) ) $url .= "&faccina=".$_GET['faccina'] ;

$cercamsg = strstr($messaggio,"<");
$cercamsg2 = strstr($messaggio,">");

if (empty($messaggio))
{
Header("Location: $url");
}
elseif ($cercamsg & $cercamsg2)
{
Header("Location: $url");
}
else
{
$apri_file=fopen($file_mess,"a+");
fputs($apri_file,$commento);
fclose($apri_file);
Header("Location: $url");
}
?>
</body>
</html>
[/PHP]

ayrton2001

@massimux said:

se è una chat vengono aggiunti

Beh, non è detto. Se l'ha scritta qualche pazzo magari apre il file, carica i dati in una variabile, concatena il nuovo messaggio e risalva sovrascrivendo.
Ormai mi aspetto di tutto

ayrton2001

@giulio3x said:

ciao bruno ahimè quella funzione l'ho già provata ma non dà nessun risultato, io vi posso postare la pagina insert.php che inserisce i messaggi, magari ditemi cosa potrei fare...

[PHP]<html>
<head>
<title>Inserimento del messaggio</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<?
include("config.php");
$nick=$_GET['nick'];
$password=$_GET['password'];
$messaggio=$_GET['messaggio'];
$messaggio=stripslashes($messaggio);
$messaggio=str_replace("\n"," ",$messaggio);
$commento="[".date("H:i - d/m/Y")."] ".$nick."|".$messaggio."\n";
if ( !empty($_GET['faccina']) ) $commento = $_GET['faccina']." ".$commento ;
$url="chat.php"."?nick=".$nick."&password=".$password;
if ( !empty($_GET['faccina']) ) $url .= "&faccina=".$_GET['faccina'] ;

$cercamsg = strstr($messaggio,"<");
$cercamsg2 = strstr($messaggio,">");

if (empty($messaggio))
{
Header("Location: $url");
}
elseif ($cercamsg & $cercamsg2)
{
Header("Location: $url");
}
else
{
$apri_file=fopen($file_mess,"a+");
fputs($apri_file,$commento);
fclose($apri_file);
Header("Location: $url");
}
?>
</body>
</html>
[/PHP]

io seguirei il consiglio di bruno61.
Prova a cambiare le il codice cosi:
[php]$apri_file=fopen($file_mess,"a+");
if (flock($apri_file, LOCK_EX)) {
fwrite($apri_file,$commento);
flock($apri_file, LOCK_UN);
} else {
echo "Errore lock";
}
fclose($apri_file);
Header("Location: $url");[/php]

bruno61

Ciao,
giusto una prova, correggi l'elseif in

elseif ($cercamsg && $cercamsg2)

L'AND binario e' sensibile al tipo di dato (dovrebbe lavorare correttamente solo su interi), non vorrei che la condizione venisse male interpretata e qualche messaggio capitasse per qualche ragione li' dentro.

Ciao
Bruno

giulio3x

@Bruno61 said:

Ciao,
giusto una prova, correggi l'elseif in

elseif ($cercamsg && $cercamsg2)

L'AND binario e' sensibile al tipo di dato (dovrebbe lavorare correttamente solo su interi), non vorrei che la condizione venisse male interpretata e qualche messaggio capitasse per qualche ragione li' dentro.

Ciao
Bruno

ciao bruno, mi piacerebbe capire bene cosa fa questa riga cioè quale funzione ha e perchè ora mi hai detto di aggiungere una & in più, grazie per le info che mi darai...

giulio3x

@ayrton2001 said:

io seguirei il consiglio di bruno61.
Prova a cambiare le il codice cosi:
[php]$apri_file=fopen($file_mess,"a+");
if (flock($apri_file, LOCK_EX)) {
fwrite($apri_file,$commento);
flock($apri_file, LOCK_UN);
} else {
echo "Errore lock";
}
fclose($apri_file);
Header("Location: $url");[/php]

io avevo inserito questo codice che è leggeremente diverso voi che dite?
[PHP]if (empty($messaggio))
{
Header("Location: $url");
}
else
{
$apri_file=fopen($file_mess,"a+");
if (@flock($apri_file, LOCK_EX)) { //blocco, uso esclusivo in scrittura
fputs($apri_file,$commento);
@flock($apri_file, LOCK_UN); // rilascia il il blocco
} else {
fclose($apri_file);
Header("Location: $url_mess");//torna alla chat portandosi dietro il messaggio
}
fclose($apri_file);
Header("Location: $url");
}
?>[/PHP]

e avevo tolto queste per mettere il pezzo sopra:
[PHP]$cercamsg = strstr($messaggio,"<");
$cercamsg2 = strstr($messaggio,">");

if (empty($messaggio))
{
Header("Location: $url");
}
elseif ($cercamsg & $cercamsg2)
{
Header("Location: $url");
}
else
{
$apri_file=fopen($file_mess,"a+");
fputs($apri_file,$commento);
fclose($apri_file);
Header("Location: $url");
}
?>[/PHP]

bruno61

Ciao,
@giulio3x said:

ciao bruno, mi piacerebbe capire bene cosa fa questa riga cioè quale funzione ha e perchè ora mi hai detto di aggiungere una & in più, grazie per le info che mi darai...

a dire il vero volevo chiederti io che funzione avesse. Cosi' come e' messa elimina (non ricade nell'ultimo else, quindi non scrive nel file) i messaggi che contengono i caratteri < e > contemporaneamente.

A meno tu abbia qualche ragione (non sembra, da quel che dici) ti consiglierei di levarla del tutto.

Riguardo agli AND, il singolo & compara i bit di ogni operando e ritorna un valore non-zero se i due operandi hanno dei bit in comune. All'interno di un if questo e' generalmente un errore di battitura (non sempre ovviamenete, qualche volta serve cosi') ed ha un comportamento diverso dal doppio &&, che significa AND logico. Ad esempio, 4 & 8 e' uguale a 0 (100b e 1000b non hanno bit in comune) quindi l'if risultera' falsa, mentre 4 && 8 e' uguale a TRUE (i due operandi sono diversi da 0 o null) e quindi l'if viene eseguita.

Ciao
Bruno

giulio3x

ok ma per ovviare al mio problema cosa mi consigliate? il pezzo di flock che mi avete postato è un pochino diverso da quello che usavo io...ehm mi potete quindi dire esattamente quale dei due è meglio che io usi? o magari avete altri suggerimenti? apsetto fiducioso....

ayrton2001

@giulio3x said:

ok ma per ovviare al mio problema cosa mi consigliate? il pezzo di flock che mi avete postato è un pochino diverso da quello che usavo io...ehm mi potete quindi dire esattamente quale dei due è meglio che io usi? o magari avete altri suggerimenti? apsetto fiducioso....

credo non cambi niente poiche fputs è un alias di fwrite, inoltre tu richiami la funzione flock in safe-mode.
Con questo codice cosa accade? perdi sempre messaggi, la chat non funzione, non viene inserito nessun messaggio o cos'altro?

giulio3x

utilizzando il flock ho notato che ad un utente capitava che il suo messaggio veniva scritto due volte, mentre ad altri continuava a accadere che a volte il loro messaggio non compariva più (quindi bisognava riscriverlo) e questa cosa accadeva pure a me, ma ripeto questa cosa non accadeva e continua ad accadere non sempre ma a volte,....porca l'oca cosa potrebbe essere vuoi che non ci sia una spiegazione? e lo spero...un modo per risolvere questo problema definitivamente? (premetto che avevo aggiunto qualche pezzo di codice flock pure sulla pagina chat.php).

giulio3x

aggiungo che se ci sono alri modi piuttosto che l'uso di flok ditemi perchè voglio risolvere una volta per tutte questo problema.:?

ayrton2001

@giulio3x said:

utilizzando il flock ho notato che ad un utente capitava che il suo messaggio veniva scritto due volte, mentre ad altri continuava a accadere che a volte il loro messaggio non compariva più (quindi bisognava riscriverlo) e questa cosa accadeva pure a me, ma ripeto questa cosa non accadeva e continua ad accadere non sempre ma a volte,....porca l'oca cosa potrebbe essere vuoi che non ci sia una spiegazione? e lo spero...un modo per risolvere questo problema definitivamente? (premetto che avevo aggiunto qualche pezzo di codice flock pure sulla pagina chat.php).

Una curiosità. Magari non è inerente al problema, ma dopo il submit del messaggio disabiliti il reinvio? Se la risposta è no i doppioni possono presentarsi credo

ayrton2001

@giulio3x said:

aggiungo che se ci sono alri modi piuttosto che l'uso di flok ditemi perchè voglio risolvere una volta per tutte questo problema.:?

Non credo esista altro metodo. Secondo me con l'uso di flock non è possibile perdere messaggi.
A questo punto mi viene il dubbio se il problema sia la registrazione su file del messaggio o un problema che si verifica prima.
Per curiosità, quanto pesa il file?