Scriptbewertung

[glan]

Hallo,
wie findet ihr diesen Script Teil?
ich habe ihn mit Hilfe vom Selfhtml Forum (siehe http://forum.de.selfhtml.org/archiv/2006/12/t142391/) hergestellt.

Ich atte leider noch keine Zeit ihn einzurücken und zu strukturieren:

echo "<tr><td></td><td><input type="text" name="compa" value="";
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta != klebusta AND $azazei == 8)
{
echo "$zufallG[1]$zufallG[2]$zufallG[3]$zufallG[4]$zufallG[5]$zufallG[6]$zufallG[7]$zufallG[8]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta != klebusta AND $azazei == 9)
{
echo "$zufallG[1]$zufallG[2]$zufallG[3]$zufallG[4]$zufallG[5]$zufallG[6]$zufallG[7]$zufallG[8]$zufallG[9]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta != klebusta AND $azazei == 10)
{
echo "$zufallG[1]$zufallG[2]$zufallG[3]$zufallG[4]$zufallG[5]$zufallG[6]$zufallG[7]$zufallG[8]$zufallG[9]$zufallG[10]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta != klebusta AND $azazei == 11)
{
echo "$zufallG[1]$zufallG[2]$zufallG[3]$zufallG[4]$zufallG[5]$zufallG[6]$zufallG[7]$zufallG[8]$zufallG[9]$zufallG[10]$zufallG[11]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta != klebusta AND $azazei == 12)
{
echo "$zufallG[1]$zufallG[2]$zufallG[3]$zufallG[4]$zufallG[5]$zufallG[6]$zufallG[7]$zufallG[8]$zufallG[9]$zufallG[10]$zufallG[11]$zufallG[12]";
}
if($eico == co AND $klebusta == klebusta AND $spezei != spezei AND $grobusta != grobusta AND $azazei == 8)
{
echo "$zufallK[1]$zufallK[2]$zufallK[3]$zufallK[4]$zufallK[5]$zufallK[6]$zufallK[7]$zufallK[8]";
}
if($eico == co AND $klebusta == klebusta AND $spezei != spezei AND $grobusta != grobusta AND $azazei == 9)
{
echo "$zufallK[1]$zufallK[2]$zufallK[3]$zufallK[4]$zufallK[5]$zufallK[6]$zufallK[7]$zufallK[8]$zufallK[9]";
}
if($eico == co AND $klebusta == klebusta AND $spezei != spezei AND $grobusta != grobusta AND $azazei == 10)
{
echo "$zufallK[1]$zufallK[2]$zufallK[3]$zufallK[4]$zufallK[5]$zufallK[6]$zufallK[7]$zufallK[8]$zufallK[9]$zufallK[10]";
}
if($eico == co AND $klebusta == klebusta AND $spezei != spezei AND $grobusta != grobusta AND $azazei == 11)
{
echo "$zufallK[1]$zufallK[2]$zufallK[3]$zufallK[4]$zufallK[5]$zufallK[6]$zufallK[7]$zufallK[8]$zufallK[9]$zufallK[10]$zufallK[11]";
}
if($eico == co AND $klebusta == klebusta AND $spezei != spezei AND $grobusta != grobusta AND $azazei == 12)
{
echo "$zufallK[1]$zufallK[2]$zufallK[3]$zufallK[4]$zufallK[5]$zufallK[6]$zufallK[7]$zufallK[8]$zufallK[9]$zufallK[10]$zufallK[11]$zufallK[12]";
}
if($eico == co AND $spezei == spezei AND $grobusta != grobusta AND $klebusta != klebusta AND $azazei == 8)
{
echo "$zufallS[1]$zufallS[2]$zufallS[3]$zufallS[4]$zufallS[5]$zufallS[6]$zufallS[7]$zufallS[8]";
}
if($eico == co AND $spezei == spezei AND $grobusta != grobusta AND $klebusta != klebusta AND $azazei == 9)
{
echo "$zufallS[1]$zufallS[2]$zufallS[3]$zufallS[4]$zufallS[5]$zufallS[6]$zufallS[7]$zufallS[8]$zufallS[9]";
}
if($eico == co AND $spezei == spezei AND $grobusta != grobusta AND $klebusta != klebusta AND $azazei == 10)
{
echo "$zufallS[1]$zufallS[2]$zufallS[3]$zufallS[4]$zufallS[5]$zufallS[6]$zufallS[7]$zufallS[8]$zufallS[9]$zufallS[10]";
}
if($eico == co AND $spezei == spezei AND $grobusta != grobusta AND $klebusta != klebusta AND $azazei == 11)
{
echo "$zufallS[1]$zufallS[2]$zufallS[3]$zufallS[4]$zufallS[5]$zufallS[6]$zufallS[7]$zufallS[8]$zufallS[9]$zufallS[10]$zufallS[11]";
}
if($eico == co AND $spezei == spezei AND $grobusta != grobusta AND $klebusta != klebusta AND $azazei == 12)
{
echo "$zufallS[1]$zufallS[2]$zufallS[3]$zufallS[4]$zufallS[5]$zufallS[6]$zufallS[7]$zufallS[8]$zufallS[9]$zufallS[10]$zufallS[11]$zufallS[12]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta == klebusta AND $azazei == 8)
{
echo "$zufallGK[1]$zufallGK[2]$zufallGK[3]$zufallGK[4]$zufallGK[5]$zufallGK[6]$zufallGK[7]$zufallGK[8]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta == klebusta AND $azazei == 9)
{
echo "$zufallGK[1]$zufallGK[2]$zufallGK[3]$zufallGK[4]$zufallGK[5]$zufallGK[6]$zufallGK[7]$zufallGK[8]$zufallGK[9]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta == klebusta AND $azazei == 10)
{
echo "$zufallGK[1]$zufallGK[2]$zufallGK[3]$zufallGK[4]$zufallGK[5]$zufallGK[6]$zufallGK[7]$zufallGK[8]$zufallGK[9]$zufallGK[10]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta == klebusta AND $azazei == 11)
{
echo "$zufallGK[1]$zufallGK[2]$zufallGK[3]$zufallGK[4]$zufallGK[5]$zufallGK[6]$zufallGK[7]$zufallGK[8]$zufallGK[9]$zufallGK[10]$zufallGK[11]";
}
if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta == klebusta AND $azazei == 12)
{
echo "$zufallGK[1]$zufallGK[2]$zufallGK[3]$zufallGK[4]$zufallGK[5]$zufallGK[6]$zufallGK[7]$zufallGK[8]$zufallGK[9]$zufallGK[10]$zufallGK[11]$zufallGK[12]";
}
if($eico == co AND $grobusta == grobusta AND $klebusta == klebusta AND $spezei == spezei AND $azazei == 8)
{
echo "$zufallGKS[1]$zufallGKS[2]$zufallGKS[3]$zufallGKS[4]$zufallGKS[5]$zufallGKS[6]$zufallGKS[7]$zufallGKS[8]";
}
if($eico == co AND $grobusta == grobusta AND $klebusta == klebusta AND $spezei == spezei AND $azazei == 9)
{
echo "$zufallGKS[1]$zufallGKS[2]$zufallGKS[3]$zufallGKS[4]$zufallGKS[5]$zufallGKS[6]$zufallGKS[7]$zufallGKS[8]$zufallGKS[9]";
}
if($eico == co AND $grobusta == grobusta AND $klebusta == klebusta AND $spezei == spezei AND $azazei == 10)
{
echo "$zufallGKS[1]$zufallGKS[2]$zufallGKS[3]$zufallGKS[4]$zufallGKS[5]$zufallGKS[6]$zufallGKS[7]$zufallGKS[8]$zufallGKS[9]$zufallGKS[10]";
}
if($eico == co AND $grobusta == grobusta AND $klebusta == klebusta AND $spezei == spezei AND $azazei == 11)
{
echo "$zufallGKS[1]$zufallGKS[2]$zufallGKS[3]$zufallGKS[4]$zufallGKS[5]$zufallGKS[6]$zufallGKS[7]$zufallGKS[8]$zufallGKS[9]$zufallGKS[10]$zufallGKS[11]";
}
if($eico == co AND $grobusta == grobusta AND $klebusta == klebusta AND $spezei == spezei AND $azazei == 12)
{
echo "$zufallGKS[1]$zufallGKS[2]$zufallGKS[3]$zufallGKS[4]$zufallGKS[5]$zufallGKS[6]$zufallGKS[7]$zufallGKS[8]$zufallGKS[9]$zufallGKS[10]$zufallGKS[11]$zufallGKS[12]";
}
if($eico == co AND $klebusta == klebusta AND $spezei == spezei AND $grobusta != grobusta AND $azazei == 8)
{
echo "$zufallKS[1]$zufallKS[2]$zufallKS[3]$zufallKS[4]$zufallKS[5]$zufallKS[6]$zufallKS[7]$zufallKS[8]";
}
if($eico == co AND $klebusta == klebusta AND $spezei == spezei AND $grobusta != grobusta AND $azazei == 9)
{
echo "$zufallKS[1]$zufallKS[2]$zufallKS[3]$zufallKS[4]$zufallKS[5]$zufallKS[6]$zufallKS[7]$zufallKS[8]$zufallKS[9]";
}
if($eico == co AND $klebusta == klebusta AND $spezei == spezei AND $grobusta != grobusta AND $azazei == 10)
{
echo "$zufallKS[1]$zufallKS[2]$zufallKS[3]$zufallKS[4]$zufallKS[5]$zufallKS[6]$zufallKS[7]$zufallKS[8]$zufallKS[9]$zufallKS[10]";
}
if($eico == co AND $klebusta == klebusta AND $spezei == spezei AND $grobusta != grobusta AND $azazei == 11)
{
echo "$zufallKS[1]$zufallKS[2]$zufallKS[3]$zufallKS[4]$zufallKS[5]$zufallKS[6]$zufallKS[7]$zufallKS[8]$zufallKS[9]$zufallKS[10]$zufallKS[11]";
}
if($eico == co AND $klebusta == klebusta AND $spezei == spezei AND $grobusta != grobusta AND $azazei == 12)
{
echo "$zufallG[1]$zufallG[2]$zufallG[3]$zufallG[4]$zufallG[5]$zufallG[6]$zufallG[7]$zufallG[8]$zufallG[9]$zufallG[10]$zufallG[11]$zufallG[12]";
}
echo "" readonly></td></tr>\n";

Ist daran eigentlich etwas verboten (html-Tag unterbrechen...)?

MfG

[lulu]

... sind die Schulferien bald wieder vorbei.

[glan]

Hehe, der war gut.

[Wilhelm Turtschan]

habe d'ehre lulu

... sind die Schulferien bald wieder vorbei.

frueher hat man mit Papi um diese Zeit an der Krippe rumgebaut. Neues Dach, neue Laempchen, auf dem Christkindlmarkt neue Schaefchen und Kuehe gekauft, vielleicht auch einen neuen Josef, froehliche Liedchen getraellert, selbstgebackene Plaetzchen in sich reingestopft (auch mit der falschen Zimtsorte) und ist sonst niemanden auf den Nerv gegangen. Sch***technik heutzutage.

Waren das noch ruhige und besinnliche Zeiten. :-)

man liest sich
Wilhelm

[Reiner]

hi,

frueher hat man mit Papi um diese Zeit an der Krippe rumgebaut. Neues Dach, neue Laempchen, auf dem Christkindlmarkt neue Schaefchen und Kuehe gekauft, vielleicht auch einen neuen Josef, froehliche Liedchen getraellert, selbstgebackene Plaetzchen in sich reingestopft (auch mit der falschen Zimtsorte) und ist sonst niemanden auf den Nerv gegangen. Sch***technik heutzutage.

Waren das noch ruhige und besinnliche Zeiten. :-)

ja, heute sitzen die Papas vor den Rechnern und ärgern sich mit dummdreisten fremden Kindern rum, um die sich niemand kümmert. Der Friede wird aber wieder einkehren, wenn in drei Tagen dann endlich die neue Playstation/XBox ausgepackt werden kann.

;-)

Gruß
Reiner

[_hkl]

Hallo !

ja, heute sitzen die Papas vor den Rechnern und ärgern sich mit dummdreisten fremden Kindern rum, um die sich niemand kümmert. Der Friede wird aber wieder einkehren, wenn in drei Tagen dann endlich die neue Playstation/XBox

Nee, dann fangen mit Chance die Maedels an zu meckern. Lara Croft war naemlich offenbar beim Friseur...
Spiele die die Welt nicht braucht

:-)

ausgepackt werden kann.

;-)

Gruß
Reiner

Gruesse und allseits frohes Fest !

Holger

[RuD]

Seid gegrüßt!

In Anbetracht dessen  kann ich dir nur mehr als zu stimmen!

[gigoX]

Sorry, in Baden-Württemberg ist morgen erst der letzte Schultag. Woher willst du wissen ob er gerade Ferien hat?
Denn diesen Code hätte ich auch schnell nach der Schule schreiben können, oder sogar in der Mittagspause. Weil sehr durchdacht sieht dieser Code auch nicht aus...

Und um glans Frage zu beantworten: Der Code ist - ääähhmm - .... naja ... schlecht? Kann man das so sagen? Auf jeden Fall nicht annehmbar...

Du kannst doch Mehrfachverzweigungen nehmen, also if() in if(). Oder sogar switch() und case wenn dir das lieber ist. Aber so würde ich den Code nich lassen, nimmer nie. Stell dir vor, du siehst dir den Code nach einem oder zwei Monaten wieder an...

Bei Mehrfachverweigungen nehme ich immer als erste Abfrage die Variable, die am wenigsten Werte annehmen kann, bei dir ist das $eico.
Danach die die am meisten Werte annehmen kann, bei dir ist das $azazei.
Der ganze Rest wird bei mir danach abgefragt.

Aber machen kannst du mit deinem Code natürlich was du willst.

Gruß
gigoX

[seth_not@home]

gudn tach!

wie findet ihr diesen Script Teil?

eklig.

Ich atte leider noch keine Zeit ihn einzurücken und zu strukturieren:

dann nimm sie dir. dadurch muessen etwaige leser jeweils(!) weniger zeit aufbringen.

ich habe nur einen kurzen blick riskiert (weil keine zeit und so, gell). es sieht so aus wie
if(a AND b and c and $azazei == 8) mach_wadd(1 bis 8);
if(a AND b and c and $azazei == 9) mach_wadd(1 bis 9);
if(a AND b and c and $azazei == 10) mach_wadd(1 bis 10);
...

das geht auch kuerzer und huebscher:
if(a AND b AND c){
  if($azazei == 8) mach_wadd(1 bis 8);
  elseif($azazei == 9) mach_wadd(1 bis 9);
  elseif($azazei == 10) mach_wadd(1 bis 10);
}

und das geht noch mal kuerzer und huebscher:
if(a AND b AND c)
  mach_wadd(1 bis $azazei);

Ist daran eigentlich etwas verboten (html-Tag unterbrechen...)?

praezisiere die frage.

prost
seth

[glan]

Ist daran eigentlich etwas verboten (html-Tag unterbrechen...)?

praezisiere die frage.

Es gibt doch z.B. in html verbotene Sachen, wie Formulare verschachteln etc.
Ist so ein ähnlicher Fehler hier?

MfG

[seth]

gudn tach!

Ist daran eigentlich etwas verboten (html-Tag unterbrechen...)?

praezisiere die frage.
Es gibt doch z.B. in html verbotene Sachen, wie Formulare verschachteln etc.
Ist so ein ähnlicher Fehler hier?

um auf sowas zu achten, ist mir der code zu unuebersichtlich.

prost
seth

[Felix Riesterer]

Liebe(r) glan,

wie findet ihr diesen Script Teil?

absolut unübersichtlich und unnötig aufgebläht: schlechte Wertung von mir.

if($eico == co AND $grobusta == grobusta AND $spezei != spezei AND $klebusta != klebusta AND $azazei == 8)

Pfui! Was bitte ist das hier?

$eico == co

Du verwendest hier offensichtlich eine hoffentlich zuvor definierte Konstante namens "co". Aber ich bezweifle, dass Du das gemeint hast... Gleiches gilt für die restlichen Ausdrücke mit Ausnahme des letzten.

Dass man das alles viel übersichtlicher und enorm kürzer notieren kann, wurde Dir ja schon längst gesagt.

Liebe Grüße aus Ellwangen,

Felix Riesterer.

[Blaubart]

Tach.

Ich atte leider noch keine Zeit ihn einzurücken und zu strukturieren:

Das macht nichts. Viel schlimmer ist, daß du nirgendwo eval() benutzt.

[_hkl]

Hallo !

Ich atte leider noch keine Zeit ihn einzurücken und zu strukturieren:

(?)
Wenn Du einen Emacs zur Verfuegung hast:

Esc+"<" zum Dokument-Anfang
Strg+Space zum Setzen der Marke
Esc+">" zum Dokument-Ende
Strg+Alt+AltGr+"" (Das ist _kein_ Scherz ! ) rueckt den  Text ein.

Arbeite grade unter Windows, sonst wuerde ich's selbst machen.

Gruesse

Holger