Forum PHP.pl

Witam,

Chcę zrobić losowanie nazwy broni która gracze będą dostawać.

Tabele wyglądają tak:

nazwa_bron

1 | Trujący
2 | Hipnotyczny

itp.

typ_bron

1 | Pistolet
2 | Rewolwer


Kod wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
 
error_reporting(E_ALL);
ini_set('display_errors', 1);
 
include ("config.php");
 
$nazwa_bron = mysql_fetch_array(mysql_query("select * from nazwa_bron") or die(mysql_error());
$typ_bron = mysql_fetch_array(mysql_query("select * from typ_bron") or die(mysql_error());
 
// Zliczanie wszystkich wyników w tabeli
 
$liczenie1 = mysql_num_rows(mysql_query("select * from nazwa_bron")) or die(mysql_error());
$liczenie2 = mysql_num_rows(mysql_query("select * from typ_bron")) or die(mysql_error());
 
// Losowanie nazwy broni
 
$losowanie_nazwy = rand(1,$liczenie1);
$losowanie_typu = rand(1,$liczenie2);
 
// Wylosowanie broń
 
$wylosowana_nazwa = mysql_fetch_array(mysql_query("select * from nazwa_bron where id='$losowanie_nazwy'") or die(mysql_error());
$wylosowany_typ = mysql_fetch_array(mysql_query("select * from typ_bron where id='$losowanie_typu'") or die(mysql_error());
 
echo ("$wylosowana_nazwa $wylosowany_typ");
 
 
?>
[PHP] pobierz, plaintext 

Kiedy odpalam kod, nic się nie dzieje. Żadem błąd nie wyskakuje tylko biała pusta strona. O co chodzi?

var_dump($wylosowana_nazwa);
Już coś wyskakuje?

[PHP] pobierz, plaintext 
echo $wylosowana_nazwa['nazwa_broni'];
echo  $wylosowany_typ['typ_broni'];
 
[PHP] pobierz, plaintext 

tam gdzie masz nazwa broni wstawiasz nazwę kolumny z tabeli w której przechowujesz nazwe tej broni analogicznie z tym drugim

Ten post edytował kadlub 22.04.2011, 08:27:24

Nawiasów brak:

[PHP] pobierz, plaintext 
$nazwa_bron = mysql_fetch_array(mysql_query("select * from nazwa_bron") or die(mysql_error()));
$typ_bron = mysql_fetch_array(mysql_query("select * from typ_bron") or die(mysql_error()));
[PHP] pobierz, plaintext 

i tu:

[PHP] pobierz, plaintext 
$wylosowana_nazwa = mysql_fetch_array(mysql_query("select * from nazwa_bron where id='$losowanie_nazwy'") or die(mysql_error()));
$wylosowany_typ = mysql_fetch_array(mysql_query("select * from typ_bron where id='$losowanie_typu'") or die(mysql_error()));
[PHP] pobierz, plaintext 

Polecam jakieś dobre IDE

Btw, po co tyle zapytań jeśli możesz w mysql losować?:

[PHP] pobierz, plaintext 
 $wylosowana_nazwa = mysql_fetch_array(mysql_query("select * from nazwa_bron  ORDER BY RAND()")) or die(mysql_error());
$wylosowany_typ = mysql_fetch_array(mysql_query("select * from typ_bron ORDER BY RAND()")) or die(mysql_error());
[PHP] pobierz, plaintext 

Ten post edytował Hpsi+ 22.04.2011, 08:36:30

Bo niestety to mało optymalne przy dużej liczbie rekordów. To zależy też co ile się takie losowanie odbywa, bo ja często korzystam z RAND() w MySQL jeśli skrypt jest odpalany raz dziennie przed crona

zdaje sobię sprawę z tego ze to średnio optymalne jest. Ale w tym konrketym przypadku: ile może być rodzajów broni 200-300? , ile typów broni 60/100 ? przy takiej ilości rekordów to jest wg mnie dopuszczalne ale to moja opinia

Zrobiłem wersje z losowaniem w mysql

teraz kod wyglada tak:

[PHP] pobierz, plaintext 
<?php
 
 
error_reporting(E_ALL);
ini_set('display_errors', 1);
 
include ("config.php");
 
$nazwa_bron = mysql_fetch_array(mysql_query("select * from nazwa_bron") or die(mysql_error()));
$typ_bron = mysql_fetch_array(mysql_query("select * from typ_bron") or die(mysql_error()));
 
 
// Losowanie broni
 
$wylosowana_nazwa = mysql_fetch_array(mysql_query("select * from nazwa_bron  ORDER BY RAND()")) or die(mysql_error());
$wylosowany_typ = mysql_fetch_array(mysql_query("select * from typ_bron ORDER BY RAND()")) or die(mysql_error());
 
 
 
echo $wylosowana_nazwa['nazwa'];
echo " ";
echo $wylosowany_typ['nazwa'];
 
 
?>
[PHP] pobierz, plaintext 

Wywala błąd:



Jak wyłączę wyświetlanie błędów wszystko działa.

Jedynie mam problem z kodowaniem

Plik php jest zapisany w UTF-8 bez BOM, w bazie danych Metoda porównywania napisów utf8_general_ci a mimo wszystko nie pobiera dobrze poslkich nazw.

Przykładowo: GĹ�oĹ�ny Pistolet

Po pierwsze
Pierwsza dwa zapytania są zbędne:

[PHP] pobierz, plaintext 
$nazwa_bron = mysql_fetch_array(mysql_query("select * from nazwa_bron") or die(mysql_error()));
$typ_bron = mysql_fetch_array(mysql_query("select * from typ_bron") or die(mysql_error()));
[PHP] pobierz, plaintext 

W tym wypadku nie są potrzebne, wiec możesz je wywalić.
Wpisywałeś dane do bazy z pma czy z poziomu strony www?
bo czesto sie zdarza ze jeśli masz stronę kodowana w utf8 , i wrzucasz dane do bazy przez pma to wychodza krzaczki na stronie

A bład ci wywalał przez brak nawiasów tam

Nospor - ja bym raczej płakał

Ten post edytował Hpsi+ 22.04.2011, 09:08:44

Nie wiadomo czy tu się śmiać czy płakać
Błąd zapewne generują ci zapytania, które poprzednik kazał ci usunąć. Nie kasuj nigdy wyświetlania błędów, tylko usuwaj te błędy

Znaczy to bardzo trudno nam stwierdzić gdy nie znamy szczegółów, bo jeśli te losowanie będzie generowane przez użytkowników 1k razy na sekund to może spocić jednak serwer

Ok, już wszystko działa. Usunąłem te dwa pierwsze zapytania bo faktycznie były zbędne.

Co dziwne kodowanie samo się naprawiło. Wyświetla się już poprawnie.


A tak na przyszłość, to w jaki sposób dodawać wpisy do bazy danych aby kodowanie było dobre? Skopiowanie z Notepada++ w kodowaniu UTF8 do pma pewnie nic nie da?

Jak jest z tym losowaniem w MYSQL? Bo luq napisał, że nie jest to optymalne rozwiązanie a mój skrypt będzie bardzo dużo razy dziennie odpalany. Powininenem zrobić to po staremu, czy może zostać losowanie w MYSQL?

w bazie utf_polish_ci
i przy połączniu z bazą
mysql_query('SET NAMES utf8');

Tego nie próbowałem więc nie powiem, ale jedna rada - napisz panel administratora gdzie możesz takie wpisy przez przeglądarke dodawać i problemu nie będzie