Tabela em foreach

[RollySantos 0]

Gente, tudo bem?

 

Estou tentando colcoar uma tabela para funcionar em foreach e não estou conseguindo. Certa vez me ajudaram a fazer em while e eu consegui fazer funcionar beleza.

Mas agora, precisava fazer em foreach, tem como alguem me dar uma força?

<table width="auto" border="0" cellspacing="0" cellpadding="0">
    <?php foreach ($imagens as $id =>$image): ?>
       <?php if (preg_match($CodigoCli, $image) != ""){ ?>
          <?php if (preg_match($tamanhoH, $image) || preg_match($tamanhoV, $image)){ ?>
          
			<?php	if ($contador >= 0) { ?>
		  			<tr>
		   		<?php } ?> 
    				<td><img src="<?php echo $Diretorio ?>/<?php echo $image ?>"><br /><label><input type="checkbox" name="imagens[<?php echo $id ?>]" value="<?php echo $image ?>"></label> 
    - <?php echo $image ?></td>
            
      		<?php $contador ++;
      		 if ($contador >= 3) { ?>
		  	</tr>
            <?php $contador = 0;
		   } 
      	} 
	 } 
     endforeach ?>
    </table>

[ESerra 744]

Entenda a lógica da coisa... copiar/colar sem ler e entender te leva do nada a lugar algum...

 

Enfim, primeiro, o que o script está fazendo que não deveria ou não está fazendo e que deveria fazer?

[RollySantos 0]

Não me deram o script da outra vez. Apenas me ajudaram em como era o processamento e onde eu estava errando.

E no que estou fazendo agora, ele está quase funcionando como eu planejava. A diferença é que quando eu fiz em while coloquei um if e else if calculando o loop. E o script ficou enorme. Então, estou fazendo esse que é bem diferente no foreach.

 

Só que o que acontece é que estou colocando para fazer uma célula na coluna 1, ele fecha o <td> e vai para baixo.
Minha intenção é fazer a coluna 1, 2 e 3, então o contador vai para 3 e fecha o <tr> passando para próxima linha.

 

Então ao invés de fazer três colunas e passar para próxima linha, ele está fazendo tudo em uma linha só.

[ESerra 744]

Troque:

<?php if ($contador >= 0) { ?>

Por:

<?php if ($contador == 0) { ?>

E lembre-se de criar a variável $contador com valor igual a zero antes de iniciar o laço.

[RollySantos 0]

Troque:

<?php if ($contador >= 0) { ?>

Por:

<?php if ($contador == 0) { ?>

E lembre-se de criar a variável $contador com valor igual a zero antes de iniciar o laço.

 

 

Eu criei as variáveis, é q o script completo é grande.

 

Valeu!

Eu só não estava enxergando onde estava errando!