Problem mit PHP code :/

Marv · 16. Januar 2009

Hey, ich habe einen PHP Code geschrieben, der alle Teiler einer Zahl, die man eingibt anzeigen soll.
Leider lädt die Seite ewig, wenn ich den Teiler berechnen will.
Hier erstmal der Code:

PHP

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
   	"http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<title>Teilerliste</title>
<meta http-equiv="content-type" content="text/html; charset=iso-8859-1">
<link rel="stylesheet" type="text/css" href="phpkid.css">
</head>
<body>
<h2>Teilerliste</h2><br />
<p>
<form action="teiler.php" method="POST">
Zahl: <input type="Text" name="zahl"><br />
<input type="submit" value="Teiler Berechnen">
<br />
<br />
</form>
<?php
if(isset($_POST['zahl'])) {
  $teiler = 2;
  $zahl = $_POST['zahl'];
  while($teiler < $zahl) {
    	$r = round($zahl / $teiler);
    	if($r == 0) {
      	echo "$teiler <br />\n";
      	$teiler = $teiler + 1;
    	}
  }
}
?>
</body>
</html>

Alles anzeigen

Also mit Teiler meine ich, die Zahlen durch die eine Zahl ohne Rest geteilt werden kann. ein Beispiel:
Zahl: 12
Ergebnisse:

2
3
4
6

In dem Code verhindere ich, dass 1 angezeigt wird, da jede Zahl durch 1 teilbar ist.

Die Fehlermeldung nach einiger Zeit:

Fatal error: Maximum execution time of 60 seconds exceeded in C:\xampp\htdocs\Php\Teilerliste\teiler.php on line 24

Leon · 16. Januar 2009

Überleg mal, was passiert, wenn du 12 eingibts und das Formular abschickst:
In der Schleife wird zuerst überprüft, ob 2 kleiner als 12 ist. Weil das Stimmt, wird in $r 12/2 = 6 gespeichert. Weiter passiert nichts, da $r == 0 FALSE ist. Jetzt beginnt die Schleife aber wieder von vorne. Also bleibt in ner Endlos-Schleife und wird vom Server nach 60 Sekunden durch die max_execution_time beendet.

btw: Wenn du das echt brauchst, empfehl ich ma die Modulo-Division per %

Marv · 16. Januar 2009

Hm.
Gibt es einen PHP Befehl, der eine Zahl auf Rest überprüft?
Also, dass überprüft wird ob rest da ist, wenn ja erhält man true wenn nein halt false ?

**TimWolla** · 16. Januar 2009

Ja, den Modulo: $zahl % $teiler.

Marv · 16. Januar 2009

Also wird dann auch true herausgegeben, wenn rest bleibt?

**TimWolla** · 16. Januar 2009

Der Modulo Operator gibt dir den Rest aus.

14 % 7 = 0

13 % 7 = 6

Marv · 16. Januar 2009

Müsste ich also nur das

PHP

<?phpif(isset($_POST['zahl'])) {  $teiler = 2;  $zahl = $_POST['zahl'];  while($teiler < $zahl) {        $r = round($zahl / $teiler);        if($r == 0) {          echo "$teiler <br />\n";          $teiler = $teiler + 1;        }  }}?>

In das

PHP

<?php
if(isset($_POST['zahl'])) {
  $teiler = 2;
  $zahl = $_POST['zahl'];
  while($teiler < $zahl) {
        $r = $zahl % $teiler;
        if($r == 0) {
          echo "$teiler <br />\n";
          $teiler = $teiler + 1;
        }
  }
}
?>

Alles anzeigen

Ändern?

**TimWolla** · 16. Januar 2009

PHP

<?php
if(isset($_POST['zahl'])) {
  $teiler = 2;
  $zahl = $_POST['zahl'];
  while($teiler < $zahl) {
        $r = $zahl % $teiler;
        if($r == 0) echo "$teiler <br />\n";
  $teiler++;
        }
  }
}
?>

Alles anzeigen

So.

Marv · 16. Januar 2009

aja xD man kann der wert ja auch mit ++ erhöhen x,x
schande über mich

ich glaube du hast ne klammer vergessen

PHP

<?php
if(isset($_POST['zahl'])) {
  $teiler = 2;
  $zahl = $_POST['zahl'];
  while($teiler < $zahl) {
        $r = $zahl % $teiler;
        if($r == 0) echo "$teiler <br />\n"; //Muss hinter die if-Bedingung nicht noch ein '{' ?
  $teiler++;
        }
  }
}
?>

Alles anzeigen

**TimWolla** · 17. Januar 2009

Nein, muss nciht unbedingt, kann aber. Hättest aber einfach testen können.

E:/ Ich hab aber eine zuviel gemacht:

PHP

<?php
if(isset($_POST['zahl'])) {
  $teiler = 2;
  $zahl = $_POST['zahl'];
  while($teiler < $zahl) {
        $r = $zahl % $teiler;
        if($r == 0) echo "$teiler <br />\n"; //Muss hinter die if-Bedingung nicht noch ein '{' ?
  $teiler++;
  }
}
?>

Alles anzeigen