wyświetlanie wpisów z bazy danych - PHP

Question

Witam, tworzę projekt PHP+mysql i chciałbym zrobić pętlę do wyświetlania wpisów - jak na razie mam coś takiego (wklejam cały kod aby oszczędzić wam czarowania z fusów)

function ile() {	
	mysqli_report(MYSQLI_REPORT_STRICT);
	try {
			$db_connect = mysqli_connect(db_host,db_user,db_password,db_name);
		if ($db_connect->connect_errno!=0){
			throw new Exception(mysqli_connect_errno());
		} else {						
			$result = mysqli_query($db_connect, "SELECT * FROM jake_posts");
			$rows = mysqli_num_rows($result);
			if ($rows>0) {
				return $rows;
			} else {
				return "Nie znaleziono wpisów!";
			}
			$db_connect->close();
		}
	} catch(Exception $e_db) {
		echo s_server_error;
	}
}
function posts(){
        mysqli_report(MYSQLI_REPORT_STRICT);
	try {
		$db_connect = mysqli_connect(db_host,db_user,db_password,db_name);
		if ($db_connect->connect_errno!=0){
			throw new Exception(mysqli_connect_errno());
		} else {						
			$result = mysqli_query($db_connect, "SELECT * FROM jake_posts");
			$rows = mysqli_num_rows($result);
			if ($rows>0) {
				for ($i = 1; $i <= $rows; $i++){
				$line = mysqli_fetch_assoc($result);
					$post_id = $line['id'];								
				
					echo $post_id;
				}
			} else {
				echo "Nie znaleziono wpisów!";
			}
			$db_connect->close();
		}
	} catch(Exception $e_db) {
		echo s_server_error;
	}
}

jak można by to zoptymalizować aby ograniczyć to do tego co chcę wyświetlić? dokładniej chodzi mi o to że nie chciałbym otwierać kilku połączeń do bazy danych tylko zrobić to raz i później z tego korzystać - może oddzielna funkcja? Nie mam pojęcia, więc proszę o podsunięcie pomysłu jak można to najlepiej zrobić. Jeśli napisałem coś nie jasno przepraszam, dopiero się uczę pisać w php i nie umiem tego dokładnie opisać/nazwać.

Answer 1

Zastosuj dependency injection i wstrzykuj połączenie z bazą za pomocą argumentu funkcji, ot 1 optymalizacja

 

<?php


function fetchProducts(\PDO $pdo) : array
{
     
     $sql = "SELECT name,price,something FROM products";
     $result = $pdo->query($sql)->fetchAll();

     return $result;

}
 $connection = new PDO(..) // oczywiście odpowiednie zabezpieczenia..

print_r(fetchProducts($connection));

Tadam

kolejna: Użyj na litość PDO + fetchAll i unikaj SELECT * tylko pobieraj to co potrzeba.

Comments

Nie wiem skąd się uczysz, ale sugeruję wpisać w google phptherightway 

---

<?php
function fetchPosts(\PDO $pdo) : array
{
   // http://php.net/manual/en/pdostatement.fetchall.php
   $sql = "SELECT id,postTitle,postContent, something FROM jake_posts";
   // In general, the only time that you'll want to catch and handle a query exception
   // is when you want to do something else if the query fails like insert update del
   return $pdo->query($sql)->fetchAll();

}

 

Trochę się głupio czuję minimalizując tą funkcję tak bardzo.. ;P