[SOLUCIONADO] Fallo en mostrar n pares de array

[HackID1]

Hola a toda la comunidad!.

Bueno posteo la pregunta del ejercicio que estoy realizando:

2.-) Se considera un array tab como el siguiente: 4, -5, 5, 7, 1,-8 , 9, -10, 4, 15
Guardar en un array tabPar aquellos valores de tab que sean pares y en un
array tabImpar aquellos valores impares.
Visualizar ambos arrays: tabPar y tabImpar.

La solución que hice es esta.

Código PHP:

//Variables
$tab = array( 4, -5, 5, 7, 1, -8, 9, -10, 4, 15);
 
$tabPar = array();
$tabImpar = array();
  //funcion para saber si es par o impar y sumarlo en el array vacío.
    function solucion2 ($tab, &$tabPar, &$tabImpar) {
             
                for ($i=0; $i <= count($tab); $i++) {
                 
                    if ($tab[$i] % 2 == 0 ) {
                     
                        $tabPar[] += $tab[$i] ;      
                    }
                    else {
                     
                        $tabImpar[] += $tab[$i];   
                    }
                }
        //La funcion devuelve los pares e Impares del array.
        for ($i=0; $i <= count($tabPar); $i++) {
            echo "Par : ";
            echo $tabPar[$i] . "<br/>";
            echo " Impares : ";
            echo $tabImpar[$i] . "<br/>";
        }
    }
 
//Llamar la funcion
solucion2 ($tab, $tabPar, $tabImpar);
 

Funciona solo que me devuelve 2 números PARES de más en la solución.

Si se fijan cuando ejecuto el código.

Como puedo quitar este fallo ???? Gracias a todos.

[jonni09lo]

Código PHP:

<?php
//Variables
$tab = array( 4, -5, 5, 7, 1, -8, 9, -10, 4, 15);
$tabPar = array();
$tabImpar = array();
//funcion para saber si es par o impar y sumarlo en el array vacío.
function solucion2 ($tab, &$tabPar, &$tabImpar) {
    for ($i=0; $i < count($tab); $i++) {
        if ($tab[$i] % 2 == 0 ) {
            $tabPar[] = $tab[$i] ;      
        } else {
            $tabImpar[] = $tab[$i];   
        }
    }
    //La funcion devuelve los pares e Impares del array.
    $numTabPar = count($tabPar);
    echo "Par : <br>";
    for ($i=0; $i < $numTabPar; $i++) {
        echo $tabPar[$i] . "<br/>";
    } 
    
    $numTabImpar = count($tabImpar);
    echo " Impares : <br>";
    for ($i=0; $i < $numTabImpar; $i++) {
        echo $tabImpar[$i] . "<br/>";
    }
} 
//Llamar la funcion
solucion2 ($tab, $tabPar, $tabImpar); ?>
 

Tienes varios problemillas:

Cuando usas for para trabajar con arrays el limite debe ser menor a la cantidad de elementos del array, no menor o igual

Código PHP:

for ($i=0; $i <= count($tab); $i++) {
 

Por

Código PHP:

for ($i=0; $i <= count($tab); $i++) {
 

Segundo es recomendable que trabajes con arrays separados para saber el contenido de cada uno de ellos ya que pueden tener diferentes longitudes.

Tercero es una pequeña optimización, cuando trabajes con for no coloques un count dentro de este ya que siempre se ejecutará con cada llamada que se haga al for

Edito: Hay un caso adicional. Usaste += para asignar un nuevo valor al array, esto es incorrecto ya que += se usa para sumar valores. En este caso debes de usar = que es para asignación.

Saludos

[HackID1]

Iniciado por jonni09lo

Tienes varios problemillas:

Cuando usas for para trabajar con arrays el limite debe ser menor a la cantidad de elementos del array, no menor o igual

Código PHP:

for ($i=0; $i <= count($tab); $i++) {
 

Por
AQUI CREO QUE Querías decir < no <= de nuevo. :)

Código PHP:

for ($i=0; $i < count($tab); $i++) {
 

Segundo es recomendable que trabajes con arrays separados para saber el contenido de cada uno de ellos ya que pueden tener diferentes longitudes.

Tercero es una pequeña optimización, cuando trabajes con for no coloques un count dentro de este ya que siempre se ejecutará con cada llamada que se haga al for

Edito: Hay un caso adicional. Usaste += para asignar un nuevo valor al array, esto es incorrecto ya que += se usa para sumar valores. En este caso debes de usar = que es para asignación.

Saludos

Gracias por tus consejos y por tu ayuda.
Una pregunta que tengo con respecto a lo que me has dicho de no usar count dentro del for( )..
No es igual el poner $i < $numTabPar como aquí

Código PHP:

$numTabPar = count($tabPar);
        
        for ($i=0; $i < $numTabPar; $i++) {
 

Que poner el count dentro??

Se que en una es $i < (variable con el count($tabPar) , pero que diferencia tiene esto o en que puede ayudar, entiendo que has dicho que es para que no se repita el count cada vez en el for, pero no lo haría también de esa forma, ya que la Variable tiene un count($tabPar) dentro????

Saludos y muchas gracias , tendré más en cuenta el < y no el <= en los próximos ejercicios con arrays y con bucles for

[jonni09lo]

Hola, si tenias razón un error de typo

Bueno lo que quiero decirte es que si el ciclo for se repite 500 veces, al tener el count() dentro de la declaración del for, se repetirá 500 veces. Pero si lo tienes por fuera solo se ejecura una sola vez el count().

Código PHP:

//Ejemplo A
$numMax = count($data) //retorna 500 por ejemplo
 
for($i=0; $i < $numMax; $i++){
    //operaciones
}
 
//Ejemplo B
for($i=0; $i < count($data); $i++){ //Php no sabe cuanto vale count($data) así que ejecutará siempre el count para saber cuando llega al limite
    //operaciones
}
 

Espero sea mas claro

Saludos

[HackID1]

Gracias por tomarte el tiempo en aclarar mi duda, ahora lo entiendo mucho mejor, estos fallos son cosas que mi maestro en clases ni nos explica la verdad, y dudo que si se lo preguntará a él nos diera una respuesta clara.

Gracias de nuevo! Tema Solucionado.