foreach循環中對象的重新實例化

Question

我正在處理某人的代碼。代碼中有一個函數。

它像以下：

function send_notification($device_token) 
{ 
    $apn = new APN(); 
    $apn->payloadMethod = 'enhance'; // you can turn on this method for debuggin purpose 
    $apn->connectToPush(); 
    ............................ 
    ............................ 
} 

然後foreach循環裏面，他調用該函數。

foreach($alluser as $user) 
{ 
    send_notification($user['device_token']); 
} 

現在，如果我運行上面的代碼，那麼它說APN Failed to connect: Something wrong with context。

所以我在代碼中改變了一些東西。

$apn = new APN(); 
foreach($alluser as $user) 
{ 
    $apn->payloadMethod = 'enhance'; // you can turn on this method for debuggin purpose 
    $apn->connectToPush(); 
    ............................ 
    ............................ 
} 

我創建foreach循環之外的類的對象然後它工作。
但事實是，我必須在每個地方編寫上面的代碼（本頁面包含其他foreach）。

那麼我怎麼才能以一種聰明的方式解決上述問題？

FULL CODE (Just some part)

<?php 
foreach($alluser as $user) 
{ 
    send_notification($user['device_token']); 
} 

function send_notification($device_token) 
{ 
    $apn = new APN(); 
    $apn->payloadMethod = 'enhance'; // you can turn on this method for debuggin purpose 
    $apn->connectToPush(); 
    ............................ 
    ............................ 
} 
?> 

旁註：什麼，我想知道的是，當我每次創建新的類實例，那麼爲什麼它不工作？

Answer 1

你可以只讓APN比如你功能的必需參數和之前的唯一實例循環。

<?php 
$apn = new APN(); 
foreach($alluser as $user) 
{ 
    send_notification($user['device_token'], $apn); 
} 

function send_notification($device_token, APN $apn) 
{ 
    $apn->payloadMethod = 'enhance'; // you can turn on this method for debuggin purpose 
    $apn->connectToPush(); 
    ............................ 
    ............................ 
} 

另一種方法是使用一個單身：

class APN { 
    private static $instance = null; 
    public static function getInstance() { 
     if (null === self::$instance) { 
      self::$instance = new self; 
     } 
     return self::$instance; 
    } 
    //.... whatever your class does 
} 

foreach($alluser as $user) 
{ 
    send_notification($user['device_token'], $apn); 
} 

function send_notification($device_token) 
{ 
    $apn = APN::getInstance(); 
    $apn->payloadMethod = 'enhance'; // you can turn on this method for debuggin purpose 
    $apn->connectToPush(); 
    ............................ 
    ............................ 
} 

艱難的注意，Singleton pattern還帶有其缺點一樣緊耦合，這使得測試更加努力，也隱藏依賴關係：

Singleton Antipattern

所以我的建議將是第一種方法。

Answer 2

您可以使用使用對象全局，那麼你不需要一次又一次地創建對象，例如

<?php 
$apn = new APN(); 
foreach($alluser as $user) 
{ 
    send_notification($user['device_token']); 
} 

function send_notification($device_token) 
{ 
    global $apn; 
    $apn->payloadMethod = 'enhance'; // you can turn on this method for debuggin purpose 
    $apn->connectToPush(); 
    ............................ 
    ............................ 
} 
?>