Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [php]Pierwsze funkcje
Forum PHP.pl > Forum > Przedszkole
redelek
23.04.2008, 10:40:31
Witam,

Piszę prostą biblioteczkę w php:).
Bardzo bym prosił o pomoc czy jest to poprawne czy raczej nie bawić się w coś takiego. Będę wdzięczny za wskazówki 
[PHP] pobierz, plaintext 
  1. <?php
  2. $dbhost = 'localhost';
  3. $dbuser = 'root';
  4. $dbpass = 'dropsy';
  5. $dbname = 'link';
  6. $polacz = @mysql_connect($dbhost, $dbuser, $dbpass);
  7. ########## FUNKCJE ##########
  8. function polaczMySQL(){
  9.  
  10. if (!$polacz) {
  11. 	 die('Nie można się połączyć: ' . mysql_error());
  12.  }else
  13. 	 echo 'OK'.'<br>';
  14. 	 mysql_select_db($dbname);
  15. }
  16.  
  17. function zamknijMySQL()
  18. {
  19. $zamknij= mysql_close($polacz);
  20.  
  21. if (!$zamknij){
  22. 	die ('Nie mogę się rozłączyć z bazą: ' . mysql_error());
  23.  }else
  24. 	  echo 'Nastąpiło rozłączenie :)';
  25. }
  26. ?>
[PHP] pobierz, plaintext


Z tą zmienną połącz musiałem wyskoczyć poza funkcję, bo jak była w niej to nie działała, mi funkcja zamknijMySQL().
Coś pewnie o globalnych zmiennych trzeba by było poczytać , ale jeszcze nie wiem nic o tym i zrobiłem tak jak wyżej.

Dzięki za wskazówki

Pozdrawiam
Redelek
MMPrime
23.04.2008, 10:43:46
Przekaż wszelakie dane przez parametry funkcji i będzie grało.
Kod
function polaczMySQL( $polacz, $dbname )
{
  // ....


Postaraj się też ustandaryzować nazewnictwo zmiennych, teraz stosujesz polskie nazwy i angielskie.
webdice
23.04.2008, 10:45:48
Poczytaj sobie jeszcze o zasięgu zmiennych w funkcjach bo to nie ma prawa działać. Pamiętaj że musisz przekazywać do funkcji zmienne jako parametry jeśli chcesz z nich korzystać.

EDIT: Po co pisać funkcje do łączenie z bazą danych skora tak naprawdę robi coś całkiem innego?
l0ud
23.04.2008, 10:46:48
- funkcji polacz() powinieneś przekazywać dane do bazy, a nie już gotowy link. Przecież ona ma się łączyć z bazą (jak nazwa mówi) a nie tylko sprawdzić czy jest połączenie i wybrać bazę.

[PHP] pobierz, plaintext 
  1. <?php
  2. function polacz($dbhost, $dbuser, $dbpass, $dbname) {
  3. ?>
[PHP] pobierz, plaintext
redelek
23.04.2008, 11:07:34
Cytat(l0ud @ 23.04.2008, 09:46:48 ) *
- funkcji polacz() powinieneś przekazywać dane do bazy, a nie już gotowy link. Przecież ona ma się łączyć z bazą (jak nazwa mówi) a nie tylko sprawdzić czy jest połączenie i wybrać bazę.

[PHP] pobierz, plaintext 
  1. <?php
  2. function polacz($dbhost, $dbuser, $dbpass, $dbname) {
  3. ?>
[PHP] pobierz, plaintext 


[PHP] pobierz, plaintext 
  1. <?php
  2. function polaczMySQL ($dbhost, $dbuser, $dbpass, $dbname){
  3. $polacz = @mysql_connect($dbhost, $dbuser, $dbpass);
  4. if (!$polacz) {
  5. 	 die('Nie można się połączyć: ' . mysql_error());
  6.  }else
  7. 	 echo 'OK'.'<br>';
  8. 	 mysql_select_db($dbname);
  9. }?>
[PHP] pobierz, plaintext


A w skrypcie mam ją wywoływać tak
[PHP] pobierz, plaintext 
  1. <?php
  2. polaczMySQL('localhost','root', 'dropsy','link');
  3. ?>
[PHP] pobierz, plaintext


czy to o to chodzi?
ayeo
23.04.2008, 11:14:19
Mi się wydaje, że funkcja polaczMYSQL(); jest bez sensu bo jest identyczna w działaniu z mysql_connect(); Chyba, że zapiszesz na sztywno dane (host, login, hasło) w funkcji. Funkcja powinna zwracać uchwyt połączenia lub FALSE w przypadku niepowodzenia. 

[PHP] pobierz, plaintext 
  1. <?php
  2. function polaczMySQL()
  3. {
  4. $dbhost = "localhost";
  5. $dbuser = "root";
  6. $dbpass = "dupa8"; // ;)
  7. $dbname = "linki";
  8. $polacz = mysql_connect($dbhost, $dbuser, $dbpass);
  9. if (!$polacz) 
  10. {
  11. 	return FALSE;
  12. }
  13. else
  14. {
  15.   mysql_select_db($dbname);
  16.   return $polacz;
  17. }?>
[PHP] pobierz, plaintext


Pozdrawiam!
redelek
23.04.2008, 11:52:37
Cytat(ayeo @ 23.04.2008, 10:14:19 ) *
Mi się wydaje, że funkcja polaczMYSQL(); jest bez sensu bo jest identyczna w działaniu z mysql_connect(); Chyba, że zapiszesz na sztywno dane (host, login, hasło) w funkcji. Funkcja powinna zwracać uchwyt połączenia lub FALSE w przypadku niepowodzenia. 

[PHP] pobierz, plaintext 
  1. <?php
  2. function polaczMySQL()
  3. {
  4. $dbhost = "localhost";
  5. $dbuser = "root";
  6. $dbpass = "dupa8"; // ;)
  7. $dbname = "linki";
  8. $polacz = mysql_connect($dbhost, $dbuser, $dbpass);
  9. if (!$polacz) 
  10. {
  11. 	return FALSE;
  12. }
  13. else
  14. {
  15.   mysql_select_db($dbname);
  16.   return $polacz;
  17. }?>
[PHP] pobierz, plaintext


Pozdrawiam!


Mi właśnie o to chodzi żeby był jeden plik z konfiguracją.
Chciałem go podzielić na sekcje
#### ZMIENNE ####
#### FUNKCJE ####
Plik bym sobie dołączał do wszystkich stron i zamiast pisać całego mysql_connect. będę pisał polaczMYSQL(); i po sprawie
Tak jak ty napisałeś to po zmianie hasła do bazy będę musiał przelecieć wszystkie funkcje i wyszukać w nich $dbpass i zmienić to.
ayeo
23.04.2008, 12:05:00
Jakie wszystkie funkcje? Właśnie tylko w tej jednej!
piotrooo89
23.04.2008, 12:51:00
jeśli moglbym Ci coś poradzić z czego sam kożystam to stworzenie sobie pliczku do laczenia sie z bazą danych.

oto on:

[PHP] pobierz, plaintext 
  1. <?php
  2. $dbhost = "localhost";
  3. $dbuser = "root";
  4. $dbpass = "dupa8"; // ;)
  5. $dbname = "linki";
  6. $polacz = mysql_connect($dbhost, $dbuser, $dbpass);
  7. if (!$polacz) 
  8. {
  9. 	return FALSE;
  10. }
  11. else
  12. {
  13.   mysql_select_db($dbname);
  14.   return $polacz;
  15. }?>
[PHP] pobierz, plaintext


nazywasz go sobie np.: mysql.inc

i includujesz go sobie w pliczku .php np:

[PHP] pobierz, plaintext 
  1. <?php
  2. include('mysql.inc');
  3. ?>
[PHP] pobierz, plaintext


jak zrobisz taki myk to przy znianie jakis danych w np dostępie do bazy edytujesz tylko ten plik i masz spokoj.

mam nadzije że Ci sie to przyda smile.gif
l0ud
23.04.2008, 13:21:03
Cytat
jak zrobisz taki myk to przy znianie jakis danych w np dostępie do bazy edytujesz tylko ten plik i masz spokoj.


Masz spokój, dopóki ktoś nie odnajduje pliku mysql.inc, ściąga i nie rozwala Ci bazy winksmiley.jpg Wg. mnie podstawą jest stosowanie rozszerzenia .php w takich plikach.
ayeo
23.04.2008, 13:24:42
To jeszcze zależy gdzie ten plik będziesz trzymał i jak masz skonfigurowany serwer.
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.