Witam, mam skrypt do zarządzania MENU oparty o pliki TXT. Mój plik wygląda tak:



Ale przy zarządzaniu tego menu występuje błąd - robi się coś takiego:


Chciałbym, aby to co się powtórzyło - 1||Menu Główne||menu_glowne||||1||file||1|| - zostało usunięte przez skrypt. I każde inne powtórzenie... Z góry dzięki za pomoc ;-)

Przydał by się jakiś kod, gdzie wykonujesz operacje na tym pliku, to może wtedy zlokalizujemy błąd.

[PHP] pobierz, plaintext 
<?php
if ($krok == "prawo") {
 if(isset($_GET["panel_id"])) {
	$filename = GLOWNY."baza_".$data_instalacji."/menu.txt";
 
	$nr_y = $_GET["panel_id"];
	$dane = file($filename);
	$wyciagnij = explode("||", $dane[$nr_y-1]);
 
	$nowy = "".$wyciagnij[0]."||".$wyciagnij[1]."||".$wyciagnij[2]."||".$wyciagnij[3]."||4||".$wyciagnij[5]."||".$wyciagnij[6]."";
 
	$plik = file($filename);
	$plik[$nr_y-1] = $nowy."r\n".$plik[$nr_y-1];
	$plik = implode("",$plik);
 
	$fp = fopen($filename , "w");flock($fp,2);
	fputs( $fp , $plik);
	flock($fp,3);fclose($fp);
 
	$linia_do_przeniesienia = $_GET["panel_id"];
	$dane = file($filename);
	unset($dane[$linia_do_przeniesienia]);
 
	$f = fopen($filename, "w");
	foreach($dane as $linia){
		   fputs($f, $linia);
	}
	fclose($f);
	header("Location: ".ACC_SELF.$link_autoryzacyjny."");
 }
}
?>
[PHP] pobierz, plaintext 

To ma za zadanie przesuwać menu w prawo, link wygląda tak:
panele.php?krok=prawo&panel_id=1

tylko, że to wygląda tak: raz robi się powtórzenie, raz nie ;-)

Jestem początkującym programistą, ale dla mnie trochę ten kod zakręcony Nie wiem za bardzo jak to ma działać, ale np w tym miejscu:

[PHP] pobierz, plaintext 
<?php
$plik = file($filename);
$plik[$nr_y-1] = $nowy.&#092;"r\n\".$plik[$nr_y-1];
?>
[PHP] pobierz, plaintext 

zmienne $nowy i $plik[$nr_y-1] chyba mają taką samą zawartość, z tym że $nowy nie ma na końcu '||'. Czyli zerowy element tablicy $plik ma tak jakby dwa takie same wiersze oddzielone powrotem karetki i nową linią. Tak ma być?
A tutaj:

[PHP] pobierz, plaintext 
<?php
$wyciagnij = explode("||", $dane[$nr_y-1]);
 
	$nowy = "".$wyciagnij[0]."||".$wyciagnij[1]."||".$wyciagnij[2]."||".$wyciagnij[3]."||4||".$wyciagnij[5]."||".$wyciagnij[6].""
?>
[PHP] pobierz, plaintext 

nie rozumiem po co rozbijasz wiersz za pomocą explode jak potem łączysz to z powrotem?
Widze, że na końcu w $nowy nie ma tylko tych dwóch kreseczek '||', wiec wystarczyło użyć funkcji

[PHP] pobierz, plaintext 
<?php
substr($dane[$nr_y-1], 0 , strlen($dane[$nr_y-1])-2)
?>
[PHP] pobierz, plaintext 

Nie wiem, czy dobrze napisałem ale chodzi o to, że funkcja zwraca stringa "wyciętego" z $dane[nr_y-1] od znaku zerowego do drugiego od końca.

[PHP] pobierz, plaintext 
<?php
$wyciagnij = explode("||", $dane[$nr_y-1]);
 
	$nowy = "".$wyciagnij[0]."||".$wyciagnij[1]."||".$wyciagnij[2]."||".$wyciagnij[3]."||4||".$wyciagnij[5]."||".$wyciagnij[6].""
?>
[PHP] pobierz, plaintext 

Ty wygląda tak:
1||Menu Główne||menu_glowne||||1||file||1||

To oznacza, że menu ma być po lewej stronie, a tamto zmienia każdą zawartość oprócz $wyciagnij[5]

i po zmianie powstaje
1||Menu Główne||menu_glowne||||4||file||1||

Nie wiem jak logicznie obmyśliłeś jak ta strona ma działać, do czego służą te wartości w pliku, więc w tym ci niestety nie pomogę.
BTW. Po wykonaniu dwukrotnie tego twojego skryptu na pliku

[HTML] pobierz, plaintext 
1||Menu Główne||menu_glowne||||1||file||1||
2||Logowanie||panel_uzytkownika||||4||file||1||
[HTML] pobierz, plaintext 

wyszło mi coś takiego:

[HTML] pobierz, plaintext 
1||Menu Główne||menu_glowne||||4||file||1
1||Menu Główne||menu_glowne||||4||file||1
2||Logowanie||panel_uzytkownika||||4||file||1||
[HTML] pobierz, plaintext 

Jak chcesz usunąć powtarzające się wiersze, to wczytaj plik za pomocą "file()" do jakiejś tablicy tak jak robiłeś i w pętli for przejdź przez całą tą tablice. W pętli porównuj dwa kolejne wiersze, jeśli będą różne to zapisuj je do pliku, jeśli takie same to nic nie rób i dajesz następny obrót pętli.

Jak byś mógł to zamienić na kod... ;D

Tak na szybko teraz mogę napisać bez uruchamiania, jak bede miał trochę więcej czasu to edytuje i postaram się poprawić co będzie źle. Mniej więcej ma to wyglądać jakos tak:

[PHP] pobierz, plaintext 
<?php
$dane = file($filename);
$wsk = fopen($filename , &#092;"w\");
 
for($i=0; $i<=count($dane)-2; $i++)
{
	 if($dane[$i] !== $dane[$i+1])
		fwrite($wsk, $dane[$i], strlen($dane[$i]));
}
fwrite($wsk, $dane[count($dane)-1], strlen($dane[count($dane)-1]));
?>
[PHP] pobierz, plaintext 

Ale coś mi się wydaje, że może nie usuwać ostatnich dwóch takich samych wierszy, bo w ostatnim jest jeszcze znak końca pliku i te dwa wiersze może uznać za różne. Ale nie wiem czy tak rzeczywiście jest . Potem zobacze, a może ktoś inny pomoże w międzyczasie


Hmmm to usuwanie powtarzających wierszy powinno działać, ale wcześniej w tym swoim skrypcie powinieneś przed fclose($f) dodać do pliku jeszcze znak końca lini fputs($f, "\n") albo fputs($f, "\r\n"). Ja to łopatologicznie robiłem tak, że otwierałem plik w notatniku, dawałem kursor na koniec ostatniego wiersza i wciskałem enter. Wtedy ładnie usuwał. Bo inaczej nie wiem jak rozwiązać ten problem końca pliku