PHP - Spiel [Anfänger]

Suru · 8. April 2011, 03:47

[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-Quellcode

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 · 8. April 2011, 07:33

Und evt. die DB?

Suru · 8. April 2011, 08:27

Ups !

Sorry habe ich vergessen

Die Datenbank :)

PHP-Quellcode

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 · 8. April 2011, 08:59

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 · 8. April 2011, 09:10

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 · 8. April 2011, 10:35

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 · 8. April 2011, 10:40

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

Shizuka · 8. April 2011, 11:01

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

ulle · 8. April 2011, 15:42

Scharock;229980 schrieb:

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 · 8. April 2011, 19:25

Antwort

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 :)

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

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-Quellcode

<?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: '.' '.$benutzer.'<br /> ';
echo '<a href="viewer.php">Zum Login Bereich !</a>';
}else {
echo 'Passwort oder Benutzername stimmen nicht !';
}
}
?>

Alles anzeigen

Register.php

PHP-Quellcode

<?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 · 8. April 2011, 22:01

Danke ich werde mir das mal genauer ansehen :)

ulle · 8. April 2011, 22:32

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 · 8. April 2011, 22:33

@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 · 10. April 2011, 07:57

@Push

Würde mich über neue Updates freuen

Lg.Suru ^1

yannickmama · 10. April 2011, 10:27

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:

Quellcode

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

Wo ist da der Sinn? ^^

Quellcode

$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

Quellcode

$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:

Quellcode

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

Quellcode

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 · 10. April 2011, 10:52

Okay Yannick !

Sicherheit ist das A & O ^^

Ich versuche es besser zu machen

@All

Würde mich auf Re-Ups freuen ^^

Suru · 11. April 2011, 12:16

Push gogo re-ups ich warte xD

ulle · 12. April 2011, 13:19

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]

PHP - Spiel [Anfänger]

PHP-Quellcode

PHP-Quellcode

Scharock;229980 schrieb:

Antwort

PHP-Quellcode

PHP-Quellcode

Quellcode

Quellcode

Quellcode

Quellcode

Quellcode

Teilen

Benutzer online 1