[MySQL][PHP] Funkcje co robię żle, 2 Funkcje odpowiedzialne za wyswietlanie stron Opcje

[imashjan82]

Ta funkcja która jest w stopce odpowiedzialna za wyswietlenie stopki strony

[PHP] pobierz, plaintext 
function footer()
    {
	      $link = "link";
          $ask = mysql_query
		  ("SELECT * FROM footer");
		  while ($wiersz = mysql_fetch_array($ask))
		  {
		  print" <a href='ask.php?=".$wiersz['link']."'>".$wiersz['tytul']."</a> |";
         }
		  print"<br /><br /><br />  <br /><br />";
	}
[PHP] pobierz, plaintext 

Ta funkcja ma wyswietlic strone o danym tytule link wyglada tak www.strona.pl/ask.php?=o-nas

[PHP] pobierz, plaintext 
function ask($link)
		{
		 if (isset($_POST['$link']))
		 {
		  $pytanie =mysql_query
		  ("SELECT tytul_strony , tresc FROM footer WHERE link='$link'");	  
          while ($kod = mysql_fetch_array($pytanie))
		  {
		  print"<b>".$kod['tytul_strony']."</b><br /><br />".$kod['tresc']."<br />";
		  }}
		  else{
		  print"Podana strona nie istnieje";
		  }
		  }
 
[PHP] pobierz, plaintext 

co robie nie tak? ze caly czas wyswietla mi Podana Strona nie istnieje? jakaś chocby mała podpowiedz.

Ten post edytował imashjan82 7.04.2010, 22:02:18

[darko]

tu: if (isset($_POST['$link']))

[imashjan82]

szukałem w google i dalej nie wiem co? :-(

[Kshyhoo]

darko już Ci napisał. Popatrz, co przekazujesz w $_POST.

[imashjan82]

Przekazuje $link czyli to : ask.php=$link ? jak usunel znaczek $ to nie dziala wogole i pokazuje blad.

[janiek_j]

if (isset($_POST['$link']))

poprawa na

if (isset($_POST[$link]))

[imashjan82]

to tez nie to dalej pokazuje ze strona nie istnieje

[janiek_j]

a href='ask.php?TU=".$wiersz['link']."'>".$wiersz['tytul']."</a> |";


tam gdzie ci napisałem "TU" powinno coś być, zgadnij co, bo to zbyt proste

[mortus]

Pierwszy kod linia 8

[PHP] pobierz, plaintext 
 print "<a href='ask.php?link=" . $wiersz['link'] . "'>" . $wiersz['tytul'] . "</a> |";
[PHP] pobierz, plaintext 

[imashjan82]

nie dziala

[PHP] pobierz, plaintext 
print" <a href='ask.php?=".$_POST[$link]."'>".$wiersz['tytul']."</a> |";
[PHP] pobierz, plaintext 

nie dziala

[PHP] pobierz, plaintext 
print" <a href='ask.php?=".$_POST['$link']."'>".$wiersz['tytul']."</a> |";
[PHP] pobierz, plaintext 

nie dziala

[PHP] pobierz, plaintext 
print" <a href='ask.php?=".$_POST['link']."'>".$wiersz['tytul']."</a> |";
[PHP] pobierz, plaintext 

nie dziala

[PHP] pobierz, plaintext 
print" <a href='ask.php?=".$_POST[link]."'>".$wiersz['tytul']."</a> |";
[PHP] pobierz, plaintext 

i za kadzym razem adres wyglada tak : ask.php= i tyle

dodam ze to wszystko jest w jednym pliku, plik stopki jest includowany

Ten post edytował imashjan82 7.04.2010, 23:05:44

[mortus]

Pierwszy kod linia 8 w funkcji footer powinna wyglądać tak:

[PHP] pobierz, plaintext 
print "<a href='ask.php?link=" . $wiersz['link'] . "'>" . $wiersz['tytul'] . "</a> |";
[PHP] pobierz, plaintext 

Nie wiem, czy ja po chińsku piszę?

Funkcja druga (ask) linia 3:

[PHP] pobierz, plaintext 
if(isset($_POST['link']))
[PHP] pobierz, plaintext 

Jak to nie zadziała, to wklej cały kod.

[imashjan82]

NIe piszesz po chińsku. Zrobiłem tak ale dalej jest to samo.

[mortus]

Skoro wszystko jest w jednym pliku, to wklej tutaj kod. Ciekaw jestem, jak Ty tę funkcję ask wywołujesz.

[imashjan82]

tak wywoluje
plik z funkcjami

[PHP] pobierz, plaintext 
function menu_lewe()
   {
      $ask = mysql_query 
	  ("SELECT * FROM menu_lewe ORDER BY id LIMIT 0,15");
              while ($wiersz = mysql_fetch_array($ask))
              {
                print"<a href='http://worcester.pl/".$wiersz['link']."'>".$wiersz['tytul']."</a><br />";
               }
    }
///////////////////////////////////////	WIADOMOSCI NA GLOWNEJ STRONIE ////////////
function news_home()
        {
		   print"<div class='title'>Wiadomość z UK & Worcestershire</div> <br />";
 
	       $ask = mysql_query
		   ("SELECT * FROM news ORDER BY ID DESC LIMIT 0,7");
		   while ($wiersz = mysql_fetch_array($ask))
		   {
		    print"<a href='news/".$wiersz['ID'].",".$wiersz['link'].".html'>".$wiersz['tytul']."</a><br />";
			}
			print"<a href='http://www.worcester.pl/news/'>Archiwum Aktualnosci</a>";
		}	
///////////////////////////////////////	STOPKA WSZEDZIE ////////////////////////////	
function footer()
    {
	      $link = "link";
          $ask = mysql_query
		  ("SELECT * FROM footer");
		  while ($wiersz = mysql_fetch_array($ask))
		  {
		  print" <a href='ask.php?link=" .$wiersz['link']. "'>" .$wiersz['tytul']. "</a> |";
         }
		  print"<br /><br /><br /><br /><br />";
	}
//////////////////////////////////// GALERIA NA GŁOWNEJ STRONIE//////////////////////
function gallery_home()
    {
			$ask = mysql_query
			("SELECT * FROM gallery  ID LIMIT 0,5");
			while ($wiersz = mysql_fetch_array($ask))
				{
				 print"<a href='gallery/=".$wiersz['ID']."'><img src='gallery/worcester/".$wiersz['ID'].".JPG' width='151' border='0' height='75'></a>";
				 }
	  }	
////////////////////////////////// o serwisie ///////////////////////////////////////	  	
function ask($link)
		{
		 if (isset($_POST[$link]))
		 {
		  $pytanie =mysql_query
		  ("SELECT tytul_strony , tresc FROM footer WHERE link='$link'");	  
          while ($kod = mysql_fetch_array($pytanie))
		  {
		  print"<b>".$kod['tytul_strony']."</b><br /><br />".$kod['tresc']."<br />";
		  }}
		  else{
		  print"Podana strona nie istnieje";
		  }
		  }
 
[PHP] pobierz, plaintext 

plik ask

[PHP] pobierz, plaintext 
 
require_once('function/function.php');
 
lacz_bd();
 
$page_title = 'tytul strony';
include "templates/header.html";
 
include "templates/header_top.html";
print"<tr><td>";
?>
<script language='JavaScript' type='text/javascript' src='http://www.iiads.eu/adx.js'></script>
<script language='JavaScript' type='text/javascript'>
<!--
   if (!document.phpAds_used) document.phpAds_used = ',';
   phpAds_random = new String (Math.random()); phpAds_random = phpAds_random.substring(2,11);
 
   document.write ("<" + "script language='JavaScript' type='text/javascript' src='");
   document.write ("http://www.iiads.eu/adjs.php?n=" + phpAds_random);
   document.write ("&amp;what=zone:1");
   document.write ("&amp;exclude=" + document.phpAds_used);
   if (document.referrer)
      document.write ("&amp;referer=" + escape(document.referrer));
   document.write ("'><" + "/script>");
//-->
</script><noscript><a href='http://www.iiads.eu/adclick.php?n=a9320cbb' target='_blank'><img src='http://www.iiads.eu/adview.php?what=zone:1&amp;n=a9320cbb' border='0' alt=''></a></noscript>
<?
print"</tr></td>";
 
include"templates/top_spacer.html";
 
print"<tr>
<td width='15' bgcolor='#e8e7e2'>&nbsp;</td>
<td width='204' bgcolor='#dbdbcf' class='left_column' valign='top'>"; menu_lewe(); 
print"</td><td width='336' class='news_home' bgcolor='#e8e7e2'>"; ask($link); 
print"</td><td width='204' class='right_column' bgcolor='#dbdbcf'></td>
<td width='22' bgcolor='#e8e7e2'>&nbsp;</td>
</tr></table>";
include"templates/footer.html";
print"</body></html>";  	
 
[PHP] pobierz, plaintext 

plik stopki

[HTML] pobierz, plaintext 
<table width="781" cellpadding="0" cellspacing="0" align="center" bgcolor="#e8e7e2">
 <tr>
  <td width="775" align="center">
 
  <? footer(); ?>
 
  </td>  
 </tr> 
</table>
 
[HTML] pobierz, plaintext 

Ten post edytował imashjan82 8.04.2010, 00:04:39

[AdIoS_Neo]

Zaraz zaraz Przecież jeśli on chce pobrać zmienną z linku [www.strona.pl/ask.php?link=o-nas] to to $_GET[] jest a nie $_POST[]... oO btw. nie ma to jak szczegółowo opisać na czym polega problem ...

[mortus]

Zaraz zaraz Przecież jeśli on chce pobrać zmienną z linku [www.strona.pl/ask.php?link=o-nas] to to $_GET[] jest a nie $_POST[]... oO btw. nie ma to jak szczegółowo opisać na czym polega problem ...

To i tak nic nie da. Do powyższego kodu trzeba zastosować dwie zmiany:
1. w pliku ask.php w linii 35 zamiast ask($link); powinno być ask($_GET['link']);,
2. w funkcji ask() w trzeciej linijce powinno być if(isset($_GET['link']) && $_GET['link'] == $link) zamiast if(isset($_POST['link'])).

Poza tym mam zastrzeżenia co do przeplatania kodu PHP i (X)HTML w pliku ask.php, ale to odrębna kwestia i mniej znacząca.

[imashjan82]

Proszę proszę pisać o tych zastrzeżeniach co do pliku ask.php. Nie powinieniem go drukować w printach tak?

[mortus]

Proszę proszę pisać o tych zastrzeżeniach co do pliku ask.php. Nie powinieniem go drukować w printach tak?

Nie o to chodzi, że nie powinieneś, ale skoro już przeplatasz kod (X)HTML i PHP to możesz to robić znacznie bardziej czytelnie (bez używania print). To chyba nie jest cały plik ask.php, bo nigdzie nie widzę znaczników rozpoczynających <html>, sekcji <head></head> itp. rzeczy, ale podany fragment można zapisać w taki sposób:

[PHP] pobierz, plaintext 
            <?php
            require_once('function/function.php');
 
            lacz_bd();
 
            $page_title = 'tytul strony';
 
            include "templates/header.html";
 
            include "templates/header_top.html";
            ?>
            <tr>
                <td>
                    <script language='JavaScript' type='text/javascript' src='http://www.iiads.eu/adx.js'></script>
                    <script language='JavaScript' type='text/javascript'>
                    <!--
                    if(!document.phpAds_used) document.phpAds_used = ',';
                    phpAds_random = new String (Math.random()); phpAds_random = phpAds_random.substring(2,11);
 
                    document.write ("<" + "script language='JavaScript' type='text/javascript' src='");
                    document.write ("http://www.iiads.eu/adjs.php?n=" + phpAds_random);
                    document.write ("&amp;what=zone:1");
                    document.write ("&amp;exclude=" + document.phpAds_used);
                    if (document.referrer) document.write ("&amp;referer=" + escape(document.referrer));
                    document.write ("'><" + "/script>");
                    //-->
                    </script>
                    <noscript>
                        <a href='http://www.iiads.eu/adclick.php?n=a9320cbb' target='_blank'>
                            <img src='http://www.iiads.eu/adview.php?what=zone:1&amp;n=a9320cbb' border='0' alt=''>
                        </a>
                    </noscript>
                </td>    
            </tr>
            <?php include"templates/top_spacer.html"; ?>
            <tr>
                <td width='15' bgcolor='#e8e7e2'>&nbsp;</td>
                <td width='204' bgcolor='#dbdbcf' class='left_column' valign='top'><?php menu_lewe(); ?></td>
                <td width='336' class='news_home' bgcolor='#e8e7e2'><?php ask($link); ?></td>
                <td width='204' class='right_column' bgcolor='#dbdbcf'></td>
                <td width='22' bgcolor='#e8e7e2'>&nbsp;</td>
            </tr>
        </table>
        <?php include"templates/footer.html"; ?>
    </body>
</html>
[PHP] pobierz, plaintext 

Bynajmniej nie chodzi tu o formatowanie, ale o samo wywoływanie skryptów (funkcji) PHP. Po prostu wstawiamy odpowiedni kod PHP w miejscach, w których jest to absolutnie konieczne.

[imashjan82]

To jest cały plik. poprostu znaczniki są w pliku header.html potem góra strony jest w header_top.html. I już wiedzę co muszę zmienić. część z iiads.eu musze dodac to pliku header_top.html bo to jest część strony która się nie zmienia. Tak samo boczne menu znaczy lewa strona. Zmienne jest tylko centrum strony i prawe menu. Bardzo dziękuję za poradę.