這是代碼：
ActivationService.php
<code> <?php
namespace App;
use Illuminate\Mail\Mailer;
use Illuminate\Mail\Message;
class ActivationService
{
protected $mailer;
protected $activationRepo;
protected $resendAfter = 24;
public function __construct(Mailer $mailer, ActivationRepository $activationRepo)
{
$this->mailer = $mailer;
$this->activationRepo = $activationRepo;
}
public function sendActivationMail($user)
{
if ($user->activated || !$this->shouldSend($user)) {
return;
}
$token = $this->activationRepo->createActivation($user);
$link = route('user.activate', $token);
$message = sprintf('Activate account <a href="%s">%s</a>', $link, $link);
$this->mailer->raw($message, function (Message $m) use ($user) {
$m->to($user->email)->subject('Activation mail');
});
}
public function activateUser($token)
{
$activation = $this->activationRepo->getActivationByToken($token);
if ($activation === null) {
return null;
}
$user = User::find($activation->user_id);
//下面这行是53行.
$user->activated = true;
$user->save();
$this->activationRepo->deleteActivation($token);
return $user;
}
private function shouldSend($user)
{
$activation = $this->activationRepo->getActivation($user);
return $activation === null || strtotime($activation->created_at) + 60 * 60 * $this->resendAfter < time();
}
}</code>錯誤訊息:
<code> ErrorException in ActivationService.php line 53:
Creating default object from empty value</code>53行的位置在程式碼中有註釋，程式碼是$user->activated = true;
問題：
上面的程式碼如何解決這個錯誤？
這是代碼：
ActivationService.php
<code> <?php
namespace App;
use Illuminate\Mail\Mailer;
use Illuminate\Mail\Message;
class ActivationService
{
protected $mailer;
protected $activationRepo;
protected $resendAfter = 24;
public function __construct(Mailer $mailer, ActivationRepository $activationRepo)
{
$this->mailer = $mailer;
$this->activationRepo = $activationRepo;
}
public function sendActivationMail($user)
{
if ($user->activated || !$this->shouldSend($user)) {
return;
}
$token = $this->activationRepo->createActivation($user);
$link = route('user.activate', $token);
$message = sprintf('Activate account <a href="%s">%s</a>', $link, $link);
$this->mailer->raw($message, function (Message $m) use ($user) {
$m->to($user->email)->subject('Activation mail');
});
}
public function activateUser($token)
{
$activation = $this->activationRepo->getActivationByToken($token);
if ($activation === null) {
return null;
}
$user = User::find($activation->user_id);
//下面这行是53行.
$user->activated = true;
$user->save();
$this->activationRepo->deleteActivation($token);
return $user;
}
private function shouldSend($user)
{
$activation = $this->activationRepo->getActivation($user);
return $activation === null || strtotime($activation->created_at) + 60 * 60 * $this->resendAfter < time();
}
}</code>錯誤訊息:
<code> ErrorException in ActivationService.php line 53:
Creating default object from empty value</code>53行的位置在程式碼中有註釋，程式碼是$user->activated = true;
問題：
上面的程式碼如何解決這個錯誤？
因為$activation = $this->activationRepo->getActivationByToken($token);是你透過別的拿到的一個不為null的回傳$activation->user_id可能，也可能沒值。
然後$user = User::find($activation->user_id);的時候沒值，回傳的是null，然後你53行$user->activated = true;就給一個空去指對象了。 。 。所以。 。最簡單你就是52行下面增加
<code>if(!$user){
return null;
}</code>不過這類你可以斷點一行行看返回是啥再去找問題
