anpera.net

Daher ich wieder angefnagen ahb mich in php zu probieren und mein neuster versuch leider nicht getestet werden kann und mich mir nicht 100%sicher bin das alles tut bitte ich einen der php kann sich diesen anzuschauen
[php]<?
// Saufi

require_once "common.php";
checkday();

page_header("Der Schmied des Mehrunes Indaren");
if ($HTTP_GET_VARS[op]==""){
output("Du stehst in einem Großen Raum,du betrachtest die Wände,und die fällt auf das hie rnur edle Waffen und Rüstungen hängen doch kein Preis daran");
output("Du Gehst dan an den Ladentisch der Besitzter scheint ein Kräftiger Schmied zu sein du schaust ihn eine weile an");
output(" Unfreundlich meint er: `4 Entweder du Gibst mir deine Rüstung oder Waffe damit ich sie verbessern kann, oder du Gehst wieder");
addnav("Was wirst du tun");
addnav("Wieder heraus gehen","village.php");
addnav("Rüstung verbessern lassen","schmied.php?op=ruestung");
addnav("Waffe verbessern lassen","schmied.php?op=waffe");
}else if ($HTTP_GET_VARS[op]=="ruestung"){
output("Du legst ihm deine ".$session[user][armor]."auf den Tresen.");
output("Er nimmt sie und meint das es dich 5000Gold kosten würde");
addnav("Du gehst doch wieder","village.php");
addnav("Du lässt sie verbessern","schmied.php?op=rv");
}else if ($HTTP_GET_VARS[op]=="waffe"){
output("Du legst ihm deine ".$session[user][weapon]." auf den Tresen.");
output("Er nimmt sie und meint des es dich 500Gold kosten würde");
addnav("Du gehst doch wieder","village.php");
addnav("du lässt sie verbesser","schmied.php?op=wv");
}else if ($HTTP_GET_VARS[op]=="rv"){
output("er gibt dir wieder deine ".$session[user][armor].");
output("Du bednakst dich und verlässt den Laden,deine ".$session[user][armor]." fühlt sich leichter an aber scheint Stavbiler zu sein");
$session[user][gold]-=5000;
$session[user][armor]+=1;
addnav("Wieder gehen","village.php");
}else if ($HTTP_GET_VARS[op]=="wv"){
output("ergibt dir wieder deine ".$session[user][weapon].");
output("Du bedankst duch und verlässt den LAden,deine ".$session[user][weapon]."fühlt sich leichter an aber die Klinge scheint schärfer und härter zu sein");
&session[user][gold]-=5000;
&session[user][weapon]+=1;
addnav("Wieder gehen","village.php");

}

page_footer();
?>
[/php]

_________________

ich hab das nur mal so fix angeschaut.

in zeile 27

$this->bbcode_second_pass_code('', 'output("er gibt dir wieder deine ".$session[user][armor]."); ')

iss irgendwie nen " zuviel

$this->bbcode_second_pass_code('', 'output("Er gibt Dir Deine " .$session[user][armor]. " wieder");')

das gleiche (oder das selbe - kann mir das immer noch nicht merken) ist auch bei der waffe unten

zeile 33:

$this->bbcode_second_pass_code('', 'output("er gibt dir wieder deine ".$session[user][armor]."); ')

$this->bbcode_second_pass_code('', 'output("Er gibt Dir Deine ".$session[user][weapon]." wieder");')

zeile 35 & 36

$this->bbcode_second_pass_code('', '&session[user][gold]-=5000;
&session[user][weapon]+=1; ')

$this->bbcode_second_pass_code('', '$session[user][gold]-=5000;
$session[user][weapon]+=1;')

wegen den rechtschreibfehlern habe ich jetzt nicht so geschaut, aber es sind fast in jeder zeile welche drin

greetz BlackWarrior

Servus, hier kleinere und schönere Version, hab einige stellen verbessert rest musst du selbst machen auch was die Rechtschreibung angeht!

Ps:
Für klenigkeiten am Script, musst du selbst gucken habs jetzt nur grob bearbeitet und nicht extrem optimiert, geht zwar noch bissel aber trotzdem.


€dit:
BlackWarrior, Array Elemente zb ['...'] immer in ' setzen oder "
Sonst entstehen leicht konflikte, kann auch bei der page_footer funktion passieren, habs bereits selbst gemerkt

Hm.. ja, die idee ist schon ganz nett, aber.... das ist doch zimlich ähnlich schon in der Orkburg verbaut, oder? Naja, ansonst kann ich dir raten:
Erst ein paar mal auf Rechtschreibung lesen und einrücken, dann sieht das galeich viel übersichtlicher aus.

mfg
Auric

PS: Ich würde die Preise levelabhängig machen, ist eigentlich fairer.

_________________
Mehr oder minder inaktiv

Ein kleiner Überblick über die Datenbankfelder in LoGD:


attack, integer: Aktueller Angriffswert inklusive Waffenschaden
defence, integer: Aktueller Defwert inklusive Rüstungsschutz

weapon, string: Waffenname
weapondmg, integer: Schaden, welcher Waffe verursacht.
weaponvalue, integer: Wert der Waffe

armor, string: Rüssiname
armordef, integer: Schaden, welcher die Rüssi verursacht.
armorvalue, integer: Wert der Rüssi


Daraus folgt:
[php]$weapon = "Schwert";
$weapon = $weapon + 1; // Das gleiche wie $weapon+=1;
print $weapon; // Gibt 1 aus, von Schwert ist nix mehr da[/php]

Na, fällt was auf?

Folgliche Bugs:

1.) Waffenname wird überschrieben
2.) Rüssiname wird überschrieben
3.) Waffenwert nicht angepasst
4.) Rüssiwert nicht angepasst
5.) Waffenschaden nicht erhöht
6.) Rüssischutz nicht erhöht
7.) Spielerstärke nicht erhöht
8.) Spielerschutz nicht erhöht
9.) Keine Kontrolle, ob Spieler genügend Gold hat

mir geht es bei dme script weniger um die rechtschreibung sonder das er tut naja also in den sorce von Lotgd ist mitt im satz das mit " naja ich geb doch im prinzip doch der waffe was dazu dafür muss ich sie ja nicht umbennnen des +1 is doch störend naja des was Auric angesprochen hat dafür fhelt mir elider der befehl wollte eher so einbauen um so stärker man shcon ist desto mehr kostet es aber levl abhängig ist auch gut allerdigns ist mir da weshalb bei uns bei der Gilde das rausgenommen wurde aufgefallen das bei fist mann 0 bezhalne muss

@Kevs öm du has ja dne scirpt komplett umgedreht der anfang steht unten O.o[/quote]

_________________

Hast du meinen Post eigentlich gelesen & begriffen?


Du überschreibst den Waffennamen mit der Ziffer 1, du machst sie nicht stärker...

Hab doch extra um es verständlicher zu machen die nötigen Felder & deren Bedeutung aufgeschrieben.


Die Waffe WIRD NICHT STÄRKER, du benennst sie einfach um in 1.


Nun Begriffen oder schreib ich immer noch eine Wand an?


PS: Lies deinen Post vor absenden mindestens 3mal durch, behebe die gröbsten Typos, vertippsler und so, mach Absätze - Danke.
Runen kann ich nur schwer entziffern.

loool ich hab den waffennamen umschreibenlassen naja ok also msuss ich attace udn defence einfügen oder wie? aber des show stimmt doch das dan der aktuele waffennamen angezeigtt wird?

_________________

Lies das durch...

Da sind die Felder, die du brauchst, in Deutschen Worten erkärt, was sie für das Script bringen.

Damit kannst du im den primitivsten Zügen die Waffe verbessern lassen.

nein warum des am ende steht versteh ich nciht wirklich


zu eliwood(hoffe ich habs richtig geschrieben)
9.) Keine Kontrolle, ob Spieler genügend Gold hat <---da hasst du recht aber ich weis nit wo ich die ifabfrage einbauen solltewollte nen extra addnav nochmal einbauen aber ich glaub des würd die user nur "nerven" wenn die 4 mal klicken müssten bis se die waffe verbessert haben

_________________

[php]if($session['user']['gold']>=5000) addnav("Waffe verbesssern","{filename}.{extension}");[/php]

dan sag ich ma danke werd des dan bei dem von kevz einfügen und dan nochma hier posten

_________________

UND du wirst das überschreiben der Variablen weapon und armor beheben, meine Posts helfen dir dabei.

ja das werd ich ja =) hab erstma was gegessen

_________________

[php]<?
<?
// Saufi in Hilfe von Kevz und besonder Eliwood

require_once './common.php';

checkday();
page_header("Der Schmied des Mehrunes Indaren");

switch($_GET['op']) {
case 'wv':
output("Er gibt dir deine ".$session['user']['weapon']." wieder.");
output("Du bedankst dich und verlässt den Laden, deine ".$session['user']['weapon']." fühlt sich leichter an aber die Klinge scheint schärfer und härter zu sein.");
$ession['user']['weapondmg'] += 1;
$session['user']['weaponvalue']+=1000;
$session['user']['attack'] +=1;
$session['user']['gold'] -=5000;
addnav("Wieder gehen","village.php");
case 'waffe':
utput("Du legst ihm deine ".$session['user']['weapon']."auf den Tresen, er nimmt deine Waffe und begutachtet sie ");
output("und sagt dir das er dafür 5000 Totenköpfe verlangen würde!");
addnav("Du gehst doch wieder","village.php");
if($session['user']['gold']>=5000) addnav("Du lässt sie verbessern","schmied.php?op=wv");
break;
case 'rv':
output("Er gibt dir deine ".$session['user']['armor']." wieder.");
output("Du bedankst dich und verlässt den Laden, deine ".$session['user']['armor']." fühlt sich leichter an aber scheint stabiler zu sein.");
$session['user']['armordef'] += 1;
$session['user']['armorvalue'] +=1000;
$seesion['user']['defence'] +=1;
$session['user'['gold'] -=5000;
addnav("Wieder gehen","village.php");
case 'ruestung':
output("Du legst ihm deine ".$session['user']['armor']."auf den Tresen, er nimmt deine Rüstung und begutachtet sie ");
output("und sagt dir das er dafür 5000 Totenköpfe verlangen würde!");
addnav("Du gehst doch wieder","village.php");
if($session['user']['gold']>=5000) addnav("Du lässt sie verbessern","schmied.php?op=rv");
break;
default:
output("Du stehst in einem großen Raum, du betrachtest die Wände,und dir fällt auf das hier nur edle Waffen und Rüstungen hängen doch kein Preis daran");
output("Du Gehst dann an den Ladentisch, der Besitzter scheint ein Kräftiger Schmied zu sein, du schaust ihn eine Weile an");
output(" Unfreundlich meint er: `4 Entweder du Gibst mir deine Rüstung oder Waffe damit ich sie verbessern kann, oder du gehst wieder");
addnav("Was wirst du tun");
$nav = array('Wieder heraus gehen'=>'village.php', 'Rüstung verbessern lassen'=>'schmied.php?op=ruestung', 'Waffe verbessern lassen'=>'schmied.php?op=waffe');
foreach($nav as $text => $link) addnav($text, $link);
}

page_footer();
?> [/php]

so habs einmal nochma rausgemacht und ersetzt aber es tut leider nicht kann mir jemand sagen warum?

_________________

will mir keiner helfen?

_________________

guck nochmal nach.. da fehlt mindestens 2 mal ein "break;"
Und mit "geht nicht" kann man nix anfangen, du könntest den Fehler ruhig mal etwas genauer beschreiben...

_________________
Mehr oder minder inaktiv

och ums ganz genau zu beschreiben der bildschirm is dan weis

edit: die breaks wurdne gesetzt, das problem mit der weisen seite ist aber immernoch

_________________

Ich kann fast zu 80% sagen, dass eine Ausgabe vor <?php steht. (Nach ?> ist es da egal, da page_footer sowieso das Script abbricht).

d.h. Vor deinem ersten <?php steht Entweder ein Leerzeichen, ein Absatz over son sonstiges Zeichen.