Forum PHP.pl > [MySQL][PHP]Błąd mysql_fetch_assoc() expects parameter 1 to be resource, null given in

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [MySQL][PHP]Błąd mysql_fetch_assoc() expects parameter 1 to be resource, null given in

kecajs

19.08.2015, 22:08:47

Witam,
walczę z pewnym kodem i wyskakuje mi taki właśnie błąd: mysql_fetch_assoc() expects parameter 1 to be resource, null given in ścieżka on line 47.
Kod ten ma za zadanie wyciągnąć z bazy tylko dane jednego artykułu, na podstawie zmiennej LacznikID.
Oto kod strony pokaz.php, odpowiedzialnej za wyświetlanie tychże danych.

[PHP] pobierz, plaintext 
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"><html>
<head>
  <meta http-equiv="Content-type" content="text/html; charset=utf-8">
  <link rel="stylesheet" href="style.css" type="text/css">
</head>
 
<body>
 
<h1><center>Szczegóły asortymentu</center></h1><br>
 
<table border="1" width="90%" align=center cellpadding=2> 
            <tr> 
                <td width="20" class="table_small">Lp</td> 
                <td width="130" class="table_small">Nazwa</td> 
                <td width="40" class="table_small">Materiał</td> 
                <td width="60" class="table_small">Dostawca</td> 
                <td width="60" class="table_small">Producent</td> 
                <td width="50" class="table_small">Lokacja</td> 
                <td width="40" class="table_small">Waga</td> 
		<td width="40" class="table_small">Stan</td> 
            </tr>
                <?php
                include_once ("db_con.php");							
		$connection = dbconn::instance();
 
		// tutaj wywolanie zapytania do bazy
         	if (isset($_GET['LacznikID']) && $_GET['LacznikID'] != '') {
		$tut_nr = mysql_real_escape_string($_GET['LacznikID']);
		$wynik = $connection->query('
			SELECT 
	  		        LacznikID, 
			        Nazwa, 
				Material, 
				DostawcaID, 
				ProducentID, 
				Lokacja, 
				Waga, 
				Stan 
			FROM 
				tb_laczniki_main 
			WHERE 
				LacznikID='.$tut_nr
		    );
		}								
 
		while ($r = mysql_fetch_assoc($wynik)) {
                  echo '<tr>';
                  echo '<td width="20" class="table_small">' . $r['LacznikID'] . '</td>';
                  echo '<td width="130" class="table_small">' . $r['Nazwa'] . '</td>';
                  echo '<td width="40" class="table_small">' . $r['Material'] . '</td>';
                  echo '<td width="60" class="table_small">' . $r['DostawcaID'] . '</td>';
                  echo '<td width="60" class="table_small">' . $r['ProducentID'] . '</td>';
                  echo '<td width="50" class="table_small">' . $r['Lokacja'] . '</td>';
                  echo '<td width="40" class="table_small">' . $r['Waga'] . '</td>';
		  echo '<td width="40" class="table_small">' . $r['Stan'] . '</td>';
                  echo "</tr>";
                } 
                ?>
  </table>
 
</body>
</html>
[PHP] pobierz, plaintext

Z kolei kod w pliku, z którego następuje wywołanie konkretnego artykułu wygląda tak.

[PHP] pobierz, plaintext 
<!doctype html >
<html>
    <head>
        <meta http-equiv="Content-type" content="text/html;charset=utf-8">
        <link rel="stylesheet" href="style.css"/>
    </head>
 
    <body>
        <center><strong>Zestawienie łączników</strong></center><br><br>
        <table border="1" width="90%" align=center cellpadding=2> 
            <tr> 
                <td width="20" class="table_small">Lp</td> 
                <td width="130" class="table_small">Nazwa</td> 
                <td width="40" class="table_small">Materiał</td> 
                <td width="60" class="table_small">Dostawca</td> 
                <td width="60" class="table_small">Producent</td> 
                <td width="50" class="table_small">Lokacja</td> 
                <td width="40" class="table_small">Waga</td> 
		<td width="40" class="table_small">Stan</td> 
            </tr>
                <?php
                include_once ("db_con.php");							
		$connection = dbconn::instance();
                $wynik = $connection->query("
                  SELECT 
                      LacznikID, 
                      Nazwa, 
                      Material, 
                      DostawcaID, 
                      ProducentID, 
                      Lokacja, 
                      Waga, 
                      Stan 
                  FROM 
                      tb_laczniki_main 
                  ORDER BY 
                      LacznikID ASC
                ");
		while ($r = mysql_fetch_assoc($wynik)) {
                echo '<tr>';
                    echo '<td width="20" class="table_small"><a href="index.php?sr=pokaz&artykul_id='.$r['LacznikID'].'">'.$r['LacznikID'].'</a></td>';
                    echo '<td width="130" class="table_small">' . $r['Nazwa'] . '</td>';
                    echo '<td width="40" class="table_small">' . $r['Material'] . '</td>';
                    echo '<td width="60" class="table_small">' . $r['DostawcaID'] . '</td>';
                    echo '<td width="60" class="table_small">' . $r['ProducentID'] . '</td>';
                    echo '<td width="50" class="table_small">' . $r['Lokacja'] . '</td>';
                    echo '<td width="40" class="table_small">' . $r['Waga'] . '</td>';
  		    echo '<td width="40" class="table_small">' . $r['Stan'] . '</td>';
                    echo "</tr>";
                } 
                ?>
                </table>
    </body>
</html>
[PHP] pobierz, plaintext

Gdzie mam szukać przyczyn tego komunikatu o błędzie? Poza tym, funkcja mysql_fetch_assoc jest odpowiedzialna za wyświetlanie tylu wierszy, ile wierszy spełnia zadane warunki - jeśli się nie mylę. A czy tą samą funkcję zastosować w moim przypadku, kiedy wiem, że nie będę miał więcej niż jeden wiersz lub gdybym chciał te dane zaprezentować w innym układzie niż tabela z nagłówkami i wierszami, np. w postaci formularza (tabela z kilkoma wierszami, bez nagłówków, opisy pól i wartości obok siebie)? Przeszukałem Google w obu kwestiach, jednak nie znalazłem odpowiedzi dokładnie na moje pytania...

I jeszcze mam pytanie o samo połączenie z bazą.
Mam dość dziwny twór, który ktoś kiedyś mi napisał

[PHP] pobierz, plaintext 
<?php
 
abstract class Singleton {
 
    protected static $_instance;
 
    protected function __construct() {
 
    }
 
    public static function instance() {
 
    }
 
}
 
class dbconn extends Singleton {
 
    private $_conn;
    private $server = 'xxx';
    private $login = 'xxx';
    private $password = 'xxx';
    private $database = 'mz_warehouse';
 
    protected function __construct() {
        $this->_conn = mysql_connect($this->server, $this->login, $this->password) or die(mysql_error());
        mysql_select_db($this->database, $this->_conn) or die(mysql_error()); mysql_query("SET NAMES 'utf8'");
    }
    public static function instance() {
        if (is_null(self::$_instance)) {
            self::$_instance = new dbconn();
        }
        return self::$_instance;
    }
 
    public function query($query) {
        return mysql_query($query);
    }
 
    public function insert($query) {
        if (mysql_query($query)) {
            return mysql_affected_rows();
        } else {
            return 0;
        }
    }
 
    public function select($query) {
        return mysql_num_rows(mysql_query($query));
    }
 
    public function escape($string) {
        return htmlspecialchars(strip_tags($string));
    }
 
}
 
?>
[PHP] pobierz, plaintext

Czy zamiast tego mogę zastosować zdecydowanie krótszą formę znalezioną na którymś z forów

[PHP] pobierz, plaintext 
<?php
$db_connection = null;
function polaczenie() {
    global $db_connection;
    $mysql_server = "xxx";
    $mysql_admin = "xxx";
    $mysql_pass = "xxx";
    $db_connection = mysql_connect($mysql_server, $mysql_admin, $mysql_pass) or die('Brak połączenia z serwerem MySQL.');
    @mysql_select_db($db_connection);
}
    ?>
[PHP] pobierz, plaintext

W jakim celu został użyty ten zwrot

[PHP] pobierz, plaintext 
$connection = dbconn::instance();
[PHP] pobierz, plaintext

Czy samo inkludowanie nie wystarczy?

Dzięki za wyrozumiałość i cierpliwość :)
Pozdrawiam,
kecajs

kapslokk

20.08.2015, 07:21:56

1. Sprawdź czy zapytanie nie generuje błędu - jeśli tak to mysql_query zwróci false i chyba stąd cały problem.
2.

[PHP] pobierz, plaintext 
$connection = dbconn::instance();
[PHP] pobierz, plaintext

jest użyte dlatego, że klasa dbconn wykorzystuje singleton'a.
http://phpedia.pl/wiki/Singleton
3. Możesz użyć:

[PHP] pobierz, plaintext 
$db_connection = null;
function polaczenie() {
    global $db_connection;
    $mysql_server = "xxx";
    $mysql_admin = "xxx";
    $mysql_pass = "xxx";
    $db_connection = mysql_connect($mysql_server, $mysql_admin, $mysql_pass) or die('Brak połączenia z serwerem MySQL.');
    @mysql_select_db($db_connection);
}
[PHP] pobierz, plaintext

ale będziesz musiał przepisać kod bo $connection->query() nie będzie już działać.

MESSIAH :)

20.08.2015, 07:22:39

Mysql oczekuje jakiś jeden parametr a otrzymał Goowno. Innymi słowy :

[PHP] pobierz, plaintext 
mysql_fetch_assoc ( $wynik ))
[PHP] pobierz, plaintext

podstaw odpowiedni parametr do $wynik zobacz w dokumentacji - pewnie chodzi o połączenie z bazą danych.

kecajs

20.08.2015, 17:12:37

Cytat(kapslokk @ 20.08.2015, 08:21:56 )

1. Sprawdź czy zapytanie nie generuje błędu - jeśli tak to mysql_query zwróci false i chyba stąd cały problem.

A czy muszę robić to z wykorzystaniem funkcji mysql_fetch_assoc, czy nie? Zawsze będzie wyciągana z bazy informacja o konkretnym artykule i zawsze będzie tylko jedna. W jaki sposób mogę zaprezentować dane aby wyglądały jak formularz. Przykładowo:
wiersz 1. - Lp. 2 | Nazwa Przykładowa nazwa artykułu
wiersz 2. - Materiał jakiś materiał | Norma norma produktu
wiersz 3. - Stan stan produktu | Lokacja rozmieszczenie w magazynie | Waga waga produktu
wiersz 4. - Dostawca Dostawca | Producent Producent
a nie w postaci tabeli.

Cytat

[PHP] pobierz, plaintext 
$connection = dbconn::instance();
[PHP] pobierz, plaintext

jest użyte dlatego, że klasa dbconn wykorzystuje singleton'a.
http://phpedia.pl/wiki/Singleton

Czy to rozwiązanie jest dużo lepsze od przedstawionego poniżej? Bo Twój przykład poniżej, a mój kod wstawiony w poprzedniej wiadomości znacznie się różnią wielkością.

Cytat

3. Możesz użyć:

[PHP] pobierz, plaintext 
$db_connection = null;
function polaczenie() {
    global $db_connection;
    $mysql_server = "xxx";
    $mysql_admin = "xxx";
    $mysql_pass = "xxx";
    $db_connection = mysql_connect($mysql_server, $mysql_admin, $mysql_pass) or die('Brak połączenia z serwerem MySQL.');
    @mysql_select_db($db_connection);
}
[PHP] pobierz, plaintext

ale będziesz musiał przepisać kod bo $connection->query() nie będzie już działać.

Czyli zamiast

[PHP] pobierz, plaintext 
$wynik = $connection->query()
[PHP] pobierz, plaintext

mam wpisać

[PHP] pobierz, plaintext 
$wynik = $db_connection->query()
[PHP] pobierz, plaintext

?
Dzięki za pomoc :)

Cytat(MESSIAH :) @ 20.08.2015, 08:22:39 )

Mysql oczekuje jakiś jeden parametr a otrzymał Goowno. Innymi słowy :

[PHP] pobierz, plaintext 
mysql_fetch_assoc ( $wynik ))
[PHP] pobierz, plaintext

podstaw odpowiedni parametr do $wynik zobacz w dokumentacji - pewnie chodzi o połączenie z bazą danych.

Ale zmienna $wynik odwołuje sie do

[PHP] pobierz, plaintext 
$wynik = $connection->query()
[PHP] pobierz, plaintext

, zaś $connection do

[PHP] pobierz, plaintext 
$connection = dbconn::instance();
[PHP] pobierz, plaintext

.
W takim razie jak zmienię skrypt logowania na przykład na ten zaproponowany przez kapslokk i powinno być wszystko ok?

viking

20.08.2015, 17:57:37

Ogólnie cały ten kod singletona, twojej klasy db nadaje się do kosza chyba tylko. Przede wszystkim stare mysql_ zamiast PDO albo Mysqli. Zero możliwości, niezabezpieczony i tworzy kłopoty. Jest jakiś powód dla którego piszesz w ten sposób?

kecajs

20.08.2015, 18:36:47

Cytat(viking @ 20.08.2015, 18:57:37 )

Nie, nie ma żadnego. Swego czasu ktoś mi napisał taki skrypt do łączenia się z bazą i go używam do teraz.
Czy w zupełności wystarczy mi skrypt zaproponowany przez kapslokk?

viking

20.08.2015, 18:45:42

Kapslokk zaproponował gołą funkcję łączącą się z bazą. Zrób to nowocześnie i bezpiecznie.

kecajs

20.08.2015, 20:02:15

A jak to zrobić nowocześnie i bezpiecznie? Czy dla celów testowych wystarczy ta goła funkcja łącząca się z bazą?

viking

20.08.2015, 20:21:34

Odpal mysql_error() w kodzie po kolejnych krokach oraz zapoznaj się z Temat: Jak poprawnie zadac pytanie oraz przygotowac srodowisko pracy jak wyświetlać błędy. Nowocześnie to PDO + prepared statements.

kecajs

20.08.2015, 20:22:07

Trochę pozmieniałem.
Do pliku db_con.php wrzuciłem to, co zaproponował kapslokk. Z kolei kod w pliku pokaz.php wygląda teraz tak. (od części nawiązującej do bazy danych).

[PHP] pobierz, plaintext 
<?php
 
								require "db_con.php";
								$connection = connection();
									// tutaj wywolanie zapytania do bazy
								if (isset($_GET['LacznikID']) && $_GET['LacznikID'] != '') {
								$tut_nr = mysql_real_escape_string($_GET['LacznikID']);
								$zapytanie = "
									SELECT 
										LacznikID, 
										Nazwa, 
										Material, 
										DostawcaID, 
										ProducentID, 
										Lokacja, 
										Waga, 
										Stan 
									FROM 
										tb_laczniki_main 
									WHERE 
										LacznikID=.$tut_nr";
								$wynik = $connection->query($zapytanie);
								}								
 
								while ($r = mysql_fetch_assoc($wynik)) {
                    echo '<tr>';
                    echo '<td width="20" class="table_small">' . $r['LacznikID'] . '</td>';
                    echo '<td width="130" class="table_small">' . $r['Nazwa'] . '</td>';
                    echo '<td width="40" class="table_small">' . $r['Material'] . '</td>';
                    echo '<td width="60" class="table_small">' . $r['DostawcaID'] . '</td>';
                    echo '<td width="60" class="table_small">' . $r['ProducentID'] . '</td>';
                    echo '<td width="50" class="table_small">' . $r['Lokacja'] . '</td>';
                    echo '<td width="40" class="table_small">' . $r['Waga'] . '</td>';
		    echo '<td width="40" class="table_small">' . $r['Stan'] . '</td>';
                    echo "</tr>";
                } 
                ?>
[PHP] pobierz, plaintext

Teraz mam taki komunikat:
Fatal error: Call to a member function query() on a non-object in ...
Jak to ugryźć?

viking

21.08.2015, 06:56:02

$connection = connection(); // tu wywołujesz funkcję

$connection->query($zapytanie); // tutaj powinien być obiekt.

Poczytaj jakiś kurs o podstawach. Tworzenie obiektów a zamiast tak rzeźbić użyj PDO.

kecajs

25.08.2015, 21:24:54

Znalazłem coś takiego na Wikibooks:

[PHP] pobierz, plaintext 
<?php
$mysql_host = 'localhost'; //lub jakiś adres: np sql.nazwa_bazy.nazwa.pl
$port = '3307'; //domyślnie jest to port 3306
$username = 'login';
$password = 'hasło';
$database = 'nazwa_bazy'; //'produkty'
 
try{
	$pdo = new PDO('mysql:host='.$mysql_host.';dbname='.$database.';port='.$port, $username, $password );
	echo 'Połączenie nawiązane!';
}catch(PDOException $e){
	echo 'Połączenie nie mogło zostać utworzone.<br />';
}
?>
[PHP] pobierz, plaintext

Podstawiłem tu moje dane i jest to plik db_con.php

Znalazłem tu także kod umożliwiający połączenie się z bazą:

[PHP] pobierz, plaintext 
<?php
 
   try
   {
      $pdo = new PDO('mysql:host=localhost;dbname=produkty', 'root', 'root');
      $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
      $stmt = $pdo->query('SELECT id, nazwa, opis FROM produkty');
      echo '<ul>';
      foreach($stmt as $row)
      {
          echo '<li>'.$row['nazwa'].': '.$row['opis'].'</li>';
      }
      $stmt->closeCursor();
      echo '</ul>';
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
   }
?>
[PHP] pobierz, plaintext

I teraz zastanawiam się, jak podmienić ten mój kod w pliku pokaz.php i wykorzystać ten powyższy kod, jednocześnie nie budować kompletnie mi w tym przypadku nie potrzebnej tabeli z wierszami wynikowymi, bo jak pisałem we wcześniejszych postach - w tym przypadku wynik będzie zawsze jednowierszowy.
I proszę mnie potraktować trochę ulgowo, bo to co robię, ma na celu jedynie przedstawić możliwości i ogólny zarys funkcjonowania aplikacji. Po akceptacji osób decyzyjnych i tak zajmie się tym programista. Ja jestem początkującym amatorem próbującym coś tam klecić, a nie uważam się za programistę. Jestem ekonomistą :)

Pozdrawiam

Hmm, no cóż, wygląda na to, że pozostałem sam na polu boju... Nic nowego. Temat można zamknąć, bo po co ma wisieć martwy.

viking

26.08.2015, 05:36:16

Ale jaki jest tym razem problem konkretnie? Piszesz że nie chcesz budować tabeli - i nie budujesz. Masz generowana listę ul-li.

kecajs

26.08.2015, 12:55:41

Chodzi o to, w jaki sposób przedstawić wynik dla jednego konkretnego artykułu. Dla większej liczby budujemy tabelę funkcją mysql_fetch_assoc(), zaś dla jednego, konkretnego wyniku jak pokazać te informacje?
Z racji tego, że chodzi mi tylko o zaprezentowanie funkcjonalności, a nie tworzę aplikacji do użytkowania, więc sposób łączenia z bazą pozostawiam tak, jak w początkowych wpisach przedstawiłem - nie wykorzystuję pdo i mysqli, tylko mysql.
Dzięki za odpowiedź.

viking

26.08.2015, 13:28:45

W zapytaniu LIMIT 1. Wybranie jednego rekordu w PHP http://www.php.net/manual/en/pdostatement.fetch.php

kecajs

26.08.2015, 15:01:47

OK, ale czy do tego celu muszę stosować funkcję mysql_fetch_array(), czy też mogę zaprezentować te dane w inny sposób? W ogóle nie chcę ich prezentować w układzie tabelarycznym (nagłówek u góry i dane poniżej), a bardziej w układzie formularza (nazwa pola i obok dane, np. |Nr artykułu| 1|Nazwa artykułu|Śruba M16 x 110| i tak dalej. Jak to mogę zaprezentować?

nospor

26.08.2015, 15:05:04

fetch() zwraca ci jeden rekord w postaci tablicy. Jak ty to sobie wyswietlisz i gdzie to tylki i wylacznie Twoja sprawa.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.