1

I've been learning PHP for a couple of days now and I came across a strange issue for me. I have two files :

login.php that stores database access data (user, pw, host, db)
security.php that have functions that manipulate strings/queries to prevent sql/xss injections and also stores salt prefix and suffix for password hashing.

Then I have my index.php, register.php and signin.php. My problem is : when I put require_once '../login.php'; in register.php I can normally call a variable or function like $var. On the other hand, exactly the same pattern in signin.php is not working, PHP tells that that variables are not declared. I tried to do : global $var, and it was working but I have feeling that it's completely insecure.

1) What is the proper way to solve this problem?
2) Is it connected with app architecture?
3) Should all functions (and etc.) that use db connection be written in a single file?

Guy
  • 10,931
  • 5
  • 36
  • 47
Tomek Fiechowski
  • 37
  • 6
  • See also [Reference: What is variable scope, which variables are accessible from where and what are "undefined variable" errors?](http://stackoverflow.com/q/16959576) – mario Jul 09 '15 at 11:04

2 Answers2

0

Step 1 - Create a php file , write all database connection into it(example- db.php)
Step 2 - Include this php (db.php) wherever you have a database related operation.

require 'db.php';  //add your database php file here..


This will be easy to organize, and minimizes errors when you migrate your app from one server to another in php there is require and include. require is good since it will stop executing code if file is not available.

Also this is easy to organize and minimizes the errors caused when migrating from one server to another

Unni Babu
  • 1,839
  • 12
  • 16
0
  1. install composer
  2. Put all your code into functions in classes with your namespace (as static functions if you must).
  3. Save the classes like specified in psr4 in lib dir.
  4. Create composer.json file as specified on composer website and include your lib dir as autoload as it says in composer docs.
  5. require_once projectroot/vendor/autoload.php in your files.

Call your class methods (with namespace) instead of functions. Be lazy. Avoid require/include hell.

Community
  • 1
  • 1
OIS
  • 9,833
  • 3
  • 32
  • 41