PHP - defining classes inside a function
Solution 1
Yes it is bad practice; no it's not.
You're likely going to get both answers, and here's why:
If you use __autoload
(or equivalent), calling:
function someFunc()
{
$n = new UndefinedClassName();
}
is equivalent to:
function someFunc()
{
include('path/to/UndefinedClassName.php');
//may be require_once, include_once, include, or require
//depending on how autoload is written
$n = new UndefinedClassName();
}
But you'll get better performance out of your code if you avoid using __autoload
. And to maintain your code, it's better to put all your includes
at the top of the script like you would for import
statements in other languages.
include('path/to/UndefinedClassName.php');
...code...
function someFunc()
{
$n = new UndefinedClassName();
}
I would suggest consistency. If you consistently call the include
in the functions, you shouldn't have too many issues, but I would choose imports at the beginnings of files, or as autoloads
.
Solution 2
I would avoid it.
It is not what another developer would expect, and, as such, would reduce the maintainability of your code.
Solution 3
This is how class loaders work. This is not necessarily bad practice.
Depends on what the function does and why you are doing this. Using autoloading may be more appropriate.
Solution 4
This is generally a bad practice and should be avoided. You should probably consider using an autoloader instead.
Related videos on Youtube
Comments
-
Alex almost 2 years
Is this a bad practice?
like:
function boo(){ require_once("class.moo.php"); } ...
?
-
zzzzBov over 13 years"it's not what another developer would expect" unless of course the other developer is doing the same thing...
-
Reed Copsey over 13 years@zzzzBov: Well, I mean a "normal" developer, of course. Non-standard practices are bad practices, IMO.
-
zzzzBov over 13 yearsi agree with what you're saying on some levels, but it's important to distinguish which standard you're talking about. There isn't an official standard, there is only the way other programmers have written code in the past. The closest thing to an official standard is the coding specification and syntax, which is liberal for a reason.
-
keithjgrant over 13 yearsYes, but autoloader doesn't have the same readability/maintainability issues.
-
Reed Copsey over 13 yearsTrue - I'm not talking about an "official standard", but more of a "most common practice". That being said, I never mentioned a "standard" in my answer for a reason ;)
-
Reed Copsey over 13 years@zzzzBov: I'd see it as similar to why I would say it's bad practice to use any of these without very good reason: stackoverflow.com/questions/1995113/strangest-language-feature
-
Dai over 5 yearsWhat if there are other declarations inside
UndefinedClassName.php
that are syntactically illegal when used inside a function definition?