[Erledigt] $_REQUEST übergabe geht nicht - Seite 2

Arne Drews · 29.01.2011, 19:18

Das stimmt. Hab nur keine Lus seinen Code jetzt komplett umzuschreiben...

Scheffi · 29.01.2011, 22:30

so geht das auch nicht hier einfach mal die php

PHP-Code:

  <?php

 
$suchenickname = (isset($_POST['submit']) && 'Spieler Suchen' == trim($_POST['submit']))? $_POST['suchenickname']: false;  

 
if(!isset($_SESSION["playerlistpage"])) { $_SESSION["playerlistpage"] =25; }

 
    if(isset($_GET['sps']))

    {

          $_SESSION["playerlistpage"] =$_GET['sps'];

    }

    if(isset($_GET['user']))

    {

    

    } else {

        if($_GET['order'] == 7) {            

            include("funktion/browse_ext.php");    

            

            

            $page = $_GET["page"];

            if(!isset($page))

               $page = 1;

            

            $eps = $_SESSION["playerlistpage"];

            $start = $page * $eps - $eps;

            

            $users= mysql_query("SELECT * FROM users ORDER BY level DESC LIMIT $start, $eps");

        }

        if($_GET['order'] == 8) {            

            

            

            $eps = 10;

                $users= mysql_query("SELECT * FROM users WHERE name LIKE '%$suchenickname%' ORDER BY Level DESC");

                $anzahlnamen = mysql_num_rows($users);

                echo"$suchenickname <____________";

        }

        

        

        ?>

        

        <div class="box">

          <div class="headlines">

            <h2><span>Spielerliste</span></h2>

            <a href="#" class="show-filter">Spielersuche</a>

          </div>

          <!-- filter -->

          <div class="filter">

             <form action="index.php?seite=playerlist&order=8" method="post">

            <input type="text" value="Spielername" name="suchenickname" />

            <input type="submit"  name="Spieler Suchen" value="Spieler Suchen" /> </form>

          </div>

          <!-- /filter -->

 
          <!-- table -->

          <table class="tab tab-drag">

            <tr class="top nodrop nodrag">

              <th class="dragHandle">&nbsp;</th>

              <th class="checkbox"><input type="checkbox" name="" value="" class="check-all" /></th>

              <th>Spielername</th>

              <th>Job</th>          

              <th>On/Off</th>

              <th class="action">Action</th>

            </tr>

            <?php 

            while($row1 = mysql_fetch_object($users)) {

            

            $on = $row1->online;

            if($row1->Job == 0) { $job='Arbeitslos'; }

            if($row1->Job == 1) { $job='Polizist'; }

            if($row1->Job == 2) { $job='Mechaniker'; }

            if($row1->Job == 3) { $job='Transporter'; }

            if($row1->Job == 4) { $job='Autoverkaeufer'; }

            if($row1->Job == 5) { $job='Chauffeur'; }

            if($row1->Job == 6) { $job='Sanitaeter'; }

            if($row1->Job == 7) { $job='Fast-Food Verkaeufer'; }

            if($row1->Job == 8) { $job='Fahrschullehrer'; }

            if($row1->Job == 9) { $job='Sicherheitsdienst'; }

            if($row1->Job == 10) { $job='Kurierfahrer'; }

            if($row1->Job == 11) { $job='Immobilienmakler'; }

            if($row1->Job == 12) { $job='Vermieter'; }

            if($row1->Job == 13) { $job='Müllmann'; }

            if($row1->Job == 14) { $job='Feuerwehr'; }

            if($row1->Job == 15) { $job='Hartz 4 Empfeanger'; }

            if($row1->Job == 16) { $job='Sweeper'; }

            

            ?>

            <tr>

              <td class="dragHandle">&nbsp;</td>

              <td class="checkbox"><input type="checkbox" name="" value="" /></td>

              <td><?php echo"<a href=index.php?seite=player&show=$row1->reg_id>$row1->name</a>"; ?></td>

              <td><?php echo"$job"; ?></td>  

              <?php if($on == 1) { echo'<td><font color="#006600">Online</td></font>'; } else { echo'<td><font color="#FF0000">Offline</font></td>'; } ?>  

              

              <td class="action">

                <a href="" title="Spieler Sperren" class="ico ico-delete">Delete</a>

                <a href="" title="Spieler bearbeiten" class="ico ico-edit">Edit</a>

              </td>

            </tr>

            

            <?php } ?>

          </table>

          <!-- /table -->

 
          <!-- box-action -->  

          <div class="tab-action">

            <select class="select">

              <option>Aktion wählen</option>

              <option>Markierte löschen</option>

            </select>

            <input type="submit" value="Aktion Starten" class="submit" />

          </div>

          <!-- /box-action -->

 
          <!-- /pagination -->

           <div class="pagination">

              <ul>

             <?php

             if($_GET['order'] == 7) {                            

                $user_delete_result = mysql_query("SELECT COUNT(*) FROM users");

                $menge = mysql_fetch_row($user_delete_result);

                $menge = $menge[0];

                

                $wieviele_pages = ceil($menge / $eps);

            

                echo blaetterfunktion($page, $wieviele_pages, "index.php?seite=playerlist&order=7");        

             }

            ?>

            </ul>

            

          </div>

          <!-- /pagination --> 

       </div>

       <!-- /box -->

        

        <?php

    }

        

 
?>

ich komm net weiter

Harry_X · 29.01.2011, 22:43

watt n chaos...

tr0y · 29.01.2011, 23:10

PHP-Code:

  if($row1->Job == 0) { $job='Arbeitslos'; } 

            if($row1->Job == 1) { $job='Polizist'; } 

            if($row1->Job == 2) { $job='Mechaniker'; } 

            if($row1->Job == 3) { $job='Transporter'; } 

            if($row1->Job == 4) { $job='Autoverkaeufer'; } 

            if($row1->Job == 5) { $job='Chauffeur'; } 

            if($row1->Job == 6) { $job='Sanitaeter'; } 

            if($row1->Job == 7) { $job='Fast-Food Verkaeufer'; } 

            if($row1->Job == 8) { $job='Fahrschullehrer'; } 

            if($row1->Job == 9) { $job='Sicherheitsdienst'; } 

            if($row1->Job == 10) { $job='Kurierfahrer'; } 

            if($row1->Job == 11) { $job='Immobilienmakler'; } 

            if($row1->Job == 12) { $job='Vermieter'; } 

            if($row1->Job == 13) { $job='Müllmann'; } 

            if($row1->Job == 14) { $job='Feuerwehr'; } 

            if($row1->Job == 15) { $job='Hartz 4 Empfeanger'; } 

            if($row1->Job == 16) { $job='Sweeper'; }

So nachdem ich mir die 2 Liter tränen aus den Augen gewischt hab, zeig ich dir mal wie du deine Finger behältst wenn das mal 500 Job-Arten werden ^^

PHP-Code:

  // Jobs, each indexed by auto incremental integer key

$possibleJobs = array('Arbeitslos', 'Polizist', 'Mechaniker', 'Transporter', 

               'Autoverkaeufer', 'Sanitaeter', 'Fast-Food Verkaeufer', 'Fahrschullehrer', 

               'Sicherheitsdienst', 'Kurierfahrer', 'Immobilienmakler', 'Vermieter', 'Müllmann', 

               'Feuerwehr', 'Hartz 4 Empfaenger', 'Sweeper');

 
// get job

if ( array_key_exists((integer)$row1->Job, $possibleJobs) ) $job = $possibleJobs[(integer)$row1->Job];

Noch besser, die daten über die Job bezeichnung in die Datenbank und mit JOIN per SQL abholen, spart arbeit in PHP, das da is nur mal nen "chaos fix".

Desweiteren solltest du mit hoher Dringlichkeitsstufe mal Kommentare benutzen und vor allem Eindeutig Identifizierbare $variablenNamen sowie Array-Keys, gewöhne es dir an, das du nach 3-4 Wochen nicht reinsehn in den Code, den Code zur Entschlüsselung ans BKA senden musst damit du noch durchblickst.

PHP-Code:

  $suchenickname = (isset($_POST['submit']) && 'Spieler Suchen' == trim($_POST['submit']))? $_POST['suchenickname']: false;

PHP-Code:

  if ( isset($_POST['submit'],$_POST['suchenickname']) ) {

   if ( strlen($_POST['suchenickname']) > 0 ) {

      $suchenickname = mysql_real_escape_string($_POST['suchenickname']);

   }

}

Warum auch immer du nach der präsenz eines Submit-Buttons suchst.. es sei dir gegönnt, was drin steht sollte für deine Operationen egal sein. Nimm sonst halt eindeutige Bezeichner für die Formular-Felder.

Scheffi · 30.01.2011, 00:27

Zitat:

Zitat von tr0y

PHP-Code:

  if($row1->Job == 0) { $job='Arbeitslos'; } 

            if($row1->Job == 1) { $job='Polizist'; } 

            if($row1->Job == 2) { $job='Mechaniker'; } 

            if($row1->Job == 3) { $job='Transporter'; } 

            if($row1->Job == 4) { $job='Autoverkaeufer'; } 

            if($row1->Job == 5) { $job='Chauffeur'; } 

            if($row1->Job == 6) { $job='Sanitaeter'; } 

            if($row1->Job == 7) { $job='Fast-Food Verkaeufer'; } 

            if($row1->Job == 8) { $job='Fahrschullehrer'; } 

            if($row1->Job == 9) { $job='Sicherheitsdienst'; } 

            if($row1->Job == 10) { $job='Kurierfahrer'; } 

            if($row1->Job == 11) { $job='Immobilienmakler'; } 

            if($row1->Job == 12) { $job='Vermieter'; } 

            if($row1->Job == 13) { $job='Müllmann'; } 

            if($row1->Job == 14) { $job='Feuerwehr'; } 

            if($row1->Job == 15) { $job='Hartz 4 Empfeanger'; } 

            if($row1->Job == 16) { $job='Sweeper'; }

So nachdem ich mir die 2 Liter tränen aus den Augen gewischt hab, zeig ich dir mal wie du deine Finger behältst wenn das mal 500 Job-Arten werden ^^

PHP-Code:

  // Jobs, each indexed by auto incremental integer key

$possibleJobs = array('Arbeitslos', 'Polizist', 'Mechaniker', 'Transporter', 

               'Autoverkaeufer', 'Sanitaeter', 'Fast-Food Verkaeufer', 'Fahrschullehrer', 

               'Sicherheitsdienst', 'Kurierfahrer', 'Immobilienmakler', 'Vermieter', 'Müllmann', 

               'Feuerwehr', 'Hartz 4 Empfaenger', 'Sweeper');

 
// get job

if ( array_key_exists((integer)$row1->Job, $possibleJobs) ) $job = $possibleJobs[(integer)$row1->Job];

Noch besser, die daten über die Job bezeichnung in die Datenbank und mit JOIN per SQL abholen, spart arbeit in PHP, das da is nur mal nen "chaos fix".

ich glaube mal leiber 500 mal ne zeile als alles in ein array den dan weiß ich gleich granicht mehr welche nummer was war und da die nummern sich je nach dem wie der pawno scripter das macht sich die nummer auch ändern können

und Kommentare ... hmm irgend wie nicht so meins ich selber weiß was ich wann geschrieben habe vielicht nicht sofort aber nach 2 min ganz sicher
so hab ich auch keine angst das einer was mit meinen Scripten anstellt wenn er nicht weiß was was ist

und noch was was habt ihr alle immer gegen meine anpassung wie ich weiß nciht jeder könnt immer heulen in mein codes

Harry_X · 30.01.2011, 00:33

Zitat:

Zitat von Scheffi

ich glaube mal leiber 500 mal ne zeile als alles in ein array den dan weiß ich gleich granicht mehr welche nummer was war und da die nummern sich je nach dem wie der pawno scripter das macht sich die nummer auch ändern können

das soll wohl n witz sein. Und was ist ein "pawno scripter"?

tr0y · 30.01.2011, 00:40

PHP-Code:

  $possibleJobs = array('Arbeitslos', 'Polizist', 'Mechaniker', 'Transporter',  
               'Autoverkaeufer', 'Sanitaeter', 'Fast-Food Verkaeufer', 'Fahrschullehrer',  
               'Sicherheitsdienst', 'Kurierfahrer', 'Immobilienmakler', 'Vermieter', 'Müllmann',  
               'Feuerwehr', 'Hartz 4 Empfaenger', 'Sweeper');

is the same as

PHP-Code:

  $possibleJobs = array(
   [0] => 'Arbeitslos', 
   [1] => 'Polizist', 
   [2] => 'Mechaniker', 
   [3] => 'Transporter',  
   [4] => 'Autoverkaeufer',
   [5] => 'Sanitaeter',
   [6] => 'Fast-Food Verkaeufer',
   [7] => 'Fahrschullehrer',  
   [8] => 'Sicherheitsdienst',
   [9] => 'Kurierfahrer',
   [10] => 'Immobilienmakler',
   [11] => 'Vermieter',
   [12] => 'Müllmann',  
   [13] => 'Feuerwehr',
   [14] => 'Hartz 4 Empfaenger',
   [15] => 'Sweeper');

Um die Zahl der zeilen gehts nicht, deine IF's sind 234989932894234 If-Blöcke, eigentlich ist das nur einer und der rest elseif, andere möglichkeit select(..) { case ...: break; default: ... }

Aber wie ich schon sagte: Diese "Daten" die du da aus deinem IF-Wolkenkratzer abholst pack lieber in eine datenbank und hol beim Query des jeweiligen Eintrags wo du den "Beruf" zugeordnet haben willst, gleich mit raus. ( SELECT ... JOIN ... )

Je öfter du den Quellcode erweiterst, je mehr Chaos entsteht da. Man dokumentiert seinen Quellcode, damit man jederzeit 2 Minuten, 2 Stunden oder 3 Wochen sparen kann. Damit man SOFORT weiß wo wie wann wer ist.

Das mit dem IF-Ding ist das kleinstes Übel.. Aber ein Anfang.

Scheffi · 30.01.2011, 04:00

Zitat:

Zitat von Harry_X

das soll wohl n witz sein. Und was ist ein "pawno scripter"?

das ist ein naja wie soll ich sagen vielicht sagt dir das was

http://sa-mp.com/

@tr0y danke versteh jetzt die logik

Wolla · 30.01.2011, 12:42

Zitat:

und Kommentare ... hmm irgend wie nicht so meins ich selber weiß was ich wann geschrieben habe vielicht nicht sofort aber nach 2 min ganz sicher
so hab ich auch keine angst das einer was mit meinen Scripten anstellt wenn er nicht weiß was was ist

Ah ja, aber uns hier mutest du zu, in deinem Spaghetticode irgendwas zu verstehen? Bei solchen Codewüsten bin ich nicht der einzige, der kommentarlos zum nächsten Beitrag geht. Sei froh, dass wenigstens tr0y sich die Mühe gemacht hat.

Abgesehen davon könntest du dir auch mit der deutschen Sprache etwas mehr Mühe geben. dann versteht man auch, was du willst.

tr0y · 30.01.2011, 13:38

Zitat:

Zitat von Wolla

Ah ja, aber uns hier mutest du zu, in deinem Spaghetticode irgendwas zu verstehen? Bei solchen Codewüsten bin ich nicht der einzige, der kommentarlos zum nächsten Beitrag geht. Sei froh, dass wenigstens tr0y sich die Mühe gemacht hat.

Abgesehen davon könntest du dir auch mit der deutschen Sprache etwas mehr Mühe geben. dann versteht man auch, was du willst.

Insbesondere im Quellcode

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Übergabe eines Werts aus einem DropDown in ein Input-Feld	MiBro	PHP Tipps 2010	14	26.02.2010 09:56
Variablen Übergabe innerhalb 2 php-Scripts	hakkinen-ffm	PHP Tipps 2009	4	15.11.2009 14:43
Fehler bei Übergabe von Variablen aus nem Formular...	MK1717	PHP Tipps 2009	3	01.06.2009 23:49
[Erledigt] Probleme mit "this.value" - Übergabe bei onchange	espressoboy	JavaScript, Ajax und mehr	3	27.01.2009 01:21
Fehler bei Übergabe von Variablen aus MySQL in Firefox	jensoweber	Datenbanken	0	13.07.2008 13:14
GET Übergabe durch vorherigen md5 hash checken		PHP Tipps 2005-2	4	27.10.2005 15:55
link - übergabe		PHP Tipps 2005-2	9	24.09.2005 22:14
Übergabe via URL => Scriptfehler		PHP Tipps 2005-2	9	24.09.2005 13:12
Parameter Übergabe	seven-12	PHP Tipps 2005-2	0	05.08.2005 11:06
POST Problem: Übergabe von Formulardaten	Asipak	PHP Tipps 2005-2	19	19.07.2005 11:12
Übergabe Cookie unter PHP 5.0 nicht möglich ??		PHP Tipps 2005	1	20.03.2005 18:33
Übergabe aus Textarea		PHP-Fortgeschrittene	2	01.03.2005 21:42
[Erledigt] Übergabe on Variablen über mehrere Seiten		PHP Tipps 2005	13	01.02.2005 21:13
Parameter übergabe bei meta http-equiv="refresh	MadCro	PHP Tipps 2004	3	19.07.2004 11:46
Probleme bei Übergabe von Parametern		PHP Tipps 2004	21	17.06.2004 11:08

29.01.2011, 19:18
Arne Drews Erfahrener Benutzer Registriert seit: 22.04.2009 Beiträge: 3.486 PHP-Kenntnisse: Anfänger	Das stimmt. Hab nur keine Lus seinen Code jetzt komplett umzuschreiben... __________________ Phantasie ist wichtiger als Wissen, denn Wissen ist begrenzt. Probleme kann man niemals mit derselben Denkweise lösen, durch die sie entstanden sind. Albert Einstein


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

29.01.2011, 22:30
Scheffi Benutzer Registriert seit: 16.10.2010 Beiträge: 37 PHP-Kenntnisse: Anfänger	so geht das auch nicht hier einfach mal die php PHP-Code: <?php $suchenickname = (isset($_POST['submit']) && 'Spieler Suchen' == trim($_POST['submit']))? $_POST['suchenickname']: false; if(!isset($_SESSION["playerlistpage"])) { $_SESSION["playerlistpage"] =25; } if(isset($_GET['sps'])) { $_SESSION["playerlistpage"] =$_GET['sps']; } if(isset($_GET['user'])) { } else { if($_GET['order'] == 7) { include("funktion/browse_ext.php"); $page = $_GET["page"]; if(!isset($page)) $page = 1; $eps = $_SESSION["playerlistpage"]; $start = $page * $eps - $eps; $users= mysql_query("SELECT * FROM users ORDER BY level DESC LIMIT $start, $eps"); } if($_GET['order'] == 8) { $eps = 10; $users= mysql_query("SELECT * FROM users WHERE name LIKE '%$suchenickname%' ORDER BY Level DESC"); $anzahlnamen = mysql_num_rows($users); echo"$suchenickname <____________"; } ?> <div class="box"> <div class="headlines"> <h2><span>Spielerliste</span></h2> <a href="#" class="show-filter">Spielersuche</a> </div> <!-- filter --> <div class="filter"> <form action="index.php?seite=playerlist&order=8" method="post"> <input type="text" value="Spielername" name="suchenickname" /> <input type="submit" name="Spieler Suchen" value="Spieler Suchen" /> </form> </div> <!-- /filter --> <!-- table --> <table class="tab tab-drag"> <tr class="top nodrop nodrag"> <th class="dragHandle"> </th> <th class="checkbox"><input type="checkbox" name="" value="" class="check-all" /></th> <th>Spielername</th> <th>Job</th> <th>On/Off</th> <th class="action">Action</th> </tr> <?php while($row1 = mysql_fetch_object($users)) { $on = $row1->online; if($row1->Job == 0) { $job='Arbeitslos'; } if($row1->Job == 1) { $job='Polizist'; } if($row1->Job == 2) { $job='Mechaniker'; } if($row1->Job == 3) { $job='Transporter'; } if($row1->Job == 4) { $job='Autoverkaeufer'; } if($row1->Job == 5) { $job='Chauffeur'; } if($row1->Job == 6) { $job='Sanitaeter'; } if($row1->Job == 7) { $job='Fast-Food Verkaeufer'; } if($row1->Job == 8) { $job='Fahrschullehrer'; } if($row1->Job == 9) { $job='Sicherheitsdienst'; } if($row1->Job == 10) { $job='Kurierfahrer'; } if($row1->Job == 11) { $job='Immobilienmakler'; } if($row1->Job == 12) { $job='Vermieter'; } if($row1->Job == 13) { $job='Müllmann'; } if($row1->Job == 14) { $job='Feuerwehr'; } if($row1->Job == 15) { $job='Hartz 4 Empfeanger'; } if($row1->Job == 16) { $job='Sweeper'; } ?> <tr> <td class="dragHandle"> </td> <td class="checkbox"><input type="checkbox" name="" value="" /></td> <td><?php echo"<a href=index.php?seite=player&show=$row1->reg_id>$row1->name</a>"; ?></td> <td><?php echo"$job"; ?></td> <?php if($on == 1) { echo'<td><font color="#006600">Online</td></font>'; } else { echo'<td><font color="#FF0000">Offline</font></td>'; } ?> <td class="action"> <a href="" title="Spieler Sperren" class="ico ico-delete">Delete</a> <a href="" title="Spieler bearbeiten" class="ico ico-edit">Edit</a> </td> </tr> <?php } ?> </table> <!-- /table --> <!-- box-action --> <div class="tab-action"> <select class="select"> <option>Aktion wählen</option> <option>Markierte löschen</option> </select> <input type="submit" value="Aktion Starten" class="submit" /> </div> <!-- /box-action --> <!-- /pagination --> <div class="pagination"> <ul> <?php if($_GET['order'] == 7) { $user_delete_result = mysql_query("SELECT COUNT(*) FROM users"); $menge = mysql_fetch_row($user_delete_result); $menge = $menge[0]; $wieviele_pages = ceil($menge / $eps); echo blaetterfunktion($page, $wieviele_pages, "index.php?seite=playerlist&order=7"); } ?> </ul> </div> <!-- /pagination --> </div> <!-- /box --> <?php } ?> ich komm net weiter

29.01.2011, 22:43
Harry_X Erfahrener Benutzer Registriert seit: 08.10.2009 Beiträge: 681 PHP-Kenntnisse: Anfänger	watt n chaos...

29.01.2011, 23:10
tr0y Erfahrener Benutzer Registriert seit: 26.07.2010 Beiträge: 4.874 PHP-Kenntnisse: Fortgeschritten	PHP-Code: if($row1->Job == 0) { $job='Arbeitslos'; } if($row1->Job == 1) { $job='Polizist'; } if($row1->Job == 2) { $job='Mechaniker'; } if($row1->Job == 3) { $job='Transporter'; } if($row1->Job == 4) { $job='Autoverkaeufer'; } if($row1->Job == 5) { $job='Chauffeur'; } if($row1->Job == 6) { $job='Sanitaeter'; } if($row1->Job == 7) { $job='Fast-Food Verkaeufer'; } if($row1->Job == 8) { $job='Fahrschullehrer'; } if($row1->Job == 9) { $job='Sicherheitsdienst'; } if($row1->Job == 10) { $job='Kurierfahrer'; } if($row1->Job == 11) { $job='Immobilienmakler'; } if($row1->Job == 12) { $job='Vermieter'; } if($row1->Job == 13) { $job='Müllmann'; } if($row1->Job == 14) { $job='Feuerwehr'; } if($row1->Job == 15) { $job='Hartz 4 Empfeanger'; } if($row1->Job == 16) { $job='Sweeper'; } So nachdem ich mir die 2 Liter tränen aus den Augen gewischt hab, zeig ich dir mal wie du deine Finger behältst wenn das mal 500 Job-Arten werden ^^ PHP-Code: // Jobs, each indexed by auto incremental integer key $possibleJobs = array('Arbeitslos', 'Polizist', 'Mechaniker', 'Transporter', 'Autoverkaeufer', 'Sanitaeter', 'Fast-Food Verkaeufer', 'Fahrschullehrer', 'Sicherheitsdienst', 'Kurierfahrer', 'Immobilienmakler', 'Vermieter', 'Müllmann', 'Feuerwehr', 'Hartz 4 Empfaenger', 'Sweeper'); // get job if ( array_key_exists((integer)$row1->Job, $possibleJobs) ) $job = $possibleJobs[(integer)$row1->Job]; Noch besser, die daten über die Job bezeichnung in die Datenbank und mit JOIN per SQL abholen, spart arbeit in PHP, das da is nur mal nen "chaos fix". Desweiteren solltest du mit hoher Dringlichkeitsstufe mal Kommentare benutzen und vor allem Eindeutig Identifizierbare $variablenNamen sowie Array-Keys, gewöhne es dir an, das du nach 3-4 Wochen nicht reinsehn in den Code, den Code zur Entschlüsselung ans BKA senden musst damit du noch durchblickst. PHP-Code: `$suchenickname = (isset($_POST['submit']) && 'Spieler Suchen' == trim($_POST['submit']))? $_POST['suchenickname']: false;` PHP-Code: `if ( isset($_POST['submit'],$_POST['suchenickname']) ) { if ( strlen($_POST['suchenickname']) > 0 ) { $suchenickname = mysql_real_escape_string($_POST['suchenickname']); } }` Warum auch immer du nach der präsenz eines Submit-Buttons suchst.. es sei dir gegönnt, was drin steht sollte für deine Operationen egal sein. Nimm sonst halt eindeutige Bezeichner für die Formular-Felder. __________________ Lasse mir ohne Anwendung von Gewalt Dinge schenken, Amazon weiß darüber bald mehr. Geändert von tr0y (29.01.2011 um 23:20 Uhr).

30.01.2011, 00:40
tr0y Erfahrener Benutzer Registriert seit: 26.07.2010 Beiträge: 4.874 PHP-Kenntnisse: Fortgeschritten	PHP-Code: `$possibleJobs = array('Arbeitslos', 'Polizist', 'Mechaniker', 'Transporter', 'Autoverkaeufer', 'Sanitaeter', 'Fast-Food Verkaeufer', 'Fahrschullehrer', 'Sicherheitsdienst', 'Kurierfahrer', 'Immobilienmakler', 'Vermieter', 'Müllmann', 'Feuerwehr', 'Hartz 4 Empfaenger', 'Sweeper');` is the same as PHP-Code: $possibleJobs = array( [0] => 'Arbeitslos', [1] => 'Polizist', [2] => 'Mechaniker', [3] => 'Transporter', [4] => 'Autoverkaeufer', [5] => 'Sanitaeter', [6] => 'Fast-Food Verkaeufer', [7] => 'Fahrschullehrer', [8] => 'Sicherheitsdienst', [9] => 'Kurierfahrer', [10] => 'Immobilienmakler', [11] => 'Vermieter', [12] => 'Müllmann', [13] => 'Feuerwehr', [14] => 'Hartz 4 Empfaenger', [15] => 'Sweeper'); Um die Zahl der zeilen gehts nicht, deine IF's sind 234989932894234 If-Blöcke, eigentlich ist das nur einer und der rest elseif, andere möglichkeit select(..) { case ...: break; default: ... } Aber wie ich schon sagte: Diese "Daten" die du da aus deinem IF-Wolkenkratzer abholst pack lieber in eine datenbank und hol beim Query des jeweiligen Eintrags wo du den "Beruf" zugeordnet haben willst, gleich mit raus. ( SELECT ... JOIN ... ) Je öfter du den Quellcode erweiterst, je mehr Chaos entsteht da. Man dokumentiert seinen Quellcode, damit man jederzeit 2 Minuten, 2 Stunden oder 3 Wochen sparen kann. Damit man SOFORT weiß wo wie wann wer ist. Das mit dem IF-Ding ist das kleinstes Übel.. Aber ein Anfang. __________________ Lasse mir ohne Anwendung von Gewalt Dinge schenken, Amazon weiß darüber bald mehr.