recorrido de array multidimensional

[bjlv00]

He escrito el siguiente código que tiene que recorrer un array de 5 filas y $asientos columnas. Tiene que buscar el número mayor del array, cambiar su valor por 0 y buscar el número mayor de los que quedan $asientos veces. El problema es que sólo lo hace una vez y no sé porqué ¿Me podéis ayudar?

Código PHP:

  for ($k=0; $k<=$asientos; $k++){ 
for ($i=0; $i<=4; $i++){ 
for ($j=0; $j<=$asientos; $j++){ 
if ($mayor<$votos[$i][$j]){ 
$mayor=$votos[$i][$j]; 
$votos[$i][$j]=0; 
break; 
} 
} 
} 
}  
   

Gracias.

[Vallu]

Eso es por culpa del break: http://php.net/manual/es/control-structures.break.php

[bjlv00]

Iniciado por Vallu

Eso es por culpa del break: [url]http://php.net/manual/es/control-structures.break.php[/url]

Gracias por la respuesta, pero no. Lo he quitado y sigue igual.

Un saludo.

[Vallu]

Código PHP:

  
for($i=0;$i<5;$i++){
        $bool=false;
    for(j=0;$j<$asientos;$j++){
        $mayor=max($votos[$i]);
        if(($mayor==$votos[$i][$j])&&($bool==false)){ 
            $votos[$i][$j]=0;
            $mayor=max($votos[$i]);
                        $bool=true;
        }
    }
}  
   

Esto lo que en teoría hace es buscar el numero mayor dentro de cada porción de array y sustituirlo por 0, luego recoges en la variable $mayor el nuevo numero mas grande de esa porcion del array.
No sé si es lo que pides pero es lo que entendí.

[Nexus10]

Hola bjlv00.

Así, de un primer vistazo, veo varios errores en tu código.

El primer bucle, el que quieres que repita $asientos veces, lo está repitiendo $asientos +1 veces porque empiezas desde cero. Debes poner $k<$asientos o empezar desde $k=1.

La instrucción break que tienes dentro del if no sirve para nada. Una instrucción break debe ir dentro de un bucle for.

Con respecto a lo que preguntas no he probado el código todavía. Voy a ver si lo pruebo.

Un saludo.

[abimaelrc]

Es dificil decirte que podrías hacer, escribe lo que tengas en las variables.

[Nexus10]

Perdón, lo que he puesto del break es una tontería, se me ha ido la olla.

[Nexus10]

¿Podría ser algo así?

Código PHP:

  for ($k=0; $k<$asientos; $k++){  
  for ($i=0; $i<5; $i++){  
    for ($j=0; $j<$asientos; $j++){  
      if ($mayor<$votos[$i][$j]){  
        $mayor=$votos[$i][$j]; 
        $x=$i; 
        $y=$j;         
      }      
    }  
  } 
  $votos[$x][$y]=0; 
  $mayor=0; 
}  
   

Creo que tu problema estaba en la variable $mayor, que no la inicializas al acabar el bucle más externo y después de la primera iteración ya no entra en el if.

[bjlv00]

Iniciado por Vallu

Código PHP:

   
for($i=0;$i<5;$i++){ 
        $bool=false; 
    for(j=0;$j<$asientos;$j++){ 
        $mayor=max($votos[$i]); 
        if(($mayor==$votos[$i][$j])&&($bool==false)){  
            $votos[$i][$j]=0; 
            $mayor=max($votos[$i]); 
                        $bool=true; 
        } 
    } 
}  
   

Esto lo que en teoría hace es buscar el numero mayor dentro de cada porción de array y sustituirlo por 0, luego recoges en la variable $mayor el nuevo numero mas grande de esa porcion del array.
No sé si es lo que pides pero es lo que entendí.

Muchas gracias por tu interés, pero tampoco funcioma. Sólo cambia por cero el mayor valor de cada columna. Además, no puedo usar la función max(). Llevo horas y horas intentando descifrar el problema y no soy capaz. Me estoy desesperando.

Un saludo y muchas gracias.

[Nexus10]

Hola bjlv00,
¿Has probado el código que he puesto? Es que como hemos posteado casi a la vez no sé si lo has visto.

Un saludo.

[bjlv00]

Iniciado por Nexus10

Hola bjlv00.

Así, de un primer vistazo, veo varios errores en tu código.

El primer bucle, el que quieres que repita $asientos veces, lo está repitiendo $asientos +1 veces porque empiezas desde cero. Debes poner $k<$asientos o empezar desde $k=1.

La instrucción break que tienes dentro del if no sirve para nada. Una instrucción break debe ir dentro de un bucle for.

Con respecto a lo que preguntas no he probado el código todavía. Voy a ver si lo pruebo.

Un saludo.

Muchas gracias. Tienes razón en todo lo que dices. Sigo sin solución.

Quizás debería explicar un poco mejor el problema:

La tabla consta de 5 filas y $asientos columnas. Debo buscar el mayor valor de toda la tabla y sustituirla por un 0. Luego, volver a empezar el recorrido por toda la tabla buscando el mayor, ya que el primero de todos ya no existe, por ser 0. Así, sucesivamente, hasta encontrar los $asientos valores mayores.

Un saludo a todos y muchísimas gracias.

[Nexus10]

¿Cuando vuelves a repetir buscando el mayor ya no lo tienes que sustituir por cero?. Entonces no está bien el código que te he puesto. Voy a volver a pensarlo.

[bjlv00]

Iniciado por Nexus10

Hola bjlv00,
¿Has probado el código que he puesto? Es que como hemos posteado casi a la vez no sé si lo has visto.

Un saludo.

¡Sí! Nexux10, funciona! No sé cómo agradecertelo. Muchísimas gracias.

Un saludo.

[Nexus10]

Ah! Vale, entonces lo había entendido bien, :).

De nada, un saludo.