Why absolute path constants __DIR__ and __FILE__ should not be used in Symfony

拈花ヽ惹草 提交于 2019-12-21 09:15:10

问题


I use SensioLabs Insight to control my code quality.

For a simple file upload, I have to get the absolute path of my uploads directory:

protected function getUploadRootDir()
{
    // the absolute directory path where uploaded
    return __DIR__.'/../../../../web/'.$this->getUploadDir();
}

Code directly coming from official documentation (How to handle file uploads with Doctrine)

But SLInsight raises a warning if the code analysed contains __DIR__ or __FILE__ PHP magic constants:

__DIR__ and __FILE__ constants may conflict with the Symfony resource overriding system.

How usage of this constants can causes conflicts with Symfony?

And how can I avoid them in my code?


回答1:


In the case of the file upload class, you can probably ignore this error message. But in other cases, it's better o use the Symfony file locator instead of hardcoding file paths. For example:

$path = $this->get('kernel')->locateResource('@AppBundle/Resources/config/services.xml');

Instead of:

$path = __DIR__.'/../../../src/Acme/AppBundle/Resources/config/services.xml'



回答2:


Well, this is actually something that SensioLabs Insight does not handles properly. It warns against using the constants because of the resource overriding system, but in many cases, these constants are used in places which are unrelated to the resource overriding system (and this is probably the case for your code here). So you can ignore the warning in this case




回答3:


If you are creating a third-party bundle and want to locate some resources, the (good) solution proposed by @Javier is not applicable as it throws an exception:

ServiceNotFoundException in ContainerBuilder.php line 816:
You have requested a non-existent service "kernel".

In this case the solution is to use $this->getPath(), a method inherited by the BundleNameBundle from the Symfony\Component\HttpKernel\Bundle\Bundle class.

This returns the same result of realpath(__DIR__).

So doing $this->getPath() . '/Resources/config/doctrine/mappings' is the same as realpath(__DIR__ . '/Resources/config/doctrine/mappings').

Originally proposed here.



来源:https://stackoverflow.com/questions/34331370/why-absolute-path-constants-dir-and-file-should-not-be-used-in-symfony

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!