fwrite schreibt doppelt

Wie sollen wir das bei dem Quellcode-Ausschnitt und den Infos beurteilen??? Fakt ist aber, dass PHP nicht von alleine alles doppelt reinschreibt.

Richtig debuggen
1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(E_ALL); und ini_set("display_errors", true);
3. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
4. An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde (oder auch nicht).
5. Schritt 3 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
6. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
7. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
8. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

Zitat

es muss an diesem befehl liegen

Das glaube ich nicht.

Ich glaube immer noch, dass es an deinem Script liegt. Sollte es aber doch ein PHP-Bug sein, dann solltest du hier was zu finden: http://bugs.php.net/search.php

Wenn es an PHP liegt, müsstest du hier auch den Text doppelt in der Datei haben:

<?php
   $text = "Dies ist ein Text\n";
 
   $file = fopen ('test.log', 'a+');
   fwrite($file, $text);    
   fclose($file);  
?>

Zitat

du kannst dir gerne den rest meines Scripts ansehen

Zitat

das ist der gesamte teil des Codes

Also wieder nicht das komplette Script.

Hast du nachgesehen, ob das ein bekannter PHP-Bug sein könnte? Hast du mein kleiner Testscript ausprobiert? Du hast "Richtig debuggen" nicht umgesetzt.

Du hast ein Problem, also bemühe du dich, um solche Dinge hier nicht offen zu lassen. Solange das nicht alles geklärt ist, verabschiede ich mich aus diesem Thread.

OHA... mumpitz²

jaja bandit ^^,
ich nehme alles zurück,...

<?php
// so lassen
$errors = array();
$logdir = "logs/";


// irgendwann schreibst du so die errors halt in den array
$errors[] = "Fehler 1!";
$errors[] = "Fehler 2!";
$errors[] = "Fehler 3!";


// und diesen teil müsstest du größtenteils wiedererkennen
if(count($errors) > 0)
{
    if(is_writeable($logdir))
    {
        $timestamp = time();

        $e_dayword = array('mon','tue','wed','thu','fri','sat','sun');
        $e_date = date("d.m.Y", $timestamp);

        $file = fopen ($logdir.$e_date.".log", "a+");
        foreach($errors as $error)
        {
            $content = $e_dayword[date('w', $timestamp)].' '.$e_date.' '.date("H:i:s", $timestamp)." ".$error."\n";
            fwrite($file,$content);
        }
        fclose($file);
    }
    else
    {
        $nerror = "The directory ".$logdir." isn't wirtable";
    }
}


foreach($errors as $error)
{
    echo '
    <div class="error" style="margin-bottom: 15px">
    <b>error:</b><br/>
    '.$error.'
    </div>';
}
if(!empty($nerror))
{
    echo '<br />
    <b>error:</b>
    '.$nerror.'<br />
    please tell us about this error';
}




?>

Ist das das was du willst?
Du solltest dich unbedingt mal mit foreach Schleifen auseinandersetzen

Gruß crAzywuLf

Zitat von crAzywuLf

Den Modus "ab" gibt es nicht! Vielleicht solltest du den richtigen für dich auswählen

Aus der Doku:

Zitat

Hinweis: Zusätzlich kann mode der Buchstabe 'b' hinzugefügt werden, der die Behandlung von Binärdateien erlaubt. Dies ist nur auf Systemen sinnvoll, welche zwischen Binär- und Text-Dateien unterscheiden (z.B. Windows. Ist bei Unix sinnlos). Wenn das Feature nicht gebraucht wird, wird es einfach ignoriert. Sie sollten den 'b' Flag verwenden, um Ihre Skripte portabler zu machen.

:D:-);-)

Zitat von bandit600

Aus der Doku:

:D:-);-)

Korb^^
okay... man lernt immer dazu...
ich hab meinen Post oben nochmal editiert!
Der Code ist die Hölle:D

Zitat von SpeedFire

ich will nicht das man das mir klaut

Ohhhhhh, du schreibst auch bestimmt ein Script, auf das sich jeder sofort stürzen und kopieren wird (ganz besonders bei dem Quellcode).
Dann bleiben aber immer noch 2 offene Punkte. Ich würde mal sagen, da wir hier nur Teilinformationen kennen, wirst du den Fehler alleine suchen müssen.

Zitat von SpeedFire

ok ich hab wirklich noch nie mit foreach gearbeitet, aber der sinn ist nicht mehr der selbe.
das script sollte bei gesetzter $error einen eintrag machen
bei einem fehler so:
(datum) - (fehler)
bei zwei fehlern so:
(datum) - (fehler) & (fehler2)
bei dem script unten würde er 2 einträge mit dem selben datum schreiben.
das war nicht der sinn

@bandit600:
ich fühl mich dabei unwohl alles hier rein zu stellen
den das ist arbeit von mehr als ein monat und ich will nicht das man das mir klaut:?
außerdem arbeitet dieser teil des scripts unabhängig und wird nur so ins hauptscript importiert.

Alles anzeigen

<?php
// so lassen
$errors = array();
$logdir = "logs/";


// irgendwann schreibst du so die errors halt in den array
$errors[] = "Fehler 1!";
$errors[] = "Fehler 2!";
$errors[] = "Fehler 3!";


// und diesen teil müsstest du größtenteils wiedererkennen
if(count($errors) > 0)
{
    if(is_writeable($logdir))
    {
        $timestamp = time();

        $e_dayword = array('mon','tue','wed','thu','fri','sat','sun');
        $e_date = date("d.m.Y", $timestamp);

        $file = fopen ($logdir.$e_date.".log", "a+");
        $content = $e_dayword[date('w', $timestamp)].' '.$e_date.' '.date("H:i:s", $timestamp)." ".implode(" & ",$errors)."\n";
        fwrite($file,$content);
        fclose($file);
    }
    else
    {
        $nerror = "The directory ".$logdir." isn't wirtable";
    }
}


foreach($errors as $error)
{
    echo '
    <div class="error" style="margin-bottom: 15px">
    <b>error:</b><br/>
    '.$error.'
    </div>';
}
if(!empty($nerror))
{
    echo '<br />
    <b>error:</b>
    '.$nerror.'<br />
    please tell us about this error';
}




?>

so besser?

Gruß crAzywuLf

Zitat von SpeedFire

@Bandit: nicht gleich ausrasten;-)
crAzywuLf: ja

Wenn bandit schriftlich austickt, sieht das anders aus^^
Aber so wie bandit das sieht ist es schon richtig, ich bin nur zu gutmütig...

Zitat von crAzywuLf

Wenn bandit schriftlich austickt, sieht das anders aus

Vielen Danke Herr Von und Zu. Ich ticke doch hier nie aus, sondern ich drücke mich nur unmissverständlich aus.

jetzt versteh ich nur noch bahnhof