¿Qué les parece este código?

  • Autor Autor FacundoGQR
  • Fecha de inicio Fecha de inicio
F

FacundoGQR

Beta
Verificación en dos pasos activada
¡Usuario con pocos negocios! ¡Utiliza siempre saldo de Forobeta!
Muy buenas!!, estoy probando un código que descargué de una web. En resumen lo que hace es almacenar una imágen en un campo longblob de la BBDD y en un carpeta del servidor.

La verdad es que me funciona de maravilla :encouragement:
Me gustaría saber sus opiniones, ¿alguna mejora que se le pueda hacer?.

Un saludo! 🙂

PHP:
<?php
	include("conectarbd.php");
		if(isset($_POST['btn1']))
			{	
				$imagen = addslashes(file_get_contents($_FILES['imagen']['tmp_name']));
				$nombre_imagen=$_FILES['imagen']['name'];
				$carpeta_destino=$_SERVER['DOCUMENT_ROOT'].'/imagenes/servicios/';
				$tipo_imagen=$_FILES['imagen']['type'];
				$size_imagen=$_FILES['imagen']['size'];

				if (empty($_FILES['imagen']['tmp_name'])){
					echo "No seleccionaste ninguna imágen";
				} else {
						if($size_imagen<=3000000){
							if($tipo_imagen=="image/jpeg" || $tipo_imagen=="image/jpg" || $tipo_imagen=="image/png" || $tipo_imagen=="image/gif" || $tipo_imagen=="image/bmp"){
								move_uploaded_file($_FILES['imagen']['tmp_name'],$carpeta_destino.$nombre_imagen);
} else {
				echo "Solo se pueden subir imágenes en los formatos: jpg/jpeg/png/gif";
					}												
						} else {
							echo "El tamaño es demasiado grande";
							}
						}

					$imagen_objetivo=fopen($carpeta_destino . $nombre_imagen, "r");
					$contenido=fread($imagen_objetivo, $size_imagen);
					fclose($imagen_objetivo);

					if(!empty($imagen)){									
					$sql="INSERT INTO tablaimagenes (imagen) VALUES ('$imagen')";
					$resultado=mysqli_query($conexion, $sql);}

					if(mysqli_affected_rows($conexion)>0){
						echo "Todo correcto";
					}else{
						echo "Algo salió mal";
					       }
					}
							
		include("desconectar.php");
?>
 
almacenar una imágen en un campo longblob de la BBDD

A la larga esto puede afectar a tu bdd. Siempre es mejor almacenar los archivos en una directorio y guardar la ruta en la bdd. Es mi opinion
 
A la larga esto puede afectar a tu bdd. Siempre es mejor almacenar los archivos en una directorio y guardar la ruta en la bdd. Es mi opinion

Tienes razón. :encouragement: Por el momento tengo pocos usuarios, por eso lo estaba utilizando así, pero pronto tendré que modificarlo.
 
Hola si amigo y debes de colocare seguridad, saludos :encouragement:
 
Atrás
Arriba