Witam, mam dwie funkcje, jedna do dodawania lokalizacji druga do kasowania lokalizacji.

Lokalizacja - jest to liczba. a tabela w której jest to umieszczone to "text" i wartość wygląda np tak 1,3,6,12,33,4 - czyli oddzielone przecinkiem.

Aby pokazać jak najepiej problem przedstawię na początku funkcję do dodawania lokalizacji:

[PHP] pobierz, plaintext 
<?php
function add_location($array){
        $id_site = $array['site'];
        $location = $array['location'];
 
        $sql ="    UPDATE ".DB_PREFIX."site
                SET id_location=IF(id_location='','$location', CONCAT(id_location,',$location'))
                WHERE id='$id_site'
                AND NOT FIND_IN_SET('$location', id_location)";
        $res =& $this->_db->query($sql);
    }
?>
[PHP] pobierz, plaintext 

Zapytanie sprawdza czy dany rekord istnieje, jeżeli nie istnieje to dodaje po przecinku.

A teraz potrzebuję funkcję która będzie usuwać daną lokalizację z bazy danych.

Aktualnie mam to rozwiązane na zasadzie:

[PHP] pobierz, plaintext 
<?php
function remove_location($array){
        $id_site = $array['site'];
        $location = $array['location'];
        
    $sql ="SELECT id_location FROM ".DB_PREFIX."site WHERE id = '$id_site' ";
                $res =& $this->_db->query($sql);
                $row = $res->fetchRow(MDB2_FETCHMODE_ASSOC);
    
        $id_array = explode(',',$row['id_location']);
        $kay_delete = array_search($location,  $id_array);
        unset($id_array[$kay_delete]);
        
        foreach ($id_array as $id){
            $id_sql .= $id.",";
        }
        $id_sql = substr($id_sql,0,-1);
        
        $sql ="    UPDATE ".DB_PREFIX."site
                SET id_location='$id_sql'
                WHERE id='$id_site'";
        $res =& $this->_db->query($sql);
    }
?>
[PHP] pobierz, plaintext 

Czyli najpierw pobieram dane na temat lokalizacji następnie obrabiam dane i dopiero robię update.


Czy jest możliwość jakoś to poprawić tzn na takim samym rozwiązaniu jak przy dodawaniu aby nie używać funkcji php a wszystko zrobić po stronie mysql ?

Ja bym proponował przeprojektowanie bazy tak, aby można było wykorzystać relacje, a nie podawać po przecinku.

Tak nie pierwsza osoba mi to mówi, ale przy obecnym projekcie jest to nie możliwe. Muszą to rozwiązać na tej zasadzie. Jeżeli się nie da zostanie tak jak jest. Nie jest to strasznie obciążają, a dodatkowo jest t wykonywane po stronie użytkownika- ajax. Ale fajnie by było zoptymalizować do maximum.

Skoro DB, to nie jest po stronie użytkownika.


Przeprojektuj bazę, bo o optymalizacji nie ma mowy. Szybciej jest wyciągnąć rekord wg zoptymalizowanego indeksu niż obrabiać przecinki w każdym z osobna.