Ho risolto così, è corretto?

codice:
if (isset($_GET["op"]))
    {
       echo "<tr><td colspan=2>";
	   switch ($_GET["op"])
	         {
		        case 'save':
				$action = 'op=save';
				$title = 'Inserisci materiale';
				$value = 'Inserisci';
				include_once ("functions/function_insert.inc.php");
				break;
				
				case 'delete':
				include_once ("functions/function_delete.inc.php");
		        break;
				
				case 'edit':
				include_once ("functions/function_edit.inc.php");
			    break;
				
				default:
				$action = 'op=save';
				$title = 'Inserisci materiale';
				$value = 'Inserisci';
				include_once ("functions/function_insert.inc.php");
				break;
				
			 }
	   echo "</td></tr>";
    }
  else
    {
	   $action = 'op=save';
	   $title = 'Inserisci materiale';
	   $value = 'Inserisci';
	}