[PHP] Oplossing nodig voor loop probleempje

[Spin1]

Ik heb weer eens een probleempje .
Een tijdje geleden heb ik een importtscript gemaakt voor het importeren van CSV bestanden dit werkte perfect. maar nu wordt het csv file op een andere manier aangeroepen en kan ik de refresh methode niet meer gebruiken.

Wat ik zelf heb geprobeerd om dit script werkend te maken is:
Een import1.php voor het file te seleteren en uit te voeren in een import2.php.
Een break in de lus zetten, dit had ook geen effect,
en een functie om de uit te voeren loop zetten dit heeft ook geen effect.

de code:

code:

<?php

fwrite(STDOUT, "Geef bestand op: ");
$num1 = rtrim(fgets(STDIN));
fwrite(STDOUT, $num1);

include_once('config.php');
$verbinding = mysql_connect(mysql_host, mysql_user, mysql_password) or die(mysql_error());
mysql_select_db(mysql_database, $verbinding) or die (mysql_error());
$row = 1;
ini_set('auto_detect_line_endings',1);
$bestand = fopen($num1, 'r');
while (($data = fgetcsv($bestand, 1000, ',')) !== FALSE) {
   $val1 = mysql_real_escape_string($data[0]); 
   $val2 = mysql_real_escape_string($data[1]); 
       
        $que = mysql_query("SELECT * FROM paginas WHERE pagina='". $val1 ."' AND parentid='". $pagina ."'");
        $resultaat = mysql_fetch_array($que);
        $pagina = $resultaat['id'];   
        $num_rows = mysql_num_rows($que);
       
       
        if ($pagina >0)
        {
        $ids['id'] = $pagina;
        foreach($ids as $q) {
        $q."<br>";
        }
        }

       

        if ($pagina<1)
        {mysql_query("INSERT INTO `paginas` (`pagina`,`tekst`,`parentid`) VALUES ('{$val1}','{$val2}','$q')");

        function printbreadcrumb($pagina) {
    $sql = "SELECT id, parentid, pagina
                FROM paginas
                WHERE id = $pagina";
    $result    = mysql_query($sql) or header("location: import.php");
    $row = mysql_fetch_array($result);
    if ($row['parentid']) {
        printbreadcrumb($row['parentid']); 
    }
    return;
}
printbreadcrumb($pagina);
          exit(); 
          
          
        }
          
        }
    


?>

Als ik de exit() weg haal wordt alles gegooid wat niet gevonden wordt in de database in 1 keer neegezet.
Als ik het csv file handmatig selecteer op de volgende wijze:

code:

$bestand = fopen(test2.csv, 'r');

dan werkt het script perfect.

Is er een mogelijkheid om deze code aan te passen:

code:

  $result    = mysql_query($sql) or header("location: import.php");

dat hij niet de hele pagina verandert maar de while loop weer overnieuw start.
Ik heb hier een break neergezet, een functie om de while loop gezet. En naar een ander file toegelinkt. Dit allemaal zonder effect.

Zou iemand mij kunnen helpen.

[ Voor 3% gewijzigd door Spin1 op 29-12-2009 11:09 ]

[Cartman!]

Lijkt me een kwestie van debuggen, als het eerst goed werkte. Je zegt dat er iets in de aanroep veranderd is, wat dan? Gewoon lekker echo-en en print_r-en en kijken waarom ie er nog een keer doorgaat.

Je laatste vraag: tuurlijk kan dat Kijk eens naar classes/functies.

[Spin1]

Cartman! schreef op dinsdag 29 december 2009 @ 11:32:
Je zegt dat er iets in de aanroep veranderd is, wat dan?

in de aanroep is dit verandert:

code:

$bestand = fopen(test2.csv, 'r');

naar dit:

code:

$bestand = fopen($num1, 'r');

zodat deze niet meer op de gewenste manier werkt (wat zeker logisch is):

code:

$result    = mysql_query($sql) or header("location: import.php");

Maar achter de or, zou ik een return willen plaatsen of iets dat de lus weer overnieuw begint.

[doeternietoe]

Wat je eerste deel betreft: kan je het probleem iets beter isoleren? Ik snap het probleem niet helemaal en de grote lap code is niet echt behulpzaam. [code=php] gebruiken zou ook fijn zijn .

Spin1 schreef op dinsdag 29 december 2009 @ 11:08:
Is er een mogelijkheid om deze code aan te passen:

code:

  $result    = mysql_query($sql) or header("location: import.php");

dat hij niet de hele pagina verandert maar de while loop weer overnieuw start.
Ik heb hier een break neergezet, een functie om de while loop gezet. En naar een ander file toegelinkt. Dit allemaal zonder effect.

Kijk eens naar een recursieve functie. Wel uitkijken dat je niet in een eeuwige loop terechtkom. Het is nog beter er voor te zorgen dat die query altijd lukt, zodat het alternatief nooit wordt uitgevoerd.

Edit: Ik denk dat ik nu ongeveer je probleem snap. Doe eens

PHP:

var_dump( $num1 );

vlak voor de declaratie van $bestand.

[ Voor 13% gewijzigd door doeternietoe op 29-12-2009 11:48 ]

[Spin1]

Edit: Ik denk dat ik nu ongeveer je probleem snap. Doe eens

PHP:

var_dump( $num1 );

vlak voor de declaratie van $bestand.

Met var_dump code krijg ik

code:

C:\wamp\www\test2.csvstring<21>"

[ Voor 13% gewijzigd door Spin1 op 29-12-2009 12:20 ]

[Manuel]

En als je gewoon:

PHP:

echo $num1 . " | ";

doet? Dan als ik jou voorbeeld zie hoor je: "C:\wamp\www\test2.csv | " dat eruit te krijgen. Hier gebruik je absolute paden en hiervoor niet. Daar deed je gewoon xxxxxx.csv

[Spin1]

Manuel schreef op dinsdag 29 december 2009 @ 12:41:
En als je gewoon:

PHP:

echo $num1 . " | ";

doet? Dan als ik jou voorbeeld zie hoor je: "C:\wamp\www\test2.csv | " dat eruit te krijgen. Hier gebruik je absolute paden en hiervoor niet. Daar deed je gewoon xxxxxx.csv

Met de code van Manuel krijg ik:
C:\wamp\www\test2.csv |
Te zien.

Maar dat is het grootste probleem volgens mij niet: volgens mij is meer het probleem dit stukje:

PHP:

$result    = mysql_query($sql) or header("location: import.php");

Met de header location kan hij natuurlijk niet meer de opgegeven variabele vinden, als de pagina gerefresht wordt. En bij een statische invoer wel.

[ Voor 18% gewijzigd door Spin1 op 30-12-2009 08:16 ]

[HuHu]

Dat is ook wel een domme constructie, zo met die header() achter de mysql_query. Doe dat eens niet. Wat probeer je daar mee te bereiken?

Ah... ik snap opeens wat je er mee probeert, je wilt de while opnieuw starten zonder de code erna uit te voeren. Daarvoor hebben ze continue uitgevonden. Alleen ben ik bang dat die niet gaat werken in zo'n recursieve functie.

[ Voor 49% gewijzigd door HuHu op 30-12-2009 11:01 ]

[HuHu]

Je code is ook fucking onleesbaar trouwens. Gebruik eens een nette opmaak, juiste inspringing met spaties en de [ php ] tag hier op Tweakers.

PHP:

  if ($pagina < 1) {
    mysql_query ( "INSERT INTO `paginas` (`pagina`,`tekst`,`parentid`) VALUES ('{$val1}','{$val2}','$q')" );
    
    function printbreadcrumb($pagina) {
      $sql = "SELECT id, parentid, pagina FROM paginas WHERE id = {$pagina}";
      
      $result = mysql_query ( $sql ) or header ('Location: import.php');
      $row = mysql_fetch_array ( $result );
      
      if ($row ['parentid']) {
        printbreadcrumb($row['parentid']);
      }
      
      return;
    }
    printbreadcrumb ( $pagina );
    exit ();  
  }

Zie ik het nu goed dat je binnen deze if een recursieve functie printbreadcrumb declareert die printbreadcrumb heet, maar helemaal niets print? Waarom? Waarom recursief? Waarom binnen een while en een if? Wat doet het daar? Wat verwacht jij dat het zou doen?

[Spin1]

HuHu schreef op woensdag 30 december 2009 @ 10:57:
Zie ik het nu goed dat je binnen deze if een recursieve functie printbreadcrumb declareert die printbreadcrumb heet, maar helemaal niets print? Waarom? Waarom recursief? Waarom binnen een while en een if? Wat doet het daar? Wat verwacht jij dat het zou doen?

Ik had de breadcrumb functie binnen de loop geplaatst met als bedoeling:
Kan geen volledig pad maken> Voer de lus overnieuw uit door header.
En als pad in orde is laat zien.

De code werkte met een statische invoer van het csv file. Maar met een invoer via een formulier of iets dergelijks werkt het natuurlijk niet .

Ik heb daarom een paar dingen geprobeerd die ik in mijn startpost heb aangegeven.
En ik zoek daar eigenlijk een oplossing voor.

[Manuel]

In welke directorie staat het bestand precies? Heb vaak genoeg vragen gezien die hadden ook zo'n probleem en bleek nou stond hun file in een andere directorie. Het is gewoon kijken wat de code doet, hier en daar een echo en je hebt het probleem zo te pakken.

[Spin1]

Manuel schreef op woensdag 30 december 2009 @ 14:02:
In welke directorie staat het bestand precies? Heb vaak genoeg vragen gezien die hadden ook zo'n probleem en bleek nou stond hun file in een andere directorie. Het is gewoon kijken wat de code doet, hier en daar een echo en je hebt het probleem zo te pakken.

Het probleem is niet dat het file niet gevonden wordt. Deze wordt wel gevonden.
Maar het probleem is .
Als er een naam geïmporteerd is.
Moet de lus stoppen.
Nieuwe database gegevens ophalen.
En hetzelfde bestand weer inscannen.
net zolang totdat alle items uit het csv file geimporteerd zijn.

[Spin1]

Zou iemand misschien een functie weten waarmee ik bovenstaande voor elkaar krijg.

[RedRose]

Spin1 schreef op dinsdag 05 januari 2010 @ 14:16:
Zou iemand misschien een functie weten waarmee ik bovenstaande voor elkaar krijg.

Ja hoor: IfNameImportedExitLoopAndGetRecordsThenScanFileAgainUntilEOF()

[Woy]

Spin1 schreef op dinsdag 05 januari 2010 @ 14:16:
Zou iemand misschien een functie weten waarmee ik bovenstaande voor elkaar krijg.

Wat heb je nou feitenlijk al zelf bedacht? Je geeft een lijst dingen die je wilt, en je verwacht dat daar iemand een kant en klare functie voor heeft gemaakt? Zo werkt het natuurlijk niet met programmeren.

Je zult je probleem op moeten delen in kleine sub-problemen, en die oplossen. Daarna kun je die sub-oplossingen aan elkaar knopen met standaard taal constructies.