PHP - gdzie jest błąd w funkcji?

Question

Hej,

przerabiam własnie książkę Gajdy, pt. "PHP. Praktyczne projekty".

Mam w niej poniższą funkcję: 

function stringArrayIsOK($s, $separator = "|")
{
	$linie = explode("\n", trim("$s"));
	$liniec = count($linie);
	$pierwsza = count(explode($separator, trim($linie[0])));
	$bledy = array();
	for ($i = 0; $i < $liniec; $i++) {
		$linia = explode($separator, trim($linie[$i]));
		$liniac = count($linia);
		if ($pierwsza != $liniac) {
			$bledy[] = trim($linie[$i]);
		}	
	}
	if (empty($bledy)) {
		return true;
	} else {
		return $bledy;
	}
}

Czy możecnie mi podpowiedzieć dlaczego mimo błędów w pliku rzeki.txt poniższy kod zwraca wartość "Plik poprawny!"?

require_once "vh-array.inc.php";

$p = file_get_contents("rzeki.txt");

$czy_ok = stringArrayIsOK($p);

if ($czy_ok == true) {
	echo "Plik poprawny!";
} else {
	echo "Błędy w wierszach!";
	foreach ($czy_ok as $blad) {
		echo $blad;
	}
}

 

Answer 1

Pokaż plik wejściowy

Prawdopodobnie warunek $pierwsza != $liniac nie jest nigdy spełniony. Czyli zawsze są równe. Najlepiej przed warunek wstaw sobie coś w stylu

echo $pierwsza.' --- '.$liniac.'<br>';

żeby sprawdzić, co obie zmienne zawierają.

 

Swoją drogą, ten kod ma sprawdzać, czy w każdym wierszu jest tyle samo elementów?

Comments

Plik wejściowy poniżej:

Amazonka|7040|7200
Amu-daria|2620|465| || ||||||| ||| ||||| ||
Amur|4440|1855
Brahmaputra|2940|935
Churchill|1610|280

A tutaj wynik wskazanego przez Ciebie polecenia:

3 --- 3
3 --- 23
3 --- 3
3 --- 3
3 --- 3

Ten kod rozbija plik na osobne linie, a następnie każdą linię na poszczególne elementy. Jeśli w którejś linii jest mniej lub więcej elementów niż w pierwszej, to ma zwrócić błąd i wskazać, o którą linię chodzi.

Dzięki za zainteresowanie!

---

Hmm...

A spróbuj wewnątrz if $pierwsza != $liniac wstawić 

var_dump($linie[$i]);

 

---

Jezus, głupi jestem, już wiem, o co chodzi.

Przecież jak sprawdzasz warunek $czy_ok == true, a w $czy_ok masz tablicę z błędami, to to JEST równe true. Niepusta tablica == true. Musisz sprawdzać albo operatorem identyczności (===, $czy_ok === true), albo w inny sposób.

---

Ja jeszcze głupszy! :D

W oryginale są właśnie 3 znaki równa się ;]

Dziękuję za pomoc!