PHP - Spiel [Anfänger]

Suru

[COLOR="red"]

Hallo ,

Ich habe aus langerweile heute mal ein kleines Spiel besser gesagt er sollte ein Spiel

werden ich arbeite noch aktiv weiter dran und vlt wollt ihr es auch weiter bearbeiten

oder vlt lernt einer von euch auch gerade PHP in zusamm arbeit mit Msql

Ich habe jetzt beim coden nicht auf die sicherheit geachtet aber man kein dies simpel

hinzufügen welche Funktion diese Script´s (Spiel)(Test xD) bis jetzt hat

Ein Login mit daten aus einer datenbank + Session´s

Eine Gold Funktion + Viewer wieviel gold man gerade hat

Ein Invi Viewer mit information über euren Account wie

Benutzername:
Passwort :
ID:
Cash:-> GOLD :D
Wieviel Items man hat

Man kann bis jetzt 4 Items haben also hat man 4 Plätze

die ich mit Slots benannt habe

Eine Passwort Changer mit dem sie ihr passwort changen können

und schon ein kleiner script für den shop

Warum stelle ich es als download bereit ?

Weil ich gerne sehen wurd was ihr darmit macht ihr hönnt es ja wieder re-Uploaden

vlt kann ich mich den daran weiter entwickeln

Download-LINK

Und die DATENBANK ->

PHP

CREATE TABLE IF NOT EXISTS `reg` (
  `id` int(34) NOT NULL AUTO_INCREMENT,
  `user` varchar(56) NOT NULL,
  `pass` varchar(34) NOT NULL,
  `cash` int(40) NOT NULL,
  `items` int(50) NOT NULL,
  `slot1` varchar(50) NOT NULL,
  `slot2` varchar(50) NOT NULL,
  `slot3` varchar(50) NOT NULL,
  `slot4` varchar(50) NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin1 AUTO_INCREMENT=4 ;

Alles anzeigen

Es sind nur PHP datein deswegen kann ich den VT weglassen

wurde mich auf neue Re-Uploads freuen

Lg.Suru[Anfänger];D

[/COLOR]

Scharock

Und evt. die DB?

Suru

Ups !

Sorry habe ich vergessen

Die Datenbank :)

PHP

CREATE TABLE IF NOT EXISTS `reg` (
  `id` int(34) NOT NULL AUTO_INCREMENT,
  `user` varchar(56) NOT NULL,
  `pass` varchar(34) NOT NULL,
  `cash` int(40) NOT NULL,
  `items` int(50) NOT NULL,
  `slot1` varchar(50) NOT NULL,
  `slot2` varchar(50) NOT NULL,
  `slot3` varchar(50) NOT NULL,
  `slot4` varchar(50) NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin1 AUTO_INCREMENT=4 ;

Alles anzeigen

Scharock

so habs mal getestet.Abgesehn vom Shop funzt alles so weit, jedoch kann man nicht wirklich etwas damit anfangen.
viewer.php würd ich zu index.php ändern.
Und diverse Falschnachrichten schleichen sich ein.
Ich bin eingeloggt, geh auf PW ändern steht unten: Anwendung war nicht erfolgreich !

Suru

Zitat

so habs mal getestet.Abgesehn vom Shop funzt alles so weit, jedoch kann man nicht wirklich etwas damit anfangen.
viewer.php würd ich zu index.php ändern.
Und diverse Falschnachrichten schleichen sich ein.
Ich bin eingeloggt, geh auf PW ändern steht unten: Anwendung war nicht erfolgreich !

Ich entwickel ja weiter ich habe es ja nur zum Download bereit gestellt das vlt jemand der mehr erfahrung hat als ich was ändern kann und es re-Upen kann

ulle

Also dafür das du noch in der Lernphase bist, sieht es doch schon gut aus. Allerdings rate ich dir, dich mit OOP auseinander zusetzten. Das was ich sehe spricht ja für ein Browsergame. Und wenn du schon hier mit Objekten Arbeitest, erleichterst du dir die Schreib Arbeit enorm. Desweiteren würde ich eine Template Engine einsetzten. Entweder du nimmts dafür direkt PHP oder eine von den freien wie Smarty.

Ich wünsch dir aufjedenfall viel glück damit. Vielleicht bau ich es ja mal auf OOP um und poste es dann hier wieder. Viel ist es ja net :D

Lieben Gruß
Ulle

Scharock

wenn du fleißig übst, könnteste mir ja evt. beim flyff bg unter die Arme greifen^^

Shizuka

ich lerne auch gerade php und das ist ne gute möglichkeit zu üben xD
ich versuche gerade nen register rein zu machen ^^

ulle

Zitat von Scharock;229980

wenn du fleißig übst, könnteste mir ja evt. beim flyff bg unter die Arme greifen^^

Was meinst du den mit Flyff BG ? Kannste da mal genauere Angaben machen? Würde mich mal interessieren was genau du da planst :D

Suru

Antwort

Zitat

Also dafür das du noch in der Lernphase bist, sieht es doch schon gut aus. Allerdings rate ich dir, dich mit OOP auseinander zusetzten. Das was ich sehe spricht ja für ein Browsergame. Und wenn du schon hier mit Objekten Arbeitest, erleichterst du dir die Schreib Arbeit enorm. Desweiteren würde ich eine Template Engine einsetzten. Entweder du nimmts dafür direkt PHP oder eine von den freien wie Smarty.
Ich wünsch dir aufjedenfall viel glück damit. Vielleicht bau ich es ja mal auf OOP um und poste es dann hier wieder. Viel ist es ja net :D
Lieben Gruß
Ulle

Würde mich freuen :)

Zitat

wenn du fleißig übst, könnteste mir ja evt. beim flyff bg unter die Arme greifen^^

Ja gerne aber ich glaube ich muss noch ein bischen erfahrung sammeln

Zitat

ich lerne auch gerade php und das ist ne gute möglichkeit zu üben xD
ich versuche gerade nen register rein zu machen ^^

Cool ,

Ein Register script zu machen ist eig ganz simpel :)

EDIT :

Weil die frage kam wie erstelle ich jetzt einen Account ?

habe ich mal schnell ein Script gemacht wie immer nicht auf sicherheit geachtet :) gerade am Crysis 2 Suchteln ^^

Login.php Update

PHP

<?php 
session_start();
include ('./inc/db.php');
?>
<form action="#" method="post">
Benutzername:<br />
<input type="text" name="username" /><br />
Passwort: <br />
<input type="password" name="password" /><br /> 
<input type="submit" value="login" />
</form>
<br /> 
<a href="register.php"><small>Noch kein Account ? </small></a>
<?php 
$benutzer  = $_POST['username'];
$password  = $_POST['password'];


$abfrage = "SELECT user, pass FROM reg WHERE user LIKE  '$benutzer'  LIMIT 1"; 
$ergebnis = mysql_query($abfrage); 
$row = mysql_fetch_array($ergebnis);
if(isset($benutzer)){
	if($benutzer == $row['user'] And $password == $row['pass']){
		$_SESSION['user'] = $benutzer;
		$_SESSION['pass'] = $password;
		
		echo 'Willkommen: '.'&nbsp;'.$benutzer.'<br /> ';
		echo '<a href="viewer.php">Zum Login Bereich !</a>';
	}else {
		
		echo 'Passwort oder Benutzername stimmen nicht !';
	}
}
?>

Alles anzeigen

Register.php

PHP

<?php
include ('./inc/db.php');
?>
<form action="#" method="post">
Benutzername:<br /> 
<input type="text" name="user" /><br >
Passwort: <br >
<input type="password" name="pass" /><br />
<input type="submit" value="Anmelden" />
</form>
<?php 
$username = $_POST['user'];
$passwort = $_POST['pass'];
if(isset($username)){
	$insert = "INSERT INTO reg (user, pass)
	VALUES ('{$username}', '{$passwort}')";
	mysql_query($insert);
	echo 'Willkommen:'.$username.'!'.'<br /> ';
	echo '<a href="login.php">Zurück zum Login !</a>';
}else{
	echo 'Fehler';
}


?>

Alles anzeigen

Shizuka

Danke ich werde mir das mal genauer ansehen :)

ulle

Ich werde wenn ich dazu komme das ding auch mal morgen umbauen. Bissle gegen SQL Injections sichern und eine Template Engine einbauen.
Was ist euch den Lieber. PHP als Template Engine oder Smarty. Wobei mein Favorite auf Smarty liegt.

Würde mich freuen wenn ihr ein wenig darüber abstimmt.
Gruß
Ulle

Suru

Shizuka

Bitte beachte das dies nicht Scriptig richtig ist also er funkt aber Cross Scripting und so weiter kann man durch aus machen xD

@ALL

Würde mich auf Re-Uploads mit Updates freuen :D

Suru

@Push

Würde mich über neue Updates freuen

Lg.Suru ^1

yannickmama

Vorweg ich will nie wieder von dir hören "hab mal auf sicherheit nicht geachtet". Wäre für mich wie "hab mal bei den variablien kein $ gesetzt.
Querys prüfen ist das A und O wenn man mit datenbanken arbeitet!

zum code:

Code

$int = 1;


if($int == 1){
...
} else { echo 'iein fehler'; }

Wo ist da der Sinn? ^^

Code

$logout ='<form action="#" method="get">
<input type="submit" name="logut" value="Ausloggen" />
</form>';
echo '<br /> <br /> ';
echo $logout;

Benutzt da doch einfach den <a> Tag

Code

$abfrage = "SELECT user, pass FROM reg WHERE user LIKE  '$benutzer'  LIMIT 1"; 
$ergebnis = mysql_query($abfrage); 
$row = mysql_fetch_array($ergebnis);
if(isset($benutzer)){
if($benutzer == $row['user'] And $password == $row['pass']){
...
...
}
}

1. Wieso führst du erst den Query aus und überprüfst dann ob $_POST['benutzer'] übergeben wurde? o.O
2. Der Query ist total unpassend. Mach doch einfach "SELECT user, pass FROM reg WHERE user = $_POST['user'] AND pass = $_POST['pass']"

Aufjedenfall mysql_real_escaped_string() benutzen um den query sicher auszuführen.

Bei den Items arbeite doch mit einem 2dimensionalen Array also anstatt:

Code

if($_SESSION['slot4'] == ""){
    echo 'Item-Slot4:&nbsp; LEER'.'<br />';
}else{
    echo 'Item-Slot4: '.$_SESSION['slot4'].'<br />';
}

Code

for($i=0;$i>COUNT($_SESSION['item']);$i++)
{
if($_SESSION['item'][$i] == "") {
  echo 'Item-Slot'.$i.' LEER<br />';
} else {
  echo 'Item-Slot'.$i.': '.$SESSION["item"][$i].'<br />';
}
}

Dann wieso machst du öfters echo ' texttext '.' weiterer text ';
Das '.' brauchst du bei reinem Text (also keine Variablen, konstanten, funkction usw).

Beim Passwort Change Script würde ich noch nie Session killen und auf die login seite redirecten.

Zu guterletzt bei der db.php würde ich anstatt echo 'Fehler'; lieber die('Fehler'); benutzen. Die beendet das Script an dieser Stelle komplett.
Wenn deine Datenbank nicht zu erreichen ist würde sonst das Script weiterlaufen und bei einigen Selects haufenweise Error's ausspucken.

Suru

Okay Yannick !

Sicherheit ist das A & O ^^

Ich versuche es besser zu machen

@All

Würde mich auf Re-Ups freuen ^^

Suru

Push gogo re-ups ich warte xD

ulle

Ganz im ernst mach du doch auch weiter und Poste deine Erfolge. Verlange doch nicht von der Community das diese dir deine Updates bringen ;)

Learning by Doing sag ich nur

PHP - Spiel [Anfänger]

Jetzt mitmachen!

SCHNELLZUGRIFF

HILFE & COMMUNITY

RECHTLICHES