phpBB2 by Przemo

picer · Posty: 12

Chciałem zrobić skrypt na forum dzięki któremu można ułożyć pionki na szachownicy. Na krasnalu mi wszystko piknie chodzi ale już jak wrzucę na właściwy serv to nie, dlaczego? Załączam kodzik . I jeszcze jedno pytanko czy da się tak zrobić by zbiór funkcji(lub wszystkie funkcje) był przeszukiwany i określone wartości zmieniały się na inne np. przy pomocy switcha?

Gadatliwa Kasia

koziolek · **Pomógł:** 675 razy Posty: 1925

OoooOooO00... :shock:

na początek kilka dobrych rad na temat standardów, optymalizacji, itp

[php] echo nie potrzebuje nawiasów, wystarczy tak:
Kod:

echo "<center><textarea rows=12 name=tresc cols=50 style=\"border: 1 solid #000000\">
<table border=0>
<tr><td bgcolor=E8CEA5 width=32px height=32px>";
[php] zamiast cudzysłowów " można używać apostrofów '
- w ciągach objętych cudzyslowami php szuka zmiennych, a apostrofani - nie czyli jest troche szybciej
- jeśli chcesz wyświetlić cudzysłów w ciągu objętym cudzysłowami musisz dawać slash przed cudysłowem, z apostrofami nie ma takiego problemu
przykład:
Kod:

echo '<center><textarea rows=12 name=tresc cols=50 style="border: 1 solid #000000">
<table border=0>
<tr><td bgcolor=E8CEA5 width=32px height=32px>';
[html] wszystkie wartości atrybutów muszą być w cudzysłowach, przykład:
Kod:

<center><textarea rows=12 name=tresc cols=50 style="border: 1 solid #000000">
<table border=0>
<tr><td bgcolor=E8CEA5 width=32px height=32px>
- żle

Kod:

<center><textarea rows="12" name="tresc" cols="50" style="border: 1 solid #000000">
<table border="0">
<tr><td bgcolor="E8CEA5" width="32px" height="32px">
- dobrze

[php] pisząc coś takiego:

Kod:

switch($a8){
case b_p:
echo '<img src="gen/img/b_p.gif">';
break;
case b_s:
echo '<img src="gen/img/b_s.gif">';
break;
[...]
}

odnosisz się do stałych b_p i b_s, a przecierz takich nie ma

trzeba je wziąźć w apostrofy lub cudzysłowy

Kod:

switch($a8){
case 'b_p':
echo '<img src="gen/img/b_p.gif">';
break;
case 'b_s':
echo '<img src="gen/img/b_s.gif">';
break;
[...]
}

[php] skoro obrazki nazywają się tak samo jak wartość zmiennej np. $a8 to kod:

Kod:

switch($a8){
case b_p:
echo("<img src=\"gen/img/b_p.gif\">");
break;
case b_s:
echo("<img src=\"gen/img/b_s.gif\">");
break;
case b_g:
echo("<img src=\"gen/img/b_g.gif\">");
break;
case b_w:
echo("<img src=\"gen/img/b_w.gif\">");
break;
case b_h:
echo("<img src=\"gen/img/b_h.gif\">");
break;
case b_k:
echo("<img src=\"gen/img/b_k.gif\">");
break;
case c_p:
echo("<img src=\"gen/img/c_p.gif\">");
break;
case c_s:
echo("<img src=\"gen/img/c_s.gif\">");
break;
case c_g:
echo("<img src=\"gen/img/c_g.gif\">");
break;
case c_w:
echo("<img src=\"gen/img/c_w.gif\">");
break;
case c_h:
echo("<img src=\"gen/img/c_h.gif\">");
break;
case c_k:
echo("<img src=\"gen/img/c_k.gif\">");
break;
}

nożna zastąpić:

Kod:

echo '<img src="gen/img/' . $a8 . '.gif">';

lub

Kod:

echo "<img src=\"gen/img/$a8.gif\">";

z całą resztą podobnie

to wszystko

Kod:

echo("<br><br><div align=center><table border=0><tr><td><form method=post action=gentest.php>
<table width=100%>
8.<SELECT name=a8> <OPTION selected>-=brak=-</OPTION><OPTION value=b_p>biały pion</OPTION><OPTION value=b_s>biały skoczek</OPTION><OPTION value=b_g>biały goniec</OPTION><OPTION value=b_w>biała wieża</OPTION><OPTION value=b_h>biały hetman</OPTION><OPTION value=b_k>biały król</OPTION><OPTION value=c_p>czarny pion</OPTION><OPTION value=c_s>czarny skoczek</OPTION><OPTION value=c_g>czarny goniec</OPTION><OPTION value=c_w>czarna wieża</OPTION><OPTION value=c_h>czarny hetman</OPTION><OPTION value=c_k>czarny król</OPTION></SELECT>
<SELECT name=b8> <OPTION selected>-=brak=-</OPTION><OPTION value=b_p>biały pion</OPTION><OPTION value=b_s>biały skoczek</OPTION><OPTION value=b_g>biały goniec</OPTION><OPTION value=b_w>biała wieża</OPTION><OPTION value=b_h>biały hetman</OPTION><OPTION value=b_k>biały król</OPTION><OPTION value=c_p>czarny pion</OPTION><OPTION value=c_s>czarny skoczek</OPTION><OPTION value=c_g>czarny goniec</OPTION><OPTION value=c_w>czarna wieża</OPTION><OPTION value=c_h>czarny hetman</OPTION><OPTION value=c_k>czarny król</OPTION></SELECT>

[...]
[...]
[...]

</table><center><input type=hidden name=proces value=ok><input type=submit value=generuj></table>");

można zastąpić pętlami

Kod:

<?php
$literki = array('a', 'b', 'c', 'd', 'e', 'f', 'g', 'h');
$opcje = array(
'' => '-=brak=-',
'b_p' => 'biały pion',
'b_s' => 'biały skoczek',
'b_g' => 'biały goniec',
'b_w' => 'biała wieża',
'b_h' => 'biały hetman',
'b_k' => 'biały król',
'c_p' => 'czarny pion',
'c_s' => 'czarny skoczek',
'c_g' => 'czarny goniec',
'c_w' => 'czarna wieża',
'c_h' => 'czarny hetman',
'c_k' => 'czarny król'
);

echo '<br><br><div align="center"><form method="post" action="gentest.php">';

for($numerek = 8; $numerek > 0; $numerek--)
{
echo $numerek . '.';
foreach($literki as $litera)
{
echo '<select name="' . $litera . $numerek . '">';
foreach($opcje as $value => $nazwa)
{
echo '<option value="' . $value . '">' . $nazwa . '</option>';
}
echo '</select>';
}
echo '<br>';
}

echo '<center><input type="hidden" name="proces" value="ok"><input type="submit" value="generuj"></div>';
?>

to chyba tyle

teraz rozwiązanie problemu:

odwołujesz się do zmiennej np. $a8, ale skąd ona się wzieła? odpowiesz "Z formularza" - tak, ale takie odwoływanie się działa tylko przy włączonej opcji register_globals na "On" w pliku php.ini na wielu serwerach jest ona wyłączona ze względów bezpieczeństwa i trzeba się odwoływać tak:

Kod:

$_POST['a8']

KONIEC

mam nadzieje że pomogłem, jakbyś czegoś nie rozumiał to pytaj

[ Dodano: 06-12-2007, 09:43 ]
Polecam:
:arrow:

http://pl.php.net/manual/pl/ - Maual php
:arrow:

http://www.kurshtml.boo.pl/ - Kurs HTML + CSS
:arrow:

http://webmaster.helion.pl/kurshtml/ - Kurs HTML

arcy · **Pomógł:** 509 razy Posty: 3761

koziolek, brawo...
To się nazywa wyczerpująca odpowiedź

picer · Posty: 12

Bardzo mi już pomogłeś. Jeszcze jedna sprawa tylko. Poprawiłem cały kod i skoro formulaże można zrobić pentlą to wyświetlanie też i będzie to bardziej praktyczne. Problem w tym że szachownica ma różnokolorowe pola, czyli numerki parzyste zaczynają tworzyć tabele od jasnego koloru, a nieparzyste ciemnego. Teraz pola tabeli z literkami kolorują się naprzemian według schematu który zaczęły numerki.

Kod:

<?php
if($proces=='')
{
$literki = array('a', 'b', 'c', 'd', 'e', 'f', 'g', 'h');
$literki_1 = array('a', 'c', 'e', 'g');
$opcje = array(
'' => '-=brak=-',
'b_p' => 'biały pion',
'b_s' => 'biały skoczek',
'b_g' => 'biały goniec',
'b_w' => 'biała wieża',
'b_h' => 'biały hetman',
'b_k' => 'biały król',
'c_p' => 'czarny pion',
'c_s' => 'czarny skoczek',
'c_g' => 'czarny goniec',
'c_w' => 'czarna wieża',
'c_h' => 'czarny hetman',
'c_k' => 'czarny król'
);

echo '<br><br><div align="center"><form method="post" action="gen.php">';

for($numerek = 8; $numerek > 0; $numerek--)
{
echo $numerek . '.';
foreach($literki as $litera)
{
echo '<select name="' . $litera . $numerek . '">';
foreach($opcje as $value => $nazwa)
{
echo '<option value="' . $value . '">' . $nazwa . '</option>';
}
echo '</select>';
}
echo '<br>';
}

echo '<center><input type="hidden" name="proces" value="ok"><input type="submit" value="generuj"></div>';
}
if($proces=='ok')
{
echo '<center>
<textarea rows="12" name=:tresc: cols="50" style="border: 1 solid #000000">
<table border="0">';
for($numerek = 8; $numerek > 0; $numerek--)
{
echo $numerek . '.';
foreach($literki as $litera)
{
if($numerek%2 == 0) //jeżeli numerek jest parzysty
{
if($litera == $literki_1{
echo '<td bgcolor="E8CEA5" width="32" height="32"><img src="img/' . $litera . $numerek . '"></td>';} //białe
else{
echo '<td bgcolor="A58B5A" width="32" height="32"><img src="img/' . $litera . $numerek . '"></td>';} //czarne
}
else
{
if($litera == $literki_1{
echo '<td bgcolor="A58B5A" width="32" height="32"><img src="img/' . $litera . $numerek . '"></td>';} //czarne
else{
echo '<td bgcolor="E8CEA5" width="32" height="32"><img src="img/' . $litera . $numerek . '"></td>';} //białe
}
}
echo '</tr>'
}
echo '</table></textarea></center>'
}
?>

Gdzie ja tu mam błąd? Niby w lini 54.

koziolek · **Pomógł:** 675 razy Posty: 1925

picer · Posty: 12

Poprawiłem tą 2 część kodu i pokazuje mi, że w foreach() jest zły argument :evil:

(Warning: Invalid argument supplied for foreach() in c:\usr\krasnal\www\genek\gen.php on line 49).

Kod:

if($proces=='ok')
{
echo '<center>
<textarea rows="12" name=:tresc: cols="50" style="border: 1 solid #000000">
<table border="0">';
for($numerek = 8; $numerek > 0; $numerek--)
{
echo $numerek . '.';
foreach($literki as $litera)
{
if($numerek%2 == 0) //jeżeli numerek jest parzysty
{
if (in_array($litera, $literki_1)){
echo '<td bgcolor="E8CEA5" width="32" height="32"><img src="img/' . $_POST[$litera . $numerek] . '"></td>';} //białe
else{
echo '<td bgcolor="A58B5A" width="32" height="32"><img src="img/' . $_POST[$litera . $numerek] . '"></td>';} //czarne
}
else
{
if (in_array($litera, $literki_1)){
echo '<td bgcolor="A58B5A" width="32" height="32"><img src="img/' . $_POST[$litera . $numerek] . '"></td>';} //czarne
else{
echo '<td bgcolor="E8CEA5" width="32" height="32"><img src="img/' . $_POST[$litera . $numerek] . '"></td>';} //białe
}
}
echo '</tr>';
}
echo '</table></textarea></center>';
}

Widmo · **Pomógł:** 180 razy Posty: 2435

picer, tak na szybko patrzac:

nie masz zadeklarowanych literek:

Kod:

$literki = array('a', 'b', 'c', 'd', 'e', 'f', 'g', 'h');

zadeklaruj tablice w tym drugim if`ie a najlepiej zadeklaruj to przed wszystkimi ifami

koziolek · **Pomógł:** 675 razy Posty: 1925

tak, tak

przenieś wszystkie 3 tablice $literki, $literki_1, $opcje zaraz po
<?php

picer · Posty: 12

Poprawiłem to, miałem jeszcze pare komplikacji, ale postanowiłem je sam rozwiązać, a nie odrazu tu wrzucać

. Dodałem jeszcze pare rzeczy i wszystko cacy ale... Nie działa mi na servie http://www.chess.yoyo.pl/generator.php

Kod:

<?php
$literki = array('a', 'b', 'c', 'd', 'e', 'f', 'g', 'h');
$literki_1 = array('a', 'c', 'e', 'g');
$opcje = array(
'none' => '-=brak=-',
'b_p' => 'biały pion',
'b_s' => 'biały skoczek',
'b_g' => 'biały goniec',
'b_w' => 'biała wieża',
'b_h' => 'biały hetman',
'b_k' => 'biały król',
'c_p' => 'czarny pion',
'c_s' => 'czarny skoczek',
'c_g' => 'czarny goniec',
'c_w' => 'czarna wieża',
'c_h' => 'czarny hetman',
'c_k' => 'czarny król'
);

if($proces=='')
{
echo '<br><br><div align="center"><form method="post" action="generator.php">';

for($numerek = 8; $numerek > 0; $numerek--)
{
echo $numerek . '.';
foreach($literki as $litera)
{
echo '<select name="' . $litera . $numerek . '">';
foreach($opcje as $value => $nazwa)
{
echo '<option value="' . $value . '">' . $nazwa . '</option>';
}
echo '</select>';
}
echo '<br>';
}

echo '<br><center><input type="hidden" name="proces" value="ok"><input type="submit" value="generuj"></div>';
}
if($proces=='ok')
{
//kod pozycji
echo '<center>
<textarea rows="12" name=:tresc: cols="50" style="border: 1 solid #000000">
<table border="0">';
for($numerek = 8; $numerek > 0; $numerek--)
{
foreach($literki as $litera)
{
if($numerek%2 == 0) //jeżeli numerek jest parzysty
{
if (in_array($litera, $literki_1)){
echo '<td bgcolor="E8CEA5" width="32" height="32">'; //białe
if ($_POST[$litera . $numerek] != 'none'){ //jeżeli pole nie jest puste
echo '<img src="img/' . $_POST[$litera . $numerek] . '">';}
echo '</td>';}
else{
echo '<td bgcolor="A58B5A" width="32" height="32">'; //czarne
if ($_POST[$litera . $numerek] != 'none'){ //jeżeli pole nie jest puste
echo '<img src="img/' . $_POST[$litera . $numerek] . '">';}
echo '</td>';}
}
else
{
if (in_array($litera, $literki_1)){
echo '<td bgcolor="A58B5A" width="32" height="32">'; //czarne
if ($_POST[$litera . $numerek] != 'none'){ //jeżeli pole nie jest puste
echo '<img src="img/' . $_POST[$litera . $numerek] . '">';}
echo '</td>';}
else{
echo '<td bgcolor="E8CEA5" width="32" height="32">'; //białe
if ($_POST[$litera . $numerek] != 'none'){ //jeżeli pole nie jest puste
echo '<img src="img/' . $_POST[$litera . $numerek] . '">';}
echo '</td>';}
}

echo '</tr>';
}
}
echo '</table></textarea></center>';
}
?>

Zastanawiałem się czy nie powinno być jeszcze:

Kod:

if($_POST[$proces=='ok]')

ale wtedy nie działa mi na krasnalu. Wiecie dlaczego to się nie klei :?:

koziolek · **Pomógł:** 675 razy Posty: 1925

picer · Posty: 12

Jest jeszcze uno problemo. Niektóre obrazki się nie wyświetlają, bo nie wiem skąd w ich ścieżce pojawia się "%3E" --> http://www.chess.yoyo.pl/viewtopic.php?p=6982#6982
W poście tego wcale nie ma, więc skąd się wzięło :?:

Widmo · **Pomógł:** 180 razy Posty: 2435

picer, HASŁO!

picer · Posty: 12

Kurka zapomniałem sorka

Już nie ma hasła

PS. (Ten problem chyba nie jest związany ze skryptem?)

Widmo · **Pomógł:** 180 razy Posty: 2435

Kod:

$_POST[$litera . $numerek]

Kod:

zamien na

Kod:

trim($_POST[$litera . $numerek])

Gadatliwa Kasia

picer · Posty: 12

Poprawiłem to, ale tu nie ma winy skryptu. Poprostu nie wiem dlaczego przy wyświetlaniu dodaje się ta końcówka "%3E"(i obraz się nie wyświetla) ale w kodzie postu(tabelki[HTML]) niema tego.

Kod: