[php]Pierwsze funkcje, Zobaczcie czy jest to poprawnie Opcje

[redelek]

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 
<?php
$dbhost = 'localhost';
$dbuser = 'root';
$dbpass = 'dropsy';
$dbname = 'link';
$polacz = @mysql_connect($dbhost, $dbuser, $dbpass);
########## FUNKCJE ##########
function polaczMySQL(){
 
if (!$polacz) {
	 die('Nie można się połączyć: ' . mysql_error());
 }else
	 echo 'OK'.'<br>';
	 mysql_select_db($dbname);
}
 
function zamknijMySQL()
{
$zamknij= mysql_close($polacz);
 
if (!$zamknij){
	die ('Nie mogę się rozłączyć z bazą: ' . mysql_error());
 }else
	  echo 'Nastąpiło rozłączenie :)';
}
?>
[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

Ten post edytował redelek 23.04.2008, 10:41:57

[MMPrime]

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]

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]

- 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 
<?php
function polacz($dbhost, $dbuser, $dbpass, $dbname) {
?>
[PHP] pobierz, plaintext 

[redelek]

- 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 
<?php
function polacz($dbhost, $dbuser, $dbpass, $dbname) {
?>
[PHP] pobierz, plaintext 

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

A w skrypcie mam ją wywoływać tak

[PHP] pobierz, plaintext 
<?php
polaczMySQL('localhost','root', 'dropsy','link');
?>
[PHP] pobierz, plaintext 

czy to o to chodzi?

Ten post edytował redelek 23.04.2008, 11:11:02

[ayeo]

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 
<?php
function polaczMySQL()
{
$dbhost = "localhost";
$dbuser = "root";
$dbpass = "dupa8"; // ;)
$dbname = "linki";
$polacz = mysql_connect($dbhost, $dbuser, $dbpass);
if (!$polacz) 
{
	return FALSE;
}
else
{
  mysql_select_db($dbname);
  return $polacz;
}?>
[PHP] pobierz, plaintext 

Pozdrawiam!

[redelek]

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 
<?php
function polaczMySQL()
{
$dbhost = "localhost";
$dbuser = "root";
$dbpass = "dupa8"; // ;)
$dbname = "linki";
$polacz = mysql_connect($dbhost, $dbuser, $dbpass);
if (!$polacz) 
{
	return FALSE;
}
else
{
  mysql_select_db($dbname);
  return $polacz;
}?>
[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]

Jakie wszystkie funkcje? Właśnie tylko w tej jednej!

[piotrooo89]

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 
<?php
$dbhost = "localhost";
$dbuser = "root";
$dbpass = "dupa8"; // ;)
$dbname = "linki";
$polacz = mysql_connect($dbhost, $dbuser, $dbpass);
if (!$polacz) 
{
	return FALSE;
}
else
{
  mysql_select_db($dbname);
  return $polacz;
}?>
[PHP] pobierz, plaintext 

nazywasz go sobie np.: mysql.inc

i includujesz go sobie w pliczku .php np:

[PHP] pobierz, plaintext 
<?php
include('mysql.inc');
?>
[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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[l0ud]

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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Wg. mnie podstawą jest stosowanie rozszerzenia .php w takich plikach.

[ayeo]

To jeszcze zależy gdzie ten plik będziesz trzymał i jak masz skonfigurowany serwer.