Sessionstart PHP

[Pixelmaster]

Hallo,
wer kann mir sagen, weshalb dieser Sessionstart fehlerhaft ist? Bei der Weitergabe der Daten mittels Session funkt die letzte Zeile aus der externen Datei, immer rein.

Generelle Frage. Ist im unten stehenden Codeschnipsel die Syntax falsch oder ist diese im Ganzen "richtig"?

Codeschnipsel:

<?php
session_start( );

$id = $_GET['id'];
if (isset($_SESSION[$id])){
$_SESSION[$id]++;
} else {
$_SESSION[$id]=1;
}
?>

Danke für jegliche Hilfe!

[Guin]

Moin,
die Syntax ist richtig (gaebe ja sonst auch einen Fehler).

Der Schnipsel funktioniert auch soweit.
Welche Probleme gibt es denn?

Vielleicht ist das eine Loesung:
$id = '_'.$_GET['id'];

[Pixelmaster]

Hallo Guin,

Was bedeutet dieses '_' in Deinem Codebeispiel? In meinem Kontroll-Konstrukt print_r() erscheint mir immer folgendes:

Array
(
[] => 14
[_] => 2
[_5344256377] => 1
)

Beachte nur die []=>14! Dieser "leere" Teil funkt mir da immer dazwischen und ich bin mir nicht sicher, ob dies mit meiner Syntax beim Sessionstart im Zusammenhang steht oder nicht? Übrigens in der Klammer sollte die Artikelnummer stehen und die "14" ist die Anzahl Artikel.

[Guin]

Ach so

Das kommt daher, dass $_GET['id'] auch mal leer ist.

PHP-Code:

$id = intval($_GET['id']);  // wenn nur integer gebraucht werden, ist das besser so
if(!empty($id)){
  if (isset($_SESSION[$id])){
    $_SESSION[$id]++;
  } else {
    $_SESSION[$id]=1;
  }
} 


[Pixelmaster]

Danke Guin! Werde es mal so ausprobieren und schauen, ob's nun klappt!!!

[DSB]

Ich verstehe davon ehrlich gesagt kein Wort.
Meinst Du sowas?

$id = intval($_GET['id']);   
if($id>0))
{ 
  if (isset($_SESSION['id'])){ 
    $_SESSION['id']++; 
  } else { 
    $_SESSION['id']=1; 
  } 
}

Wenn Du eine Variable in einer Session ablegst, dann musst Du sie auch über ihren Namen refrenzieren. in Deinem Beispiel refrenzierst Du einen nicht existierenden Index ($_SESSION[14]).

[Guin]

Wenn der Index noch nicht existiert, wird er angelegt.
Es sollen ja verschiedene Artikel(nummern) als Index dienen; der Wert gibt die Anzahl an.

[Pixelmaster]

Könnte es sein, dass wenn z.B. alle Artikelnummern alphanumerisch (4r54r33w) sind, übertragen werden und Artikelnummern, die nur numerisch (576332) sind, nicht weitergeleitet werden? Müsste man da etwas bei nachstehendem Codeschnipsel ändern?

if(isset($_GET['id'])){
  $id = $_GET['id'];
  if (isset($_SESSION[$id])){
    $_SESSION[$id]++;
  } else {
    $_SESSION[$id]=1;
  }
}

Falls ja, dann wie?

[Guin]

Wenn es alphanumerisch ist, das "intval" weglassen (Kommentar in meinem Schnipsel beachten). Die Uberpruefung auf korrektem Inhalt kann an einer anderen Stelle erfolgen. Man koennte mit "htmlspecialchars" einmal druebergehen. Aber ob das sinnvoll ist, musst du wissen.
Kommt halt darauf an, was du mit der Artikelnummer noch alles anstellst.
Eine weitere Moeglichkeit waere, die Laenge der Artikelnummer zu begrenzen...(das sind alles nur Sicherheitsuebrlegungen und haben nichts mit der Funktion an sich zu tun)

Mit "isset($_GET[$id])" kann man hier nicht so gut arbeiten. Dann hast du naemlich das gleich Problem wie zuvor.
"!empty($_GET[$id])" ist hilfreicher.

Könnte es sein, dass wenn z.B. alle Artikelnummern alphanumerisch (4r54r33w) sind, übertragen werden und Artikelnummern, die nur numerisch (576332) sind, nicht weitergeleitet werden?

Wenn das wirklich Probleme macht, kommt wieder mein ersteer Vorschlag zum Tragen.

PHP-Code:

$id = ' '.$_GET['id']; 


[steffenk]

mit nichtnumerischen keys würde ich das einfacher abhandeln

PHP-Code:

$id = $_GET['id'] ? $_GET['id'] : uniqid('key_');
if ($_SESSION[$id]) {
   $_SESSION[$id]++;
} else {
  $_SESSION[$id] = 1;
} 


[Pixelmaster]

Danke für die Hinweise! Mal eine Zwischenfrage für folgendes Codeschnipsel:

<?php
session_start( );

if(isset($_GET['delete'])){
unset($_SESSION[$_GET['id']]);
if(isset($_GET['change'])){
$_SESSION[$_GET['id']] = $_GET['anzahl'];
}
}

?>

Das obere if funktioniert problemlos. Nun beim zweiten if möchte ich die Anzahl nachträglich ändern. Ist die Syntax

$_SESSION[$_GET['id']] = $_GET['anzahl'];

so richtig, wenn ich sagen will, ersetze die bereits vorhandene Anzahl durch die neue Anzahl, sobald ich den Button "Anzahl ändern" angeklickt habe? Oder gäbe es da eine spezielle Funktion dafür? Hab im PHP-Handbuch nichts gefunden, dass mir etwas zusagen würde.