[PHP]if set problem, if set Opcje

[Grzes978]

witam napisałem sobie taki scrypcik

<?php
include("include/menu.php");

$host = 'localhost';
$user = 'planeta_test';
$passwort = 'planeta_test';
$baza = 'planeta_test';


if(is_set($_GET["v"])
$id=$_GET["v"]; // tutaj wywala bład na tej lini
else
$id=1;


$polaczenie = mysql_connect($host, $user, $passwort)
or die ( 'nie można sie połaczyć z MSQL');
$db = mysql_select_db($baza,$polaczenie);

$query=mysql_query("select id, videoTyp, title, videoId from video where id=$id limit 1",$db)
$result=mysql_fetch_row($query);


$id = $result[0]; // id filmu
$type = $result[1]; //typ filmu
$title = $result[2]; // title filmu
$videoid = $result[3]; // id filmu z URLA

echo( if ($type == 1)

$kod = '<object width="640" height="360">
<param name="allowfullscreen" value="true" />
<param name="allowscriptaccess" value="always" />
<param name="movie" value="http://vimeo.com/moogaloop.swf?clip_id='.$videoid '&amp;server=vimeo.com&amp;show_title=1&amp;show_byline=1&amp;show_portrait=1&amp;color=&amp;fullscreen=1&amp;autoplay=0&amp;loop=0" />
<embed src="http://vimeo.com/moogaloop.swf?clip_id='.$videoid '&amp;server=vimeo.com&amp;show_title=0&amp;show_byline=0&amp;show_portrait=0&amp;color=&amp;fullscreen=1&amp;autoplay=1&amp;loop=0" type="application/x-shockwave-flash" allowfullscreen="true" allowscriptaccess="always" width="640" height="360">
</embed>
</object>';
else
$kod = '<object width="480" height="385">
<param name="movie" value="http://www.youtube.com/v/'.$videoid '&amp;hl=pl_PL&amp;fs=1"></param>
<param name="allowFullScreen" value="true"></param><param name="allowscriptaccess" value="always"></param>
<embed src="http://www.youtube.com/v/'.$videoid '&amp;hl=pl_PL&amp;fs=1" type="application/x-shockwave-flash" allowscriptaccess="always" allowfullscreen="true" width="480" height="385">
</embed>
</object>';
?>


<center>
<table>
<tr>
<td>
<a href=<?php echo '"./test.php?v=' . --$id .'" ';
$id+=2;
?>>Poprzedni film</a>
</td>
<td>
<?php
echo $kod.'<br/>';
echo $title;
?>
</td>
<td>
<a href=<?php echo '"./test.php?v=' . $id .'"'; ?>>Następny film</a>
</td>
</tr>
</table>
</center>

<?php
include("include/footer.php");
?>

i wywala mi bład na lini 25 czyli linijka pi if secie, nie wiem w czym jest rzecz

ma ktos pomysł?

[Mostrom]

Chyba dlatego, że funkcja ta nie nazywa się is_set tylko isset, ale nie jest pewien.
Spróbuj tak:

[PHP] pobierz, plaintext 
if(is_set($_GET["v"]){
@$id=$_GET["v"]; }
[PHP] pobierz, plaintext 

Ten post edytował Mostrom 9.09.2010, 19:43:37

[nospor]

To tez jest bardzo ciekawe:
echo( if ($type == 1)


@Grzes978
1) Zacznij uzywac bbcode
2) Jak wyswietla ci błąd to należy ten błąd dokładnie podać.
3) Zanim napiszesz kolejnego posta zastosuj się proszę do podanych tu porad:
Temat: Jak poprawnie zada pytanie

[Mostrom]

4) Naucz się liczyć, bo wg mnie linia 25 wypada jakoś tak właśnie na to co kolega podał, ale ja tam nie wiem ile linijek usunąłeś przed napisaniem posta

[bastard13]

[PHP] pobierz, plaintext 
if(is_set($_GET["v"])
[PHP] pobierz, plaintext 

Na wszelki wypadek sprawdzałem jeszcze manuala, ale tam też nie ma nic o funkcji is_set. Zmień na isset().

[sadistic_son]

I domknij nawias okrągły!! isset ma domkniety nawias u Ciebie w kodzie ale if już nie!

if ( isset ($_GET['costam']) )

A błąd, który Ci wywala to błąd, że napotkano nieznany string T_STRING. Dzieje się tak bo kolejna linijka po if traktowana jest jako ciąg dalszy warunku logicznego if, a nie jako następne instrukcje. Właśnie przez brak domkniętego nawiasu!

Ten post edytował sadistic_son 10.09.2010, 00:09:09

[Grzes978]

[quote name='nospor' date='9.09.2010, 20:53:58 ' post='788120']
To tez jest bardzo ciekawe:
echo( if ($type == 1)


poprzednie linijki już poprawiłem i faktycznie teraz wywala mi tą linijkę którą zacytował nospor

w czym problem?

[nospor]

w czym problem?

W tym, ze nie raczyłeś wpierw przejrzeć podstaw php.
W echo nie wstawia się zadnych IF
To w IF robi się instrukcje
nie: echo (IF blabla)
a: if (cośtam) echo 'blabla';

http://pl2.php.net/manual/pl/control-structures.if.php

[Grzes978]

no jasne echo jest niepotrzebne w tym momencie, ;-)

[nospor]

no jasne echo jest niepotrzebne w tym momencie, ;-)

To po co je dajesz? żeby ładniej wyglądało?

[Grzes978]

przeoczyłem, czasami tak jest że sie człowiek na coś gapi i nie widzi tego

ale ale, mam nowy orzech do rozgryzienia

otóż

mam cos takiego, co pokazuje że jesli zmienna GEt nie ma wartości to deafault jest 1, moje pytanie brzmi jak zrobić by dafault była ostatnia ID z bazy danych ID jest autoicrement

if(isset($_GET["v"]))
$id=$_GET["v"];
else
$id=1;

myślałem o czymś takim

if(isset($_GET["v"]))
$id=$_GET["v"];
else

$query1=mysql_query("SELECT id from video DESC limit 1",$polaczenie);
$wynik=mysql_result($query1);
$id=$wynik;

oczywiscie mam połaczenie z bazą

$host = 'localhost';
$user = 'planeta_test';
$passwort = 'planeta_test';
$baza = 'planeta_test';

$polaczenie = mysql_connect($host, $user, $passwort)
or die ( 'nie można sie połaczyć z MSQL');
$db = mysql_select_db($baza,$polaczenie)
or die ( 'nie można sie połaczyć z bazą');

niestety cos nie funga

może ma ktos pomysł na to

sorrki za upierdliwość ale na początku wszystko jest czarną magia,

[Mostrom]

Czy użycie BBcode też jest czarną magią?

1)Instrukcje w ELSE też trzeba ująć w klamrach.
2)password pisze się z literką "d" na końcu ;p
3)Zmieniłem ci metodę połączenia z bazą, bo nie wiem czy zmienne będą działać w taki sposób, nie sprawdzałem (najwyżej zmienisz).
4)Z bazą łączysz się na początku kodu a nie na końcu!
5)Porządek w kodzie:

[PHP] pobierz, plaintext 
$polaczenie = @mysql_connect("localhost", "planeta_test", "planeta_test") or die("Nie można nawiązać połączenia z bazą");
$db = @mysql_select_db("planeta_test", $polaczenie) or die ("nie można sie połaczyć z bazą");
 
 
if(isset($_GET["v"])){
   $id=$_GET["v"];
}
else{
   $query1=mysql_query("SELECT id from video DESC limit 1");
   $wynik=mysql_result($query1);
   $id=$wynik;
}
[PHP] pobierz, plaintext 

Ten post edytował Mostrom 10.09.2010, 17:16:03

[Grzes978]

tak wygląda pełny kod a i tak wyskauje mi bład w linii 21 i 31

dokładnie taki

Warning: mysql_result() expects at least 2 parameters, 1 given in C:\wamp\www\test\proby.php on line 21

Warning: mysql_fetch_row() expects parameter 1 to be resource, boolean given in C:\wamp\www\test\proby.php on line 31

[PHP] pobierz, plaintext 
<?php 
			include("include/menu.php"); 
 
			$host = 'localhost';
			$user = 'planeta_test';
			$passwort = 'planeta_test';
			$baza = 'planeta_test';
 
			$polaczenie = mysql_connect($host, $user, $passwort) 
				or die ( 'nie można sie połaczyć z MSQL');
			$db = mysql_select_db($baza,$polaczenie)
				or die ( 'nie można sie połaczyć z bazą');
 
			if(isset($_GET["v"]))
				{
				$id=$_GET["v"];
				}
			else
				{
				$query1=mysql_query("SELECT id from video DESC limit 1",$polaczenie);
				$wynik=mysql_result($query1);
				$id=$wynik;
				}
 
			$polaczenie = mysql_connect($host, $user, $passwort) 
				or die ( 'nie można sie połaczyć z MSQL');
			$db = mysql_select_db($baza,$polaczenie)
				or die ( 'nie można sie połaczyć z bazą');
 
			$query=mysql_query("SELECT id, videoType, title, videoId FROM video where id={$id} LIMIT 1",$polaczenie);
			$result=mysql_fetch_row($query);
 
			$id = $result[0];   
			$type = $result[1];        
			$title = $result[2];     
			$videoid = $result[3];    
 
			if ($type == 1)
 
				$kod = '<object width="640" height="360">
			<param name="allowfullscreen" value="true" />
			<param name="allowscriptaccess" value="always" />
			<param name="movie" value="http://vimeo.com/moogaloop.swf?clip_id='.$videoid. '&amp;server=vimeo.com&amp;show_title=1&amp;show_byline=1&amp;show_portrait=1&amp;color=&amp;fullscreen=1&amp;autoplay=0&amp;loop=0" />
			<embed src="http://vimeo.com/moogaloop.swf?clip_id='.$videoid. '&amp;server=vimeo.com&amp;show_title=0&amp;show_byline=0&amp;show_portrait=0&amp;color=&amp;fullscreen=1&amp;autoplay=0&amp;loop=0" type="application/x-shockwave-flash" allowfullscreen="true" allowscriptaccess="always" width="640" height="360">
			</embed>
			</object>';
 
			 else 
 
				$kod = '<object width="480" height="385">
			<param name="movie" value="http://www.youtube.com/v/'.$videoid. '&amp;hl=pl_PL&amp;fs=1"></param>
			<param name="allowFullScreen" value="true"></param><param name="allowscriptaccess" value="always"></param>
			<embed src="http://www.youtube.com/v/'.$videoid. '&amp;hl=pl_PL&amp;fs=1" type="application/x-shockwave-flash" allowscriptaccess="always" allowfullscreen="true" width="480" height="385">
			</embed>
			</object>';  
			?>
<center>
				<table>
					<tr>
						<td>
							<a href=<?php echo '"./proby.php?v=' . --$id .'" '; 
										$id+=2;
									?>>Poprzedni film</a>
						</td>
						<td>
						<center>
							<?php 
								echo $kod.'<br/>';
								echo $title;
							?>
						</center>
						</td>
						<td>
							<a href=<?php echo '"./proby.php?v=' . $id .'"'; ?>>Następny film</a>
						</td>
					</tr>
				</table>
				</center>
 
		<?php
		include("include/footer.php");
		?>
[PHP] pobierz, plaintext 

Ten post edytował Grzes978 10.09.2010, 17:34:16

[Mostrom]

Po co robisz to w każdym zapytaniu?:

[PHP] pobierz, plaintext 
$polaczenie)
[PHP] pobierz, plaintext 

Twój kod jest strasznie pokręcony =] Takie rzeczy nie ułatwiają dobrze napisanego kodu:

[PHP] pobierz, plaintext 
<a href=<?php echo '"./proby.php?v=' . --$id .'" ';
$id+=2;
?>>Poprzedni film</a>
[PHP] pobierz, plaintext 

Pozatym nei możesz dwa razy łączyć się z bazą! Wywal to co drugi raz napisałeś:

[PHP] pobierz, plaintext 
$polaczenie = mysql_connect($host, $user, $passwort)
or die ( 'nie można sie połaczyć z MSQL');
$db = mysql_select_db($baza,$polaczenie)
or die ( 'nie można sie połaczyć z bazą');
[PHP] pobierz, plaintext 

A na końcu kodu pamiętaj o zakończeniu połączenia z bazą

[PHP] pobierz, plaintext 
mysql_close():
[PHP] pobierz, plaintext 

Ten post edytował Mostrom 10.09.2010, 18:06:43

[bastard13]

ad 1) funkcja mysql_result() przyjmuje dwa parametry: wynik zapytania i numer wiersza, opcjonalnie 3 - nr pola:
http://php.net/manual/pl/function.mysql-result.php

ad 2) dodaj sobie var_dump($query). W zmiennej nie masz obiektu z wynikiem tylko wartość boolowską. Pewnie źle jest skonstruowane zapytanie, ponieważ dla selecta mysql_query() zwraca bool, ale tylko false - gdy zapytanie jest niepoprawne. Sprawdź nazwy kolumn i tabeli.

[Grzes978]

po wielu bojach wkońcu się udało dzieki wszystkim za pomoc