[php]Nadpisywanie zmiennej Opcje

[kristaps]

Witam, mam taki mały problem i chyba przez późną porę nie mogę wpaść na jakieś sensowne rozwiązanie.

Mam kod:

[PHP] pobierz, plaintext 
                if($_SESSION['idWholesale'] == 1){
 
			$table = "<tr><td>".$data[0]."</td><td>".$data[1]."</td><td>".$data[2]."</td><td>".$data[3]."</td><td>".$data[4]."</td><td>".$data[5]."</td></tr>";
			$separator = ";";
			$firstLine = true;
		}
 
		if($_SESSION['idWholesale'] == 2){
 
			$table = "<tr><td>".$data[2]."</td><td>".$data[1]."</td><td>".$data[0]."</td><td>".$data[5]."</td></tr>";
			$separator = "|";
			$firstLine = true;
		} 
 
		echo "Podgląd: " . $_SESSION['nameWholesale'] . "</br>";
 
		echo "<table>";
 
		if (($handle = fopen($_SESSION['locationANDfile'], "r")) !== FALSE){
 
 
			while (($data = fgetcsv($handle, 1000, $separator)) !== FALSE){
 
			if($firstLine) {$firstLine = false; continue;}
 
				$num = count($data);
				for ($c=0; $c < $num; $c++){
 
					$data[$c];
 
				}
 
			echo $table;
 
 
			}
 
		fclose($handle);
[PHP] pobierz, plaintext 

W zależności od wyboru w formularzu (idWholesale = 1, 2, itd.) chciałbym generować inne kolumny. Plik CSV rozdzielony jest separatorem, a ja chciałbym wybrać, które kolumny mają zostać wyświetlone.
Wszystko byłoby ok, gdyby nie fakt, że zmienna $table zawiera tablicę $data, a ta zmienna jest zarezerwowana dla

[PHP] pobierz, plaintext 
$data = fgetcsv($handle, 1000, $separator)) !== FALSE
[PHP] pobierz, plaintext 

, więc po wygenerowaniu wynik jest pusty, chyba, że warunki if dam zaraz pod powyższym kodem.

Rozwiązanie pewnie jest proste, a ja najwidoczniej za długo na to patrzę.
Byłbym wdzięczny za pomoc.

Ten post edytował kristaps 26.12.2013, 01:47:15

[Turson]

Jeśli nazwa zmiennej już istnieje i niepotrzebnie ją nadpisujesz to co za problem nazwać ją inaczej?

[kristaps]

[PHP] pobierz, plaintext 
if($_SESSION['idWholesale'] == 1){
 
$table = "<tr><td>".$data[0]."</td><td>".$data[1]."</td><td>".$data[2]."</td><td>".$data[3]."</td><td>".$data[4]."</td><td>".$data[5]."</td></tr>";
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
...
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
while (($data = fgetcsv($handle, 1000, $separator)) !== FALSE){
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
...
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$num = count($data);
	for ($c=0; $c < $num; $c++){
 
	$data[$c]; // <------------ 
 
	}
[PHP] pobierz, plaintext 

bo inaczej nie wyświetlę wyników.

Więc ktoś ma jakiś pomysł na to?

Ten post edytował kristaps 26.12.2013, 11:14:10

[com]

nie bardzo widzę z czym Ty masz problem, pomijając:

[PHP] pobierz, plaintext 
$num = count($data);
	for ($c=0; $c < $num; $c++){
 
	$data[$c]; // <------------ 
 
}
[PHP] pobierz, plaintext 

fakt że ta pętla jest bez sensu , bo nic z tymi danymi w niej nie robisz ponadto prosi się o jakiś foreach a nie for

[kristaps]

Dzięki com za odpowiedź. Pętla faktycznie jest bez sensu, była mi wcześniej potrzebna i zapomniałem usunąć. To może inaczej:

W zależności co wybierzemy w formularzu tworzona jest sesja idWholesale. Więc IFem sprawdzam co zostało wybrane, a dla każdego są ustawione inne separatory, kolumny, itd. Zdjęcie lepiej to obrazuje:



Problem w tym, że echo $table zwraca mi tylko htmla zmiennej $table danego ifa. To pewnie przez to, że zmienna $data jest wykorzystywana wyżej

[PHP] pobierz, plaintext 
while (($data = fgetcsv($handle, 1000, $separator)) !== FALSE){
[PHP] pobierz, plaintext 

i jest nadpisywana.

Dlatego proszę o radę jak to obejść.

PS. Jeżeli dam warunki w pętlę to jest ok. No ale to nie rozwiązanie.

Ten post edytował kristaps 27.12.2013, 00:18:44

[com]

bo do $table masz dopisywać a nie go zastępować czyli $table .=...

poza tym $_SESSION['idWholesale'] zawiera przecież jakias liczbę wiec zwraca to który warunek spełnił, a nie wszystkie bo tylko 1 jest prawdziwy

Ten post edytował com 27.12.2013, 19:05:58

[kristaps]

poza tym $_SESSION['idWholesale'] zawiera przecież jakias liczbę wiec zwraca to który warunek spełnił, a nie wszystkie bo tylko 1 jest prawdziwy

No tak, ale warunek jest spełniany tylko raz. Czyli prawidłowo. Wybiera jedną z powyższych opcji (czerwona bądź czerwona kreska), czyli wszystko jest ok. Niestety dopisywanie również nie działa.

[PHP] pobierz, plaintext 
if($_SESSION['idWholesale'] == 1){ 
			$separator = ";";
			$firstLine = true;
			$table .= '<tr><td>'.$data[0].'</td></tr>';
		}
 
		if($_SESSION['idWholesale'] == 2){ 
			$separator = "|";
			$firstLine = true;
			$table .= "<tr><td>".$data[0]."</td><td>".$data[1]."</td></tr>";
		} 
 
		echo "Podgląd faktury z hurtowni: " . $_SESSION['nameWholesale'] . "<HR>";
 
		echo "<table border = 1>";
 
		if (($handle = fopen($_SESSION['locationANDfile'], "r")) !== FALSE){
 
			while (($data = fgetcsv($handle, 1000, $separator)) !== FALSE){
 
				if($firstLine){$firstLine = false; continue;}
 
 
				echo $table;	
 
			}
 
			fclose($handle);	
 
		echo "</table>";
[PHP] pobierz, plaintext 

Nadal zwraca tylko htmla.

Uprościłem maksymalnie kod:

Plik test.csv:

nazwa;ilosc;cena;
pomidor;1;10;
jabłko;2;11;

Plik index.php

[PHP] pobierz, plaintext 
                        $separator = ";";
			$firstLine = true;
			$table = '<tr><td>'.$data[0].'</td><td>'.$data[1].'</td></tr>';
 
		echo "<table border = 1>";
 
		if (($handle = fopen("test.csv", "r")) !== FALSE){
 
			while (($data = fgetcsv($handle, 1000, $separator)) !== FALSE){
 
				if($firstLine){$firstLine = false; continue;}
 
 
				echo $table;	
 
			}
 
			fclose($handle);	
 
		echo "</table>";
 
		}
[PHP] pobierz, plaintext 

Byłbym bardzo wdzięczny gdybyś zerknął, bo być może źle opisuję problem. Zmienna $table (ta pierwsza) zwraca tylko htmla.

Ten post edytował kristaps 27.12.2013, 22:14:22

[com]

bo ja Cie chyba źle zrozumiałem.. to co jest nie tak ?

tzn chodzi Ci że data[0] itd jest pusta? bo teraz jak na to patrze to Ty najpierw się odwołujesz do data a potem dopiero ją tworzysz ale może gdzieś ja tworzysz wcześniej tylko tego nie widzę..

[kristaps]

Tak, jest pusta, pętla zwraca tylko htmla (<td></td>). No dokładnie, odwołuję się, wiem, że to raczej nie ma prawa działać, ale szukam sposobu na obejście tego. Bo tak jak wyżej pisałem i wklejałem kod, budowa tabel jest różna dla różnych wyborów w formularzu, a raczej bez sensu jest robić osobne pętle dla różnych $_SESSION['idWholesale'].

[com]

poco chcesz robić oddzielne pętle?
foreach po data i dynamicznie dodajesz td tyle ile jest wystąpień w danej lini o ile dobrze widzę

[kristaps]

No właśnie nie. Bo różne $_SESSION['idWholesale'] to i różne separatory i budowa CSV. Tam gdzie w jednym pliku $date[0] oznacza nazwę towaru, to w innym załóżmy cenę. Dlatego trzeba to najpierw określić.

[com]

no to przecież ja wiem ale iteracja po data nie pyta się o to jaki masz separator czy $_SESSION['idWholesale'] bo zawartość data tworzysz tu

[PHP] pobierz, plaintext 
while (($data = fgetcsv($handle, 1000, $separator)) !== FALSE){ //etc }
[PHP] pobierz, plaintext 

A w kodzie który pokazałeś jedyna różnica jest w ilości komórek w wierszu przynajmniej ja to tak rozumie, a co oznacza $date[0] co tez nie jest ważne bo on przecież wydrukuje to co tam jest w tym array

Ten post edytował com 27.12.2013, 23:02:38

[kristaps]

Masz rację, ale niektóre wartości będą dodawane, mnożone, itd. A wtedy zaczynają się schody. Dlatego w tych warunkach IF, muszę później określić które to będą.

[com]

no to teraz zmienia to wgl w zasadzie wszytko... To musisz to określić bo tak to nigdy nie dojdziemy do porozumienia bo samo wyświetlenie jest proste ale operacje na danych tak już załatwić się nie da...

[kristaps]

A co sugerujesz? Pierwotnie widziałem to tak:

[PHP] pobierz, plaintext 
 
		if($_SESSION['idWholesale'] == 1){ 
 
 
                       	$suma_brutto = ($data[2] * $data[3]) * 1.23;
			$suma_brutto = round($suma_brutto, 2);
			$separator = ";";
			$firstLine = true;
			$table = '<tr><td>'.$data[0].'</td><td>'. $suma_brutto.'</td><td>'.$data[7].'</td></tr>';
		}
 
		if($_SESSION['idWholesale'] == 2){ 
 
                        ----> A tutaj załóżmy podobne równanie, tylko na innych $data[] <----
                        $separator = "|";
			$firstLine = true;
			$table = "<tr><td>".$data[0]."</td><td>".$data[1]."</td><td>".$data[2]."</td><td>".$data[3]."</td><td>".$data[4]."</td><td>".$data[5]."</td></tr>";
		} 
 
 
[PHP] pobierz, plaintext 

Kwestia tego bym mógł określić to jeszcze przed wykonaniem pętli.

Ten post edytował kristaps 27.12.2013, 23:21:21

[com]

teraz patrze na ten twój zrzut i widzę ze robisz to w klasie, to moje pytanie czy Ty to teraz chcesz wszytko robić w tej jednej metodzie? bo to trochę bez sensu.. zrób to tak pobierz jakaś metoda x data potem do wyświetlania masz ta co już masz, tylko dasz dane pobrane z metody x, a do tych operacji zrób nowa gdzie bd operował na danych które pobrałeś z metody x i tyle

[kristaps]

com, mógłbyś sklecić jakiś szybki kod, jak ewentualnie miałoby to wyglądać? Bo zrobiłem osobną funkcję, w której trzymam strukturę tabeli oraz resztę danych, ale nic się nie zmieniło - być może źle Cię zrozumiałem.

[com]

a masz ten kod?

pseudo kod

[PHP] pobierz, plaintext 
class cos
{
private $dane = [];
private function get($data)
{
//tutaj pobierasz do $dane
}
public function show()
{
// tutaj wyświetlasz $dane
}
//itd
}
[PHP] pobierz, plaintext 

[kristaps]

Witam, muszę odkopać temat i wrócić do skończenia tego skryptu, dlatego proszę o pomoc. Streszczę cały ten wątek.

Chcę wyświetać różny układ tabel w jednej pętli - w zależności od wybranego pliku i opcji. Załóżmy mam plik.csv oraz plik2.csv

plik.csv

Separator: ;
Budowa pliku: nazwa towaru, ilość, cena

Kod

jabłko;2;11

plik2.csv

Separator: |
Budowa pliku: ilość, nazwa towaru, cena

Kod

2|jabłko|11

Niżej kod:

[PHP] pobierz, plaintext 
if($_SESSION['idWholesale'] == 1){ 
 
                       	$suma_brutto = ($data[1] * $data[2]) * 1.23;
 
                        $separator = ";";
			$firstLine = true;
			$table = '<tr><td>'.$data[0].'</td><td>'. $data[1].'</td><td>'.$suma_brutto.'</td></tr>';
		}
 
if($_SESSION['idWholesale'] == 2){ 
 
	                $suma_brutto = ($data[0] * $data[2]) * 1.23;
 
                        $separator = "|";
			$firstLine = true;
			$table = "<tr><td>".$data[1]."</td><td>".$data[0]."</td><td>'.$suma_brutto.'</td></tr>';
		} 
[PHP] pobierz, plaintext 

Jak widać wyżej mnożymy na różnych $data, w zależności od tego jaką hurtownię wybiorę. No i mam pętlę i warunki:

[PHP] pobierz, plaintext 
if (($handle = fopen($_SESSION['locationANDfile'], "r")) !== FALSE){
 
			while (($data = fgetcsv($handle, 1000, $separator)) !== FALSE){
 
				if($firstLine){$firstLine = false; continue;}
			        echo $table;	
 
			}
 
			fclose($handle);	
 
echo "</table>";
[PHP] pobierz, plaintext 

Skrypt wyświetla pustą tabelę html, bez danych. Błąd jest rzecz jasna w tym, że tablica

[PHP] pobierz, plaintext 
$data
[PHP] pobierz, plaintext 

tworzona jest tutaj:

[PHP] pobierz, plaintext 
$data = fgetcsv($handle, 1000, $separator)
[PHP] pobierz, plaintext 

Stąd moje pytanie jak to obejść, bo hurtowni mam sporo, a nie widzę sensu tworzenia osobnych pętli dla każdej z nich. com próbował pomóc, jednak nie do końca łapię jak miałbym zastosować jego ostatni post.

Będę wdzięczny za pomoc,
dzięki.

Ten post edytował kristaps 27.04.2014, 21:14:18

[session]

Nie orientuję się dobrze w Twoim temacie, ale z tego co widzę, najpierw deklarujesz parametry, a potem chcesz zastąpić zmienne, jednak PHP działa tak jakby od góry do dołu, więc wartości wykorzystywane w jednym miejscu nie będą dynamicznie aktualizowane jeśli gdzieś niżej zmienisz wartość zmiennej. Za to jest funkcja str_replace i możesz zrobić tak:

[PHP] pobierz, plaintext 
$table = '<tr><td>%data0%</td><td>%data1%</td><td>%suma%</td></tr>';
/* skrypt */
str_replace('%data0%',$data[0],$table);
str_replace('%data1%',$data[1],$table);
[PHP] pobierz, plaintext 

Natomiast co do sumy, możesz zmiennej $suma_brutto nadać jakieś wartości całkowite i korzystając ze switcha ( oczywiście switch w środku tej pętli, która czyta plik, i obliczoną wartość przypisujesz do $suma, a potem str_replace ) stosować odpowiednie obliczenia w zależności od tego jaką wartość ma $suma_brutto. Możesz również postąpić tak samo jak z $data korzystając z str_replace, a następnie zastoswać eval() (wiem może nie zbyt wydajne, ale za to będzie działać) tylko wtedy $suma_brutto ma być stringiem, a nie samymi obliczeniami.

W dodatku możesz śmiało dodać str_replace dla wszystkich elementów tablicy $data, ponieważ nawet jeśli nie będziesz z jakiejś korzystał to str_replace po prostu jej nie znajdzie.

Ten post edytował session 27.04.2014, 21:46:20