แนวทางปฏิบัติที่ดีที่สุดของระบบไฟล์

ฉันกำลังทำงานกับส่วนขยายของ Magento 2 ที่ต้องอ่านไฟล์จากระบบไฟล์
เมื่อใช้ PHP ดมกลิ่นโดยใช้มาตรฐาน ECGM2 ก็บ่นเกี่ยวกับความจริงที่ว่าฉันใช้ฟังก์ชั่นเหมือนหรือ basenamedirname

ห้ามใช้ฟังก์ชั่น dirname ()

หรือ

ห้ามใช้งานฟังก์ชันพื้นฐาน ()

ฉันควรใช้ wrapper แบบไหนเพื่อให้ได้เอฟเฟกต์เดียวกัน

[แก้ไข]
นี่คือรหัสบางส่วน แต่ไม่เกี่ยวข้องกับคำถาม
ฉันมีคลาสคอลเล็กชันที่ขยาย\Magento\Framework\Data\Collection\Filesystemคลาสและฉันต้องการแสดงรายการคอลเล็กชันนี้ในกริด (องค์ประกอบ UI) และหนึ่งในแอ็คชั่นในกริดคือการดาวน์โหลด
สำหรับสิ่งนี้ฉันต้องได้รับชื่อจริงของไฟล์เพื่อที่ฉันจะได้ส่งไฟล์ไปยังการดำเนินการดาวน์โหลด

    // here $file is dynamic and it can be
    // folder/filename.xml or folder/subfolder/file.tar.gz
    //so there is no strict number of folders and subfolders.
    $file = $downloader->getRelativePath($packageName);
    $relativeFile = UmcFilesystem::VAR_DIR_NAME . '/' .$file;
    $absoluteFile = $rootDir->getAbsolutePath($relativeFile);
    if ($rootDir->isFile($relativeFile) && $rootDir->isReadable($relativeFile)){
        //I don't want to use `explode` just for the sake of avoiding basename
        $fileName = basename($absoluteFile);
        $this->fileFactory->create(
            $fileName,
            null,
            DirectoryList::VAR_DIR,
            'application/octet-stream',
            $rootDir->stat($relativeFile)['size']
        );

        $resultRaw = $this->resultRawFactory->create();
        $resultRaw->setContents($rootDir->readFile($relativeFile));
        return $resultRaw;
    } else {
       ...
    }

ฉันต้องการบางสิ่งเช่นนั้นเมื่อเร็ว ๆ นี้ ทางออกเดียวที่ฉันพบเพื่อรับbasenameและdirnameใช้:

\ วีโอไอพี \ Framework \ Filesystem \ ไอโอ \ ไฟล์

protected function someFunction()
{
    /** @var \Magento\Framework\Filesystem\Io\File $fileSystemIo **/
    $fileInfo = $this->fileSystemIo->getPathInfo('<absolutePath>');
    $basename = $fileInfo['basename'] 
    $dirname = $fileInfo['dirname'];
}

ก่อนหน้านั้นฉันลองใช้Magento\Framework\Filesystem\Directory\WriteและgetDriver()ไม่ประสบความสำเร็จ กับพวกเขาคุณจะได้รับทุกอย่างสวยมาก basenameแต่ไม่

โชคดีที่คอมไพล์ให้เราดูเมื่อมีการห้ามใช้ชื่อเล่นและชื่อเบสเหตุผลก็ชัดเจน "เพิ่มไฟล์"

หากมองปัญหาของโครงการ ECG คุณสามารถเห็นปัญหาที่ปิดเช่นสิ่งที่ไม่ดีใน file_exists หรือไม่ # 33 , ฟังก์ชั่นข้อผิดพลาด # 26 , บางสิ่งบางอย่างที่ไม่ดีในฟังก์ชั่นนี้หรือไม่? # 17 , บริบท / คำอธิบายสำหรับกฎ # 12 , การใช้ function iconv () ถูกห้าม # 14ซึ่งจะทำให้ฉันคิดว่ารายการเริ่มต้นของฟังก์ชันที่ต้องห้ามไม่ได้รับการพิจารณามากเกินไปและ magento อาจคล้อยตามการเปลี่ยนแปลง รายการที่ต้องห้าม

ค้นหาm2 codebaseแสดงผล ~ = 78 basename ผสมของตัวแปรและรหัสที่จริงเรียก basename รวมทั้งของฉันชื่นชอบ

ฉันคิดว่าถ้าฉันเป็นคุณฉันจะโพสต์ปัญหาใน github และถาม zlik ว่าเขายังคิดว่าพวกเขาอยู่ที่นั่นหรือถ้า M2 ให้เสื้อคลุม

คุณสามารถใช้วัตถุของSplFileInfo()คลาสได้

$info = new SplFileInfo('/path/to/foo.txt');
var_dump($info->getFilename())

อาจจะใช้งานได้

คุณยังสามารถดูนี้ URL

ข้อเสนอแนะของฉันจะใช้Magento/Backupโมดูลเป็นตัวอย่าง

วิธีเขียนคลาส action download น่าสนใจที่จะดูเพราะมันเกี่ยวข้องกับไฟล์จริงที่จะดาวน์โหลด:

public function execute()
{
    /* @var $backup \Magento\Backup\Model\Backup */
    $backup = $this->_backupModelFactory->create(
        $this->getRequest()->getParam('time'),
        $this->getRequest()->getParam('type')
    );

    if (!$backup->getTime() || !$backup->exists()) {
        /** @var \Magento\Backend\Model\View\Result\Redirect $resultRedirect */
        $resultRedirect = $this->resultRedirectFactory->create();
        $resultRedirect->setPath('backup/*');
        return $resultRedirect;
    }

    $fileName = $this->_objectManager->get('Magento\Backup\Helper\Data')->generateBackupDownloadName($backup);

    $this->_fileFactory->create(
        $fileName,
        null,
        DirectoryList::VAR_DIR,
        'application/octet-stream',
        $backup->getSize()
    );

    /** @var \Magento\Framework\Controller\Result\Raw $resultRaw */
    $resultRaw = $this->resultRawFactory->create();
    $resultRaw->setContents($backup->output());
    return $resultRaw;
}

สำหรับฉันคุณควรดูวิธีการสร้างไฟล์เพื่อดาวน์โหลดโดยใช้\Magento\Framework\App\Response\Http\FileFactoryและgenerateBackupDownloadNameจากMagento\Backup\Helper\Data(โปรดสังเกตการใช้ OM;)

อีกหนึ่งบิตที่น่าสนใจ

สิ่งที่น่าสนใจอีกอย่างที่คุณควรพิจารณาคือgetStorageDataวิธีการMagento\MediaStorage\Model\ResourceModel\File\Storage\Fileที่ตัวเองเรียกใช้โดยตรงdirnameและbasenameถ้าคุณเรียกวิธีหลักนั้นในโมดูลของคุณคุณจะไม่ได้รับข้อผิดพลาดต้องห้าม;)

public function getStorageData($dir = '/')
{
    $files = [];
    $directories = [];
    $directoryInstance = $this->_filesystem->getDirectoryRead(DirectoryList::MEDIA);
    if ($directoryInstance->isDirectory($dir)) {
        foreach ($directoryInstance->readRecursively($dir) as $path) {
            $itemName = basename($path);
            if ($itemName == '.svn' || $itemName == '.htaccess') {
                continue;
            }
            if ($directoryInstance->isDirectory($path)) {
                $directories[] = [
                    'name' => $itemName,
                    'path' => dirname($path) == '.' ? '/' : dirname($path),
                ];
            } else {
                $files[] = $path;
            }
        }
    }

    return ['files' => $files, 'directories' => $directories];
}

ในความคิดที่คล้ายกันก็มีcollectFileInfoต้นกำเนิดมาจากMagento\MediaStorage\Helper\File\Media