Tengo un código que está funcionando bien pero se está alargando en una parte porque repito muchas veces algunas funciones y llamados a éstas, como estoy aprendiendo y soy algo novato aún necesito ayuda para mejorar o acortar el código pero que siga funcionando bien.
Procedo a mostrar la parte del código repetitiva:
Bueno la pagina consiste básicamente en un mapa, el cual tiene marcadores, los cuales cambian de color cuando se cambian las id desde el webservice(del que consumi los datos con php). tengo las siguientes funciones que me permiten poner los marcadores cambiando el icono a otro de otro color:
function ActualMarker1(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-violet.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[0].lat, markers[0].lng], {icon: rIcon} )
.bindPopup( 'Evasión Control' ).addTo( cities );
}
}
function ActualMarker2(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-green.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[0].lat, markers[0].lng], {icon: rIcon} )
.bindPopup( 'Emisiones Diesel' ).addTo( cities );
}
}
function ActualMarker3(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-yellow.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[0].lat, markers[0].lng], {icon: rIcon} )
.bindPopup( 'Control Taximetros' ).addTo( cities );
}
}
function ActualMarker4(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-violet.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[1].lat, markers[1].lng], {icon: rIcon} )
.bindPopup( 'Evasión Control' ).addTo( cities );
}
}
function ActualMarker5(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-green.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[1].lat, markers[1].lng], {icon: rIcon} )
.bindPopup( 'Emisiones Diesel' ).addTo( cities );
}
}
function ActualMarker6(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-yellow.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[1].lat, markers[1].lng], {icon: rIcon} )
.bindPopup( 'Control Taximetros' ).addTo( cities );
}
}
function ActualMarker7(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-violet.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[2].lat, markers[2].lng], {icon: rIcon} )
.bindPopup( 'Evasión Control' ).addTo( cities );
}
}
function ActualMarker8(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-green.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[2].lat, markers[2].lng], {icon: rIcon} )
.bindPopup( 'Emisiones Diesel' ).addTo( cities );
}
}
function ActualMarker9(){
rIcon = L.icon({
iconUrl: "marcadores/marker-icon-yellow.png",
})
for ( var i=0; i < markers.length; ++i ){
L.marker( [markers[2].lat, markers[2].lng], {icon: rIcon} )
.bindPopup( 'Control Taximetros' ).addTo( cities );
}
}
Y con el siguiente código en php, llamo las funciones, que permite que luego al variar las id (que la id en este caso se llama ot) en el webservice y al recargar la página, los iconos cambian de color.
<?php
for($i=0; $i<count($f_result); $i++){
if($f_result[0]->ot == 95)
{
echo "<script>";
echo "ActualMarker1();";
echo "</script>";
}elseif($f_result[0]->ot == 98)
{
echo "<script>";
echo "ActualMarker2();";
echo "</script>";
}elseif($f_result[0]->ot == 102)
{
echo "<script>";
echo "ActualMarker3();";
echo "</script>";
}
if($f_result[1]->ot == 95)
{
echo "<script>";
echo "ActualMarker4();";
echo "</script>";
}elseif($f_result[1]->ot == 98)
{
echo "<script>";
echo "ActualMarker5();";
echo "</script>";
}elseif($f_result[1]->ot == 102)
{
echo "<script>";
echo "ActualMarker6();";
echo "</script>";
}
if($f_result[2]->ot == 95)
{
echo "<script>";
echo "ActualMarker7();";
echo "</script>";
}elseif($f_result[2]->ot == 98)
{
echo "<script>";
echo "ActualMarker8();";
echo "</script>";
}elseif($f_result[2]->ot == 102)
{
echo "<script>";
echo "ActualMarker9();";
echo "</script>";
}else{
echo "esto no esta funcionando bien";
}
}
?>
Y como dije anteriormente lo que necesito es que no sea tan repetitivo, algo así como pulir el código, espero puedan ayudarme, de antemano muchas gracias.