[MySQL][PHP] Problem Skrypt Opcje

[Pilsneer]

Witam!
Jako iż to mój pierwszy post chciałbym się przedstawić, nazywam się Bartosz, mieszkam w Wielkopolsce.
Programowaniem PHP, MySQL, HTML zajmuje się około 3 tygodni, za to bardzo dobrze potrafię C++.

Otóż mam mam skrypt, który podwyższa mi Dochody w klubie, a także pojemność Hali.
Błąd polega na tym gdyż wcisnę "Wykonaj" to wyskakuje mi "Masz za mało kasy!", jak usunę If'a odpowiadającego za Pieniądze, skrypt wykonuje się dobrze wyskakuje "Kupiłeś (100) miejsc na hali swojego klubu! " lecz UPDATE nie wykonuje się, nie żdżera mi kasy, ani nie dodaje pojemności hali i dochodów!

Proszę o pomoc!

Tutaj Kod:

[PHP] pobierz, plaintext 
<?php
include('inc/config.php');
include('inc/top.php');
 
if((!empty($_POST['action']))&&(!empty($_POST['count'])) ){
 
    $user_data = mysql_fetch_row(mysql_query("SELECT * FROM users WHERE user='".$_POST['user']."' "));
 
    if($user_data['kasa'] < 10){
        //Za mało kasy
 
        echo "Masz za mało kasy!" . " <br><a href=\"hall.php\">Powrót</a>";
    } elseif($_POST['count'] > 100) {
        //Więcej niż 100 to echo
 
        echo "Maksymalna ilośc to 100!" . " <br><a href=\"hall.php\">Powrót</a>";
    } else {
 
        switch ($_POST['action']){
            case 'rozbuduj':
                mysql_query("UPDATE users SET poj_hala=poj_hala+".$_POST['count'].", dochody=dochody+".$_POST['count'].", kasa=kasa - (10*".$_POST['count'].") WHERE user='".$_POST['user']."'");
 
                echo "Kupiłeś  (".$_POST['count'].") miejsc na hali swojego klubu! <a href=\"hall.php\">Powrót</a>";
            break;
 
            default:
                echo "Błędna Akcja";
 
            break;
        }
 
    }
 
 
}
 
?>
 
 
 
 
<form method='post' action=''>
    <table cellspacing="10">
        <tr>
            <td align="center">Rozbuduj Miejsca w Hali:</td>
 
        </tr>
        <tr>
            <td align="center">10&euro; za 1 msc.</td>
 
        </tr>
 
        <td align="center" colspan="3">
            <select name="action">
                <option value="rozbuduj">Rozbuduj o:</option>
 
            </select>
            <select name="count">
                <option value="1">1</option>
                <option value="2">2</option>
				<option value="3">3</option>
                <option value="4">4</option>
				<option value="5">5</option>
                <option value="10">10</option>
				<option value="35">35</option>
                <option value="50">50</option>
				<option value="75">75</option>
                <option value="100">100</option>
            </select>
            <input type="submit" value="Wykonaj">
        </td>
        </tr>
    </table>
</form>
 
<?php
include('inc/down.php');
?> 
[PHP] pobierz, plaintext 

[rytek]

Pokaż co wyświetla:
print_r($_POST);
umieść przed if'em i wyślij formularz.

Tam to $_POST['user'] mi nie pasuje, bo w formularzu pola brak.

Ten post edytował rytek 15.02.2010, 16:47:09

[Pilsneer]

Niestety nic to nie dało.

[pawelsz66]

wydaje mi się że zamiast

WHERE user='".$_POST['user'].

powinno być WHERE user=$id";

a gdzieś na początek wrzucić

Kod

if ( (isset($_GET['id'])) && (is_numeric($_GET['id'])) ) {
    $id = $_GET['id'];
} elseif ( (isset($_POST['id'])) && (is_numeric($_POST['id'])) ) { // Wysłanie formularza.
    $id = $_POST['id'];
} else { // Brak poprawnego identyfikatora, zakończ skrypt.
    echo '<h1>Bł±d</h1>
    <p class="error">Bł±d podczas dostępu do strony.</p><p><br /><br /></p>';

    exit();

Oczywiście moge się mylić bo dopiero zaczynam moją "przygode" z programwoaniem

[rytek]

Tej zmiennej $_POST['user'] nie przesyłasz formularzem, dlatego $user_data['kasa'] jest mniejsza od 10 i skrypt kończy pracę na tym ifie.

Proponuję dodać sobie do formularza pole hidden z id użytkownika, jak koniecznie chcesz pobierać je ze zmiennej $_POST.

Gdybyś zrobił to, o co prosiłem (print_r($_POST) to zobaczyłbyś, że ta zmienna nie jest przesyłana i dlatego występują problemy.