[MySQL][PHP]Ujmowanie i Dodawanie 2x Większej wartości niż powinno być. Opcje

[Giluś]

Cześć, sam już nie wiem dlaczego tak mi się dzieje mam taki kod:

[PHP] pobierz, plaintext 
 
<?php
if(isset($_POST['ok'])) { 
	if ($userdata['portfel'] >= $_POST['option']) // Sprawdz czy są pkt na koncie 
{
$czas = $_POST['option'] * 86400;
$opcja = $_POST['option'];
		if ($userdata['vip'] > time()) {
		// Istnieje już VIP i przedłużenie jest
				mysql_query("update fusiond5dvP_users set portfel = portfel - ".$opcja.", vip = vip + ".$czas."  where user_id = ".$userdata['user_id']);
						$txt = "Zakupiono przedłużenie konta VIP na czas: ".$opcja." dni w cenie: ".$opcja." pkt.";
										}
		else {
		// Pierwszy raz VIP
				mysql_query("update fusiond5dvP_users set portfel = portfel - ".$opcja.", vip = ".(time() + $czas)." where user_id = ".$userdata['user_id']);
						$txt = "Zakupione konto VIP na czas: ".$opcja." dni w cenie: ".$opcja." pkt.";
			}
}
else {$txt = "Nie posiadasz wystarczającej ilości pkt.";}
}
echo "<h1>".$txt."</h1>";
?>
<p>Wybierz odpowiednią ofertę dla siebie: <br /><b>Posiadasz: 
<?php 
echo "".$userdata['portfel'].""; 
?> 
punktów</b></p>
 
<form method="post">
		<input type="radio" name="option" value="1"> 24h - 1pkt<br>
		<input type="radio" name="option" checked="" value="7"> 1 tydzień - 7 pkt<br>
		<input type="radio" name="option" value="14"> 2 tygodnie - 14 pkt<br>
        <input type="radio" name="option" value="30"> 1 Miesiąc - 30 pkt<br>
		<br>
<input type="submit" name="ok" value="Gotowe!">
</form>
 
 
<?php
if($userdata['vip'] > time()){
	$pozostalo = $userdata['vip'] - time();
	echo "<script type='text/javascript'>        
        function liczCzas(ile) {
			dni = Math.floor(ile / 86400);
            godzin = Math.floor((ile - dni * 86400)/ 3600);
            minut = Math.floor((ile - dni * 86400 - godzin * 3600) / 60);
            sekund = ile - dni * 86400 - minut * 60 - godzin * 3600;
            if (godzin < 10){ godzin = '0'+ godzin; }
            if (minut < 10){ minut = '0' + minut; }
            if (sekund < 10){ sekund = '0' + sekund; }
            if (ile > 0) {
                ile--;
                document.getElementById('zegar').innerHTML = dni + ' dni ' +godzin + ':' + minut + ':' + sekund;
                setTimeout('liczCzas('+ile+')', 1000);
            } else {document.getElementById('zegar').innerHTML = '[koniec]';}}</script>
	Do końca okresu VIP pozostało: <b><span id='zegar'></span></b><script type='text/javascript'>liczCzas(".$pozostalo.")</script>  ";}
?>
[PHP] pobierz, plaintext 

Powinno odejmować tyle pkt co jest input value oraz dodawać czas również tyle co jest input value, no ale problem jest taki wyświetla się że pobrało niby np.: 1 pkt i dodało 1 dzień, ale w MySQL odejmuję 2x więcej pkt oraz jeśli to 1x VIP czyli else warunku: if ($userdata['vip'] > time()) {} to normalnie wyświetla się w skrypcie do liczenia, że to jest 24h, ale gdy już przedłużam VIP czyli ten warunek if ($userdata['vip'] > time()) {} to robi 2x więcej czasu oraz tak jak wcześniej wspomniałem 2x więcej pkt pobiera..

Gdzie jest błąd bo już sam nie mam pojęcia...

Pozdrawiam i z góry bardzo dziękuje



Edit.:
Gdy daję aby zwróciło tablicę $_POST to wyświetla tak jak powinno option, wiec coś to musi być w php.., ale ja tam nie widzę nic dziwnego aby ujmowało 2x więcej pkt.. oraz 2x więcej dodawało czasu w przedłużeniu vipa..

Ten post edytował Giluś 27.06.2012, 13:06:58

[Sephirus]

Linijka #13 - masz złą konstrukcję klamr {} - przyjrzyj się temu...

[Giluś]

Nie wiem co masz dokładnie na myśli, ale spróbowałem wyrzucić całkowicie klamry z else - nic nie pomogło, dodać w mysql przed końcowym nawiasem ."" - też nie pomogło..

Dzięki za odpowiedz, może o czymś innym mówiłeś ?.


Edit:
Zrobiłem test na takim kodzie:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['ok'])) { 
	if ($userdata['portfel'] >= $_POST['option']) // Sprawdz czy są pkt na koncie 
{
$czas = $_POST['option'] * 86400;
$opcja = $_POST['option'];
		if ($userdata['vip'] > time()) {
		mysql_query("update fusiond5dvP_users set portfel = portfel - ".$opcja.", vip = vip + ".$czas."  where user_id = ".$userdata['user_id']."");
		$txt = "Zakupiono przedłużenie konta VIP na czas: ".$opcja." dni w cenie: ".$opcja." pkt.";
										}
}
else {$txt = "Nie posiadasz wystarczającej ilości pkt.";}
}
echo "<h1>".$txt."</h1>";
?>
<p>Wybierz odpowiednią ofertę dla siebie: <br /><b>Posiadasz: 
<?php 
echo "".$userdata['portfel'].""; 
?> 
punktów</b></p>
 
<form method="post">
		<input type="radio" name="option" value="1"> 24h - 1pkt<br>
		<input type="radio" name="option" checked="" value="7"> 1 tydzień - 7 pkt<br>
		<input type="radio" name="option" value="14"> 2 tygodnie - 14 pkt<br>
        <input type="radio" name="option" value="30"> 1 Miesiąc - 30 pkt<br>
		<br>
<input type="submit" name="ok" value="Gotowe!">
</form>
 
 
<?php
if($userdata['vip'] > time()){
	$pozostalo = $userdata['vip'] - time();
	echo "<script type='text/javascript'>        
        function liczCzas(ile) {
			dni = Math.floor(ile / 86400);
            godzin = Math.floor((ile - dni * 86400)/ 3600);
            minut = Math.floor((ile - dni * 86400 - godzin * 3600) / 60);
            sekund = ile - dni * 86400 - minut * 60 - godzin * 3600;
            if (godzin < 10){ godzin = '0'+ godzin; }
            if (minut < 10){ minut = '0' + minut; }
            if (sekund < 10){ sekund = '0' + sekund; }
            if (ile > 0) {
                ile--;
                document.getElementById('zegar').innerHTML = dni + ' dni ' +godzin + ':' + minut + ':' + sekund;
                setTimeout('liczCzas('+ile+')', 1000);
            } else {document.getElementById('zegar').innerHTML = '[koniec]';}}</script>
	Do końca okresu VIP pozostało: <b><span id='zegar'></span></b><script type='text/javascript'>liczCzas(".$pozostalo.")</script>  ";}
?>
[PHP] pobierz, plaintext 

Dalej to samo: 2x Więcej PKT pobiera oraz 2x Więcej czasu dodaje.

Ten post edytował Giluś 27.06.2012, 13:41:36

[jaslanin]

sprawdź w logach serwera czy Twoja przeglądarka nie przesyła dwa razy formularza albo Ty nie klikasz w formularz dwa razy

np. w access logach serwera, albo zapisuj wywołania do jakiegoś pliku z logami

dodatkowo ten kod nie jest zabezpieczony przed SQL INJECTION

Ten post edytował jaslanin 27.06.2012, 17:13:50

[Giluś]

Gdybym 2x Naciskał lub moja przeglądarka 2x przesyła formularz to mając 1 pkt nie kupił bym pakietu za 1pkt bo przy 1 odświeżaniu bo kupił 24h a przy drugim już by warunek zadziałał...

Wiem, że nie jest bezpieczny bo jeszcze nie dodawałem żadnych zabezpieczeń.

[peter13135]

1. Napisz swoje wypowiedzi stosjując jakieś kropki i przecinki, to może lepiej Cię zrozumiemy.
2. Zrób wcięcia w swoim kodzie, to może lepiej zrozumiemy Twój kod

[Giluś]

Poradziłem sobie sam, nie wiem czemu tak się działo zrobiłem taki kod:

[PHP] pobierz, plaintext 
if ($userdata['vip'] > time()) {
				$kasvip = $userdata['portfel'] - $opcja;
				$czasvip = $userdata['vip'] + $czas;
					mysql_query("update fusiond5dvP_users set portfel = ".$kasvip.", vip = ".$czasvip."  where user_id = ".$userdata['user_id']."");
						$txt = "Zakupiono przedłużenie konta VIP na czas: ".$opcja." dni w cenie: ".$opcja." pkt.";
header("location: #");
										}
		else {
		// Pierwszy raz VIP
				$kasavip = $userdata['portfel'] - $opcja;
				$czasvipp = time() + $czas;
					mysql_query("update fusiond5dvP_users set portfel = ".$kasavip.", vip = ".$czasvipp." where user_id = ".$userdata['user_id']."");
						$txt = "Zakupione konto VIP na czas: ".$opcja." dni w cenie: ".$opcja." pkt.";
header("location: #");
			}
[PHP] pobierz, plaintext 

Zamiast:

[PHP] pobierz, plaintext 
if ($userdata['vip'] > time()) {
		// Istnieje już VIP i przedłużenie jest
				mysql_query("update fusiond5dvP_users set portfel = portfel - ".$opcja.", vip = vip + ".$czas."  where user_id = ".$userdata['user_id']);
						$txt = "Zakupiono przedłużenie konta VIP na czas: ".$opcja." dni w cenie: ".$opcja." pkt.";
										}
		else {
		// Pierwszy raz VIP
				mysql_query("update fusiond5dvP_users set portfel = portfel - ".$opcja.", vip = ".(time() + $czas)." where user_id = ".$userdata['user_id']);
						$txt = "Zakupione konto VIP na czas: ".$opcja." dni w cenie: ".$opcja." pkt.";
			}
[PHP] pobierz, plaintext 

I Niby działa ..