Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: PHP System News'ów - problemy
Forum PHP.pl > Forum > PHP
Kam1k
1.06.2017, 14:42:56
Witam, od razu mówię, że cały system NEWS nie jest mój, autorem jest drax. Jednak temat na innym forum nie jest wspierany od kilku lat, ja postanowiłem zobaczyć jak to wszystko działa, nanoszę poprawki i estetykę, jednak nie wszystko działa jak powinno. Od razu mogę także powiedzieć, że moja wiedza na poziomie PHP jest podstawowa. Serwer postawiony na XAMPP, MySQL, Apache etc.

Jednak co do tematu. Problem jest tak, że wyskakują błędy takie jak:

Kod
Warning: mysqli_query() expects at least 2 parameters, 1 given in C:\xampp\htdocs\news\admin.php on line 110
Warning: mysqli_fetch_array() expects parameter 1 to be mysqli_result, null given in C:\xampp\htdocs\news\admin.php on line 112


Ogólnie baza danych jest postawiona jako kamik_news w phpMyAdmin, jednak ten cały Panel Administracyjny nie do końca działa - niby news dodaje, ale już w bazie go nie dodaje.

Tak jak mówiłem jest tu coś, jednak dużo do poprawki i razem z Wami chciałbym to poprawić tak aby to działało, tak aby ktoś z takiego systemu mógł korzystać, tylko trzeba wiele poprawek. Niektóre już naniosłem, ponieważ PHP poszło do przodu i niektóre zmienne jak mysql na mysqli trzeba było pozmieniać - jednak to amatorka to co robię

Liczę na Waszą pomoc (z tego względu że są ograniczenia co do liczby znaków w poście będę musiał troszke podublować odpowiedzi tak aby to wszystko było zgrane z tematem)


Plik config.php
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. function connect() {
  4.  
  5.     $mysqli_server = "localhost";
  6.     $mysqli_admin = "root";
  7.     $mysqli_pass = "haslo";
  8.     $mysqli_db = "kamik_news";
  9.  
  10.     @mysqli_connect($mysqli_server, $mysqli_admin, $mysqli_pass)
  11.     or die('Blad polaczenia.');
  12.  
  13.     @mysqli_select_db($mysqli_db)
  14.     or die('Bledna baza danych.');
  15. }
  16.  
  17. ?> 
[PHP] pobierz, plaintext


Plik functions.php
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. function showNews()
  4. {
  5. 	$sql1 = mysqli_query('SELECT * FROM news ORDER BY id DESC');
  6. 		while($news = mysqli_fetch_assoc($sql1))
  7. 			{
  8. 				$id = $news['id'];
  9.  
  10. 					$sql2 = mysqli_query('SELECT id FROM comments WHERE post_id=$id');
  11. 						$nr = mysqli_num_rows($sql2);
  12. 						echo 
  13.  
  14. 						'<h2>'.$news['tytul'].'</h2> 
  15. 						<h3>'.$news['data'].'</h3> 
  16. 						<p> '.$news['tresc'].'</p> 
  17.  
  18. 						<p align=\'right\'><a href=\'news.php?id='.$news['id'].'
  19.  
  20. 						\'>Komentarze ['.$nr.']</a>';
  21.             }
  22. }
  23.  
  24. ?>
[PHP] pobierz, plaintext


Plik form.php 
[PHP] pobierz, plaintext 
  1. <?php
  2.  #wygląd formularza
  3.  #pole z nickiem musi mieć name="name"
  4.  #pole z tekstem musi mieć name="message"
  5.  #akcja do której formularz prowadzi musi być 'action="'.$adres.'&amp;kom=dodaj"'
  6.  echo
  7.                 '<h4>Skomentuj</h4>
  8.                 <form action="'.$adres.'&amp;kom=dodaj" method="post">
  9.                              <label for="name">Nick (wymagane)</label>
  10.                             <input id="name" name="name" value="Nick" type="text" size="32" />
  11.                             <label for="message">Wiadomość</label>
  12.                             <textarea id="message" name="message" rows="10" cols="20" size="255"></textarea>
  13.                             <input class="button" type="submit" value="Dodaj" />
  14.                </form>'
  15. ;
  16. ?>
[PHP] pobierz, plaintext


Plik comment.php
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. # Plik reprezentuje ogólny wygląd komentarzy. To co jest między {} po while oznacza wygląd jednego komentarza.
  4. #zmienne:
  5. # $nr - Liczba komentarzy
  6. # $adres - adres strony do kotwicy
  7. # $kome['id'] - id komentarza
  8. # $kome['data'] - data dodania komentarza
  9. # $kome['tresc'] - treść komentarza
  10. # $kome['ip'] - IP osoby która komentuje
  11.  
  12. echo '<h4>'.$nr.' wpisów</h4>';
  13. 	while($kome = mysqli_fetch_assoc($sql2)) {
  14. 		echo  
  15. 		'<fieldset><legend><a name="comment'.$kome['id'].'" href="'.$adres.'#comment'.$kome['id'].'">'.$kome['autor'].'</a> Pisze:</legend><br/><small>'.$kome['data'].'</small><p>'.$kome['tresc'].'</p></fieldset>';
  16.  
  17. 	} 
  18.  
  19. ?>
[PHP] pobierz, plaintext



Plik news.php
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. ob_start();
  4. require ('config.php');
  5. $adres = $_SERVER['REQUEST_URI'];
  6. $a = $_GET['kom'];
  7. $id = $_GET['id'];
  8. if(empty($id))
  9. {
  10. header('Location: index.php'); # Przekierowanie, kiedy zmienna $_GET['id'] jest pusta
  11. exit;
  12. }
  13. $sql1 = mysqli_query("SELECT * FROM news WHERE id=$id LIMIT 1");
  14. $sql2 = mysqli_query("SELECT * FROM comments WHERE post_id=$id");
  15. $nr = mysqli_num_rows($sql2);
  16. $news = mysqli_fetch_assoc($sql1);
  17.  
  18.             echo'<h2>'.$news['tytul'].'</h2>
  19.             <h3>'.$news['data'].'</h3>
  20.                     <p>    '.$news['tresc'].'</p>
  21.                     <p align="right"><a href="news.php?id='.$news['id'].'">Komentarze ['.$nr.']</a>';
  22.  
  23. if(isset($a) && $a == 'dodaj')
  24. {
  25. $nazwa = substr(mysqli_real_escape_string(htmlspecialchars($_POST['name'])),0,32);
  26. $msg = substr(mysqli_real_escape_string(htmlspecialchars($_POST['message'])),0,255);
  27. $data = date("F j, Y , g:i a");
  28. $ip=$_SERVER['REMOTE_ADDR'];
  29. if(empty($_POST['name']) && ($_POST['message']))
  30. {
  31. echo'Wpisz nick i wiadomość';
  32. }
  33. elseif(empty($_POST['name']))
  34. {
  35. echo'Wpisz nick';
  36. }
  37. elseif(empty($_POST['message']))
  38. {
  39. echo'wpisz wiadmość';
  40. }
  41. if(!empty($_POST['name']) && ($_POST['message']))
  42. {
  43. $sql = mysqli_query("INSERT INTO comments VALUES (0,'$id','$nazwa','$data','$msg','$ip')");
  44. }
  45.  
  46. if($sql)
  47. {
  48. header('Location: news.php?id='.$id.'');
  49. exit;
  50. }
  51. else echo mysqli_error();
  52. }
  53. if($nr == 0)
  54. {
  55. echo 'Brak komentarzy.<br />';
  56. }
  57. else
  58. {
  59. include 'templates/comment.php';
  60. }
  61.  
  62. include 'templates/form.php';
  63.  
  64. ob_end_flush();
  65. ?>
[PHP] pobierz, plaintext


Plik admin.php
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. session_regenerate_id();
  4.     error_reporting (E_ALL ^ E_NOTICE);
  5.  
  6. ///////////////////////////////////////////////////////////////////////////////////////		
  7.  
  8. $user = 'test';
  9. $haslo = 'test';
  10. $cookie = hash(sha512, $haslo);
  11.  
  12. ///////////////////////////////////////////////////////////////////////////////////////	
  13.  
  14. $a = $_GET['id'];
  15. $b = $_GET['nr'];
  16. $c = $_GET['co'];
  17. $d = $_GET['n'];
  18.  
  19. ///////////////////////////////////////////////////////////////////////////////////////	
  20.  
  21. 	if($_GET['action']=="logout") {
  22.  
  23. 		session_unset();
  24. 		session_destroy();
  25. 		session_start();
  26. 		setcookie("login", 0, time()); 
  27.  
  28. 	}	elseif($_GET['action']=="login") {
  29.  
  30. 		if ($_POST['username']== $user&&$_POST['password']==$haslo)  $_SESSION['login']=$cookie;
  31. 			if (isset($_POST['zapamietaj'])) setcookie("login", $cookie, time()+60*60*24*31, NULL, NULL, NULL, TRUE); header('Location: admin.php'); exit;
  32. 		}
  33.  
  34. ///////////////////////////////////////////////////////////////////////////////////////	
  35.  
  36. 	if ($_SESSION['login'] != $cookie AND $_COOKIE['login'] != $cookie) {
  37.  
  38. 		die('<html>
  39. 		<head>
  40. 		<meta charset="UTF-8">
  41. 		<title>System newsów</title>
  42. 		</head>
  43. 		<body class="log">
  44. 			<fieldset class="login"><form method="POST" action="?action=login">
  45. 				<label for="username">Nazwa użytkownika</label><input type="text" id="username" name="username" size="20" />
  46. 				<label for="password">Hasło</label><input id="password" name="password" type="password" size="20" />
  47. 				<label for="zapamietaj">Zapamiętaj logowanie</label><input id="zapamietaj" name="zapamietaj" type="checkbox"/>
  48. 				<input type="submit" value="Loguj">
  49. 			</form>
  50. 			</fieldset></body></html>');
  51. 	}
  52.  
  53. ///////////////////////////////////////////////////////////////////////////////////////	
  54.  
  55. require ('config.php');
  56.  
  57. 	echo'<div style="width:800px;margin:20px auto;"><div style="float:left;"><a href="admin.php?id=dodaj">Dodaj Newsa</a><br><a href="admin.php?id=pokaz">Zarządzaj Newsami</a><br><a href="admin.php?action=logout">Wyloguj</a></div><div style="float:right;">';
  58.  
  59. ///////////////////////////////////////////////////////////////////////////////////
  60.  
  61. 	if (isset($a) && $a == 'add') {
  62.  
  63. 			$tytul = $_POST['tytul'];
  64. 			$tresc = $_POST['tresc'];
  65. 			$data = date ("F j, Y , g:i a");
  66.  
  67. 			mysqli_query("INSERT INTO news VALUES(0,'$tytul','$tresc','$data')");
  68. 			echo '<fieldset><legend>Udana Akcja</legend>Pomyślnie dodałem newsa o tytule: <b>'.$tytul.'</b> i o treści: <b>'.$tresc.'</b></fieldset>';
  69.  
  70. 	}	elseif(isset($a) && $a=='usun' && isset($b)) {
  71.  
  72. 		   $sql = mysqli_query("DELETE FROM news WHERE id='$b'");
  73. 			$sql = mysqli_query("DELETE FROM comments WHERE post_id='$b'");
  74. 			if($sql)echo '<fieldset><legend>Udana Akcja</legend>Pomyślnie usunąłem newsa numer '.$b.' oraz komentarze do niego!</fieldset>';
  75. 			else echo '<fieldset><legend>Akcja Nie udana</legend>Nie udało się usunąć newsa<br> '.mysql_error().'</fieldset>';
  76.  
  77. 		}
  78.  
  79. ///////////////////////////////////////////////////////////////////////////////////////	
  80.  
  81. 	if (isset($a) && $a == 'dodaj') {
  82.  
  83. 			echo '<fieldset><legend>Dodaj Newsa</legend>';
  84. 			echo '<form method="POST" action="admin.php?id=add">
  85. 					<label for="tytul">Tytuł:</label><input id="tytul" type="text" name="tytul" size="64"><br>
  86. 					<label for="tresc">Tresć:</label><textarea id="tresc" rows="5" name="tresc" cols="42"></textarea><br>
  87. 					<input type="submit" value="Dodaj"></form></fieldset>';
  88. 	}
  89.  
  90. ///////////////////////////////////////////////////////////////////////////////////////	
  91.  
  92. 	if (isset($a) && $a=='edytuj' && isset($b))
  93. 	{
  94. 			$link=mysqli_query("SELECT * FROM news WHERE id='$b'");
  95. 			$wiersz=mysqli_fetch_array($link, MYSQL_ASSOC);
  96.  
  97. 			echo '<fieldset><legend>Edytuj Newsa</legend><br>';
  98. 			echo '<form method="POST" action="admin.php?id=wyedytuj&amp;nr='.$wiersz['id'].'">
  99. 					<label for="tytul">Tytuł:</label><input id="tytul" type="text" name="tytul" size="64" value="'.$wiersz['tytul'].'"><br>
  100. 					<label for="tresc">Tresć:</label><textarea id="tresc" rows="5" name="tresc" cols="42">'.$wiersz['tresc'].'</textarea><br>
  101. 					<input type="submit" value="Edytuj">
  102. 				  </form></fieldset>';
  103. 	}
  104.  
  105. ///////////////////////////////////////////////////////////////////////////////////////	
  106.  
  107. 	if(isset($a) && $a=='pokaz') {
  108.  
  109. 			echo '<fieldset><legend>Lista Newsów:</legend><br>';
  110. 			$link=mysqli_query('SELECT * FROM news ORDER BY id desc');
  111.  
  112. 		while($wiersz=mysqli_fetch_array($link, MYSQL_ASSOC))  {    
  113.  
  114. 			$numer = $wiersz['id'];
  115. 			$sql = mysqli_query("SELECT id FROM comments WHERE post_id='$numer'");
  116. 			$nr = mysql_num_rows($sql);
  117. 					echo $wiersz['tytul'].' - '.$wiersz['data'].'| <a href="admin.php?id=edytuj&amp;nr='.$wiersz['id'].'">Edytuj</a> | <a href="admin.php?id=usun&amp;nr='.$wiersz['id'].'">Usuń</a> | <a href="admin.php?id=komentarze&amp;nr='.$numer.'">Zarządzaj Komentarzami['.$nr.']</a><br>'; 
  118. 		}
  119.  
  120. 		} elseif(isset($a) && $a=='komentarze' && isset($b)) {
  121.  
  122. 			$sql = mysqli_query("SElECT * FROM comments WHERE post_id='$b'");
  123.  
  124. 				while($wiersz = mysqli_fetch_assoc($sql)) {
  125. 					echo'<fieldset><legend>'.$wiersz['id'].'. Autor: '.$wiersz['autor'].' Data:'.$wiersz['data'].'</legend><p><strong>IP:</strong> '.$wiersz['ip'].'</p><p>'.$wiersz['tresc'].'</p><p><a     href="admin.php?id=komentarze&amp;co=usun&amp;n='.$wiersz['id'].'">Usuń Komentarz </a></p></fieldset>';
  126. 				}
  127.  
  128. 		} elseif(isset($a) && $a=='komentarze' && isset($c) && $c=='usun' && isset($d)) {
  129.  
  130. 			$sql = mysqli_query("DELETE FROM comments WHERE id='$d'");
  131.  
  132. 				if($sql)
  133. 					echo '<fieldset><legend>Udana Akcja</legend>Pomyślnie usunąłem komentarz!</fieldset>';
  134. 					else 
  135. 						echo'<fieldset><legend>Akcja Nie udana</legend>'.mysql_error().'</fieldset>';
  136.  
  137. 		} elseif(isset($a) && $a=='wyedytuj' && isset($b)) {
  138.  
  139. 				$tytul=$_POST['tytul'];
  140. 				$tresc=$_POST['tresc'];
  141.  
  142. 				$sql = mysqli_query("UPDATE news SET tytul='$tytul' tresc='$tresc' WHERE id='$b'");
  143.  
  144. 				if($sql)
  145. 					echo '<fieldset><legend>Udana Akcja</legend>Pomyślnie wyedytowałem newsa! <br> Jego nowy tytuł to: <b>'.$tytul.'</b>, a treść: <b>'.$tresc.'</b></fieldset>';
  146. 					else echo '<fieldset><legend>Akcja Nie udana</legend>Nie udało się edytować newsa<br> '.mysql_error().'</fieldset>';
  147. 		}
  148.  
  149. echo'</div></div>';
  150.  
  151. ?>
[PHP] pobierz, plaintext


Czekam na waszą pomoc - łatki, poprawki, rozwiązywanie problemów, tak aby to pięknie funkcjonowało smile.gif
To będzie też dobry temat dla tych co szukają takiego systemu, odświeżonego z tego roku wink.gif
Tomplus
1.06.2017, 18:44:10
Zamiast wrzucać tuzin skryptów, to wystarczyło umieścić fragment kodu gdzie jest błąd, a błąd jest tylko w skrypcie admin.php w linii 110.
Jednakże sugerując się resztą kodu. Pewnie gdybyś w funkcji connect() wyłączył @ to miałbyś więcej błędów.
Sprawdź czy połącznie z bazą jest poprawne, a jeżeli nie... to zapraszam do manuala: http://php.net/manual/en/mysqli.query.php pozycja Procedural style
plej
1.06.2017, 19:25:51
http://php.net/manual/en/mysqli.real-escape-string.php
http://php.net/manual/en/mysqli.query.php


[PHP] pobierz, plaintext 
  1. $cos = mysqli_query($poloczenie, $zapytanie);
  2. $cos = mysqli_real_escape_string($polaczenie, $zmienna);
[PHP] pobierz, plaintext


W powyższych linkach masz jak pisać w mysqli. Do niektórych funkcji musisz dodać połączenie z bazą.
Tomplus
2.06.2017, 05:39:56
W sumie dalem mu to samo, tylko chciałbym aby zaczął uczyć się obsługi manuala PHP.
Kam1k
2.06.2017, 08:17:40
No dobrze, jednak w admin.php w linii 110 kod jest wg. wzoru, którego podaliście

[PHP] pobierz, plaintext 
  1. $link = mysqli_query('SELECT * FROM news ORDER BY id desc');
[PHP] pobierz, plaintext


Zrobiłem zgodnie z manulem nowy config.php

[PHP] pobierz, plaintext 
  1. <?php 
  2.  
  3. $mysqli = new mysqli("localhost", "root", "haslo", "kamik_news");
  4.  
  5. // Sprawdzam połączenie
  6. if ($mysqli->connect_errno) {
  7.     printf("Blad Polaczenia, kod bledu: %s\n", $mysqli->connect_error);
  8.     exit();
  9. }
  10.  
  11. $mysqli->close();
  12. ?>
[PHP] pobierz, plaintext


Tak samo, mam błąd gdy chce dodać news'a

[PHP] pobierz, plaintext 
  1. Warning: mysqli_query() expects at least 2 parameters, 1 given in C:\xampp\htdocs\news\admin.php on line 69
[PHP] pobierz, plaintext


Jednak wiadomość pokazuje że dodało news'a, a w bazie nie widnieje
plej
2.06.2017, 17:47:41
jeszcze raz ci dam linka:
http://php.net/manual/en/mysqli.query.php

Procedural style
[PHP] pobierz, plaintext 
  1. mixed mysqli_query ( mysqli $link , string $query [, int $resultmode = MYSQLI_STORE_RESULT ] )
[PHP] pobierz, plaintext


tak masz w manualu.

Nie zrobiłeś jak ci mówiliśmy bo masz dalej to samo :
[PHP] pobierz, plaintext 
  1.     $link = mysqli_query('SELECT * FROM news ORDER BY id desc');
[PHP] pobierz, plaintext


a powyżej z manuala widać że musisz dać 2 parametry. Nawet błąd ci o tym mówi.

wystarczy:
[PHP] pobierz, plaintext 
  1.     $link = mysqli_query($mysqli, 'SELECT * FROM news ORDER BY id desc');
[PHP] pobierz, plaintext


Teraz widzisz o co chodzi? w zwykłym mysql nie musiałeś podawać połączenia do zapytania, ale w mysqli w niektórych funkcjach musisz.


Kam1k
5.06.2017, 08:12:29
Pomogło. Jednak pojawił się kolejny błąd

[PHP] pobierz, plaintext 
  1. Warning: mysqli_query(): Couldn't fetch mysqli in C:\xampp\htdocs\news\admin.php on line 97
[PHP] pobierz, plaintext


Linia 97:
[PHP] pobierz, plaintext 
  1. 			mysqli_query($mysqli, 'INSERT INTO news VALUES(0, \'$tytul\', \'$tresc\', \'$data\')');
[PHP] pobierz, plaintext


Plus dodatkowo dostaje komunikat zwrotny, że news został dodany - jednak to tylko "goły komunikat", nic nie wchodzi do bazy.
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.
Invision Power Board © 2001-2025 Invision Power Services, Inc.