Na hora de desenvolver um projeto, ou criar um script auxiliar, é essencial estar calmo acima de tudo, porque escrever códigos com mais de 100
linhas já começa a ser um bocado complicado na hora de encontrar }
, ou ;
no meio de tantas funções, variáveis e argumentos, no teu código, confesso que não foi diferente, acima de tudo não tinha notas
o que costuma a ser bom às vezes até mesmo para o próprio autor do código.
O que se segue são algumas possíveis melhorias que introduzi no teu script
, de modo a tornar mais objetivo/eficiente.
Funções
<?php
$pdo = new PDO("mysql: hostname=localhost; dbname=example;", "root", "");
// Nesta função trabalhamos uma vez mais a array, de modo a obtermos a mesma ID
function array_final($array, $id){
if(is_array($array)){
foreach($array as $im){
$f[] = array("imagem"=>$im, "id"=>$id);
}
$valores = array();
foreach($f as $p){
$valores = array_merge($valores, array_values($p));
}
return $valores;
} else {
return false;
}
}
function imagens($array, $nome=null){
if(!empty($array) && isset($array)){
// Esta linha imprime o nome dos ficheiros em maiusculo, para o caso de $nome for diferente de vazio
if(isset($nome)){
// Para multiplos ficheiros (album)
if(is_array($array["name"])){
foreach($array["name"] as $id=>$item){
$imagem[] = $item;
}
// Para o caso de o ficheiro selecionado ser apenas 1 (CAPA)
} else {
$explode = explode(".", $array["name"]);
return strtoupper($explode[0]);
}
if(isset($imagem)){
foreach($imagem as $id=>$img){
$explode[] = explode(".", $img);
}
for($i=0; $i<count($explode);$i++){
$explodes[] = strtoupper(array_shift($explode[$i]));
}
// Retorna uma array contento o nome e a extensao dos arquivos
return $explodes;
}
// Para o caso de $nome for igual a 0, ou vazio
} else {
// Para multiplos ficheiros (album)
if(is_array($array["name"])){
foreach($array["name"] as $id=>$item){
$imagem[] = $item;
}
// Para o caso de o ficheiro selecionado ser apenas 1 (CAPA)
} else {
return $array["name"];
}
if(isset($imagem)){
// Retorna uma array contento o nome e a extensao dos arquivos
return $imagem;
}
}
} else {
return false;
}
}
function enviar($ficheiro, $caminho){
// Arquivo existe ?
if(count($ficheiro) <= 0){
return false;
} else {
// Para multiplos ficheiros (album)
if(is_array($ficheiro["name"])){
foreach($ficheiro["name"] as $id=>$imagem){
$upload = $caminho . basename($imagem);
$estado = move_uploaded_file($ficheiro["tmp_name"][$id], $upload) ? true : false;
}
// Para o caso de o ficheiro selecionado ser apenas 1 (CAPA)
} else {
$upload = $caminho . basename($ficheiro["name"]);
$estado = move_uploaded_file($ficheiro["tmp_name"], $upload) ? true : false;
}
// se o ficheiro for movido com sucesso, retorna true como ultima declaracao
if($estado == true){
return true;
}
}
}
function criar_album($album, $capa){
// chamar a váriavel pdo atravez do escopo global
global $pdo;
// Caminho/diretorio para onde vao os ficheiros
$caminho = 'up/';
$qtd = count($_FILES["imagem"]["name"]);
if ($qtd > 0){
$nomeArquivoCapa = $_FILES["capa"]["name"];
$upload = $caminho . basename($nomeArquivoCapa);
// Se o ficheiro de capa e o(s) ficheiro(s) forem movidos com sucesso
if(enviar($_FILES["capa"], $caminho, "capa") && enviar($_FILES["imagem"], $caminho, "imagem")){
// Prepara a primeira consulta SQL
$inserir = $pdo->prepare("INSERT INTO {$capa} (nome, imagem) VALUES (:nome, :imagem)");
// Passando multiplos parametros atravez do metodo execute do PDO
// inserir->execute(array(":bind1"=>valor1, "bind2"=>valor2, ...));
$inserir->execute(array(":nome"=>imagens($_FILES["capa"], 1), ":imagem"=>imagens($_FILES["capa"])));
// Captar a ultima ID inseriida
$id = $pdo->lastInsertId();
// Aqui iniciamos uma transação para maior fluidez
$pdo->beginTransaction();
// calcular o numero de placeholders para a query em questao
foreach(imagens($_FILES["imagem"]) as $p){
$interr[] = '(?,?)';
}
// Aqui concatenamos os valores anteriormente trabalhados, incluindo os placeholders
$sql = "INSERT INTO {$album} (imagem, idCapa) VALUES ". implode(",", $interr);
echo $sql;
// Preparar a consulta SQL
$inserir = $pdo->prepare($sql);
try{
// Executar a consulta, com multiplos parametros
// INSERT INTO tabela (campo1, campo2) VALUES (?,?),(?,?)...
$inserir->execute(array_final(imagens($_FILES["imagem"]), $id));
// Capturar a exceção
} catch (PDOException $e){
// Imprimir a exceção, se existir
$e->getMessage();
}
// Confirmar inclusao
$pdo->commit();
}
}
}
?>
Para este caso, o que se queria, era cadastrar no banco de dados, um conjunto de valores de uma só vez.
No entanto era necessário construir uma query
desse gênero:
INSERT INTO tabela (campo1, campo2) VALUES (?, ?), (?, ?)...
Uma vez que se trabalhou com placeholders
colocou-se então os pontos de interrogação
no lugar onde seriam substituídos os valores.
Sendo que o conjunto de resultados para a consulta efetuada foi algo assim:
Array ( [0] => usr1.jpg [1] => 55 [2] => usr2.jpg [3] => 55 )
(imagem) (id) (imagem) (id)
Segundo o site realizar consulta utilizando esse processo pode vir até a ser mais rápido que as habituais.
Acima de tudo, só espero não ter-te complicado a vida com essas
funções novas que adicionei, não tarda edito a minha resposta e
adiciono mais alguns detalhes a explicar o que se estava a passar com
o teu código original, ou como solucionar o problema nele.
Script Original
<?php
$pdo = new PDO("mysql: hostname=localhost; dbname=example;", "root", "");
function criar_album($album, $capa){
global $pdo;
$caminho = 'up/';
$nome = $_POST['nome'];
$qtd = count($_FILES["imagem"]["name"]);
if ($qtd > 0){
$nomeArquivoCapa = $_FILES["capa"]["name"];
$nomeTemporarioCapa = $_FILES["capa"]["tmp_name"];
$tamanhoArquivoCapa = $_FILES["capa"]["size"];
for ($i=0; $i < $qtd; $i++) {
$nomeArquivo = $_FILES["imagem"]["name"][$i];
$nomeTemporario = $_FILES["imagem"]["tmp_name"][$i];
$tamanhoArquivo = $_FILES["imagem"]["size"][$i];
if (!empty($nomeArquivoCapa)) {
$arquivoArrayCapa = explode(".", $nomeArquivoCapa);
$extensaoCapa = end($arquivoArrayCapa);
$arquivoCapa = $caminho.md5(time().rand(3212, 15452)).'.'.$extensaoCapa;
// Descomente a linha para guardar a imagem com o nome original no banco de dados
//$arquivo = $nomeArquivoCapa;
if (move_uploaded_file($nomeTemporarioCapa, $arquivoCapa)) {
$inserir = $pdo->prepare("INSERT INTO $capa(nome, imagem) VALUES('$nome', '$arquivoCapa')");
$inseri = $inserir->execute(); // Execute a inserção
$last = $pdo->lastInsertId();
}
if (!empty($nomeArquivo)) {
$arquivoArray = explode(".", $nomeArquivo);
$extensao = end($arquivoArray);
$arquivo = $caminho.md5(time().rand(3212, 15452)).'.'.$extensao;
// Descomente a linha para guardar a imagem com o nome original no banco de dados
//$arquivo = $nomeArquivo;
if (move_uploaded_file($nomeTemporario, $arquivo)) {
$inserir = $pdo->prepare("INSERT INTO $album(idCapa, imagem) VALUES('$last', '$arquivo')");
$inseri = $inserir->execute(); // Execute a inserção
}
else {
echo '<div class="alert alert-danger" role="alert">Erro, tente novamente mais tarde!</div>';
}
}
}
}
echo '<div class="alert alert-success" role="alert">Salvo com sucesso!</div>';
}
}
?>
Se fores ler as linhas com atenção, vais reparar que não troquei praticamente nada, apenas 1 sinal
foi removido, o que estava no looping for
.
De:
for ($i=0; $i <= $qtd; $i++) {
...
Para:
for ($i=0; $i < $qtd; $i++) {
...
Mudei também o nome valor da variável caminho, que era uploads
, para up
, o resto está a mesma coisa. Confesso que ainda me baralho a tentar entender o teu código.
Contudo, aqui está a realizar o upload
e cadastro
com sucesso, restando apenas citar o erro.
O mais provável é que o erro esteja na estrutura da tua tabela. É assim que a(s) minha(s) tabela(s) está(ao) estruturada:
--
-- Estrutura da tabela `albumimagens`
--
CREATE TABLE IF NOT EXISTS `albumimagens` (
`id` int(11) NOT NULL AUTO_INCREMENT,
`idCapa` int(11) NOT NULL,
`imagem` varchar(36) NOT NULL,
PRIMARY KEY (`id`),
KEY `idCapa` (`idCapa`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=17 ;
CREATE TABLE IF NOT EXISTS `capa` (
`id` int(11) NOT NULL AUTO_INCREMENT,
`nome` varchar(36) NOT NULL,
`imagem` varchar(36) NOT NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=77 ;
Apesar dos valores nos campos varchar
serem insuficientes para a hash
gerada para o nome das imagens, as imagens estavam a ser salvas à metade.
Outra coisa é, se fores reparar nas minhas duas tabelas, ambas têm um campo ID
independente primary key
e com auto_increment
, que para o caso em questão é obrigatório e noutros, é simplesmente recomendado. Por isso, se possível, apague e crie novamente a tabela albumimagens
e experimenta novamente.
Até, mais e boa sorte.
Outras fontes:
PDO Prepared insert with mult. rows...