2

Olá!

Meu código é uma agenda e funciona da seguinte forma, existe uma array com alguns horários dentro:

$horarios = ["08:00", "09:00", "10:00", "11:00", "12:00", "13:00", "14:00", "15:00", "16:00", "17:00"];

Faço então um SELECT no banco procurando por agendamentos e armazeno os dados dentro de uma array através de um foreach:

$stmt = $dbh->query("SELECT *, date_format(hora_agendamento, '%H%:%i') as hora_agendamento FROM agendamentos WHERE data_agendamento='$data_agendamento_convert' AND medico='$medico_completo'");
  $result = $stmt->fetchAll();

Após isso através de um for eu ando dentro da array e verifico se cada horário da array $horarios pode ser encontrado dentro da array $result:

for ($i=0; $i <= (count($horarios) - 1); $i++) {
  if (in_array($horarios[$i], $result[$i])) { ?>
    <tr>
      <td><?php echo $horarios[$i]; ?></td>
      <td><?php echo $result[$i]['nome']; ?></td>
      <td><?php echo $result[$i]['descricao']; ?></td>
      <td>Editar</td>
    </tr>
 <?php } else { ?>
    <tr>
       <td><?php echo $horarios[$i]; ?></td>
       <td></td>
       <td></td>
       <td>Editar</td>
     </tr>
 <?php } }?>

A primeira linha encontrada é impressa corretamente, depois acredito que pelo fato de ter 10 horários em $horarios e apenas 3 matches em $result o código começa a me retornar o erro abaixo:

Notice: Undefined offset: 3 in C:\wamp64\www\admin\agenda1.php on line 91

Imagino que se eu melhorar esse meu if eu consiga resolver o problema, mas nâo estou sabendo o que usar pra melhorar ele, alguém pode me dar uma luz?

Obrigado!

10
  • Se código pode conter problemas inclusive na elaboração, porque, fetchAll() não já te retornar um array ? para fazer um for? qual é a linha 96?
    – novic
    15/05/2018 às 14:13
  • 1
    Não precisa gerar os dados se você tem os dados!? compreende isso. a variavel result é a mesma coisa que a variavel agendamentos
    – novic
    15/05/2018 às 14:20
  • 1
    Aqui o próximo problema: if (in_array($horarios[$i], $result[$i])) { ?> porque o result[$i] você está acessando o registro e não o array que precisa, se você precisar comparar horários tem que ter outra estratégia. Qual é a finalidade dessa parte?
    – novic
    15/05/2018 às 14:39
  • 1
    Desta forma ele imprime as linhas certinho uma após a outra, porém preciso agora que ele verifique o horário do agendamento e preencha a linha correta, por isso havia aquele if, pra quando der match entre o horário da $horarios e o horário da $result ele preencher aquela linha com os dados, quando não desse match, executava o else e preenchia a linha em branco, assim eu tinha vista da agenda com todos os horários usados e os disponíveis também, essa é a ideia. 15/05/2018 às 14:52
  • 1
    Então faz assim: if (in_array($result[$i]['hora_agendamento'], $horarios)) { ?> ou seja, é só inverter as variaveis dentro da função in_array
    – novic
    15/05/2018 às 15:10

1 Resposta 1

2

Faz isso:

if (isset($result[$i]) && in_array($result[$i], $horarios)) { ?>

Assim esse erro irá parar de aparecer pois o isset($result[$i]) verifica se EXISTE a variável com o índice dentro array $result. Que retornará true caso exista e desde que esse valor não seja nulo.

$array = array(0 => "oi", 1 => "", 2 => null, 3 => false);

isset( $array [0] ) // true
isset( $array [1] ) // true
isset( $array [2] ) // false
isset( $array [3] ) // true

Além disso, conforme citado pelo Virgilio Novic nos comentários, o certo é:

in_array ("valor que eu procuro", array());

Veja a documentação

Portanto, você precisa colocar o $result[$i] antes de $horarios. Como $horarios já é um array(), você não precisa indicar um índice em $horarios.

EDIT

Conforme a imagem mostrada, você pode fazer um loop dentro do seu loop, para analisar cada caso. Veja:

for($i = 0; $i < count($horarios); $i++){
    $status = true;
    foreach($result as $resultado){
        if($horarios[$i] == $resultado['hora_agendamento']){
        ?>
            <tr>
              <td><?php echo $horarios[$i]; ?></td>
              <td><?php echo $resultado['nome']; ?></td>
              <td><?php echo $resultado['descricao']; ?></td>
              <td>Editar</td>
            </tr>
        <?php 
            $status = false;
            break;
        }
    }
    if($status){ ?>

    <tr>
       <td><?php echo $horarios[$i]; ?></td>
       <td></td>
       <td></td>
       <td>Editar</td>
     </tr>

<?php }}?>

EDIT 2

Apesar da outra forma atender, este exemplo abaixo, com array_search é mais otimizado.

$horariosCheios = array();
foreach($result as $resultado){
    $horariosCheios[] = $resultado['hora_agendamento'];
}

for ($i=0; $i <= (count($horarios) - 1); $i++) {
  if (($indice = array_search($horarios[$i], $horariosCheios)) !== false) { ?>
    <tr>
      <td><?php echo $horarios[$i]; ?></td>
      <td><?php echo $result[$indice]['nome']; ?></td>
      <td><?php echo $result[$indice]['descricao']; ?></td>
      <td>Editar</td>
    </tr>
 <?php } else { ?>
    <tr>
       <td><?php echo $horarios[$i]; ?></td>
       <td></td>
       <td></td>
       <td>Editar</td>
     </tr>
 <?php } }?>
7
  • Compreendo, porém surge um problema, se com o if eu procurar o conteúdo de $result dentro de $horarios vai acabar exibindo pra mim primeiro os 3 resultados encontrados, depois os que caíram no else, como se vê na imagem abaixo, vou pensar em algo pra contornar: imgur.com/a/xCCIvrN 15/05/2018 às 17:52
  • 1
    Show de bola, não podia ter dado mais certo, estava fazendo algo do gênero mas meio cético, sou sempre receoso pra acumular muitos loops, mas funcionou perfeitamente, muito obrigado, além do código peguei dicas valiosas, tanto suas @Andrei Coelho quanto do Virgilio Novic. 15/05/2018 às 18:32
  • 1
    @FernandoGross eu dei essa sugestão mas ela não deve ser a melhor. Isso é só um exemplo, pois dependendo da quantidade você irá fazer muitos loops desnecessários talvez. Você precisa depois refatorar esse código e fazer uns testes. Pesquise também sobre array_search. Poderá te ajudar mais. Abraço! 15/05/2018 às 18:34
  • 1
    @FernandoGross coloquei no edit 2 um exemplo melhor. 15/05/2018 às 18:51
  • 1
    Ótimo! Quando eu estava procurando por qual função utilizar pra procurar os horários dentro da array eu esbarrei nessa função lá na documentação, mas não cheguei a testar ela, o princípio é bem parecido. 16/05/2018 às 14:35

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .