Forum PHP.pl

Mam problem ze skryptem, a raczej czescia skryptu sondy. Wyswietla on opcje i pytanie sondy, pobrane wczesniej z bazy danych.

[PHP] pobierz, plaintext 
<? 
include &#092;"common.php\";
global $id_polaczenia;
$id_polaczenia = db_connect();
if(!$id_polaczenia) error_message(sql_error());
 
$query = &#092;"SELECT p_option, p_hl, apoll_date FROM sondy WHERE p_id = 'ttt'\";
$wynik = mysql_query($query);
if(!$wynik) {
    echo &#092;"Brak sondy\";
}
else {
    $eto =  mysql_fetch_array($wynik);
    $naglowek = $eto[&#092;"p_hl\"];
    $data = $eto[&#092;"apoll_date\"];
    ?>
    <table cellpadding=\"1\" cellspacing=\"1\" width=\"175\">
<tr><td colspan=\"4\" class=\"pollTitle\"><? echo &#092;"$naglowek\"; ?></td></tr>
 
<form method=\"POST\" action=\"skrypt_sondy.php\" target=\"result\">
<?
$ile = &#092;"0\";
$q = &#092;"SELECT count(*) FROM sondy WHERE p_id = 'ttt'\";
$w = mysql_query($q);
$ilosc_odp = mysql_fetch_row($w)
while ($poll_data = mysql_fetch_array($wynik); $ile<$ilosc_odp[0]) {
        $ile++;
        $opcje = $poll_data[&#092;"p_option\"];
?>
    <tr>
    <td width=\"10\" class=\"pollOption\"><input type=\"radio\" name=\"pollvalue\" value=\"odpowiedz<? echo &#092;"$ile\"; ?>></td>
      <td width=\"165\" class=\"pollItem\">
    <? echo &#092;"$opcje\"; ?></td>
     </tr>
    <?
    }
}
?>
[PHP] pobierz, plaintext 

php generuje taki blad:

[PHP] pobierz, plaintext 
<?php
 
Parse error: parse error, unexpected T_WHILE in c:&#092;usrkrasnalwwwmajcokdgdfg.php on line 25
 
?>
[PHP] pobierz, plaintext 

Wiec podejrzewam ze mam zle sformulowany warunek lub jakis blad w nim w petli while. Problem w tym, ze nie za bardzo wiem jak ma wygladac poprawny kod.

Mam jeszcze jeden problem odnosnie wlasnie tego skryptu sondy. Dokladnie z id danej sondy.

[PHP] pobierz, plaintext 
<?php
$query = &#092;"SELECT p_option, p_hl, apoll_date FROM sondy WHERE p_id = '(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?'\";
?>
[PHP] pobierz, plaintext 

Nie do konca wiem w jaki sposob mozna by pobierac pytanie i odpowiedzi sondy o okreslonym id (np. najnowszym). Tzn. myslalem, ze moze przy dodawaniu nowej sondy, dodajac dane do bazy wpisac takze id wlasnie dodawanej sondy do pliku .txt, czy .dat

[PHP] pobierz, plaintext 
<?
$poll_id = &#092;"blablabla\";
$count=0
do {
      $query = &#092;"INSERT INTO sondy VALUES(NULL, '$odp[$count]', '$question', curdate(), '$poll_id')\";
      $wynik = mysql_query($query);
      if(!$wynik) error_message(sql_error());
      $ce = &#092;"$odp[$count]\";
      $count++
     }while ($ce!='');
$file = fopen(sonda/poll_id.dat, &#092;"r\");
fwrite ($file, &#092;"$poll_id\");
fclose ($file);
}
?>
[PHP] pobierz, plaintext 

i pozniej na poczatku skryptu odczytywac z tego pliku id odpowiedniej sondy, jednak tu tkwi problem. W jaki sposob mam odczytac id sondy, ktora chce wyswietlic (jakies dodatkowe informacje zapisywane wraz z id do pliku, czy co?). Eh. nie wiem.....
Mysle ze w miare zrozumiale opisalem moj drugi problem.

Thx za pomoc.

PZdr

Ten post edytował Majcok12 1.11.2004, 08:45:26

Co to jest :

[PHP] pobierz, plaintext 
<?php
// ...
while ($poll_data = mysql_fetch_array($wynik); $ile<$ilosc_odp[0]) {
// ...
?>
[PHP] pobierz, plaintext 

Ten ; (średnik) w warunku jest niedozwolony. To chyba miał być jakiś operator logiczny (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[PHP] pobierz, plaintext 
<?php
 
$ilosc_odp = mysql_fetch_row($w)
while ($poll_data = mysql_fetch_array($wynik); $ile<$ilosc_odp[0]) {
		$ile++;
		$opcje = $poll_data[&#092;"p_option\"];
?>
[PHP] pobierz, plaintext 

Jeszcze dodam od siebie ze brakuje mi średnika po mysql_fetch_row($w).

Ok, wszystko byloby cacy, gdyby nie to ze cos jest nie tak z koncowa czescia funkcji add_poll():

[PHP] pobierz, plaintext 
<?
if (isset($_POST[&#092;"ilosc\"])) {
    global $ilosc;
?>
      <table border=\"0\" cellpadding=\"1\" cellspacing=\"0\" width=\"95%\" height=\"10%\" align=\"center\" valign=\"top\">
        <tr>
           <th width=\"30%\" NOWRAP><font style=\"font-family: Verdana; font-size: 10px; color: gray;\">Pytanie:</th>
           <td width=\"70%\"><input type=\"text\" size=\"20\" maxlength=\"50\" name=\"question\" class=\"pollItem\"></td>
        </tr> 
<?
 
    for ($count=0; $count<$ilosc; $count++) {
        $offset=$count;
        ?>
        <form method=\"post\" action=\"<?php echo $PHP_SELF ?>\">
             <tr>
                 <th width=\"30%\" NOWRAP><font style=\"font-family: Verdana; font-size: 10px; color: gray;\">Podaj odpowiedz nr. <? echo &#092;"$offset\"; ?>:</th>
                 <td width=\"70%\"><input type=\"text\" size=\"20\" maxlength=\"40\" name=\"odp[]\" class=\"pollItem\"></td>
             </tr> 
             <?
            }
        ?>
 
            <th width=\"30%\" colspan=\"2\" NOWRAP>
            <input type=\"submit\" class=\"pollButton\" value=\"Wyślij\" onmouseover=\"this.className='pollButton-over'\" onmouseout=\"this.className='pollButton'\">
        </tr>
     </table>
    </form>
    <?
    if (isset ($_POST[&#092;"odp\"]) && is_array ($_POST[\"odp\"]) && isset($_POST[\"question\"])) {
            global $odp, $question;
            $count=0;
            do {
                $query = &#092;"INSERT INTO sondy VALUES(NULL, '$odp[$count]', '$question', curdate())\";
                $wynik = mysql_query($query);
                if(!$wynik) error_message(sql_error());
                $ce = count($odp);
                $count++;
            }while ($ce>$count);
 
        }
?>
[PHP] pobierz, plaintext 

chodzi o to ze wysiwietla tyle pol ile zostanie podane, jednak gdy sie wpisze pytanie oraz odpowiedzi i przycisnie wyslij nie ma nic oprocz pierwotnej zawartosci strony (zadnych bledow, komunikatow sql, nic) i w bazie danych nie ma ani jednego wpisu.


pzdr

Powiem tylko jedno - dopuki nie nauczysz się oddzielać kodu php od wyglądu strony, nigdy nie nauszysz się pisać bezpiecznie i wykrywać tego typu błędy.
Po prostu kod jest NIECZYTELNY

I nie mam tu wcale na myśli zmuszania kogokolwiek do używania jakiegoś systemu szblonów. Wystarczy przecież takie proste rozwiązanie, jak np.

[PHP] pobierz, plaintext 
<?php
 
function JakasAkcja()
{
	//cos tam robimy
	 // pobieramy jakieś dane
   $GLOBALS['aWidok'] = array (' jakieś dane' =>  ..... );
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
//widok includowany na samym końcu działania skryptu
?>
html strony
 
<?php
foreach( $GLOBALS['aWidok']  as .... )
{
?>  
znowu html
<?php echo 'jakies dane ' ; ?> 
 
html 
<?php 
}//koniecz foreach 
?>
[PHP] pobierz, plaintext 

i tak dalej, i dalej.
Sposób wydajny, prosty, i całkiem prosty do edycji.
Może nie tak prosty jak większość szablonów (szczególnie dla osoby znającej ich składnię (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ) ale na pewno lepszy, niż to, co prezentuje twój skrypt.