Forum PHP.pl

Witam, czy ktoś uczynny mógłby sprawdzić co z tym kodem jest nie tak? Za cholerę nie chcę działać, pewnie przez linię 66. Wiecie jak to poprawić?

[PHP] pobierz, plaintext 
<?php
require_once('inc/system.php');
 
$h1 = "Rejestracja";
 
if(!empty($_POST)){
	if(!isset($_POST['accept'])) $msg = "<p class='error'>Musisz zaakceptować regulamin.</p>";
	elseif(empty($_POST['login'])) $msg = "<p class='error'>Musisz podać login.</p>";
	elseif(empty($_POST['email'])) $msg = "<p class='error'>Musisz podać email.</p>";
	elseif(empty($_POST['password'])) $msg = "<p class='error'>Musisz podać hasło.</p>";
	elseif(empty($_POST['passwordr'])) $msg = "<p class='error'>Musisz podać powtórne hasło.</p>";
	elseif($_POST['passwordr'] != $_POST['password']) $msg = "<p class='error'>Podane hasła różnią się.</p>";
	else {
		$login = tekst(strtolower($_POST['login']));
		$loginMD5 = md5($login);
		$pass = md5($_POST['password']);
		$email = tekst($_POST['email']);
		$wiek = tekst($_POST['wiek']);
		$plec = tekst($_POST['plec']);
		$miasto = tekst($_POST['miasto']);
		$ip = $_SERVER['REMOTE_ADDR'];
		$data_urodzenia = date($_POST['data_urodzenia']['rok'].'-'.$_POST['data_urodzenia']['miesiac'].'-'.$_POST['data_urodzenia']['dzien']);
 
 
		if((strlen($login) < 4) || (strlen($login) > 25))   $msg = "<p class='error'>Podałeś login o nieprawidłowej długości, wpisz od 4 do 25 znaków</p>";
		elseif(strlen($_POST['password']) < 6)   $msg = "<p class='error'>Hasło musi zawierać minimum 6 znaków</p>";
		elseif(strlen($_POST['wiek']) > 2)   $msg = "<p class='error'>Podano niepoprawny wiek.</p>";
		else {
			$test = mysql_num_rows(mysql_query("select * from users where loginMD5 ='".$loginMD5."' limit 1"));
			if($test != 0) $msg = "<p class='error'>Podany login jest zajęty</p>";
			else {
				mysql_query("insert into users(login, loginMD5, haslo, email, online, wiek, plec, miasto, ip) value ('".$login."','".$loginMD5."','".$pass."','".$email."',".time().",'".$wiek."','".$plec."','".$miasto."','".$ip."')");
				$ids = mysql_insert_id();
				if(empty($ids))  $msg = "<p class='error'>Wystąpił nieoczekiwany błąd skryptu, spróbuj ponownie za chwilę.</p>";
				else $msg = "<p class='info'>Założono nowe konto.</p>";
			}
		}
 
	}
}
 
$gra ="<form action='#' method='post'>
	<table style='margin:0 auto' >
	<tr>
		<td>Login:</td>
		<td><input type='text' name='login'/></td>
	</tr>
	<tr>
		<td>Adres email:</td>
		<td><input type='text' name='email'/></td>
	</tr>
	<tr>
		<td>Hasło:</td>
		<td><input type='password' name='password'/></td>
	</tr>
	<tr>
		<td>Powtórz hasło:</td>
		<td><input type='password' name='passwordr'/></td>
	</tr>	
	<tr>
		<td>Wiek:</td>
		<td><input type='text' name='wiek'/></td>
	</tr>
	<tr>
		<td>Data urodzenia:</td>
		<td><select name='data_urodzenia[dzien]'>"<?php for ($i=1; $i<=31; $i++){echo "<option value='"$i"'>"$i"</option>";}?>"</select> -  <select name='data_urodzenia[miesiac]'>"<?php for ($i=1; $i<=12; $i++){echo "<option value='"$i"'>"$i"</option>";}?>" - <select name='data_urodzenia[rok]'>"<?php for ($i=2010; $i>=1900; $i=$i-1){echo "<option value='"$i"'>"$i"</option>";} ?>"</select></td>
	</tr>
	<tr>
		<td>Płeć:</td>
		<td><select name='plec'><option value='Mężczyzna'>Mężczyzna</option><option value='Kobieta'>Kobieta</option></select></td>
	</tr>
	<tr>
		<td>Miasto:</td>
		<td><input type='text' name='miasto'/></td>
	</tr>
	<tr>
		<td><input type='checkbox' name='accept'/> Akceptuję <a href='regulamin.php'>regulamin</a></td>
		<td align='center'><input type='submit' value='załóż konto'/></td>
	</tr>
	</table>
</form>
";
 
require_once('inc/szablon.php');
?>
[PHP] pobierz, plaintext 

Proszę o pomoc

Nie:

[PHP] pobierz, plaintext 
echo "<option value='"$i"'>"$i"</option>";
[PHP] pobierz, plaintext 

Tylko:

[PHP] pobierz, plaintext 
echo '<option value="' . $i . '">' . $i . '</option>';
[PHP] pobierz, plaintext 

Zainstaluj sobie jakiś edytor z kolorowaniem składni, chociażby Notepad 2.

Linijka 66 wygląda teraz tak:

[PHP] pobierz, plaintext 
	<td><select name='data_urodzenia[dzien]'>"<?php for ($i=1; $i<=31; $i++){echo '<option value="' . $i . '">' . $i . '</option>';}?>"</select> -  <select name='data_urodzenia[miesiac]'>"<?php for ($i=1; $i<=12; $i++){echo '<option value="' . $i . '">' . $i . '</option>';}?>" - <select name='data_urodzenia[rok]'>"<?php for ($i=2010; $i>=1900; $i=$i-1){echo '<option value="' . $i . '">' . $i . '</option>';} ?>"</select></td>
[PHP] pobierz, plaintext 

A błąd wygląda tak:
Parse error: syntax error, unexpected '?' in /home/xxxxxx/public_html/rejestracja.php on line 66

Kolego, forum to nie parser. Masz błąd składniowy w linii 66 w okolicach znaku '?'. Kombinuj.

Zauważyłem, że to błąd składniowy, nie prosiłbym o rozwiązanie, gdybym sam potrafił to zrobić. Przez ostatnie 45 minut przerobiłem tę linijkę na wszystkie możliwe sposoby i zależnie od tego gdzie dałem cudzysłów otrzymywałem albo błąd przed "?" albo ">". Proszę o pomoc!

Twój problem polega na wielokrotnym otwieraniu znaków kanonicznych <?php. Nie możesz w już otwartym bloku PHP ponownie go otwierać

[PHP] pobierz, plaintext 
<?php
<?php ///Takie coś jest nieprawidłowe i powoduje błąd.
?>
?>
[PHP] pobierz, plaintext 

U Ciebie blok zaczyna się <?php i do samego końca skryptu z niego nie wychodzisz próbując otwierać nowe:

[PHP] pobierz, plaintext 
<?php
 
$gra =
<select name='data_urodzenia[dzien]'>"<?php for ($i=1; $i<=31; $i++){echo "<option value='"$i"'>"$i"</option>";}?>"</select> <select name='data_urodzenia[miesiac]'>"<?php for ($i=1; $i<=12; $i++){echo "<option value='"$i"'>"$i"</option>";}?>";
?>
[PHP] pobierz, plaintext 

Rozwiązaniem jest zastąpienie ponownego otwierania bloku PHP funkcją:

[PHP] pobierz, plaintext 
<?php
function date_form_helper($min = 0, $max = 0) {
    	$date = '';
 
		for($i = $min; $i < $max; $i++) {
			 $date .= '<option value=".$i.">'.$i.'</option>';
		}
 
		return $date;
    }
 
    $gry = '
    <select name="data_urodzenia[dzien]">'.date_form_helper(1,31).'</select>
    <select name="data_urodzenia[miesiac]">'.date_form_helper(1900,2014).'</select>';
?>
[PHP] pobierz, plaintext 

W funkcji podajesz od którego momentu ma zacząć ($min) generować liczbę i na jakim skończyć ($max).