O relacionamento existente entre sua tabelas produto e cor_fundo é de 1:N
.
Um produto pode conter muitas cores.
Pelo menos foi o que entendi da sua imagem. No entanto apesar de ter construído uma tabela separada para cores, não criou uma tabela de relacionamento entre as tabelas. Aonde seria armazenado o id da cor e o id do produto ao invés disso concatenou dentro de um campo produto_cor_fundo
da tabela produto as ids
das cores separados por virgula.
É completamente desaconselhável seguir dessa maneira!
Na sua querie você fez:
SELECT * FROM cor_fundo AS c INNER JOIN produto AS p
ON c.cor_fundo_id = p.produto_cor_fundo
WHERE c.ativo = 1 AND p.produto_id = '".$cod."'
ORDER BY c.cor_fundo_nome ASC
Descrevendo sua querie, ela faz o seguinte:
Selecionar todos os campos da tabela cor_fundo relacionando com a tabela produto aonde o campo cor_fundo_id
é igual ao campo produto_cor_fundo
.
Enquanto o campo ativo
deve ser igual a 1 e o campo produto_id
dever ser igual a variável $cod
, ordenando ascendente pelo nome da cor.
Repare que não irá funcionar porque ao invés de ter apenas uma id de cor por linha de produto, existe uma concatenação de ids no campo produto_cor_fundo
da tabela produto
Existem duas formas de corrigir esse problema:
1ª maneira
Persistir no erro de modelagem também conhecido como POG (programação orientada a gambiarra).
$checked = explode(',', $lnPint['produto_cor_fundo']);
foreach ($checked as $row){
$id = $row['cor_fundo_id'];
$sqlPint = mysql_query("SELECT cor_fundo_nome FROM cor_fundo WHERE
cor_fundo_id =".$id);
Criar um nova querie dentro do seu foreach
e buscar os nomes de todas as cores para cada linha de produto buscada na querie anterior.
O que é completamente desaconselhável porque se tiver um produto com 50 cores irá influenciar completamente a performance da sua aplicação. E mesmo que pense que são poucas cores, se houver alguns poucos acessos também vai gerar fila de espera pela consulta em seu servidor.
Consultas no banco de dados requer muita memoria, e gera gargalo por se tratar de leitura em disco rígido, lembrando que o tempo que leva para filtrar os resultados da querie, o seu servidor fica em espera de resposta e também consumindo recursos.
2ª Maneira, e a mais correta:
Crie uma nova tabela chamada produto_cores, crie um campo produto_cores_id
que sera a chave primária dessa tabela, e crie ainda chaves compostas para essa tabela: produto_id
e cor_fundo_id
, e faça um relacionamento de 1:N
usando essas duas chaves das tabelas produto e cor_fundo. A esse tipo modelagem se da o nome de chave primária composta.
Você precisa também alterar suas rotinas de inserção do produto e de update no banco de dados. Para inserir ou atualizar nessa nova tabela as ids do produto e as ids de suas cores.
Em seguida altere sua querie de consulta para isso:
SELECT * FROM produto_cores pc
INNER JOIN produto p ON p.produto_id = pc.produto_id
INNER JOIN cores_fundo cf ON cf.cor_fundo_id = pc.cor_fundo_id
WHERE cf.ativo = 1 AND p.produto_id = ".$cod."
ORDER BY cf.cor_fundo_nome ASC;
Pronto você agora tem acesso as cores de seus produtos. Repare, que vai dar um trabalhinho para corrigir, mas no entanto executara apenas uma querie para buscar os resultados pretendidos, e sobre alta demanda de acessos a performance não decaí, não gera gargalos no servidor.
É importante que consulte também sobre: (DER) DIAGRAMA DE ENTIDADES DE RELACIONAMENTO para os próximos sistemas, que desejar montar.