Magento 2: การใช้หรือไม่ใช้ ObjectManager โดยตรง?

ตกลงดังนั้นเมื่อวานนี้เราได้มีการพูดคุยใหญ่กับคนอื่น ๆ จากชุมชนวีโอไอพีเกี่ยวกับการใช้งานโดยตรงของObjectManagerในชั้นเรียน / แม่แบบ

ฉันได้ทราบถึงสาเหตุที่เราไม่ควรใช้ ObjectManager โดยตรงโดยอ้างถึง Alan Kent :

มีสาเหตุหลายประการ รหัสจะใช้งานได้ แต่เป็นแนวปฏิบัติที่ดีที่สุดที่จะไม่อ้างอิงคลาส ObjectManager โดยตรง

• เพราะเราพูดอย่างนั้น! ;-) (แสดงได้ดีกว่าเนื่องจากรหัสที่สอดคล้องกันคือรหัสที่ดี)
• รหัสสามารถใช้กับกรอบการฉีดที่แตกต่างกันในอนาคต
• การทดสอบนั้นง่ายกว่า - คุณผ่านการจำลองแบบอาร์กิวเมนต์สำหรับคลาสที่ต้องการโดยไม่ต้องให้ ObjectManager จำลอง
• มันช่วยให้การพึ่งพาชัดเจนยิ่งขึ้น - เป็นที่ชัดเจนว่ารหัสขึ้นอยู่กับผ่านรายการคอนสตรัคแทนที่จะมีการพึ่งพาที่ซ่อนอยู่ในช่วงกลางของรหัส
• มันสนับสนุนให้โปรแกรมเมอร์คิดเกี่ยวกับแนวความคิดเช่นการห่อหุ้มและการทำให้เป็นโมดูลได้ดีขึ้น - ถ้าคอนสตรัคเตอร์มีขนาดใหญ่บางทีมันอาจเป็นสัญญาณที่รหัสจำเป็นต้องทำการเปลี่ยนใหม่

จากสิ่งที่ฉันเห็นใน StackExchange ผู้คนจำนวนมากมักจะไปหาวิธีแก้ปัญหาที่ง่าย / สั้น / ไม่แนะนำเช่นบางสิ่งเช่นนี้:

<?php 
//Get Object Manager Instance
$objectManager = \Magento\Framework\App\ObjectManager::getInstance();

//Load product by product id
$product = $objectManager->create('Magento\Catalog\Model\Product')->load($id);

แทนที่จะผ่านขั้นตอนที่เจ็บปวด แต่แนะนำให้ทำดังนี้:

• สร้างโมดูล
• ประกาศค่ากำหนด
• ฉีดพึ่งพา
• ประกาศวิธีการสาธารณะ

อย่างไรและนี่มาขึ้นเขียง, ไฟล์วีโอไอพี 2 หลักมักจะเรียก ObjectManager โดยตรง ตัวอย่างอย่างรวดเร็วสามารถพบได้ที่นี่: https://github.com/magento/magento2/blob/develop/app/code/Magento/GoogleOptimizer/Block/Adminhtml/Form.php#L57

ดังนั้นนี่คือคำถามของฉัน:

• ทำไมวีโอไอพีถึงทำในสิ่งที่พวกเขาแนะนำเราไม่ให้ทำ หมายความว่ามีบางกรณีที่เราควรใช้ObjectManagerโดยตรงหรือไม่ ถ้าเป็นเช่นนั้นกรณีเหล่านี้คืออะไร?
• อะไรคือผลของการใช้ ObjectManager โดยตรง ?

คุณไม่ควรใช้ ObjectManager โดยตรง!

ข้อยกเว้นจากกฎคือ:

• ในวิธีการมายากลคงเหมือน __wakeup, serializeฯลฯ
• ในกรณีที่คุณควรจะเข้ากันได้ย้อนหลังของตัวสร้าง
• ในขอบเขตทั่วโลกเช่นในการติดตั้งการทดสอบการรวม
• ในชั้นเรียนที่ต้องการเพียงการสร้างวัตถุเช่นโรงงานพร็อกซี ฯลฯ

เหตุใด M2 จึงเข้าถึงตัวจัดการวัตถุโดยตรงเมื่อเราแนะนำต่อมัน

คำตอบที่โหดร้าย: M2 เป็นพอร์ตของ M1 - ไม่ใช่การเขียนซ้ำทั้งหมด ดังนั้นอย่าคิดว่ารหัส M2 ทั้งหมดได้รับการพอร์ตอย่างสมบูรณ์แบบ (น่าเสียดายที่) เพียงเพราะคุณพบบางสิ่งในฐานรหัส M2 นั่นไม่ได้หมายความว่า "เป็นวิธีที่ดีที่สุดในการทำ" บางครั้งมันก็แค่ "เรายังไม่ได้แก้ไขมัน"

โหดร้ายน้อยลง: ตามคำตอบอื่น ๆ บางครั้งคุณต้องใช้มันเพราะไม่มีทางเลือกอื่น บางครั้งอาจเป็นเพราะเหตุผลด้านความเข้ากันได้ย้อนหลัง และบางครั้งรหัสเฟรมเวิร์กก็สมเหตุสมผลที่จะใช้โดยตรงเพราะเป็นรหัสเฟรมเวิร์ก แต่ถ้าฉันต้องเดาโดยไม่ได้ดูโค้ดควรจะได้รับการแก้ไขมากมาย แต่ก็ยังไม่ได้รับความสำคัญสูงพอที่จะทำเช่นนั้น

เพียงจำคำแนะนำการเลี้ยงดูที่ดี: "เด็ก ๆ ทำในสิ่งที่ฉันพูดไม่ใช่สิ่งที่ฉันทำ!"

\Magento\Framework\App\ObjectManager::getInstance()คุณไม่ควรใช้
มันเอาชนะวัตถุประสงค์ของการฉีดพึ่งพา Mage::getModel()พวกเรากลับมาที่
ผู้จัดการวัตถุควรใช้เฉพาะในโรงงานและจากนั้นฉีดเข้าไปในตัวสร้าง

ข้อดีของการใช้รหัสนี้คือการเขียนน้อย แต่นี่ไม่ได้ทำให้มันโอเค
ความจริงที่ว่านี้ยังคงใช้ในแกนหลักเป็นเพราะมันยังไม่ได้รับการฟื้นฟู ฉันหวังว่ามันจะเป็น

ทำไมวีโอไอพีถึงทำในสิ่งที่พวกเขาไม่แนะนำให้ทำ หมายความว่ามีบางกรณีที่เราควรใช้ ObjectManager โดยตรง ถ้าเป็นเช่นนั้นกรณีใด

ฉันไม่รู้ว่าเรื่องทั้งหมดนี่คือการเดาของฉัน:

ในระหว่างการพัฒนา M2 ทีมวีโอไอพีที่บางขั้นตอนวิ่งสคริปต์อัตโนมัติซึ่งแทนที่การเกิดขึ้นของMage:getModel(), Mage::getSingleton(), $layout->createBlock()และอื่น ๆ เพื่อใช้ ObjectManager

การรีฟอร์เรชั่นในภายหลังควรแก้ไขสิ่งนี้ให้ใช้การฉีดแบบพึ่งพาที่เหมาะสม แต่ไม่มีเวลา / ทรัพยากรเพียงพอที่จะแปลงการเกิดขึ้นทั้งหมด

เมื่อเร็ว ๆ นี้ทีม Magento ดูเหมือนว่าจะใช้สิ่งนี้เป็นกลไกการหลบหนี แทนที่จะทำลายการนำไปปฏิบัติที่มีอยู่ (โดยต้องการเปลี่ยนนวกรรมิก) พวกเขาก็ซ่อนการพึ่งพาใหม่ผ่าน ObjectManager ฉันไม่สามารถพูดได้ว่าฉันเห็นด้วยกับวิธีการนี้ - เขียนโค้ดที่แย่กว่านั้นเพื่อหลีกเลี่ยงการหยุดพัก BC

อะไรคือผลโดยตรงของการใช้ ObjectManager โดยตรง?

ฉันคิดว่าคำถามของคุณมีเหตุผลเพียงพอแล้ว โดยทั่วไปจะสร้างการพึ่งพาที่ซ่อนอยู่ในคำอื่น ๆ การพึ่งพาอยู่ในรายละเอียดการใช้งานและไม่สามารถมองเห็นได้จากตัวสร้างเพียงอย่างเดียว

ไม่ควรใช้ Object manager โดยตรง!

ตัวอย่างเช่น

\Magento\Framework\App\ObjectManager::getInstance();

หากคุณกำลังทำงานกับผู้สังเกตการณ์เหตุการณ์หรือปลั๊กอินคุณไม่ควรใช้มันโดยตรง

คุณสามารถใช้มันในโรงงาน แต่ยกเว้นว่าคุณควรฉีดตัวจัดการวัตถุในตัวสร้างก่อนจากนั้นคุณสามารถใช้วัตถุในวิธีการของคุณ

ที่ต้องการใช้:

1) ประกาศวัตถุส่วนตัว:

private $_objectManager;

2) ฉีดในตัวสร้างและเริ่มต้น:

public function __construct(
    \Magento\Framework\ObjectManagerInterface $objectmanager
) {
    $this->_objectManager = $objectmanager;
}

3) ใช้ในวิธีการบางอย่าง:

public function create() {
    return $this->_objectManager->create(/* ......... */);
}

คำตอบนี้มีไว้สำหรับ Magento 2.2 เวอร์ชั่นต่ำกว่าดังนั้นโปรดรับทราบ ตามมาตรฐานใหม่ของวีโอไอพี 2 ตอนนี้เราไม่สามารถใช้งานอินสแตนซ์ objectManager ได้เช่นกัน เราต้องใช้ factory ของ object class หรือ repository เพื่อรับข้อมูลใด ๆ

สาเหตุหลักที่ทำให้ผู้พัฒนาไม่สนับสนุนอย่างยิ่งจากการใช้ Object Manager โดยตรงเนื่องจากการใช้ Object Manager โดยตรงทำให้ส่วนขยายไม่สามารถติดตั้งได้ในโหมดรีลีสที่คอมไพล์

ดังนั้นจึงเป็นการแบ่งย่อยสำหรับลูกค้าของคุณโดยใช้โหมดการเปิดตัวรวมถึงลูกค้าทั้งหมดบน Magento Cloud

ดูเหมือนว่านักพัฒนาส่วนใหญ่ที่มีเหตุผล (ประมาณ 75%) จะไม่ทดสอบส่วนขยายเพื่อดูว่าพวกเขาสามารถติดตั้งในโหมดวางจำหน่ายได้หรือไม่ดังนั้นจึงไม่พบปัญหาที่เกิดจากการใช้งาน ObjectManager ที่ไม่ถูกต้อง

ณ ปี 2560 ตลาดวีโอไอพีทำการทดสอบและรวบรวมการติดตั้งในส่วนขยายทั้งหมดที่จำหน่ายผ่านทาง หากส่วนขยายของคุณใช้ Object Manager โดยตรงส่วนขยายนั้นจะล้มเหลวในการทดสอบและถูกปฏิเสธจาก Marketplace จนกว่าคุณจะแก้ไขปัญหานี้และทำการโหลดซ้ำ

คุณสามารถลองโดยการสร้างวัตถุของ objectManager และไม่ควรใช้ objectManager โดยตรง

ใช้สิ่งที่ชอบ

class Example extends \Magento\Framework\View\Element\Template
{
    private $_objectManager;

    public function __construct(
        \Magento\Framework\ObjectManagerInterface $objectmanager
    ){
        $this->_objectManager = $objectmanager;
    }

    public function getExample()
    {
        $customerSession = $this->_objectManager->create("Magento\Customer\Model\Session");
        if ($customerSession->isLoggedIn()) {
            $customerData = $customerSession->getCustomer()->getData();
            /*Your logic*/
        }
    }
}