[MySQL] Odczyt i zapis w tej samej tabeli na tej samej stronie, Edytowanie menu Opcje

[v1t4n]

Witam

Chcialem zrobic mozliwosc edytowania menu. Problem pojawia sie gdy chce dodac nowy link do bazy, mianowicie na stronie ktora laczy sie z baza i dodaje wpis pojawia sie blad z wyswietlaniem menu. Problem wystepuje tylko na tej stronie - gdy przejdzie sie na inna wszystko jest w porzadku. Blad jest taki ze podaje mi pierwsze litery z nazwy i url do strony, wiec zamiast

[HTML] pobierz, plaintext 
<a href="index.php">Strona glowna</a>
[HTML] pobierz, plaintext 

pokazuje sie '

[HTML] pobierz, plaintext 
<a href="i">S</a>
[HTML] pobierz, plaintext 

Rowniez wpis do bazy ktory jest jednoczesnie tworzony jest bledny. Przekreca pierwsza litere z kazdego pola. Zamiast

Kod

Ksiega gosci

tworzy

Kod

asiega gosci

i zamiast

Kod

index.php?action=guestbook

wstawia

Kod

andex.php?action=guestbook

Nie wiem, moze wina wynika z bledu apache. Testowalem narazie tylko u siebie (korzystam z WebServa).

Kod wyswietlania menu: (w pliku cfg.txt zapisane sa zmienne dotyczace hosta, loginu, hasla i nazwy bazy mysql)

[PHP] pobierz, plaintext 
<?php
include('cfg.txt');
		$sql = mysql_connect($mysqlhost, $mysqllogin, $mysqlpassword);
		mysql_select_db($mysqldbname);
		echo(naglowek('Menu'));
		print '<div class="menu">';
		$sql2 = 'SELECT * FROM `menu` ORDER BY `id` ASC LIMIT 0, 30 ';
		$idsql2 = mysql_query($sql2);
		$menulicz=0;
		while($wiersz = mysql_fetch_array($idsql2))
			{
				$menuid[$menulicz]=$wiersz[0];
				$menuname[$menulicz]=$wiersz[1];
				$menuurl[$menulicz]=$wiersz[2];
				$menutarget[$menulicz]=$wiersz[3];
				if($menutarget[$menulicz]=1)
					{
						$menutarget[$menulicz]='';
					}
				if($menutarget[$menulicz]=0)
					{
						$menutarget[$menulicz]=' target="_blank"';
					}
				$href[0] = '--> <a href="';
				$href[1] = $menuurl[$menulicz];
				$href[2] = '"';
				$href[3] = $menutarget[$menulicz];
				$href[4] = '>';
				$href[5] = $menuname[$menulicz];
				$href[6] = '</a><BR>';
				$link = implode('', $href);
				print $link;
			}
		print '</div>';
?>
[PHP] pobierz, plaintext 

Dodawanie wpisu do bazy:

[PHP] pobierz, plaintext 
<?php
$sql3 = 'SELECT `id` FROM `menu` ORDER BY `id` DESC LIMIT 0, 1 ';
											$idsql3 = mysql_query($sql3);
											while ($wiersz = mysql_fetch_row($idsql3))
												{
													$ilosclinkow=$wiersz[0];
												}
											$ilosclinkow++;
											echo($ilosclinkow);
											$sql3 = 'INSERT INTO `menu` (`id`, `name`, `url`, `target`) VALUES (''. $ilosclinkow .'', ''. $menuname .'', ''. $menuurl .'', ''. $menutarget .'');';
											$sql3 = 'INSERT INTO `menu` (`id`, `name`, `url`, `target`) VALUES (''. $ilosclinkow .'', ''. $menuname .'', ''. $menuurl .'', ''. $menutarget .'');';
											mysql_query($sql3);
											print 'Link został pomyślnie dodany';
?>
[PHP] pobierz, plaintext 

Tekst skopiowalem ze srodka sformatowanego skryptu strony wiec jest duzo spacji.
Za ewentualna niewiedze ze stosowania podstawowych funkcji przepraszam i obiecuje sie poprawic (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Kicok]

Co do prymitywnego rozwiązania, to muszę się zgodzić.

A co do literki "a":
1. Dodajesz link do bazy na podstawie informacji pobranych z formularza. Tutaj wszystko jest OK. Zmienna $menuname jest teraz oczywiście stringiem.
2. Następnie wyświetlasz menu (Patrz: kod z twojego pierwszego posta). W tym celu używasz niezadeklarowanych zmiennych jako tablic. Ale niezadeklarowane to one są tylko z pozoru. Tak naprawdę, to zmienna $menuname istnieje => patrz punkt 1.
3. Jako, że $menuname to jest string, to pisząc: $menuname[0] odwołujesz się do pierwszego znaku tego stringu, a nie pierwszego elementu tablicy
4. W pętli nie zwiększasz zmiennej $menulicz, więc z każdym przejściem nadpisywany jest pierwszy znak zmiennej $menuname pierwszym znakiem zmiennej $wiersz[1]
5. Tak samo podczas składania linku - odwołujesz się do pierwszego znaku stringu, a nie do pierwszego elementu tablicy, więc zamiast nazwy linku wyświetla ci jakieś "a"

6. Nie wstawiaj do swojego kodu funkcji: error_reporting( E_ALL | E_STRICT ); bo się przestraszysz
7. Powodzenia w pisaniu skryptów używających register_globals