[php+mysql], newsy Opcje

[M45t3r]

Witam!

Mam taki oto kod wysiwetlający newsy:

[PHP] pobierz, plaintext 
<?php
 
$query = 'SELECT * FROM newsy ORDER BY id DESC LIMIT '.$limit.'';
$result = mysql_query($query);
if($result) {
while($row = mysql_fetch_assoc($result)) {
echo ('<b>'.$row['temat'].'</b>
			  <br><br>
			  '.$row['text'].'
			  <br>
			  <br>
			  <a href="mailto:'.$row['mail'].'">'.$row['nick']);
}
}
echo '<br><br>';
?>
[PHP] pobierz, plaintext 

i teraz mam wlasnie problem, zaciolem sie w pewnym momencie, jak zrobic, zeby na link news.php?id=ID NEWSA wyswietal sie news, probowalem w ten sposob, ale sie zaciolem :

[PHP] pobierz, plaintext 
<?php
if(!isset($wiecej))
{
$query = 'SELECT * FROM newsy ORDER BY id DESC LIMIT '.$limit.'';
$result = mysql_query($query);
if($result) {
while($row = mysql_fetch_assoc($result)) {
echo ('<b>'.$row['temat'].'</b>
			  <br><br>
			  '.$row['text'].'
			  <br>
			  <br>
			  <a href="mailto:'.$row['mail'].'">'.$row['nick']);
}
}
}
else
{
$query = 'SELECT * FROM newsy';
$result = mysql_query($query);
if($result) {
while($row = mysql_fetch_assoc($result)) {
//Ale co tutaj  include '?';  naprwde nie mam pojecia....
}
}
}
echo '<br><br>';
?>
[PHP] pobierz, plaintext 

Ten post edytował M45t3r 8.08.2005, 12:51:19

[zbig13]

[PHP] pobierz, plaintext 
<?php
 
if (!isset($_GET['id'])) {
 
$query = 'SELECT * FROM newsy ORDER BY id DESC LIMIT '.$limit.'';
$result = mysql_query($query);
if($result) {
while($row = mysql_fetch_assoc($result)) {
echo ('<b>'.$row['temat'].'</b>
			  <br><br>
			  '.$row['text'].'
			  <br>
			  <br>
			  <a href="mailto:'.$row['mail'].'">'.$row['nick']);
// Tu gdzieś daj jeszcze ten link do  news.php?id=$row['id'].
}
}
echo '<br><br>';
 
} else {
 
$query = "SELECT * FROM newsy WHERE id = " . $_GET['id'];
$result = mysql_query($query);
$row = mysql_fetch_array ($result, MYSQL_ASSOC);
// Tu normalnie wyświetlasz newsa za pomocą echo.
}
 
?>
[PHP] pobierz, plaintext 

[M45t3r]

zrobilem tak:

[PHP] pobierz, plaintext 
<?php
 
$query = "SELECT * FROM newsy WHERE id = " . $_GET['id'];
$result = mysql_query($query);
$row = mysql_fetch_array ($result, MYSQL_ASSOC);
{
echo ''.$row['tresc'].'';
}
}
?>
[PHP] pobierz, plaintext 

i niestety jak otwieram news.php?id=1 powinno pojawiac sie napis test a nie pojawia sie nic czyli cos jest źle...niestety ja nie wiem... jakby ktos nie wiedzial tutaj caly kod:

[PHP] pobierz, plaintext 
<?php
if (!isset($_GET['id'])) {
$query = 'SELECT * FROM newsy ORDER BY id DESC LIMIT '.$limit.'';
$result = mysql_query($query);
if($result) {
while($row = mysql_fetch_assoc($result)) {
echo ('<b>'.$row['temat'].'</b>
			  <br><br>
			  '.$row['text'].'
			  <br>
			  <br>
			  Dodał: <a href="mailto:'.$row['mail'].'">'.$row['nick'].'</a>.
			  <a href="news.php?id='.$row['id'].'">Więcej...</a>
			  ');
}
}
else {
 
$query = "SELECT * FROM newsy WHERE id = " . $_GET['id'];
$result = mysql_query($query);
$row = mysql_fetch_array ($result, MYSQL_ASSOC);
{
echo ''.$row['tresc'].'';
}
}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował M45t3r 8.08.2005, 13:56:59

[Ociu]

Spróbuj dać zamiast isset, empty" title="Zobacz w manualu php" target="_manual.

[M45t3r]

[PHP] pobierz, plaintext 
<?php
 
elseif ($wiecej==''.$row['id'].'')
{
$query = "SELECT * FROM newsy WHERE id = '.$wiecej.'";
$result = mysql_query($query);
$row = mysql_fetch_array ($result, MYSQL_ASSOC);
{
echo ''.$row['tresc'].'';
}
}
 
?>
[PHP] pobierz, plaintext 

rowniez nie dziala...

[nospor]

walnij se jakąś kontrolę ewentualnych bledów a bedziesz wiedzial co jest nie tak:

[PHP] pobierz, plaintext 
<?php
 
//....
else {
echo "wchodze w pokazanie newsa <br/>";
$query = "SELECT * FROM newsy WHERE id = " . $_GET['id'];
echo 'tak wyglada zapytanie:'.$query.'<br/>';
$result = mysql_query($query) or die(mysql_error());
if (mysql_num_rows($result) > 0)
{
  $row = mysql_fetch_array ($result, MYSQL_ASSOC);
  echo 'wyswietlam tresc:'.$row['tresc'].'';
}
else
   echo 'Nie znaleziono rekordu';
}
//.....
 
?>
[PHP] pobierz, plaintext 

[M45t3r]

po wklejeniu tego kodu widze:

wchodze w pokazanie newsa
tak wyglada zapytanie:SELECT * FROM newsy WHERE id =
Something is wrong in your syntax obok '' w linii 1

a jak wejde w linki news.php?id=1 zas pusta strona... co robie źle ? juz kompletnie nei wiem jak sobie z tym poradzic...

[nospor]

ehhh.
po pierwsze masz zle zageszczenie ifów, co powoduje że gdy masz id w url to nie wchodzisz tam gdzie trzeba, wogole nigdzie nie wchodzisz
To poprawiona wersja:

[PHP] pobierz, plaintext 
<?php
 
if (!isset($_GET['id'])) 
{
	$query = 'SELECT * FROM newsy ORDER BY id DESC LIMIT '.$limit.'';
	$result = mysql_query($query);
	if($result) 
	{
		while($row = mysql_fetch_assoc($result)) 
		{
			echo ('<b>'.$row['temat'].'</b>
			  <br><br>
			  '.$row['text'].'
			  <br>
			  <br>
			  Dodał: <a href="mailto:'.$row['mail'].'">'.$row['nick'].'</a>.
			  <a href="news.php?id='.$row['id'].'">Więcej...</a>
			  ');
		}
	}
}
else 
{
	echo "wchodze w pokazanie newsa <br/>";
	$query = "SELECT * FROM newsy WHERE id = " . $_GET['id'];
	echo 'tak wyglada zapytanie:'.$query.'<br/>';
	$result = mysql_query($query) or die(mysql_error());
	if (mysql_num_rows($result) > 0)
	{
		$row = mysql_fetch_array ($result, MYSQL_ASSOC);
		echo 'wyswietlam tresc:'.$row['tresc'].'';
	}
	else
		echo 'Nie znaleziono rekordu';
}
 
 
?>
[PHP] pobierz, plaintext 

Widać roznice?Ty rób se sam wcięcia czasami to zobaczysz wiele bledów.

po drugie: sprawdz czy linki ci się dobrze generują, czy jest id ustawione, czyli czy $row['id'] zwraca to co trzeba. Jak tak to juz powinno byc git

[M45t3r]

sorry... moja wina, tametejsze kody pewnie by dzialaly, gdybym ja nie mylil zmiennej zamiast row['tresc'] mialobyc row['text'] jeszcez raz przepraszam .

Ten post edytował M45t3r 8.08.2005, 15:16:46

[nospor]

sluchaj nie bylo ok. nawet jesli to co mowisz jest prawdą to i tak kichę miales w strukturze warunku. To co ci poprawilem. U ciebie gdy bylo podane id w url to nigdzie nie wchodzilo. Teraz po poprawkach wchodzi.

[M45t3r]

A jak skracać tekst ? tzn jaka funkcja/zapytanie za to odpowiada.. bo chce jeszcze skrocic tekst do 500 znakow, a potem w linki news.php?id=x caly news...

[nospor]

No z takimi pytaniami to najpierw do manuala powinienes sięgac.
left

[SQL] pobierz, plaintext 
SELECT LEFT(pole, 10) 
FROM tabela;
[SQL] pobierz, plaintext 

http://dev.mysql.com/doc/mysql/en/string-functions.html

[strife]

substr" title="Zobacz w manualu php" target="_manual w sql chyba też jakoś tak nie pamiętam dokładnie...

chyba jakoś tak..

[SQL] pobierz, plaintext 
SELECT title, SUBSTR(cos,1,1) 
FROM tabela
[SQL] pobierz, plaintext 

[M45t3r]

a macie taka instrukcje po pl ?

[strife]

Ja jeszcze nie widziałem manual'a do mysql po polsku... pewnie nie ma , ale czego nie rozumiesz przecież nospor Ci napisał przykładowe zapytanie musisz je trochę przerobić pod swoje dane, i masz..

[M45t3r]

em jeszcez jedno pytanie, na podstawie tamtego kodu zrobilem inny pliczek ktory tez ma pobierac dane z z bazy danych i wyglada on tak:

[PHP] pobierz, plaintext 
<?php
 
include 'config.php';
if (!isset($_GET['id']))
{
$query = "SELECT * FROM pages WHERE id = " . $_GET['id'];
$result = mysql_query($query);
$row = mysql_fetch_array ($result, MYSQL_ASSOC);
{
echo '
			'.$row['tresc'].'
	 ';
}
}
?>
[PHP] pobierz, plaintext 

Jak wpisze plik.php?id=4 pusta strona a w bazie w tabeli pages, id 4 mam tekst..
tym razem nie pomylilem zadnej zmiennej a kod nadal nie dziala, a jestem prawie pewein (99% ) ze powinien...

Ten post edytował M45t3r 9.08.2005, 16:10:14

[nospor]

klejne tego typu posty chyba będę zamykał. ty czasem popatrz na kod który piszesz. nie wklejaj bezmyślnie z innych plików. PRzyjrzyj się warukowi:
if (!isset($_GET['id']))
ty robisz selecta dopiero wtedy gdy nie masz w url id. powinieneś robić wlasnie gdy jest id w url. warunek ma byc taki
if (isset($_GET['id']))

parę postów wcześniej podalem ci prostą kontrolę błedów, aby sprawdzać gdzie może być błąd. Do takiej kontroli nalezy zwykłe echo w warunku aby sprawdzić czy oby na pewno do warunku wchodzi. Czy to tak ciężko samemu taką kontrolę zrobić?
Ja rozumiem że jesteś początkujący, ale to nie zwalnia cię od tego, abyś czasem zastanowił się nad kodem czy też wziął do serca nasze uwagi i porady odnośnie pisania skryptów.

[M45t3r]

ok wszystio działa pieknie ale teraz nabrało mnie na komentwanie wiec do news.php dodałem:

[PHP] pobierz, plaintext 
<?php
echo '<br>Dodaj komentarz:
<br><br>
<form action='skomentuj.php' method='post'>
<input type='hidden' name='news_id' value=''.$row['id'].''>
Nick: <input type='text' name='nick'><br><br>
GG: <input type='text' name='gg'><br><br>
Komentarz:<br><textarea name='komentarz'></textarea>
<br>
<input type='submit' value='Dodaj komentarz'>
</form>
';
?>
[PHP] pobierz, plaintext 

skomentuj.php

[PHP] pobierz, plaintext 
<?php
include 'config.php';
$query = 'INSERT INTO `komentarze` VALUES (NULL, "'.$news_id.'", "'.$nick.'", "'.$gg.'", "'.$komentarz.'")';
$result = mysql_query($query);
if($result)
		   {
			echo 'Skomentowałeś newsa :)<a href='index.php'>Powrót na stronę główną</a>';
		   }
else
					{
					 echo 'cos sie zepsulo';
					}
?>
[PHP] pobierz, plaintext 

a tabela konentarze ywglada tak:

[SQL] pobierz, plaintext 
CREATE TABLE `komentarze` (
  `id` int(11) NOT NULL DEFAULT '0',
  `news_id` varchar(255) DEFAULT NULL,
  `nick` varchar(255) DEFAULT NULL,
  `gg` varchar(255) DEFAULT NULL,
  `komentarz` text,
  PRIMARY KEY  (`id`)
) TYPE=MyISAM;
[SQL] pobierz, plaintext 

problem polega na tym, ze wpis nie dodaje sie do bazy czy moze mi ktos powiedziec, gdzie poolenilem blad ?

Ten post edytował M45t3r 28.08.2005, 14:42:31

[popo]

erm zamiast
.$news_id.
daj
.$_GET['news_id'].

reszta analogicznie
i mysl co piszesz porownaj swoj poprzedni i aktualny skrypt (skrypt musi wiedziec z kad brac wartosci zmiennych)

edit
takie podmienienie powinno dzialac aczkolwiek nie jest to rozwiazanie bezpieczne (SQL injection i takie tam)

Ten post edytował popo 28.08.2005, 14:57:06

[M45t3r]

ók nie wazne ;] zrobiłęm ;] ale dlaczego to zapytanie

[SQL] pobierz, plaintext 
$query = 'SELECT left(text, 500) * 
FROM newsy ORDER BY id DESC LIMIT '.$limit.' ';
[SQL] pobierz, plaintext 

nie działa ?

Ten post edytował M45t3r 29.08.2005, 21:21:04