Skip to main content
Code Review

Return to Answer

deleted 14 characters in body
Source Link
phaberest
  • 184
  • 3
  • 9

Already gave in the wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
} else {
 echo "Please enter a number between 1 and 20"; 
}

A single for is enough. In the if statement we check if the $_POST['number'] exists, if it's actually a number and if it is between 1 and 20.

Already gave in the wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
} else {
 echo "Please enter a number between 1 and 20"; 
}

A single for is enough. In the if statement we check if the $_POST['number'] exists, if it's actually a number and if it is between 1 and 20.

Already gave in the wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
} else {
 echo "Please enter a number between 1 and 20"; 
}

A single for is enough. In the if statement we check if the $_POST['number'] exists, if it's actually a number and if it is between 1 and 20.

edited body
Source Link
phaberest
  • 184
  • 3
  • 9

Already gave in the wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
 } else {
 echo "Please enter a number between 1 and 20"; 
 }

A single for is enough. In the if statement we check if the $POST_['number']$_POST['number'] exists, if it's actually a number and if it is between 1 and 20.

Already gave in the wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
 } else {
 echo "Please enter a number between 1 and 20"; 
 }

A single for is enough. In the if statement we check if the $POST_['number'] exists, if it's actually a number and if it is between 1 and 20.

Already gave in the wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
 } else {
 echo "Please enter a number between 1 and 20"; 
 }

A single for is enough. In the if statement we check if the $_POST['number'] exists, if it's actually a number and if it is between 1 and 20.

replaced http://stackoverflow.com/ with https://stackoverflow.com/
Source Link

Already gave in the wrong placed question wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
 } else {
 echo "Please enter a number between 1 and 20"; 
 }

A single for is enough. In the if statement we check if the $POST_['number'] exists, if it's actually a number and if it is between 1 and 20.

Already gave in the wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
 } else {
 echo "Please enter a number between 1 and 20"; 
 }

A single for is enough. In the if statement we check if the $POST_['number'] exists, if it's actually a number and if it is between 1 and 20.

Already gave in the wrong placed question, anyway here is my solution: codepad.viper-7.com/zTLliA

Copy&paste is never a good idea.

if (isset($_POST['number']) && is_numeric($_POST['number']) && (($_POST['number'] > 0) && ($_POST['number'] <= 20))) { 
 $number = $_POST['number'];
 for ( $counter = 1 ; $counter <=10; $counter++ ) {
 $temp = $number * $counter ;
 echo "$number x $counter = $temp<br />"; 
 }
 } else {
 echo "Please enter a number between 1 and 20"; 
 }

A single for is enough. In the if statement we check if the $POST_['number'] exists, if it's actually a number and if it is between 1 and 20.

Improved explanation.
Source Link
phaberest
  • 184
  • 3
  • 9
Loading
Source Link
phaberest
  • 184
  • 3
  • 9
Loading
lang-php

AltStyle によって変換されたページ (->オリジナル) /