anpera.net
http://anpera.homeip.net/phpbb3/

Erweiterung von MightyEs Waffenladen und Pegasusrüstung
http://anpera.homeip.net/phpbb3/viewtopic.php?f=43&t=5037
Seite 1 von 1

Autor:  Chaos [ Di 01 Dez, 2009 22:28 ]
Betreff des Beitrags:  Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Es ist nur ne kleine Erweiterung des Scripts das ich mal geschrieben habe. Mich hat immer gestört das man nach einem DK die alten Waffen und Rüstungen nicht mehr kaufen konnte. Mit dieser Änderung ist es möglich alle vorherigen Waffen und Rüstungen zu kaufen, so das man am Ende die komplette auswahl hat und nicht wie bisher nur die letzten 15 Waffen und Rüstungen.

Achja, evtl. müssten beim Einbau die addnavs angepasst werden, da sie nicht zur village.php führen.

Dateianhänge:
logd.zip [8.32 KIB]
68-mal heruntergeladen

Autor:  Welv [ Mi 02 Dez, 2009 15:59 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Auch hier n paar Verbesserungsvorschläge für den Code.

bei
$result = mysql_query($sql) or die(mysql_error(LINK));

ist das or die(mysql_error(LINK)) unnötig, da dies schon in der mysql_query funktion geprüft wird ;)



Dann sind auch hier wieder jede menge unnötige outputs.
Statt 2,3,4.... untereinander zu verwenden, würde es auch ein Zeilenumbruch tun.


Und Variablen sind keine Strings, also im output bitte ausklammern.
Statt output("Er entreißt dir $row['weaponname'] gewaltsam, ");
sollte z.B. output("Er entreißt dir ".$row['weaponname']." gewaltsam, "); verwendet wrden

Autor:  Chaos [ Mi 02 Dez, 2009 16:05 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Der teil den du verbessert hast ist aber nicht von mir, das ist alles von der original version... xD

auch wenn es nichts daran ändert das du damit recht hast ist lediglich die kleine erweiterung aller waffen von mir, logd zu optimieren ist ohne eine neue version eh nicht möglich daher hab ich die alten scripte auch nicht verbessert sondern nur mit dem was ich gebraucht habe erweitert ^^

kurz gesagt es ging mir nur um eine kleine erweiterung des scripts und nicht um eine verbesserung der grauenhaften logd programmierung.

Autor:  Tidus [ Mi 02 Dez, 2009 16:28 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Natürlich ist es möglich den Code zu optimieren, du könntest ihn genausogut objektorientiert umschreiben..

den Code zu optimieren ist immer gut, und wenn es nur die ' in den arrays sind und die vereinfachung der mechanismen etc.

Ich denke alle großen 0.9.7 server haben das hinter sich, und die werden bestimmt auch sagen das es wertvoll ist den code zu optimieren bzw. ihn fehlerfrei zu machen und leichter interpretierbar..

Mit einer neuen version hat das nichts zutun oO

Autor:  Chaos [ Mi 02 Dez, 2009 16:42 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

naja ich find die basis mit der common.php nicht gerade sinnvoll, und bevor ich sowas auf oop umschreibe würde ichs lieber von grund auf neu schreiben.

Und das mag vllt nur meine meinung sein, aber ich schreib mir lieber ein eigenes system als ein altes zu verbessern. Und ich denke jeder der sein logd ansatzweise anständig optimiert hat, hat bis auf die grundfunktionen keine ähnlichkeit mehr mit der 0.9.7 version und solange der Kern des spiels auf so eine art funktioniert, hab ich auch kein spass daran einzelne scripte auf den modernsten stand zu bringen.
Bzw. jemand der 0.9.7 nutzt kann gar nicht so viel wert auf rechenleistung legen.

Aber darum gehts auch gar nicht xD, wenn jemand spass daran hat es zu verbessern soll er es gerne tun ^^ hier gehts mir lediglich um die möglichkeit die waffendatenbank sinnvoll zu nutzen und nicht immer nur die letzten 15 waffen und rüstungen kaufen zu können, da ich es sinnlos finde in der datenbank um die 200 waffen stehen zu haben, man aber nur 15 zur auswahl hat und mit der änderung hat man am ende alle eingetragenen waffen zur auswahl

EDIT: @Tidus: vllt hab ich einfach nur falsch verstanden was du gemeint hast xD, mein server läuft schon lange nicht mehr und ich wollte paar erweiterung weitergeben, da sie vllt. andere sinnvoll finden ^^

Autor:  Eliwood [ Mi 02 Dez, 2009 21:20 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Tidus, es ist unsinnig, den Waffenhändler sinnvoll objektorientiert zu schreiben ohne den Kern anzupassen.

Autor:  Tidus [ Mi 02 Dez, 2009 21:59 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Eliwood, ich habe das generell auf code optimieren bezogen nicht auf dieses script, ich habe in keiner silbe gesagt, das er nur dieses script optimieren soll, :D

@Chaos, dann schreib es neu? wieso nimmst du dann 0.9.7 wenn du so gut bist und eh nen eigenes schreiben kannst/willst :P

Autor:  Auric [ Do 03 Dez, 2009 01:20 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Tidus, das geht jetzt ein bisschen in die falsche Richtung...

Toll, das hier mal wieder jemand was veröffentlicht - die Idee an sich halte ich auch für ausgesprochen sinnvoll!

Autor:  Eliwood [ Do 03 Dez, 2009 17:19 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Da fällt mir gleich auf...:

Welv hat geschrieben:
Und Variablen sind keine Strings, also im output bitte ausklammern.
Statt output("Er entreißt dir $row['weaponname'] gewaltsam, ");
sollte z.B. output("Er entreißt dir ".$row['weaponname']." gewaltsam, "); verwendet wrden


Was soll das denn? Fehler gefunden, aber definitiv falsche Begründung. Die richtige Begründung, warum das einen Fehler auswirft, ist, dass man, wenn man Variablen innerhalb von Strings verwendet, die Array-Schlüssel nicht quotieren darf. Es gibt hier also zwei Lösungen: Entweder die von dir genannte, oder man setzt Geschweifte Klammern um den Ausdruck:
$this->bbcode_second_pass_code('', ' output("Er entreißt dir {$row['weaponname']} gewaltsam, ");')

@OP: Eine gute Idee, die Waffen Goldsortiert auswählen zu lassen. Ich hätte sie persönlich nach Drachenkills geordnet, Stärke ist auch ganz praktikabel. Meines Wissens aber verwenden ausnahmslos alle LoGD-Scripte die Wrapperfuntionen db_*. Warum hast du das geändert? db_query beispielsweise macht eine ganze Menge mehr als mysql_query.

Autor:  Chaos [ Do 03 Dez, 2009 21:47 ]
Betreff des Beitrags:  Re: Erweiterung von MightyEs Waffenladen und Pegasusrüstung

Eliwood hat geschrieben:
@OP: Eine gute Idee, die Waffen Goldsortiert auswählen zu lassen. Ich hätte sie persönlich nach Drachenkills geordnet, Stärke ist auch ganz praktikabel. Meines Wissens aber verwenden ausnahmslos alle LoGD-Scripte die Wrapperfuntionen db_*. Warum hast du das geändert? db_query beispielsweise macht eine ganze Menge mehr als mysql_query.
Ich hatte es erst nach dragonkills sortiert... aber da viel mir auf das zumindest ich eh immer die teuersten gekauft habe... und mit der sortierung musste ich mir nicht immer raussuchen welche von denen ich haben will. Aber ich denk das ist geschmackssache wie man es sortieren möchte.

Und das mit dem db_query kann ich grad nichtmal begründen... das script ist paar jahre alt, ich nehm an ich hab damals nicht so wirklich verstanden was der sinn der db_query war und hab einfach mysql_query verwendet weil ich das zu der zeit nicht anders kannte. Bin mir aber nicht sicher xD.

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