[SAMMELTHREAD] Kleine Coding Fragen

Developer · 15. Februar 2014

wenn ich das grade richtig sehe fehlt dir vor dem ?> eine schließende Klammer:

PHP

<?php 
   require_once("libs/pages/Page.class.php");
   require_once("libs/SampQueryAPI.php");

   class KeyPage extends Page
   {
      public $templateName = "Key";


      public function readData() {
          parent::readData();

           if(isset($_POST['submit']))
           {

                   $key = UCP_Framework::getDB()->escapeString(@$_POST['key']);
                   $config = UCP_Framework::getConfig();

                   if( isset ( $key ) )
                   {
                       $query = UCP_Framework::getDB()->sendQuery (" SELECT * 
                                                                     FROM ". $config->usertable ." 
                                                                     WHERE ". $config->userKey ." = '". $key ."' 
                                                                     LIMIT 1");
                       if(UCP_Framework::getDB()->numRows ( $query ) == 1)
                       {
                           echo " $query ";

                       } else UCP_Framework::getTPL()->assign("error", '<div class="alert alert-error">Der eingegebene Key ist fehlerhaft!</div>');            
                   } else UCP_Framework::getTPL()->assign("error", '<div class="alert alert-error">Bitte gebe einen key ein</div>');
           } 
    }   
} 
    ?>

Alles anzeigen

Anker. · 15. Februar 2014

Ich habe noch ein Problem, unzwa soll ja eigentlich wenn das Textfeld leer ist die Meldung kommen das es ausgefüllt werden muss, aber egal was macht macht, ob man nichts, den falschen oder den richtigen Key eingibt, es kommt das der falsche Key eingeben wurde. Kann mir Wer weiter helfen? -> Bin ja noch ein relativer Neuling.

PHP

<?php 
   require_once("libs/pages/Page.class.php");
   require_once("libs/SampQueryAPI.php");

   class KeyPage extends Page
   {
      public $templateName = "Key";


      public function readData() {
          parent::readData();

           if(isset($_POST['submit']))
           {

                   $key = $_POST['key'];
                   $config = UCP_Framework::getConfig();

                   if( isset ( $key ) )
                   {
                       $query = UCP_Framework::getDB()->sendQuery (" SELECT * 
                                                                     FROM ". $config->usertable ." 
                                                                     WHERE ". $config->userKey ." = '". $key ."' 
                                                                     LIMIT 1");
                       if(UCP_Framework::getDB()->numRows ( $query ) == 1)
                       {
                           UCP_Framework::getTPL()->assign("error", '<div class="alert alert-error"> $query </div>');

                       } else UCP_Framework::getTPL()->assign("error", '<div class="alert alert-error">Der eingegebene Key ist fehlerhaft!</div>');            
                   } else UCP_Framework::getTPL()->assign("error", '<div class="alert alert-error">Bitte gebe einen key ein</div>');
           } 
    }   
} 
    ?>

Alles anzeigen

Developer · 15. Februar 2014

mach mal anstatt isset $key empty($key)

also:

PHP

<?php 
   require_once("libs/pages/Page.class.php");
   require_once("libs/SampQueryAPI.php");

   class KeyPage extends Page
   {
      public $templateName = "Key";


      public function readData() {
          parent::readData();

           if(isset($_POST['submit']))
           {

                   $key = $_POST['key'];
                   $config = UCP_Framework::getConfig();

                   if( !empty($key ) )
                   {
                       $query = UCP_Framework::getDB()->sendQuery (" SELECT * 
                                                                     FROM ". $config->usertable ." 
                                                                     WHERE ". $config->userKey ." = '". $key ."' 
                                                                     LIMIT 1");
                       if(UCP_Framework::getDB()->numRows ( $query ) == 1)
                       {
                           UCP_Framework::getTPL()->assign("error", '<div class="alert alert-error"> $query </div>');

                       } else UCP_Framework::getTPL()->assign("error", '<div class="alert alert-error">Der eingegebene Key ist fehlerhaft!</div>');            
                   } else UCP_Framework::getTPL()->assign("error", '<div class="alert alert-error">Bitte gebe einen key ein</div>');
           } 
    }   
} 
    ?>

Alles anzeigen

Beavis · 16. Februar 2014

hab mal ne kleine Frage

Wie kann ich direkt auf den Server connecten?
sprich so ähnlich wie samp://ip:port

möchte nen kleinen launcher basteln und wollte das wenn er auf start klickt direkt mit den benutzernamen auf den Server gelangt ohne das nachgefragt wird ob er den Server zu den favorieten hinzufügen will oder die play message

Ich hoffe, das ihr versteht was ich meine

VitoHusky · 16. Februar 2014

Beavis:
Du meinst also wie bspw. beim GRP Launcher ohen samp zu öffnen direkt zu connecten?

Beavis · 16. Februar 2014

Ich kenn den nicht aber ja ohne Samp zu öffnen soll er sich verbinden

Blu3scReeN · 16. Februar 2014

Das hatte ich schonmal, ich glaube gta_sa.exe ausführen und dann die samp.dll injecten

VitoHusky · 16. Februar 2014

Ahh jap

Das sollte dir helfen

Externer Inhalt www.youtube.com

Inhalte von externen Seiten werden ohne Ihre Zustimmung nicht automatisch geladen und angezeigt.

Durch die Aktivierung der externen Inhalte erklären Sie sich damit einverstanden, dass personenbezogene Daten an Drittplattformen übermittelt werden. Mehr Informationen dazu haben wir in unserer Datenschutzerklärung zur Verfügung gestellt.

serecords · 16. Februar 2014

Zitat von MnX
Ich hab ein problem und zwar will ich wenn man auf den Button klickt das er den spieler +100 punkte gibt ich hab es schon vor gemacht aber hab keine ahnung was falsch ich
PHP
$abfrage = "SELECT id FROM NNGuser_data";
$ergebnis = mysqli_query($db, $id);
$abfrage = "SELECT punkte FROM NNGuser_data";
$ergebnis = mysqli_query($db, $cash);
$missioncash = 100;
if($_POST["quest"])
{
	if($userdata["id"] == $id)
	{
		$gesamtcash = $cash + $missioncash;
		$aendern = "UPDATE NGGuser_data Set punkte = $gesamtcash WHERE id = $id";
		$update = mysqli_query($db, $aendern);
	}
}




echo '<form action="" method="post">
	  <input name="quest" type="submit" value="Test" />';
Alles anzeigen
Wenn es noobig aussieht tut mir leid ich beschäftige mich erst seit heute damit hehe

Du hast $abfrage und $ergebnis doppelt definiert.
In deiner if Abfrage definierst du nur Variablen, anstatt einen mysqli_query auszuführen.

Developer · 16. Februar 2014

Zitat von serecords

Du hast $abfrage und $ergebnis doppelt definiert.
In deiner if Abfrage definierst du nur Variablen, anstatt einen mysqli_query auszuführen.

Na wenn das zur Lösung seines Problems beiträgt, fress ich nen besen.

Spoiler anzeigen

PHP

<?php/*$abfrage = "SELECT id FROM NNGuser_data";$ergebnis = mysqli_query($db, $abrage); // Zweiter Parameter von mysqli_query ist der Query$abfrage = "SELECT punkte FROM NNGuser_data";$ergebnis = mysqli_query($db, $abfrage); // Siehe oben*/// So würde ich das nie machen. Du sendest zwei Query ab obwohl du nur einen braucht -> sinn?$abfrage = "SELECT id as user_id, punkte as user_geld FROM NNGuser_data";$ergebnis = mysqli_query($db, $abfrage);/// jetzt kannst du das so verwenden:$missioncash = 100;if(isset($_POST["quest"])) // $_POST['quest'] nur abzufragen ist nicht grade gut. immer abfragen ob es gesetzt ist{    if($userdata["id"] == $ergebnis['user_id'])    {        $gesamtcash = $ergebnis['user_geld'] + $missioncash;        $aendern = "UPDATE NGGuser_data Set punkte = $gesamtcash WHERE id = $id";        $update = mysqli_query($db, $aendern);    }}echo '<form action="" method="post">      <input name="quest" type="submit" value="Test" />      </form>'; // Hier fehlte übrigens noch das </form>?>

Spoiler anzeigen

//Edit: Doppelzitat entfernt

Slash™ · 17. Februar 2014

Zur Zeit benutze ich für meine Website hier das mit dem Get zeug also

PHP

index.php?page=home

kann ich das auch mehrmals verwenden? also so

PHP

index.php?page=home?id=1

fnL · 17. Februar 2014

Jap.

Code

index.php?name=ABC&doener=true

Slash™ · 17. Februar 2014

Frage ich das dann auch einfach mit $_GET ab?

fnL · 17. Februar 2014

Jop.

PHP

$_GET['NAME']

Beavis · 17. Februar 2014

aber der php code sollte kein & haben
statt & ein & im html/php code

Slash™ · 17. Februar 2014

Und wie sieht das da jetzt aus Beavis: ?
index.php?page=home&test=1 ??

Und warum?

Beavis · 17. Februar 2014

Zu 1 ja so is richtig wenn du es als href machst

Der Grund darin ist einfach das wenn du & einfach schreibst der als Fehler anerkannt wird
machst du aber & zeigt er den nicht als fehler an

Das kannst du ja mal testen auf der seite
http://validator.w3.org/

[BlaZZeR]BirD · 17. Februar 2014

Man sollte $_GET["blabla"]; nicht so verwenden. Es sollte lieber: filter_input verwendet werden.

Slash™ · 17. Februar 2014

Okay danke aber ich mach es dann eher als header oder so.
header('Location: index.php?page=home&id=1');

[BlaZZeR]BirD:
Warum genau?

[BlaZZeR]BirD · 18. Februar 2014

Zitat von Slash™

Okay danke aber ich mach es dann eher als header oder so.
header('Location: index.php?page=home&id=1');

[BlaZZeR]BirD:
Warum genau?

Weil $_GET eine superglobale ist.