anpera.net
https://anpera.homeip.net/phpbb3/

Turm der 7 Suenden
https://anpera.homeip.net/phpbb3/viewtopic.php?f=43&t=4966
Seite 1 von 1

Autor:  Lunastra [ Sa 01 Aug, 2009 04:44 ]
Betreff des Beitrags:  Turm der 7 Suenden

Sooo,
schönen guten Morgen es ist morgens, und ich schlaf noch halb,
also verzeiht meine eventuellen rechtschreibfehler xD
Und nein Luna hat das Script nicht heute fertig bekommen, sondern gestern.
Danke Anpera^^ das ich es nun mal hochladen darf.

So, zu dm Addon:
Es ist mein aller erstes eigenes, von Grund auf geschriebenes Script
deshalb entschuldige ch mich jetzt schon, wenn es vielleicht von der
Form her nicht perfekt ist bzw Fehler enthalten sind.^^°
Zu dem, meine Php-kenntnisse sind sehr *räusper* sehr gering.
Kleine Inspiration war vor langer Zeit mal der Turm der Elemente.
also dann

Der Turm
Der Turm an sich hat 7 Zimmer, in jedem trifft man auf eine Sünde
welche einen dann vor 3 Wahlen stellt.
Jedes Zimmer kann man erst mit einer bestimmten Anzahl an Dks betreten,
mit Ausnahme des letzten Raumes.
Je nach dem welche Option man wählt, wird sich ein anderer Effekt aufweisen,
der sich positiv oder negativ auf den Spieler auswirken kann.

Wenn man 1x in einem Raum gewesen ist, hat man erst die Möglichkeit,
diesen beim nächsten neuen Tag wieder zu betreten. (Meine Methode ihn
soweit zu "entschärfen", da er sonst wohl etwas zu leicht auszunutzen wäre)
Wer das umstellen bzw verlängern will, das stell ich mal frei^^

Zum Einbau

Eine Einbauanleitung ist zwar schon im Script enthalten,
aber ich schreib das lieber nochmal mit hinzu.

---öffne ort an dem man es verlinken möchte----

füge ein:
addnav("Turm der 7 Sünden","turm7suenden.php");

---öffne---
newday.php

suche:
$session['user']['drunkenness']=0;

füge darunter ein:
$session['user']['sueturm']=0;

----
mehr muss man nicht ändern

---SQL---
ALTER TABLE `accounts` ADD `sueturm` INT(5) NOT NULL DEFAULT '0'


So das solls gewesen sein^^
Wenn es sich jemand einbastelt, wüsst ich nur gerne bescheid.

P.S: Ein Danke an Sando und Kevz die ich damit ab und an mal genervt hatte was die
Räume anging, und Kevz, der mir das mit der Raumanzeige etwas vereinfacht hat.


UPDATE:
woa, endlich mal geschafft <<
hab jetzt die sicherheitsabfragen drin, damit auch ja keiner meckert o.o"
soo, ich hoffe nur hab sie auch richtig formuliert x.x"
naja oo bei fehlern bitte melden Dankö :3

Dateianhänge:
Dateikommentar: Sündenturm - Update mit den Sicherheitsabfragen
turm7suenden.zip [7.73 KIB]
354-mal heruntergeladen

Autor:  Tidus [ Sa 01 Aug, 2009 10:30 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

alsoo, dann mal meine kritik ^^

also überraschender weise ist es vom aufbau her ganz okay, die singlequotes scheinen schön gesetzt zu sein.

was mich aber stört ist an manchen stellen bis zu 5 output's sind für einen text, und wie jeder weiß ist output eine sehr langsame funktion die man nicht öfter aufrufen sollte als es notwendig ist. gut, manche rufen sie nur einmal im ganzen script auf, manche für jeden text einmal ist ja beides ok aber 5 mal für einen text das ist zuviel ;)

und, du solltest einen zufall einbauen, so ist es zu 100% ausnutzbar, wenn du dir merkst in welchem raum du was drücken musst um spezielle boni zu bekommen..

ich habe nur schnell drüber geschaut, korrigiert mich wenn ich falsch liege :P ach und 50 permanente lebenspunkte ... das ist zu viel, du musst besser balancieren wenn man in den raum auch jedentag rein kann, wäre es gefährlich.. vorallem ohne zufall.. denn du musst immer bedenken 50 permanente lp sind wie 10 dks oder 100 edelsteine!

grüße Tidus.

Autor:  Lunastra [ Sa 01 Aug, 2009 18:57 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

nagut erstmal danke für kritik, wie gesagt meine Kenntnisse sind noch sehr gering
(das wird aber alles noch)

zum zweite 50 permanente Lp?? oha stimmt da hab ich mich wohl vertippt.. ich besser das gleich mal aus..

EDTIT:
So, die 50 mal in 10 geändert

Autor:  Liath [ Sa 01 Aug, 2009 20:40 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

selbst wenns entschärft wird, kann man trotzdem Glück haben und jeden Tag dadurch seine LP´s erhöhen, da ist 10 immer noch nen bisschen viel... 1 LP würd doch reichen denk ich mal

Autor:  Lunastra [ So 02 Aug, 2009 04:13 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

mhm stimmt auch wieder
ich bastel nochmal dran rum

Autor:  Welv [ So 02 Aug, 2009 10:39 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Nettes Addon und ein switch($_GET['op']) würde es noch n bisschen schneller und übersichtlicher machen als die if und elses

Autor:  Lunastra [ So 02 Aug, 2009 12:51 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Welv hat geschrieben:
Nettes Addon und ein switch($_GET['op']) würde es noch n bisschen schneller und übersichtlicher machen als die if und elses


danke^^
wie gesgat ich bin noch php anfänger^^°
ich denke ich werde das Script irgendwann sowieso nochmal abändern^^

Autor:  Rikkarda [ So 02 Aug, 2009 16:29 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Mir fehlen in dem Script jegliche "Sicherheitsabfragen"

du verlierst 20 wks

du verlierst 10 cp

du verlierst 1000 gold

du verlierst 10 hitpoints

usw. ohne jegliche Prüfung ob derjenige das überhaupt besitzt.

das solltest du definitiv noch einmal überarbeiten

Autor:  Salator [ So 02 Aug, 2009 16:43 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Von der Idee her ist das Script hübsch, aber wie meinen Vorredner schon gesagt haben, man kann zu einfach zu viel Bonus bekommen. Die Abzüge sind dafür kein Ausgleich, weil man ja früher oder später weiß was man wählen muss. Und selbst wenn man es noch nicht weiß, es wird nirgends geprüft ob das was du abziehen willst überhaupt vorhanden ist.

Zitat:
suche:
$session['user']['drunkenness']=0;

Ungünstige Stelle. Das allgemeine Werte zurücksetzen ist schon weiter oben, einzig drunkenness ist eine Ausnahme, weil man bei schwerem Kater einen WK abgezogen bekommt.
Suche:
if ($_GET['resurrection']!="true" && $_GET['resurrection']!="egg" ){
$session['user']['soulpoints']=50 + 5 * $session['user']['level'];
Wenn der Turm nur einmal pro Spieltag erreichbar sein soll füge das $session['user']['sueturm']=0; darunter ein. Wenn das Betreten auch nach einer Wiedererweckung möglich sein soll, füge es darüber ein. Zu empfehlen wäre ersteres, außer man hat sehr lange Spieltage.

Zitat:
ALTER TABLE `accounts` ADD `sueturm` INT(5) NOT NULL DEFAULT '0'

Ein INT belegt 4 Bytes in der Datenbank. Reichlich overdosed für einen Wert, der nur 0 oder 1 sein kann. TINYINT würde locker ausreichen.

Zitat:
output("`c<img src='http://i25.tinypic.com/16gn56x.png'>`c`n`n",true);

Ob das wirklich so eine gute Idee ist, das Bild von tinypic.com einzubinden? Es könnte doch auch in diesem Paket mitgeliefert werden und dann in den images-Ordner vom eigenen Lotgd rein.


Ansonsten könnte das gesamte Script mal durch den Code-Beautifyer gejagt werden. Eine web-basierte Version liegt dem Release der DS-Edition bei und kann z.b. bei mir aufgerufen werden:
http://www.wyndoria.de/code_beautifier/beautify.html
(anschließend sollten die führenden Leerzeichen wieder durch Tabs ersetzt werden, in Notepad++ zu finden unter TextFX -> TextFX Edit -> Leading space to tabs or tabs to spaces)

Bei Farbcodes, die kein Standard sind und nicht aus dem Text ersichtlich sind, könntest du evtl noch dazuschreiben wie das aussehen soll. Aufgefallen ist mir `É, was ich mit Sicherheit nicht habe.

Und bei den Texten ist mir aufgefallen, dass die recht eigenwillig formatiert sind. Kurze zentrierte Zeilen würde ich ja noch als Symbolik für Turm sehen, aber dass dann jede Zeile groß anfängt, das sieht komisch aus.
Auch scheinen mir die Texte sehr auf männliche Chars gemünzt zu sein. Was machen denn die Mädels, bekommen die ne eigene Version von dem Turm?

So genug gemeckert ;) Nicht den Kopf hängen lassen, mit der Zeit kommt auch die Erfahrung.

Autor:  Lunastra [ So 02 Aug, 2009 17:19 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

gut die stelle uhm ich hab mich etwas nach dem jenovascript gerichtet XD"

ehm zu dem bild, nunja, man kann es benutzen, oder nicht
zu dem man es ändern kann, wenn man ein ganz anderes verwenden will

was die farbcodes anbelangt, nun also da kann ich nur dazu sagen das man
diese anpassen muss,
allerdings ist mir auch aufgefallen, das auf manchen servern sogar die
Grundfarben wieder komplett anders sind, deshalb kann ich darauf etwas wenig rücksicht nehmen^^°

und dei Texte, nunja, das kommt davon wenn man die immer nachtschreibt und dann noch in word, wobei manche texte ja zentriert gehalten werden, und da man sich ja ebenso
die ausrichtung ansehen will, hat word leider die macke (bei mir zumindest) bei jeder neuen zeile groß anzufangen x,x"
naja und manchmal achtet man dann nicht mehr so drauf xD"

Autor:  Linus [ Mo 10 Aug, 2009 13:12 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Dann werde ich auch mal meinen Senf dazu geben! ;)

Grundsätzlich eine schöne Idee! Ich werde es einsetzen, nach einigen kleinen Änderungen an den Boni, weil ich halt einfach eine andere Balance zwischen gut und schlecht bevorzuge. Und ebenso werde ich vor Abzug diverse Abfragen einfügen, wie Rikka es zuvor schon angemerkt hatte.

Was mir noch aufgefallen ist: Am Ende der Datei steht ein page_footer(); somit sind die page_footer(); in den einzelnen elseif{} Blöcken überflüssig und führen nur dazu das diese Funktion zweimal ausgeführt wird!

Aber sonst: Prima! Weiter so!

Grüßle Linus

Autor:  Salator [ Mo 10 Aug, 2009 21:52 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Kann sein dass es in der 0.9.7 anders ist, in der DS3 wird das Script beim Erreichen von page_footer() beendet.
Manchmal kann dieses Verhalten nützlich sein. Ich schreib dann aber trotzdem noch zur Sicherheit ein break dahinter.
Okay, in dem Turm-Script sehe ich auch keine Notwendigkeit für einen vorzeitigen Abbruch, also kann das entfallen.

Autor:  Linus [ Di 11 Aug, 2009 13:08 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Salator hat geschrieben:
Kann sein dass es in der 0.9.7 anders ist, in der DS3 wird das Script beim Erreichen von page_footer() beendet.
Stimmt! Ist in der 0.97er ebenfalls so dass das Script beendet wird. In sofern muss ich meine Aussage korrigieren! Aber, wie schon angemerkt, reicht in diesem Script das page_footer() am Ende.

Ich habe denn auch$this->bbcode_second_pass_code('', '}
if($_GET['op'] == 'ruhe'){')nach$this->bbcode_second_pass_code('', '}elseif($_GET['op'] == 'ruhe'){')
geändert. Aber das kann man auch in die Rubrik "schöner Wohnen" einordnen. :D
Eleganter wäre es das Ganze mit select($_GET['op']) zu lösen, war mir aber zu viel Arbeit für geringen Nutzen. ;)

Autor:  Lunastra [ Do 13 Aug, 2009 11:44 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Salator hat geschrieben:
Kann sein dass es in der 0.9.7 anders ist, in der DS3 wird das Script beim Erreichen von page_footer() beendet.


uhm das kann sein, aber auf meinem server läuft ja kein DS
und das script istr an sich auch nur für die 0.9.7 ausgelegt^^°


@Linus

erstmal dank, hab schon gesehn das dus eingebaut hast
freut mich wenns gefällt, kleine änderungen,
naja, an die sicherheitsabfragen die rikka und die anderen schon
angesprochen hatten, muss ich mich nochmal setzen,
und an einer switch funktion werd ich mich auch mal ransetzen
wann(!) ist nur die frage xD"
aber naja, ich lerne php usw jetzt richtig, von daher, sobald ich
das dann aucvon den grundlagen her besser intus habe
änder ich das script nochmal komplett

Autor:  Nina131bln [ Fr 11 Sep, 2009 21:57 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

Tolle Skript..kommt auf dem Server sehr gut an...hoffe man sieht bald noch mehr :)

Autor:  Lunastra [ Sa 12 Sep, 2009 15:43 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

danke freut mich zu hören^^

Autor:  Lunastra [ Di 24 Nov, 2009 16:24 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

sry für den doppelpost aber siehe oben A_A
=> für die, die eevntuell mal drüberschauen wollen..

Autor:  Welv [ Di 24 Nov, 2009 17:24 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

$this->bbcode_second_pass_code('', '
//Seitenkoerper
output("`c<img src='http://i25.tinypic.com/16gn56x.png'>`c`n`n",true); // Bildlink austauschen, wenn ihr meins nicht wollt ;)
output("`c`bTurm der 7 Sünden`b`c`n`n`0");
output("`cDu betrittst den Turm und erblickst eine riesige Wendeltreppe,`n die schier unendlich weit nach oben führt.");
output("`nAuf jeder Etage ist eine Tür, die in ein Zimmer zu führen scheint.`n Verschiedene Mächte verbieten dir den Eintritt in manche Zimmer.
`nEs scheint, als müsstest du bei diesen Zimmern`n erst eine Gewisse Stärke erreicht haben um eintreten zu dürfen.`n`n`n`c");
//Chat
')

Hierbei und bei ähnlichen Stellen würde doch auch ein output genügen

Autor:  Lunastra [ Mi 25 Nov, 2009 13:56 ]
Betreff des Beitrags:  Re: Turm der 7 Suenden

oh stimmt, hatte ich ganz übersehen oo" danke^^

Seite 1 von 1 Alle Zeiten sind UTC + 1 Stunde
Powered by phpBB® Forum Software © phpBB Group
https://www.phpbb.com/