Currently, I have the following code:
if(isset($_GET['mid']) && !empty($_GET['mid'])) {
    $mid = $_GET['mid'];
    if(is_numeric($mid) && $mid > 0) {
        if(isset($_GET['op']) && !empty($_GET['op'])) {
            $op = $_GET['op'];
            if($op == 'info') {
            }
            if($op == 'cast') {
            }
        }
    }
}
But I think it's too "complex" with if statements inside if statements, etc...
Would you handle it differently? How would you make it simpler?
I would use filter_input with the FILTER_SANITIZE_NUMBER_FLOAT filter for mid. Something like that :
$mid = filter_input(INPUT_GET, 'mid', FILTER_SANITIZE_NUMBER_FLOAT);
$op = filter_input(INPUT_GET, 'op');
if($mid > 0){
  switch($op){
    case 'info':
      // Some code
      break;
    case 'cast':
      // Some more code
      break;
    default:
      break;
  }
}
                        If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With