php / php/php-src

str_getcsv won't work correctly in some special cases

Offen
#10,566 6 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bug Extension: standard Status: Needs Triage
Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.2k
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
96

Beschreibung

Description

The function should not rebuild the string-annotation in the CSV-String by using the commas, if str_getcsv is used in the first step of the small algorithm.
The following code is motivated by the example of durik at 3ilab dot net : He mentioned: "...
Why not use explode() instead of str_getcsv() to parse rows? Because explode() would not treat possible enclosured parts of string or escaped characters correctly."

<?php

$string =<<<DOMTEST
"PHP, Java","Python"
"PHP2, Java2","Python"
"PHP3, Java3","Python"
DOMTEST;

  $data = str_getcsv($string, "\n" );
        foreach ($data as &$row) {
            $row = str_getcsv($row, ',');
        }
var_dump($data);

Resulted in this output:

array(3) {
  [0]=>
  array(3) {
    [0]=>
    string(3) "PHP"
    [1]=>
    string(5) " Java"
    [2]=>
    string(6) "Python"
  }
  [1]=>
  array(3) {
    [0]=>
    string(4) "PHP2"
    [1]=>
    string(6) " Java2"
    [2]=>
    string(6) "Python"
  }
  [2]=>
  &array(3) {
    [0]=>
    string(4) "PHP3"
    [1]=>
    string(6) " Java3"
    [2]=>
    string(6) "Python"
  }
}

But I expected this output instead:

array(3) {
  [0]=>
  array(2) {
    [0]=>
    string(8) "PHP, Java"
    [1]=>
    string(6) "Python"
  }
  [1]=>
  array(2) {
    [0]=>
    string(10) "PHP2, Java2"
    [1]=>
    string(6) "Python"
  }
  [2]=>
  &array(2) {
    [0]=>
    string(10) "PHP3, Java3"
    [1]=>
    string(6) "Python"
  }
}
PHP Version

8.2.1

Operating System

No response

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, das bereitgestellte PHP-Beispiel mit str_getcsv unter Verwendung von Zeilenumbruch- und Komma-Trennzeichen zu reproduzieren, und untersuche anschließend das bestehende Verhalten der Funktion für Kommas in Anführungszeichen und maskierte Zeichen. Das Issue nennt weder eine Implementierungsdatei noch einen Test; als erledigt gilt die Aufgabe, wenn das beabsichtigte Verhalten der verschachtelten Verarbeitung festgelegt und durch Regressionstests abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
backend-api-design
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.