1

Estoy intentado incluir un código en function para que sea más optimo pero no consigo visualizar los valores el código en cuestión es este.

    function Ciudad() {
     if ($Datos[0]["Valor1"]>10) {$txte = " Ciudad, ";}
     if ($Datos[1]["Valor1"]>10) {$txte .= "Ciudad2, ";}
     if ($Datos[2]["Valor1"]>10) {$txte .= "Ciudad3, ";}
     if ($Datos[3]["Valor1"]>10) {$txte .= "Ciudad4, ";}
     if ($Datos[4]["Valor1"]>10) {$txte .= "Ciudad5, ";}
     if ($Datos[5]["Valor1"]>10) {$txte .= "Ciudad6, ";}
     if ($Datos[6]["Valor1"]>10) {$txte .= "Ciudad7, ";}
     if ($Datos[7]["Valor1"]>10) {$txte .= "Ciudad8, ";}
     if ($Datos[8]["Valor1"]>10) {$txte .= "Ciudad9, ";}
     if ($Datos[9]["Valor1"]>10) {$txte .= "Ciudad10, ";}
     if ($Datos[10]["Valor1"]>10) {$txte .= "Ciudad11, ";}
     if ($Datos[11]["Valor1"]>10) {$txte .= "Ciudad12, ";}
     if ($Datos[12]["Valor1"]>10) {$txte .= "Ciudad13, ";}
     if ($Datos[13]["Valor1"]>10) {$txte .= "Ciudad14, ";}
     if ($Datos[14]["Valor1"]>10) {$txte .= "Ciudad15, ";}
     if ($Datos[15]["Valor1"]>10) {$txte .= "Ciudad16";}
     return $txte;
    }

Al Igual he pensado optimizarlo dado que los valores los saco de $Datos[0]["Valor1"] y la ciudad de $Datos[0]["Nombre"] había pensado en un bucle que juntase la información que cumpla con el requisito de > 10 algo así...

    function Ciudad($id, $valor) {
    for ($i = $id; $i <  16; $i++) { 
        for ($j = $valor; $j < 17; $j++) { 
            if ($Datos[$i]["Valor".($j)]>10) {
            $ciudades .= $Datos[$i]["Nombre"]; } }
    }
     return $ciudades;
    }
10
  • Necesitas acceder al Array de datos dentro de la función, puedes pasarlo como parámetro o acceder a él como variable global. Una duda, accedes al valor siempre con "Valor 1" o es variable?
    – Carmen
    Commented el 9 nov. 2018 a las 18:59
  • @Carmen cómo podría hacer eso? Es variable el Valor 1 por eso en el código lo puse como $Datos[$i]["Valor".($j)]
    – Vicente
    Commented el 9 nov. 2018 a las 19:17
  • Te doy la bienvenida a Stackoverflow. Cuando veo código como el tuyo la primera palabra que se me ocurre es justo una que mencionas en la pregunta: Optimizar. Tu código necesita ciertamente una optimización seria, pero si no se conoce bien el contexto es imposible sugerir algo que resuelva el problema de raíz. A veces para optimizar hay que retroceder, mejorando por ejemplo la forma en que se construye el objeto $Datos, o aún mucho más, porque a veces hay que optimizar incluso el origen mismo de los datos.
    – A. Cedano
    Commented el 9 nov. 2018 a las 19:19
  • @A.Cedano creo que en cuanto a optimizar con el código que estoy mencionado se está optimizando primero lo trabajo a lo brusco y después optimizo cuando funciona la cuestión es que al no saber donde falla no podía seguir.
    – Vicente
    Commented el 9 nov. 2018 a las 19:35
  • No sé. Por ejemplo la comparación > 10 me hace dudar sobre tal optimización. Como ya dije, sin conocer el contexto opinar sobre optimización es aventurarse. Pero, imaginemos un contexto donde sólo necesitas la información para esto, y que los datos vienen de una base de datos (o de otra fuente). Si sólo necesitas los que son mayor que 10, y traes todos los datos para luego desechar una parte, el código no está optimizado. Es un ejemplo que pongo para decir que, en un contexto como ese tu código no está optimizado porque traes datos sin filtrar para hacerlo luego, pudiéndolo hacer antes.
    – A. Cedano
    Commented el 9 nov. 2018 a las 19:40

2 respuestas 2

0

Le puedes pasar tus datos como parámetro. He declarado la variable que retornas en vacío para que siempre sea accesible aunque no se encuentren datos para añadir.

$Datos[0]["Valor1"]=3;
$Datos[0]["Nombre"]="Pekin";
$Datos[0]["Valor2"]=13;
$Datos[0]["Nombre"]="Tokio";
$Datos[1]["Valor1"]=3;
$Datos[1]["Nombre"]="Londres";
$Datos[1]["Valor2"]=13;
$Datos[1]["Nombre"]="Paris";

echo Ciudad(0,1,$Datos);
echo "<br>";
echo Ciudad(1,1,$Datos);

function Ciudad($id, $valor,$Datos) {
   $ciudades ="";
   for ($i = $id; $i < 2; $i++) { 
            for ($j = $valor; $j < 3; $j++) { 
                if ($Datos[$i]["Valor".$j]>10) {
                $ciudades .= $Datos[$i]["Nombre"]; } 
            }
  }
  return $ciudades;
}

Resultado:

TokioParis
Paris
12
  • Lo he probado y no consigo mostrar los valores hecho var_dump y me sale string 0
    – Vicente
    Commented el 9 nov. 2018 a las 19:36
  • @Vicente lo he probado y funciona, edito con un ejemplo completo. Comprueba los parámetros que pasas a la función.
    – Carmen
    Commented el 9 nov. 2018 a las 20:01
  • Creo que lo que el OP necesita se puede hacer con dos o tres líneas de código con alguna función como array_map combinada con otra si fuera necesario. Pero en el planteamiento no se explica con claridad lo que necesita ni cómo está estructurado el array internamente. Tu solución, en caso de que funcione, sería un parche no optimizado a una solución del problema planteada también de una forma no óptima.
    – A. Cedano
    Commented el 9 nov. 2018 a las 20:06
  • @A.Cedano es muy posible que se pueda hacer de otros modos, pero pregunta por qué no le resulta convertir el código a función, y el uso de bucles es una buena simplificación de código. Cambiar el planteamiento puede ser interesante, pero sería una cuestión diferente.
    – Carmen
    Commented el 9 nov. 2018 a las 20:22
  • 1
    Si lo que necesitas son valores separados por comas, puedes crear un array aparte con esos valores y luego usar implode.
    – A. Cedano
    Commented el 9 nov. 2018 a las 20:32
0

En el siguiente código se crea un array tomando el valor de la clave Nombre del array original en los casos en que el valor de la clave Valor del mismo array original sea mayor que 10.

No sé si es exactamente lo que quieres, pero, sea lo que sea, intuyo que es algo que se puede hacer de una forma muy simple según tus requerimientos.

Para imprimir la lista de ciudades separadas por ,, aplicamos implode sobre $partialData.

El código incluye un simulacro sobre el posible estado original de tu array.

$datos=array(
                array("Nombre"=>"Madrid",   "Valor"=>10),
                array("Nombre"=>"Tokio",    "Valor"=>15),
                array("Nombre"=>"New York", "Valor"=>5),
                array("Nombre"=>"París",    "Valor"=>10),
                array("Nombre"=>"Londres",  "Valor"=>11),
                array("Nombre"=>"Berlín",   "Valor"=>3)   
            );

foreach ($datos as $item){
    if ($item["Valor"]>10){
        $partialData[]=$item["Nombre"];
    }
}
echo implode(",",$partialData);

Salida:

Tokio,Londres

EDICIÓN:

Si en cambio el array es así:

$datos=array(
                array("Nombre"=>"Madrid",   "Valor1"=>10),
                array("Nombre"=>"Tokio",    "Valor2"=>15),
                array("Nombre"=>"New York", "Valor3"=>5),
                array("Nombre"=>"París",    "Valor4"=>10),
                array("Nombre"=>"Londres",  "Valor5"=>11),
                array("Nombre"=>"Berlín",   "Valor6"=>3)   
            );

Sólo hay que implementar un contador para construir el nombre de la clave dinámicamente:

$i=1;

foreach ($datos as $item){
    if ($item["Valor$i"]>10){
        $partialData[]=$item["Nombre"];
    }
    $i++;
}
echo implode(",",$partialData);

La salida será la misma. Si también las clave Nombre tuviesen un número secuencial, agregas el valor aquí: $partialData[]=$item["Nombre$i"];, aunque es una pena que los datos estén así, pudiendo tener un diseño de datos más fácil y estructurado...


NOTA: Si se trata de algo más complejo, has de saber que PHP cuenta con varias decenas de funciones sobre arrays capaces de hacer cualquier tipo de operación complicada de una forma sencilla. Muchas de ellas, combinadas con otras funciones del lenguaje, hacen posible cualquier tipo de comparación, operación, cálculo, construcción de nuevos arrays, etc.

7
  • La cuestión seria poder manipular el valor de las variables Valor no será Valor sera Valor1, Valor2 así lo tengo estructurado son diferentes campos por separado.
    – Vicente
    Commented el 9 nov. 2018 a las 22:30
  • @Vicente en el ejemplo de tu pregunta los campos se llaman todos igual: Valor, por eso te pedí una muestra de los datos reales. De todos modos, si los campos tiene por desgracia o por mal diseño un nombre incremental, puedes implementar un contador y construir en base a eso el nombre de la clave. Pero, viendo que esos datos vienen de una base de datos, es evidente que tu modelo de datos necesita ser revisado y mejorado. Ahora editaré la respuesta, mostrando una forma de hacerlo con un contador.
    – A. Cedano
    Commented el 9 nov. 2018 a las 22:38
  • Si mil disculpas no me explique con claridad, los campos son incrementales porqué pensé que sería la forma más básica si no serian campos con diferentes nombres que van a hacer una funcionalidad guardar valores por separado de ahí a que no deban ser iguales. Para que te hagas una idea Se compone de id, nombre, valor1, valor2, valor3, valor4, valor5, valor6, valor7, valor8, confirmado
    – Vicente
    Commented el 9 nov. 2018 a las 22:46
  • @Vicente viendo que esos datos vienen de una base de datos, me temo que voy a tener razón en lo que te dije hace un rato. Creo que empezaste diseñando mal el mismo modelo de datos (en la base de datos). Da la impresión de que quisiste meter una serie de valores en una sola tabla cuando pudiste haber implementado un modelo relacional que use varias tablas y exprese las relaciones mediante índices. De ese modo, tus columnas no se llamarían Valor1, Valor2, Valor99999 o Nombre1, Nombre2, Nombre9999, sino que se llamarían Valor y Nombre y tendrías un array asociativo con tus datos ...
    – A. Cedano
    Commented el 9 nov. 2018 a las 22:49
  • ... incluso ya organizados sin necesidad que tener que recurrir a esto o en caso de tener que recurrir a esto, hacerlo como se indica en la 1ª parte de la respuesta. Como te dije, en cuanto hay algo que se repite al estilo valor1, valor2, valor3, valor4, valor5, valor6, valor7, valor8 es una clara evidencia de un mal diseño de datos.
    – A. Cedano
    Commented el 9 nov. 2018 a las 22:50

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.