[PHP] błędy w generowaniu csv Opcje

[neurogen]

napisałem skrypt który z xml generuje csv i działa to dla większości z ponad 20.000 węzłów, ale niektóre wygenerowane linie csv są zepsute i nie mam pojęcia z jakiego powodu. wrzuciłem na hosting plik (ma 48mb ale widać w kilku miejscach te błędy gdy wyłączy się word wrap - nagle następuje przełamanie linii, albo jakiś notice) wynik

[Neutral]

Czy mógłbyś podać kod?

[SmokAnalog]

Zamiast podawać ogromny plik, który w dodatku być może zawiera wrażliwe dane, wysil się na wycięcie z niego jednej prawidłowej i jednej nieprawidłowej linii. I, jak kolega wyżej zasugerował, pokaż kod.

[Pyton_000]

Kod

<b>Notice</b>:  Trying to get property 'LastName' of non-object in <b>H:\xampp\htdocs\pubmed\csv.php</b> on line <b>71</b><br />
<br />
<b>Notice</b>:  Trying to get property 'ForeName' of non-object in <b>H:\xampp\htdocs\pubmed\csv.php</b> on line <b>73</b><br />

Jakieś notice...

Przełamania lini też są poprawne tylko Ty nie poprawnie generujesz CSV. Teksty powinny być ujęte w np. " wtedy przełamanie linni jest nie istotne.

KOD pokaż..

[neurogen]

link do skryptu zamieniającego xml (312mb) na csv. ładuje się 1-2min.
http://evidence-based-dentistry.org/csv.php
view-source:http://evidence-based-dentistry.org/csv.php

np.linie 24,918,4284


csv.php

[PHP] pobierz, plaintext 
<?php
            $articles = simplexml_load_file('2017.xml');
			$id=27;
            foreach ($articles as $text):
 
 
                $title=$text->MedlineCitation->Article->ArticleTitle;
 
$postname = strtolower($title);
$postname = str_replace(' ', '-', $postname);
$postname = preg_replace('/[^0-9a-z\-]+/', '', $postname);
$postname = preg_replace('/[\-]+/', '-', $postname);
$postname = trim($postname,'-');
$postname = substr($postname,0,200).'';
 
				$abstract=$text->MedlineCitation->Article->Abstract->AbstractText;
 
 
    			$author=$text->MedlineCitation->Article->AuthorList->Author;
         		$journallowercase=$text->MedlineCitation->Article->Journal->Title;
    			$journal = ucwords($journallowercase);  
 
			$year=$text->MedlineCitation->DateRevised->Year;
    			$month=$text->MedlineCitation->DateRevised->Month;
    			$day=$text->MedlineCitation->DateRevised->Day;
 
				$pmid=$text->PubmedData->ArticleIdList->ArticleId;
 
				#$tag1=$text->MedlineCitation->KeywordList->Keyword[0];
				#$tag2=$text->MedlineCitation->KeywordList->Keyword[1];
				#$tag3=$text->MedlineCitation->KeywordList->Keyword[2];
				#$tag4=$text->MedlineCitation->KeywordList->Keyword[3];
				#$tag5=$text->MedlineCitation->KeywordList->Keyword[4];
 
				#$tag6=$text->MeshHeadingList->MeshHeading->DescriptorName;
 
 
	if (! empty($abstract)) {
				echo $id;#id
				echo "@";
				echo "1";#post_author
				echo "@";
				echo $year;
				echo "-";
				echo $month;
				echo "-";
				echo $day;
				echo "@";#post_date
				echo $year;
				echo "-";
				echo $month;
				echo "-";
				echo $day;
				echo " 12:00:00";#post_date_gmt
				echo "@";
 
				echo "<div style='text-align:justify;'>";
				echo "<strong>Abstract:</strong> ";
				echo $abstract;#post_content
				echo "<br/><br/>";
				echo $journal;
				echo " - ";
 
				echo $text->MedlineCitation->Article->AuthorList->Author[0]->LastName;
				echo " ";
				echo $text->MedlineCitation->Article->AuthorList->Author[0]->ForeName;
				echo ", ";
 
				if (!empty($text->MedlineCitation->Article->AuthorList->Author[1]->LastName)) {
						echo $text->MedlineCitation->Article->AuthorList->Author[1]->LastName;
						echo " ";
						echo $text->MedlineCitation->Article->AuthorList->Author[1]->ForeName;
						echo ", ";
				}
				if (!empty($text->MedlineCitation->Article->AuthorList->Author[2]->LastName)) {
						echo $text->MedlineCitation->Article->AuthorList->Author[2]->LastName;
						echo " ";
						echo $text->MedlineCitation->Article->AuthorList->Author[2]->ForeName;
						echo ", ";
				}
				if (!empty($text->MedlineCitation->Article->AuthorList->Author[3]->LastName)) {
						echo $text->MedlineCitation->Article->AuthorList->Author[3]->LastName;
						echo " ";
						echo $text->MedlineCitation->Article->AuthorList->Author[3]->ForeName;
						echo ", ";
				}		
				if (!empty($text->MedlineCitation->Article->AuthorList->Author[4]->LastName)) {
						echo $text->MedlineCitation->Article->AuthorList->Author[4]->LastName;
						echo " ";
						echo $text->MedlineCitation->Article->AuthorList->Author[4]->ForeName;
						echo ", ";
				}
				if (!empty($text->MedlineCitation->Article->AuthorList->Author[5]->LastName)) {
						echo $text->MedlineCitation->Article->AuthorList->Author[5]->LastName;
						echo " ";
						echo $text->MedlineCitation->Article->AuthorList->Author[5]->ForeName;
						echo " ";
				}
 
				echo " - ";
				echo $day;
				echo "/";
				echo $month;
				echo "/";
				echo $year;
				echo " - ";
				echo "<a href='https://www.ncbi.nlm.nih.gov/pubmed/$pmid' target='_blank'>";
				echo $pmid;
				echo "</a>";
				echo "</div>";
 
				echo "@";
				echo $title;#post_title
				echo "@";
				echo $abstract;#post_excerpt
				echo "@";
				echo "publish";#post_status
				echo "@";
				echo "open";#comment_status
				echo "@";
				echo "open";#ping_status
				echo "@";
				echo "";#post_password
				echo "@";
				echo $postname;#post_name
				echo "@";
				echo "";#to_ping
				echo "@";
				echo "";#pinged
				echo "@";
				echo $year;
				echo "-";
				echo $month;
				echo "-";
				echo $day;#post_modified
				echo "@";
				echo $year;
				echo "-";
				echo $month;
				echo "-";
				echo $day;
				echo " 12:00:00";#post_modified_gmt
				echo "@";
				echo "";#post_content_filtered
				echo "@";
				echo "0";#post_parent
				echo "@";
				echo "http://localhost/ebd/?p=$id";#guid
				echo "@";
				echo "0";#menu_order
				echo "@";
				echo "post";#post_type
				echo "@";
				echo "";#post_mime_type
				echo "@";
				echo "0";#comment_count
 
 
				echo "\n";
				$id++;
 
	}
	else
	{
		echo "";
	}
 
            endforeach;
 
        ?>
[PHP] pobierz, plaintext 

[Neutral]

Podaj kod w formacie xml, a nie coś już wygenerowanego. OK, dobra już nic, nie zajrzałem do view-source.

Podaj tego jakiś wycinek, bo u mnie wywala błędy (warning).

Ten post edytował Neutral 2.06.2018, 11:34:43

[SmokAnalog]

PHP ma wbudowane funkcje do obsługi CSV, skorzystaj z nich. W Twojej wersji wystarczy, że coś ma wewnątrz znak @ i już masz błąd. Poza tym masz HTML w CSV? Na 99% coś źle kombinujesz.

[neurogen]

wstawiam html do csv i to działa (wordpress odczytuje to prawidłowo), choć pewnie można to zrobić inaczej i lepiej. w tekście nie ma znaków @ innych niż separatory kolumn bo są kasowane wcześniej przez php. pytanie mam tylko z czego wynikają te błędy. że brakuje ' lub " ?





bardzo proszę o pomoc w tym temacie.

Ten post edytował neurogen 2.06.2018, 18:32:38

[SmokAnalog]

Jeśli Author to normalna tablica, zamień sobie to:

[PHP] pobierz, plaintext 
echo $text->MedlineCitation->Article->AuthorList->Author[0]->LastName;
echo " ";
echo $text->MedlineCitation->Article->AuthorList->Author[0]->ForeName;
echo ", ";
 
if (!empty($text->MedlineCitation->Article->AuthorList->Author[1]->LastName)) {
    echo $text->MedlineCitation->Article->AuthorList->Author[1]->LastName;
    echo " ";
    echo $text->MedlineCitation->Article->AuthorList->Author[1]->ForeName;
    echo ", ";
}
 
if (!empty($text->MedlineCitation->Article->AuthorList->Author[2]->LastName)) {
    echo $text->MedlineCitation->Article->AuthorList->Author[2]->LastName;
    echo " ";
    echo $text->MedlineCitation->Article->AuthorList->Author[2]->ForeName;
    echo ", ";
}
 
if (!empty($text->MedlineCitation->Article->AuthorList->Author[3]->LastName)) {
    echo $text->MedlineCitation->Article->AuthorList->Author[3]->LastName;
    echo " ";
    echo $text->MedlineCitation->Article->AuthorList->Author[3]->ForeName;
    echo ", ";
}
 
if (!empty($text->MedlineCitation->Article->AuthorList->Author[4]->LastName)) {
    echo $text->MedlineCitation->Article->AuthorList->Author[4]->LastName;
    echo " ";
    echo $text->MedlineCitation->Article->AuthorList->Author[4]->ForeName;
    echo ", ";
}
 
if (!empty($text->MedlineCitation->Article->AuthorList->Author[5]->LastName)) {
    echo $text->MedlineCitation->Article->AuthorList->Author[5]->LastName;
    echo " ";
    echo $text->MedlineCitation->Article->AuthorList->Author[5]->ForeName;
    echo " ";
}
[PHP] pobierz, plaintext 

Na to:

[PHP] pobierz, plaintext 
echo implode(', ', array_map(function ($author) {
    return $author->LastName . ' ' . $author->ForeName;
}, $text->MedlineCitation->Article->AuthorList->Author));
[PHP] pobierz, plaintext 

[neurogen]

dzięki to pewnie skróci kod ale raczej nie zlikwiduje błędów.

[SmokAnalog]

Nie mów hop może naprawić, bo Twój błąd to odwoływanie się do nieistniejącego obiektu.

[neurogen]

<b>Warning</b>: array_map(): Argument #2 should be an array in <b>H:\xampp\htdocs\pubmed\csv.php</b> on line <b>106</b><br />
<br />
<b>Warning</b>: implode(): Invalid arguments passed in <b>H:\xampp\htdocs\pubmed\csv.php</b> on line <b>106</b><br />

[SmokAnalog]

Czyli to nie jest tablica, ale obiekt implementujący ArrayAccess. Ja bym poszedł w tym kierunku, żeby pozamieniać ten kod z wypisywaniem autorów na coś zgrabniejszego i bezpieczniejszego. Mogę Ci pomóc z ogarnięciem tego, jeśli mi powiesz gdzie mogę znaleźć tę bibliotekę.

[neurogen]

źródło xml jest tu xml a kod skryptu php tu php

[SmokAnalog]

Link do PHP nie działa.

[neurogen]

poprawka: txt

[SmokAnalog]

Spróbuj tego:

[PHP] pobierz, plaintext 
echo implode(', ', array_map(function (SimpleXMLElement $author) {
    return $author->LastName . ' ' . $author->ForeName;
}, ((array) $text->MedlineCitation->Article->AuthorList)['Author']));
[PHP] pobierz, plaintext 

Nie wiem jak zgrabniej zamienić SimpleXMLElement na tablicę dzieci, ta bilbioteka jest strasznie wkur*iająca.

[neurogen]

niestety te same błędy

[SmokAnalog]

U mnie błędów nie ma, ale prawdopodobnie korzystasz z innego pliku XML niż ja. W kodzie jest użyty 2017.xml, a wysłałeś mi 2018.xml.

[neurogen]

faktycznie nie zauważam błędów NOTICE ale te przełamania kodu nadal występują i nie wiem dlaczego. widać to w podglądzie strony np linia 24 view-source:http://evidence-based-dentistry.org/csv.php