MI CODIGO ES ESTE: funciona ... pero creo que no es lo conveniente. Por lo menos es lo que me imagino

.
Si pueden ayudarme a resolver este problema por favor se los agradeceria
// ACA EMPIEZA EL PRIMER SELECT QUE ELIGE LA MARCA Y SALTA NUEVAMENTE CON EL ID DE LA MISMA PARA BUSCAR EL MODELO POR MARCA
************************************************** *******
<select name="menu1" class="verdana" onChange="MM_jumpMenu('parent',this,0)">
<option value="index.php" selected>Seleccione la marca</option>
<?
require("link.php");
if($link){
$query = "SELECT * from marcas";
$result = mysql_db_query($base,$query,$link);
while($row=mysql_fetch_array($result)){
if ($row[id_marca]==$marca_actual) {
?>
<option value="index.php?marca_actual=<? print("$row[id_marca]");?>" selected><? print("$row[nombre_marca]");?></option>
<?
} else {
?>
<option value="index.php?marca_actual=<? print("$row[id_marca]");?>"><? print("$row[nombre_marca]");?></option>
<?
}
}
}
?>
</select>
************************************************** *******
//ACA EMPIEZA EL 2DO SELECT QUE IMPRIME LOS MODELOS CON OTRO QUERY
************************************************** *******
<select name="id_modelo" class="verdana" id="id_modelo">
<option value="0" selected>Seleccione Modelo</option>
<?
if($link){
$query = "SELECT * from modelos WHERE Id_marca=$marca_actual";
$result = mysql_db_query($base,$query,$link);
while($row=mysql_fetch_array($result)){
?>
<option value="<? print("$row[id_modelo]");?>"><? print("$row[nombre_modelo]");?></option>
<?
}
}
?>
</select>
************************************************** *******