Tijdens while loop veldwaarden vergelijken uitlog script

- spring eerst je code eens fatsoenlijk in, dit is onleesbaar
- zouden deze queries niet in een transactie moeten staan?
- controleer welke queries uitgevoerd worden
- wat bedoel je in hemelsnaam met "werkt geheel niet": wat verwacht je wat deze code doet, en wat is het (ongewenste) resultaat?
- je header is fout; waarschijnlijk bedoel je header('[color=#ff0000]Location:[/color] uitlog.html');
- daarnaast kan de UPDATE query meerdere keren worden uitgevoerd omdat header('Location: ...') je niet direct automagisch transporteert naar de nieuwe locatie, dit gebeurt pas aan het einde van het script

Ga voor jezelf eerst na wat dit script zou moeten doen, en ga dan pas dingen aanpassen.

Je hebt gelijk. Zal het geheel eerst wat fatsoeneren & het hele verhaal hier neerzetten. Is dat mogelijk?

Thomas van den Heuvel op 26/03/2016 14:30:25

- daarnaast kan de UPDATE query meerdere keren worden uitgevoerd omdat header('Location: ...') je niet direct automagisch transporteert naar de nieuwe locatie, dit gebeurt pas aan het einde van het script

Hetgeen eenvoudig op te lossen is door er een exit; achter te zetten.
<?php
// ...

header('Location: logout.html');
exit;

// ...
?>

Frank Nietbelangrijk op 26/03/2016 16:36:17

Hetgeen eenvoudig op te lossen is door er een exit; achter te zetten.

Dat zou ik pas doen als duidelijk is wat het script zou moeten doen. Mogelijk wil je die header (met exit) pas na de while-loop aanroepen.

De oplossing was nog al een puzzel maar is gevonden.

Na een eerdere post is het geheel gefatsoeneerd en aangepast en heb wat aangereikte ideetjes
uitgeprobeerd zoals een mysqli_multi_query of transaction. In combinatie met een session gaat
dit geheid een keer mis, sessie valt een keer weg, gevolg een empty_string en lege velden in
een tabel. Dit is niet tegen te gaan maar het abstractie niveau stijgt ineens want er spelen
nog meer problemen. In hetzelfde script blokkeert het verhaal en met 1 veld is er geen kader
ter referentie op te zetten. Dus je zult twee velden aan moeten maken in een tabel,de eerste
als ijkwaarde genaamd servtijd en de tweede als controlewaarde genaamd speltijd.

De opzet zoals hij nu is werkt, heb eerst de reset van beide velden in elkaar geknutseld, het
ding heet tijd0.php, volgens het bottom-up systeem opgebouwd vanwege het abstracte en om de
daarop volgende problemen tegen te gaan. De opbouw kan daarom nogal vreemd overkomen.
De opbouw is nu tijd1 -> redirect tijd0.php(reset) of tijd2.php ->(uitloggen) komt.

Dit is tijd1.php;

<?php
session_start();
include("cnnc.php");

$id = $_SESSION['id'];

$sql = "SELECT MAX(servtijd) FROM spelers";
if($result = mysqli_query($link, $sql)) {
while($rij = mysqli_fetch_array($result)) {
$klok = $rij['MAX(servtijd)'];
    }
mysqli_free_result($result);
  } else {
echo "Oeps1:\n" . mysqli_error($link);
}

if($klok >= 9999) {
header("location:tijd0.php");
} else {
$query = "UPDATE spelers SET servtijd = '$klok', speltijd = '$klok' WHERE id = $id'";
if(!mysqli_query($link, $query)) {
echo "Oeps2:\n" . mysqli_error($link);
}

$doei1 = $klok-1;

$sql = "SELECT servtijd, speltijd FROM spelers WHERE speltijd = 'doei1' AND status = 'online'";
if(mysqli_query($link, $sql)) {
$_SESSION['klok'] = $klok;
header("location:tijd2.php");
       } else {
echo "Oeps3:\n" . mysqli_error($link);
    }
}
?>

Als de klok zijn max heeft behaald redirecten naar tijd0.php, beide velden resetten naar 0 en
terugverwijzen naar tijd1.php waardoor er automagisch een loop of lus ontstaat. Tijd0.php;

<?php
include("cnnc.php");
$query = "UPDATE spelers SET servtijd = '0', speltijd = '0'";
if(mysqli_query($link, $query)) {
header("location:tijd1.php ");
} else {
echo "Meuh1:\n" . mysqli_error($link);
}
?>

Is de maxtijd niet behaald maar zit er wel verschil tussen de ijk en controlewaarde dan door
sturen middels een redirect naar tijd2.php, de moeilijkheid is hier het moment van inloggen
onder een client & de daarbij behorende variabelen; client reageert niet meer of is de netwerk
verbinding is traag? Dit is de reden van een $doei2 als maximale grens....Tijd2.php;

<?php
session_start();
include("cnnc.php");
}
$klok = $_SESSION['klok'];

$doei2 = $klok-5;    

$query = "UPDATE spelers SET servtijd='0', speltijd='0', room='none', saw='-',status='offline', 
type='0', gstat='0' WHERE speltijd='$doei2'";
if(!mysqli_query($link, $query)) {
echo "Oeps4:\n" . mysqli_error();
}
mysqli_close($link);
?>

Hoe spreek je je dit verhaal onder Jquery - ajax aan?

Thomas van den Heuvel op 26/03/2016 14:30:25

- spring eerst je code eens fatsoenlijk in, dit is onleesbaar

En je springt nog steeds niet in

if($code=='good'){
	echo 'Ik schrijf goede code';
}else{
	echo 'Ik schrijf slechte code';
}

Ow? Dus als jij in de sloot springt moet ik er maar achter aan springen?
Wie denk jij nu wel niet dat je bent? Zolang als het werkt, werkt het & ik laat me door jou echt niet de les lezen!

Yoop, als je fatsoenlijke antwoorden wil krijgen is medewerking van jouw kant wel belangrijk. Als je gevraagd wordt je onleesbare brei om te zetten naar iets leesbaars zodat je geholpen kan worden hoef je niet zo extreem te reageren. Daarnaast zul je zelf een hoop fouten al kunnen herkennen als je netjes inspringt.

Ben van Velzen op 08/04/2016 13:30:50

Als je gevraagd wordt je onleesbare brei om te zetten naar iets leesbaars zodat je geholpen kan worden

Yoop, kijk eens in de documentatie van je editor. Hier zijn functies voor.

Yoop Overmaat op 08/04/2016 12:49:52

Zolang als het werkt, werkt het & ik laat me door jou echt niet de les lezen!

Maar blijkbaar moet je toch 'de les lezen' want het werkt niet..
Je mag er ook op rekenen dat mensen die je helpen verstand hebben van zaken en praktische tips hebben.