Funckja i lista rozwijana Opcje

[koreja]

Witam. Mam mały problem, mianowicie zbudowałem taką funkcję:

[PHP] pobierz, plaintext 
<?php
function buildOptions($options, $selectedOption) {
    foreach($options as $value => $desc) {
        if($value == $selectedOption) {
            print '<option value "' . $value . '" selected="selected" name="'.$value.'">' . $desc . '</option>';
        } else {
            print '<option value"' . $value . '" name="'.$value.'">' . $desc . '</option>';
        }
    }
}
?>
[PHP] pobierz, plaintext 

Wklejam ją do poniższego formularza:
strona.php

[HTML] pobierz, plaintext 
<form action="strona.php" method="post">
    <label for="opt">Opcje:</label>
        <select id="opt" name="opt"><?php buildOptions($options); ?></select><br />
            <input type="submit" value="Dalej" name="submit" />
</form>
[HTML] pobierz, plaintext 

Gdy wchodzę na stronkę z tym formularzem, lista jest poprawna, jednak po wyborze opcji i kliknięciu na "Dalej" listy już nie ma. A sprawdzenie

[PHP] pobierz, plaintext 
<?php echo $_POST['opt']; ?>
[PHP] pobierz, plaintext 

nie pokazuje nic.

W czym może być problem?

Ten post edytował koreja 10.01.2009, 14:27:05

[Rafal Filipek]

Kod

print '<option value"' . $value . '" name="'.$value.'">' . $desc . '</option>';

brakuje = po value

[koreja]

Ehh, mały błąd (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Teraz już

[PHP] pobierz, plaintext 
<?php
$_POST['opt']
?>
[PHP] pobierz, plaintext 

wyświetla numerek, jednak po przesłaniu danych nie generuje listy po raz kolejny...

[sowiq]

jednak po przesłaniu danych nie generuje listy po raz kolejny...

A może by tak <option...><option...> objąć tagiem <select...> ?

[Rafal Filipek]

Kod

<select id="opt" name="opt"><?php buildOptions($options); ?></select>

select jest
@koreja - po kodzie ktory pokazales nie jestesmy teraz w stanie stwierdzic dlaczego tak sie dzieje. musisz chyba pokazac cos wiecej.

[koreja]

Aha (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Plik register.php

[HTML] pobierz, plaintext 
<?php include('reg.func.php');
 
if(isset($_POST['submit']) && $_POST['submit'] == "Rejestruj") {
 
?>
       		<fieldset id="register">
        	<legend>Rejestracja nowego użytkownika:</legend>
              <form action="register.php" method="post">
        	<label for="state">Województwo:</label><?php isValidState(); ?>
                    <input type="submit" name="submit" style="margin-left:150px;" value="Rejestruj" />
              </form>
            </fieldset>
<?php } else {
?>
       		<fieldset id="register">
        	<legend>Rejestracja nowego użytkownika:</legend>
              <form action="register.php" method="post">
        		<label for="state">Województwo:</label>
                	<select id="state" name="state"><?php buildOptions($stateOptions); ?></select><br />
                    <input type="submit" name="submit" style="margin-left:150px;" value="Rejestruj" />
              </form>
            </fieldset>
<?php }
?>
[HTML] pobierz, plaintext 

reg.func.php

[PHP] pobierz, plaintext 
<?php
function isValidState() {
            if($_POST['state'] != '0') {
                print '<select id="state" name="state">'; buildOptions($stateOptions, $_POST['state']); print '</select><br />';
                $valid = true;
                }
                else {
                print '<select id="state" name="state">'; buildOptions($stateOptions); print '</select><br />';
                }
            }
?>
[PHP] pobierz, plaintext 

Ten post edytował koreja 10.01.2009, 15:08:50

[sowiq]

select jest

Sorry, nie doczytałem.
@autor,
czy w kodzie HTML jest generowane cokolwiek? Bo jeśli nie, to pewnie przekazujesz w parametrze pustą tablicę, lub coś w tym stylu.

[PHP] pobierz, plaintext 
<?php
function isValidState() {
           if($_POST['state'] != '0') {
               print '<select id="state" name="state">'; buildOptions($stateOptions, $_POST['state']); print '</select><br />';
[...]
?>
[PHP] pobierz, plaintext 

Zapomniałeś o global $stateOptions

Ten post edytował sowiq 10.01.2009, 15:01:56

[koreja]

Wiesz, global nie ma sensu, bo do reg.func.php includuje plik array.php, w którym znajduje się funkcja buildOptions oraz tablice, z których generuję te opcje (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[sowiq]

Wiesz, global nie ma sensu, bo do reg.func.php includuje plik array.php, w którym znajduje się funkcja buildOptions oraz tablice, z których generuję te opcje (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

A jak chcesz wywołać z funkcji isValidState() funkcję
buildOptions($stateOptions, $_POST['state']), skoro w momencie wywołania zmienna $stateOptions nie istnieje (w kontekście funkcji isValidState() )?

Zrób:

[PHP] pobierz, plaintext 
<?php
function isValidState() {
    global $stateOptions;
    // reszta
?>
[PHP] pobierz, plaintext 

Ten post edytował sowiq 10.01.2009, 15:11:39

[koreja]

Dzięki sowiq (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Źle zrozumiałem poprzedni post. Dzięki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)