Skocz do zawartości
Pecetowicz Forum komputerowe

Czy napisany sklep z integracją Pukawka pod MyBB jest dobry?


Rekomendowane odpowiedzi

Witam, a więc na początku przepraszam jeśli zły dział.
Dziś napisałem sobie taką jakby "wtyczkę" do mybb i chciałbym się zapytać czy żadnego błędu nie popełniłem i czy wy byście coś zmienili w tym kodzie.

Tu proszę kod formularza:

<!-- POCZĄTEK -->
    <style>
    .sms {
	display: none
	}
	label,
	input[type="radio"] {
	cursor: pointer
	}
    </style>
<!-- KONIEC -->
<!-- POCZĄTEK 2 -->
	<form action="smsik.php" method="post">
		<b>Wybierz ilość dni:</b><br />
	<input type="radio" id="sms-1" class="choose" name="sms" value="1" data-target="sms-1">
	<label for="sms-1">30</label>
	<br>
	<input type="radio" id="sms-2" class="choose" name="sms" value="2" data-target="sms-2">
	<label for="sms-2">60</label>
	<br>
	<input type="radio" id="sms-3" class="choose" name="sms" value="3" data-target="sms-3">
	<label for="sms-3">90</label>
	<br>
	<input type="radio" id="sms-4" class="choose" name="sms" value="4" data-target="sms-4">
	<label for="sms-4">120</label>
		<div class="sms sms-1">Wyślij sms'a o treści <b>pukawka</b> pod numer <b>76480</b>. Koszt sms'a to <b>7.38 zł</b>.</div>
		<div class="sms sms-2">Wyślij sms'a o treści <b>pukawka</b> pod numer <b>91400</b>. Koszt sms'a to <b>17.22 zł</b>.</div>
		<div class="sms sms-3">Wyślij sms'a o treści <b>pukawka</b> pod numer <b>91900</b>. Koszt sms'a to <b>23.37 zł</b>.</div>
		<div class="sms sms-4">Wyślij sms'a o treści <b>pukawka</b> pod numer <b>92550</b>. Koszt sms'a to <b>30.75 zł</b>.</div><br />
	<label for="kod">KOD:</label><input type="text" name="kod" id="kod" size="15" />
	<center><input type="submit" value="Dodaj" /></center>
	</form>
<script src="//production-assets.codepen.io/assets/common/stopExecutionOnTimeout-58d22c749295bca52f487966e382a94a495ac103faca9206cbd160bdf8aedf2a.js"></script>
<script src='https://code.jquery.com/jquery-2.2.4.min.js'></script>
    <script>
    $(document).ready(function () {
    function sms_key(target) {
        $('.sms:visible').css({ 'display': 'none' });
        $('.' + target).css({ 'display': 'block' });
    }
    $('.choose').first().attr({ 'checked': 'checked' });
    sms_key($('.choose:checked').data('target'));
    $('.choose').change(function () {
        sms_key($(this).data('target'));
    });
	});
  //# sourceURL=pen.js
  </script>
<!-- KONIEC 2 -->

Tutaj kod smsik.php:

 <?php
	$keyapi = "2qTCY1uze2XdLVv"; // klucz API
    $code = $_POST['kod']; // kod SMS
    $get = file_get_contents("https://admin.pukawka.pl/api/?keyapi=$keyapi&type=sms&code=$code");
    if($get)
    {
	    $get = json_decode($get);
	    if(is_object($get))
	    {
		    if($get->error)
		    {
			    echo $get->error;
		    }
		    else
		    {
			    $status = $get->status;
			    if($status=="ok")
			    {
				    $kwota = $get->kwota;
				    echo "Kod jest poprawny. Do portfela wpłynęło: $kwota pkt.<br /><a href='sms.php'>Powrót</a>.";
			    }
		    	else
			    {
				    echo "Kod jest błędny lub został już wykorzysatny.<br /><a href='sms.php'>Powrót</a>.";
			    }
		    }
	    }
	    else
	    {
		    echo "Nieznany błąd API.<br /><a href='sms.php'>Powrót</a>.";
	    }
    }
    else
    {
	    echo "Błąd połączenia z API.<br /><a href='sms.php'>Powrót</a>.";
    }
?>

To jest tylko taki przykładowy sklepik, chodzi o to, czy żadnego błędu nie popełniłem i czy pisać dalej.

@Edit: podgląd - klik.

Odnośnik do odpowiedzi
Udostępnij na innych stronach

Problem wciąż nierozwiązany? Dodaj swoją odpowiedź

Jeśli chcesz dodać odpowiedź, zaloguj się lub zarejestruj nowe konto. Jedynie zarejestrowani użytkownicy mogą komentować zawartość tej strony.

Zarejestruj nowe konto

Załóż nowe konto. To bardzo proste!

Zarejestruj się

Zaloguj się

Posiadasz już konto? Zaloguj się poniżej.

Zaloguj się
×
×
  • Dodaj nową pozycję...