opiniones de mi codigo

[jcescalantes]

Saludos colegas

este es mi primer sistema en forma con php, pero quisiera que calificaran mi programa.
les dejo mi codigo:

Código:

<?php
session_start(); 
include('../a_includes/errores_session.php');
$ordenbase = $filtro =	'';

if (isset($_GET["orden"])){ //ORDENAR POR
	$paso=$_GET["orden"];	
	switch ($paso) {
		case 'nombre':
			$ordenbase=" order by operador";
			break;
		case 'status':
			$ordenbase=" order by status";
			break;
		case 'telefono1':
			$ordenbase=" order by telefono1";
			break;
		case 'vencimiento':
			$ordenbase=" order by vencimiento";
			break;
		default:
			echo "ERROR";
			break;
	}
	unset($_GET["orden"]);
}else{
	$ordenbase=" order by operador";
}

if (isset($_GET["filtraop"])){ //FILTRAR
	$filtro=' where operador like "%' . $_GET["filtraop"] . '%"';
	unset($_GET["filtraop"]);
}

if (!$_POST && !$_GET) { //SI NO HAY NADA HTML
include('../a_includes/encabezadohead.php');
?>
<link rel="stylesheet" type="text/css" href="operadores.css" />
<link rel="stylesheet" type="text/css" href="../a_estilo/general.css" />
<script src="../a_scripts/jquery-1.9.1.js"></script>
<script src="operadores.js"></script>
<?php
include ('../a_includes/encabezadobody.php');
?>
<div class="menu">
    <ul>
        <li>
            <a href="#" id="nvoop">Nuevo operador</a>
        </li>
		<li>
            <a href="#" id="filop">Filtra operador</a>
        </li>
        <li>
            <a href="../principal.php">Salir</a>
        </li>
    </ul>
</div>
<br />
<br />

<div class="titcontenedor">
    Operadores
</div>

<table border='2' class="tabla">
    <tr class='tablafila'>
        <th class="tablaenc tablacelda" style="width: 4em;"></th>
        <th class="tablaenc tablacelda" style="width: 25em;"><a href=operadores.php?orden=nombre>Nombre</a></th>
        <th class="tablaenc tablacelda" style="width: 5em;"><a href=operadores.php?orden=status>Status</a></th>
        <th class="tablaenc tablacelda" style="width: 10em;"><a href=operadores.php?orden=telefono1>Telefono1</a></th>
        <th class="tablaenc tablacelda" style="width: 6em;"><a href=operadores.php?orden=vencimiento>Venc. Lic.</a></th>
        <th class="tablaenc tablacelda" style="width: 3em;"></th>
    </tr>
</table>

<div class="subcontenedor">
    <form class="nuevoregistro" method="post" action="operadores.php">
        <label>Nuevo Operador: </label>
        <input type="text" name="operadora" class="entrada valta" placeholder="Nombre del operador" maxlength="50" size="40"/>
        &nbsp;&nbsp;&nbsp;
        <input class="boton" type="submit" id="validanombrealta" value="Alta" />
        &nbsp;&nbsp;
        <input class="boton" type="reset" id="cancela" value="Cancelar" />
    </form>
	<form class="filtraregistro" method="get" action="operadores.php">
        <label>Filtra Operador: </label>
        <input type="text" name="filtraop" class="entrada valta" placeholder="Filtrar operador" maxlength="50" size="40"/>
        &nbsp;&nbsp;&nbsp;
        <input class="boton" type="submit" id="validanombrealta" value="Filtrar" />
        &nbsp;&nbsp;
        <input class="boton" type="reset" id="cancela1" value="Cancelar" />
    </form>
	

    <table border='2' class="tabla">
        <?php
		include ('../a_includes/conexion.php');
		$consulta = $bd -> query('SELECT operador,telefono1,vencimiento,status FROM loperadoresa ' . $filtro . $ordenbase);
		$numfilas = $consulta -> num_rows;
		$hoy = date('Y-m-d');
		if ($numfilas != 0) {
			for ($x = 0; $x < $numfilas; $x++) {
				$fila = $consulta -> fetch_object();
		?>
				<tr class="tablafila">
				<td class="tablacelda" style="width:4em;"><a href="ope_cambio.php?id=<?php echo $fila -> operador ?> ">Cambio</a></td>
				<td class="tablacelda" style="width:25em;"> <?php echo $fila -> operador ?> </td>
				<td class="tablacelda" style="width:5em;"> <?php echo $fila -> status ?> </td>
				<td class="tablacelda" style="width:10em;"> <?php echo $fila -> telefono1 ?></td>
				<?php if ($fila -> vencimiento <= $hoy) { ?>
					<td class="tablacelda" style="width:6em;background:#ff0000;"> <?php echo $fila -> vencimiento ?> </td>
				<?php } else { ?>
					<td class="tablacelda" style="width:6em"> <?php echo $fila -> vencimiento ?> </td>
				<?php } ?>
				<td class="tablacelda" style="width:3em"><a href="ope_baja.php?id=<?php echo $fila -> operador ?> ">Baja</a></td>
				</tr>
		<?php
			}
		}
        ?>
    </table>
</div>
</div> <!-- cierre de contenedor que esta en include("encabezqadobody.php")-->
</body>
</html>
<?php
}else{
	if (isset($_POST['operadora'])){
		$operadora=$_POST['operadora'];
		include ('../a_includes/conexion.php');
		$consulta = $bd->query('SELECT operador FROM loperadoresa WHERE operador="' . $operadora . '"');
		
		if ($consulta->num_rows!=0){
			echo '<script type="text/javascript">
			alert("Ese operador YA existe!!!");
			history.back();
			</script>';
		}else{
			mysqli_close($bd);
			header("Location: ope_alta.php?operador=$operadora");
			die();
		}
	}
}

lo que no me gusta, es que hago varias consultas, lo pondria en un arreglo, pero si algun otro usuario da de alta operadores, estos no se mostrarian, alguna sugerencia sobre esto y sobre todo el codigo.

gracias por su apoyo