Как исключить дубли из выдачи в цикле php?

Question

[Павел]

Есть 2 таблицы , в одной ФИО , дата и часы. Во второй даты и "id дня"
циклами сравниваю дату из первой таблицы со второй и если она совпадает в зависимости от "id дня" делаю математические операции с часами из первой таблицы.
Все бы хорошо , но если даты во второй таблице нет (последний else if в коде) , тоже нужно провести математику , но в этом моменте выходит огромное количество дублей
как исключить дубли?

написал такой код:

$output = '';
                    foreach ($result as $row) { //здесь делаем двумерный массив
                        foreach ($result_days as $row_days) {
                            if ($row['date'] === $row_days['off_date'] && $row_days['off_id'] == 0) { //если выходной
                                // echo '<span>Выходной день</span><br>';
                                $sub_hours = (int)$row['sub_hours'];
                                $output = '
                        <tr>
                            <td>' . $row['name'] . '</td>
                            <td>' . ($sub_hours = $sub_hours - 0) . '</td>
                            <td>' . $row['date'] . '</td>
                            <td>Не рабочий день</td>
                            </tr>';
                                echo $output;
                            } else if ($row['date'] === $row_days['off_date'] && $row_days['off_id'] == 1) { //если сокращенка
                                $sub_hours = (int)$row['sub_hours'];
                                $sub_hours = $sub_hours - 7;
                                if ($sub_hours > 0) {
                                    $output = '
                        <tr>
                         <td>' . $row['name'] . '</td>
                         <td>' . $sub_hours . '</td>
                         <td>' . $row['date'] . '</td>
                         <td>Сокращенный день</td>
                         </tr>';
                                    echo $output;
                                }

                            } else if ($row['date'] != $row_days['off_date']) { // рабочие дни
                                $sub_hours = (int)$row['sub_hours'];
                                $sub_hours = $sub_hours - 8;
                                if ($sub_hours > 0) {
                                    $output = '
                        <tr>
                        <td>' . $row['name'] . '</td>
                         <td>' . $sub_hours . '</td>
                         <td>' . $row['date'] . '</td>
                        <td>Рабочий день</td>
                        </tr>';
                                    echo $output;
                                }

                            }
                        }

                    }

Comments

[Alams Stoyne]

Дано:
Table_1: ID | FIO | DATE | CLOCK
Table_2: ID_DATE | DATE

Вопрос уникальны ли DATE в Table 2?

Если ДА, то и это выборка именно из БД то самое элементарное изменить SQL запрос:

SELECT * FROM Table_1 INNER JOIN Table_2 using(DATE)

или

SELECT * FROM Table_1, Table_2 WHERE Table_1.DATE = Table_2.DATE;

В следствии чего результат будет:
ID | FIO | DATE | CLOCK | ID_DATE
но есть в Table_1.DATE есть данные которых нет в Table_2.DATE то запись из БД не выбериться

Советую почитать

[Павел]

Alams Stoyne, да , во второй таблице даты не повторяются , а в первой соотвецтвенно у каждого пользователя

Answer 1

[Vitsliputsli]

Так вы ведь выполняете последний else if практически для всех элементов массива. Если не менять текущую реализацию, вынесите это условие из под второго foreach и замените условие на проверку флага, флаг соответственно того, что после прохождения второго foreach не было совпадений.

Comments

[Павел]

а подскажите пожалуйста как это сделать? Уже 4 дня мучаюсь
менять можно что угодно , это мой го**о код так как я еще даже до джуна не дорос

[Vitsliputsli]

Примерно так:

<?php

define('WORKDAY',0);
define('SHORTDAY',1);
define('HOLIDAY',2);

$dayName = [
    WORKDAY => 'Рабочий день',
    SHORTDAY => 'Сокращенный день',
    HOLIDAY => 'Нерабочий день',
];

$dayType = [
    '08.05.2019' => SHORTDAY,
    '09.05.2019' => HOLIDAY,
    '10.05.2019' => HOLIDAY,
    '11.05.2019' => HOLIDAY,
];

$inputData = [
    [ 'date'=>'06.05.2019', 'fio'=>'', 'hours'=>0 ],
    [ 'date'=>'07.05.2019', 'fio'=>'', 'hours'=>8 ],
    [ 'date'=>'08.05.2019', 'fio'=>'', 'hours'=>7 ],
    [ 'date'=>'09.05.2019', 'fio'=>'', 'hours'=>0 ],
];

$outputData = [];
foreach ($inputData as $day) {
    $data = [
        'name' => $day['fio'],
        'date' => $day['date'],
    ];
    if ( isset($dayType[$day['date']]) ) {
        if ($dayType[$day['date']] === HOLIDAY) { 
            $data['type'] = HOLIDAY;
            $data['hours'] = $day['hours']; // calculate for holiday
        } else if ($dayType[$day['date']] === SHORTDAY) {
            $data['type'] = SHORTDAY;
            $data['hours'] = $day['hours']; // calculate for shortday
        } else {
            // unknown type of day, need exception
        }
    } else {
        $data['type'] = WORKDAY;
        $data['hours'] = $day['hours']; // calculate for workday
    }
    $outputData[] = $data;
}

// Print result
foreach ($outputData as $row) {
    echo "<tr><td>${row['name']}</td>";
    echo "<td>${row['hours']}</td>";
    echo "<td>${row['date']}</td>";
    echo "<td>${dayName[$row['type']]}</td></tr>";
}

[Павел]

Vitsliputsli, только у меня HOLIDAY во второй таблице не задан , от этой "лени" заполнять вторую таблицу всеми днями и пошла проблема )

[Vitsliputsli]

Павел, вы же проверяете условие $row_days['off_id'] == 0, значит нерабочие дни у вас обозначены как 0 в $row_days, разве нет?

[Павел]

Vitsliputsli, ой перепутал , именно рабочих нет (WORKDAY)

[Vitsliputsli]

Павел, ну и правильно, зачем лишние данные, поэтому в скрипте выше идет проверка на существование в массиве значения и если его нет, то по-умолчанию назначаем этот день рабочим. Наоборот если бы они там были, то в скрипт нужно было бы добавить еще одно условие или поменять существующие, а так, как раз как вы хотели.

[Vitsliputsli]

В принципе, чтобы было понятнее, можно переписать foreach так:

$outputData = [];
foreach ($inputData as $day) {
    $currentDayType = $dayType[$day['date']] ?? WORKDAY;
    $data = [
        'name' => $day['fio'],
        'date' => $day['date'],
        'type' => $currentDayType,
    ];
    switch ($currentDayType) {
        case HOLIDAY:
            $data['hours'] = $day['hours']; // calculate for holiday
            break;
        case SHORTDAY:
            $data['hours'] = $day['hours']; // calculate for shortday
            break;
        case WORKDAY:
            $data['hours'] = $day['hours']; // calculate for workday
            break;
        default:
            // unknown type of day, need exception
    }
    $outputData[] = $data;
}

в этом случае может задавать, а можете не задавать рабочие дни, если день не будет найден, то он будет считаться рабочим (строка $dayType[$day['date']] ?? WORKDAY)

[Павел]

Vitsliputsli, спасибо

Answer 2

[AUser0]

Вот код:

$output = '';
                    foreach ($result as $row) { //здесь делаем двумерный массив
                        $found = false;
                        foreach ($result_days as $row_days) {
                            if ($row['date'] === $row_days['off_date'] && $row_days['off_id'] == 0) { //если выходной
                                $found = true;
                                // echo '<span>Выходной день</span><br>';
                                $sub_hours = (int)$row['sub_hours'];
                                $output = '
                        <tr>
                            <td>' . $row['name'] . '</td>
                            <td>' . ($sub_hours = $sub_hours - 0) . '</td>
                            <td>' . $row['date'] . '</td>
                            <td>Не рабочий день</td>
                            </tr>';
                                echo $output;
                            } else if ($row['date'] === $row_days['off_date'] && $row_days['off_id'] == 1) { //если сокращенка
                                $found = true;
                                $sub_hours = (int)$row['sub_hours'];
                                $sub_hours = $sub_hours - 7;
                                if ($sub_hours > 0) {
                                    $output = '
                        <tr>
                         <td>' . $row['name'] . '</td>
                         <td>' . $sub_hours . '</td>
                         <td>' . $row['date'] . '</td>
                         <td>Сокращенный день</td>
                         </tr>';
                                    echo $output;
                                }
                            }
                        }
                        if (!$found) { // рабочие дни
                                $sub_hours = (int)$row['sub_hours'];
                                $sub_hours = $sub_hours - 8;
                                if ($sub_hours > 0) {
                                    $output = '
                        <tr>
                        <td>' . $row['name'] . '</td>
                         <td>' . $sub_hours . '</td>
                         <td>' . $row['date'] . '</td>
                        <td>Рабочий день</td>
                        </tr>';
                                    echo $output;
                                }
                            }
                    }

Comments

[Павел]

я выходит просто зря последний else if сделал? Можно просто вынести условие в первый цикл)) , спасибо открыли мне глаза