Shakes & Fidget Server - Code & Anleitung

YumYum! · 07/17/2014, 17:07

Nett! Unsauber programmiert aber egal.
Jedenfalls solltest du die mysql_* durch mysqli_ erstetzen oder direkt zu PDO umsteigen.
Mysql_* ist veraltet und sollte laut der offiziellen PHP Website nicht mehr verwendet werden und wird in Zukunft auch entfernt.

UND3RW0RLD · 07/17/2014, 23:16

Hast du dich extra registriert, um mir das mitzuteilen?

Nett von dir..

Zum Thema: Mir doch egal.. wenn sie den Support dafür streichen update ich PHP einfach nicht.. Problem gelöst.. Zur Not kann man auch ne ältere Version kompilieren.

Es macht was es soll.. darauf kommts an. Das ACP wird vllt 3x am Tag benutzt, wenn der Server gut besucht ist.. Da brauch ich keine prepared statements und so einen schicki micki.. quick and dirty und vor allem.. easy erweiterbar.. selbst von anfängern.. ^^

Was du mit unsauber meinst, weiß ich nich.. ich code welten sauberer als die meisten hier, aber wenn du ein konkretes beispiel hast, guck ich's mir nochma an und arbeite wenn nötig nach..

YumYum! · 07/18/2014, 01:13

Quote:

Originally Posted by ǝnd1ǝss-ɯonǝʎ

Hast du dich extra registriert, um mir das mitzuteilen?

Nett von dir..

Zum Thema: Mir doch egal.. wenn sie den Support dafür streichen update ich PHP einfach nicht.. Problem gelöst.. Zur Not kann man auch ne ältere Version kompilieren.

Es macht was es soll.. darauf kommts an. Das ACP wird vllt 3x am Tag benutzt, wenn der Server gut besucht ist.. Da brauch ich keine prepared statements und so einen schicki micki.. quick and dirty und vor allem.. easy erweiterbar.. selbst von anfängern.. ^^

Was du mit unsauber meinst, weiß ich nich.. ich code welten sauberer als die meisten hier, aber wenn du ein konkretes beispiel hast, guck ich's mir nochma an und arbeite wenn nötig nach..

Nein, darum hab ich mich nicht registriert, aber ist doch ein schöner, erster Post.

Nun, was ich mit unsauber meine:
Stichwort: templates
HTML und PHP sollte man immer trennen. Macht den Code 100x übersichtlicher und einfacher zu lesen.

Und dann noch kleinigkeiten, welche in PHP eigentlich Standard sind/sein sollten. (findest du bei fast jeden professionellen PHP Projekt)
* 4 Tabs statt Tabs verwenden
* Klammern falsch gesetzt
* Die Klammern bei Methoden und Klassen sind in der nächsten Zeile.
* Bei If-Abfragen, jedoch in der gleichen.

usw.

Hier ein Beispielcode

MyBB hat das ganz schön und übersichtlich hinbekommen. Schau dir mal den Source Code durch.

Es ist außerdem sinnvoll, jede Klasse in eine extra Datei zu schreiben, bei so einen großen Projekt.

Ist zwar alles kein Muss, macht jedoch den Quellcode um einiges übersichtlicher.
mfg

UND3RW0RLD · 07/18/2014, 01:36

Quote:

Originally Posted by YumYum!

Nein, darum hab ich mich nicht registriert, aber ist doch ein schöner, erster Post.

Nun, was ich mit unsauber meine:
Stichwort: templates
HTML und PHP sollte man immer trennen. Macht den Code 100x übersichtlicher und einfacher zu lesen.

Und dann noch kleinigkeiten, welche in PHP eigentlich Standard sind/sein sollten. (findest du bei fast jeden professionellen PHP Projekt)
* 4 Tabs statt Tabs verwenden
* Klammern falsch gesetzt
* Die Klammern bei Methoden und Klassen sind in der nächsten Zeile.
* Bei If-Abfragen, jedoch in der gleichen.

usw.

Hier ein Beispielcode

MyBB hat das ganz schön und übersichtlich hinbekommen. Schau dir mal den Source Code durch.

Es ist außerdem sinnvoll, jede Klasse in eine extra Datei zu schreiben, bei so einen großen Projekt.

Ist zwar alles kein Muss, macht jedoch den Quellcode um einiges übersichtlicher.
mfg

Template ist overkill für sowas.. ^^
Es gibt für 99,9% der Spieler nur den Client, den Rest sieht keiner und da muss man echt nicht mehr aufwand betreiben als ein billigstes table/css-konstrukt. WBB2 css for the win (man kann btw jedes WBB2-Css nehmen)

Was die Klammern betrifft ist es meiner Meinung nach (und weil ich es so gelernt habe und damit noch nie Probleme hatte) wie folgt besser zu lesen. Das betrifft auch die Tabs. Mit den tabs statt leerzeichen kann man $vars schneller durch die Ebenen bewegen.

Falls es dich zu sehr stört kannst du ja (vermutlich) 100.000x backspace drücken.

G.Poison · 07/18/2014, 16:21

Quote:

Originally Posted by ǝnd1ǝss-ɯonǝʎ

Template ist overkill für sowas.. ^^
Es gibt für 99,9% der Spieler nur den Client, den Rest sieht keiner und da muss man echt nicht mehr aufwand betreiben als ein billigstes table/css-konstrukt. WBB2 css for the win (man kann btw jedes WBB2-Css nehmen)

Was die Klammern betrifft ist es meiner Meinung nach (und weil ich es so gelernt habe und damit noch nie Probleme hatte) wie folgt besser zu lesen. Das betrifft auch die Tabs. Mit den tabs statt leerzeichen kann man $vars schneller durch die Ebenen bewegen.

Falls es dich zu sehr stört kannst du ja (vermutlich) 100.000x backspace drücken.

word

//Shop Update 100% Working
//Pay with honor of the Hall of Fame!
//Change Lvl from the Hall of Fame to honor

UND3RW0RLD · 07/20/2014, 14:37

I am sorry guys, but I didn't get it done.

I had to work alot this week and didn't have much time for coding..

-> all db-stuff works over secrets.php
-> domain via db, so client_cfg.php is dynamic now!!
-> tidied up 80-90% of the basic-code
-> broke down the req.php to 4 files
-> basic new requesthandler to avoid to let apache parse over 9000 LOC on every request!!
-> faster db-design user_data to 4 tables
-> system for autocleanup!!
-> some Acp-Stuff

But I give you some codesamples to see what I am working on and why it takes more time.

PHP Code:


			
<?php

ini_set('display_errors', '0');     # don't show any errors...
error_reporting(E_ALL | E_STRICT);  # ...but do log them

require_once "adminui/backend/secrets.php";
require_once "gamefuncs.php";
require_once "gameconts.php";
require_once "gameclass.php";

// TODO: GET RID OF THIS
include "dtoverride.php";
include "settings.php";
/////////////////////////

try
{
    $db = new PDO ( 'mysql:host=' . $mysql_host . ';dbname=' . $mysql_db . ';charset=' . $mysql_char . '', '' . $mysql_user . '', '' . $mysql_pass . '');
}
catch(Exception $e)
{
    exit();
}

PHP Code:


			
switch ($action)
{
    case $ACT_REGISTER :
        require_once "acts/ACT_REGISTER.php";
        break;

    case $ACT_LOGIN_FOLLOW_UP :
        require_once "acts/ACT_LOGIN_FOLLOW_UP.php";
        break;

    case $ACT_LOGIN :
        require_once "acts/ACT_LOGIN.php";
        break;

    case $ACT_LOGOUT :
        require_once "acts/ACT_LOGOUT.php";
        break;

    case $ACT_HERO :
        require_once "acts/ACT_HERO.php";
        break;

    case $ACT_ARENA_ENTER :
        require_once "acts/ACT_ARENA_ENTER.php";
        break;

    case $ACT_ARENA :
        require_once "acts/ACT_ARENA.php";
        break;

    case $ACT_BUY_STAT :
        require_once "acts/ACT_BUY_STAT.php";
        break;

    case $ACT_RANKING :
        require_once "acts/ACT_RANKING.php";
        break;

    case $ACT_SCREEN_PILZDEALER :
        require_once "acts/ACT_SCREEN_PILZDEALER.php";
        break;

PHP Code:


			
<?php
include 'dtoverride.php';
dbc();

// Base-URL -> domain
$sql = "SELECT value FROM game_settings WHERE setting = 'SYS_BASEURL' LIMIT 1";
$qry = mysql_query($sql);
$arr = mysql_fetch_array($qry);
$url = $arr["value"];
$domain = explode("//", $url);
$dmn = $domain[1];

print "1    de\n";
print "2    http://img.playa-games.com/res/sfgame/\n";
print "3    http://img.playa-games.com/res/sfgame/\n";
print "7    " . $dmn . "\n";
print "8    http://" . $dmn . "/\n";
print "9    0\n";
print "10    http://www.elitepvpers.com\n";
print "11    http://" . $dmn . "/support.php\n";
print "12    http://www.gamepsf.de/\n";
print "13    http://" . $dmn . "/rules.html\n";
print "14    http://" . $dmn . "/shop.php\n";
print "17    3\n";
print "18    " . $dmn . "\n";
print "21    3\n";
print "23    1\n";
print "25    http://" . $dmn . "/req.php?req=%1&random=%2\n";
print "29    support@" . $dmn . "\n";
print "30    http://" . $dmn . "/papaya44.swf\n";
print "31    http://" . $dmn . "/papaya_cfg.php\n";
print "32    1\n";
print "34    2\n";
print "35    http://" . $dmn . "/spenden.php\n";
print "36    gamestaff@" . $dmn . "\n";
print "42    1\n";
print "43    1337\n";
print "48    http://" . $dmn . "/res/sfgame_low/\n";
print "46    de/ar/cs/da/en/es/fr/el/hu/it/ja/nl/pl/pt/pt-br/ru/sv/tr\n";
print "47    german/arabic/czech/danish/english/spanish/french/greek/hungarian/italian/japanese/dutch/polish/portuguese/brazilian portuguese/russian/swedish/turkish\n";

print "26    166;;http://" . $dmn . "/cid/cid.php?cid=%cid%&action=166&playerid=%playerid%&mushbought=%mushbought%&bid=<bid>&token=<token>&serverid=3&gameid=1\n";
print "26    1;;http://" . $dmn . "/cid/cid.php?cid=%cid%&action=1&playerid=%playerid%&mushbought=%mushbought%&bid=<bid>&token=<token>&serverid=3&gameid=1\n";
print "//26    188;;http://" . $dmn . "/cid/cid.php?cid=%cid%&action=188&playerid=%playerid%&mushbought=%mushbought%&bid=<bid>&token=<token>&serverid=3&gameid=1\n";
?>

I will release it, when it's done.. Today I have some hours to work on it.

~~MaYeX159~~ · 07/20/2014, 20:25

***

G.Poison · 07/20/2014, 21:11

So Update von heute:
Serverauswahl funktioniert nun richtig und arbeitet mit dem Chat zusammen:

1Hit2Kill · 07/20/2014, 22:53

Kann mir jemand sagen wie ich auf das Admin Panel zugreifen kann ?

G.Poison · 07/20/2014, 23:02

localhost/adminui

~~MaYeX159~~ · 07/21/2014, 07:41

Ich habe eine Frage, was der Grund ist, dass es keine Registrierung

UND3RW0RLD · 07/21/2014, 10:10

??

Dmixior · 07/21/2014, 10:34

I can't open host.
Warning: mysql_connect() [function.mysql-connect]: Access denied for user 'USER'@'localhost' (using password: YES) in C:\xampp\htdocs\dtoverride.php on line 7
Access denied for user 'USER'@'localhost' (using password: YES)

What i should do? I don't know://

UND3RW0RLD · 07/21/2014, 11:19

Quote:

dtoverride.php on line 7

EptunLP · 07/21/2014, 11:31

Quote:

Originally Posted by ǝnd1ǝss-ɯonǝʎ

IM DYING