[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)

[uli]

Oczywiście że tak się dzieje, bo:

href[0] to 1-szy znak zmiennej href
href[1] to 2-gi znak zmiennej href

itd

Rozwiązanie:

Zamiast href[0], href[1] itd w ten sposób:

[PHP] pobierz, plaintext 
<?php
$link = '--> <a href="';
				$link .= $menuurl[$menulicz];
				$link .= '"';
				$link .= $menutarget[$menulicz];
				$link .= '>';
				$link .= $menuname[$menulicz];
				$link .= '</a><BR>';
?>
[PHP] pobierz, plaintext 

Wywalić:
$link = implode('', $href);

Widzisz już gdzie masz błąd? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[v1t4n]

ale tak dzieje sie tylko gdy dodaje nowy link, normalnie dziala dobrze...

[Kicok]

1. Przypisanie zamiast porównania:

[PHP] pobierz, plaintext 
<?php
if($menutarget[$menulicz]=1)
?>
[PHP] pobierz, plaintext 

2. Jeśli kolumna menu.id jest kluczem podstawowym z auto_increment, to nie musisz się bawić w sprawdzanie ostatniego ID i zwiększać go o jeden. W ogóle nie musisz bawić się z ID - jeśli nie wstawisz go do zapytania, to kolumna ta zostanie uzupełniona automatycznie.


Natomiast co do zamieniania pierwszej litery na "a", to nigdzie tego nie widzę. Wklej cały kod dodawania linków do bazy.

[v1t4n]

[PHP] pobierz, plaintext 
<?php
if($cat=='addlink')
			{
				if (!isset($_SESSION['zalogowany']))
						 {
							  echo(error('Musisz być zalogowany, żeby móc dodawać newsy.'));
						  }
					   else
						  {
 
							  $sql = mysql_connect($mysqlhost, $mysqllogin, $mysqlpassword);
							  mysql_select_db($mysqldbname);
							  $pytanie7 = 'SELECT `class` FROM `users` WHERE `username` = ''. $_SESSION['login'] .'' LIMIT 0, 30 ';
							  $idzapytania5 = mysql_query($pytanie7);
							  while ($wiersz = mysql_fetch_row($idzapytania5))
								 {
									 $_SESSION['classMS'] = $wiersz[0];
								 }
							  if ($_SESSION['classMS']==3||$_SESSION['classMS']==4)
								 {
							 echo(error('Nie masz uprawnień żeby dodawać nowe newsy. Żeby móc dodawać newsy musisz mieć r
angę "Klasowicz" lub "Admin". Aby zmienić rangę swojego konta poproś o to Admina.'));
								 }
							  if ($_SESSION['classMS']==1||$_SESSION['classMS']==2)
								 {
									echo(naglowek('Dodawanie linków'));
									echo(tree('3', 'Panel administracyjny --> Menu --> Dodaj link'));
									if($id==1||$id=='')
										{
											print 'Link zostanie automatycznie dodany jako ostatni. Jeżeli chcesz zmienić jego kole
jność wejdź w edytowanie menu<P>';
											print '<table><form method="post" action="index.php?action=menusettings&cat=addlink&id=2"><tr>';
											print '<td width="100"><input type="text" name="menuname" size="18"></td>';
											print '<td width="190"><input type="text" name="menuurl" size="35"></td>';
											print '<td width="50"><select name="menutarget">';
											print '<option value="0" selected>W tym oknie</option>';
											print '<option value="1">W nowym oknie</option></select>';
											print '</td></tr></table><center><input type="submit" value="Dodaj link"></form>';
										}
									if($id==2)
										{
											$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 .'');';
											mysql_query($sql3);
											print 'Link został pomyślnie dodany';
										}
								 }
						 }
			}
?>
[PHP] pobierz, plaintext 

Caly skrypt na dodawanie linkow. Podobny problem wystepuje gdy chce edytowac menu (IMG:http://forum.php.pl/style_emoticons/default/blink.gif) !? Gdy ustawie zeby menu w tym momencie w ktorym zmieniane sa wpisy w bazie sie nie wyswietlaly to dziala !? nie moge tego pojac, a poza tym wkurza mnie to prymitywne rozwiazanie ;/

[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

[v1t4n]

heh dzieki za pomoc (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ) ale jak sie robi po nocach to sie nie pamieta gdzie jaka zmienna byla (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) )