Besonderer Downloadzähler

[mrhappiness]

wie rufst du das denn auf?
poste mal den link auf den geklickt werden soll

[adihas]

link: download.php?cat=gaestebuch?id=1

[mrhappiness]

zwei "?" is böse

download.php?cat=gaestebuch&id=1 is besser

und wie schon (mehrmals) gesagt: mach im Skript aus $id $_GET['id']

[adihas]

also, wenn ich diesen Link aufrufe:

download.php?cat=gaestebuch&id=1

Löscht es alles in der TXT Datei!?

Und das hier ist der Code:

<?
$kategorie = "kategorie/$cat.txt";
$datei = fopen ($kategorie, "w");
$zeilen=file('kategorie/gaestebuch.txt');

$i=1;
while ($i<count($zeilen))
{
  $zeile=explode('|',$zeilen[$i]);
  if ($zeile[0]==$_GET['id'])
  {
     $zeile[count($zeile)-1]++;
     $zeilen[$i]=implode('|',$zeile);
     break;
  }
  $i++;
}
$save = implode ("\n", $zeilen);

fputs($datei, $save);
fclose($datei);

?>

[mrhappiness]

du öffnest die datei ja zuerst und leerst sie, danach liest du die (jetzt leere) Datei ein und schreibst den (logischerweise) auch leeren string wieder in die datei.

Was lernen wir daraus?
Erst denken (und Manual lesen, wozu poste ich denn die Links?) und dann machen

btw: hast du das irgendwo online?

[adihas]

Ja, hier: http://webmasters-archiv.ch/downloads/ download.php ist hier die new.php Datei.

[mrhappiness]

Nachtrag

Musst natürlich erst die datei einlesen ($zeilen=file(...) ) und danach die datei öffnen und leeren

wo hast du eigentlich das $i=1 vor der while-schleife her?

[adihas]

Das hast Du mir mal gegeben!?

[mrhappiness]

Original geschrieben von mrhappiness

$i=0;
while ($i<count($zeilen)
{
  $zeile=explode('|',$zeilen[$i]);
  if ($zeile[0]==$id)
  {
     $zeile[count($zeile)-1]++;
     $zeilen[$i]=implode('|',$zeile);
     break;
  }
  $i++;
} 

Sicher, dass ich dir $i=1 gegeben hab. Solltest wenigstens richtig Copy&Pasten können...

[adihas]

aber hier:

$i=0;
while ($i<count($zeilen)
{
$zeile=explode('|',$zeilen[$i]);
if ($zeile[0]==$id)
{
$zeile[count($zeile)-1]++;
$zeilen[$i]=implode('|',$zeile);
break;
}
$i++;
}

fehlkt ja bei
while ($i<count($zeilen) noch eine Klammer --> while ($i<count($zeilen))

[mrhappiness]

und deswegen änderst du die 0 in eine 1???????????????????????????????

[adihas]

nein, sorry

[adihas]

Aber es gaht auch nicht wenn ich diesen Code habe:

<?
$kategorie = "kategorie/$cat.txt";
$datei = fopen ($kategorie, "w");
$zeilen=file('kategorie/gaestebuch.txt');

$i=0;
while ($i<count($zeilen))
{
  $zeile=explode('|',$zeilen[$i]);
  if ($zeile[0]==$_GET['id'])
  {
     $zeile[count($zeile)-1]++;
     $zeilen[$i]=implode('|',$zeile);
     break;
  }
  $i++;
}
$save = implode ("\n", $zeilen);

fputs($datei, $save);
fclose($datei);

?>

[mrhappiness]

Re: Nachtrag

Original geschrieben von mrhappiness
Musst natürlich erst die datei einlesen ($zeilen=file(...) ) und danach die datei öffnen und leeren

Was davon hast du nichtverstanden?!

Also entweder liest du nicht richtig, oder ich erklär zu schlecht oder ich setz zuviel voraus...

[adihas]

Sorry, stimmt es jetzt aber so:

<?
$kategorie = "kategorie/$cat.txt";
$zeilen=file('kategorie/gaestebuch.txt');

$i=0;
while ($i<count($zeilen))
{
  $zeile=explode('|',$zeilen[$i]);
  if ($zeile[0]==$_GET['id'])
  {
     $zeile[count($zeile)-1]++;
     $zeilen[$i]=implode('|',$zeile);
     break;
  }
  $i++;
}
$save = implode ("\n", $zeilen);

$datei = fopen ($kategorie, "w");

fputs($datei, $save);
fclose($datei);

?>

?

Es speichert jetzt alles richtig, nur setzt es nach jedem aufruf des Links: ein paar zeilen mehr zwischen die 1. und 2. Id...new.php?cat=gaestebuch&id=1