为什么我会收到错误：未定义的变量？

Question

我创建了一个输入脚本。我将名字和脚本帖子名称写入数据库。但我有错误 - ErrorException [ Notice ]: Undefined variable: result。

有我的控制器：

class Controller_About extends Controller_Template{ 
    public function action_index() 
    { 
     if(!empty($_POST['name'])){ 
      $name = Model::factory('index')->insert_names($_POST['name']);; 
      $result= $name; 
     } 
     $this->template->site_name = Kohana::$config->load('common')->get('site_name'); 
     $this->template->site_description = Kohana::$config->load('common')->get('site_description'); 
     $this->template->page_title = 'About'; 
     $this->template->content = View::factory('about/about')->set('result', $result); 
     $this->template->styles[] = 'index/index'; 
    } 
} 

还有就是我的观点：

<form action=""> 
    <input type="text" name="name" /> 
</form> 

而且是我的模型：

Class Model_Index Extends Model { 

    public static function insert_names($name){ 
     $query = DB::query(DATABASE::INSERT, 'INSERT INTO names (name) VALUES (:name)')->parameters(array(':name' => $name)); 
    } 
} 

问题出在哪里？

编辑＃1

我编辑控制器：

class Controller_About extends Controller_Template{ 
    public function action_index() 
    {$result = ''; 
     if(!empty($_POST['name'])){ 
      $name = Model::factory('index')->insert_names($_POST['name']);; 
      $result= $name; 
     } 
     $this->template->site_name = Kohana::$config->load('common')->get('site_name'); 
     $this->template->site_description = Kohana::$config->load('common')->get('site_description'); 
     $this->template->page_title = 'About'; 
     $this->template->content = View::factory('about/about')->set('result', $result); 
     $this->template->styles[] = 'index/index'; 
    } 
} 

但这不工作，因为当我输入名字，他们没有付诸数据库。

Answer 1

可能是因为一个空值传递给name并且该变量没有被初始化，除非它非空。但它可以让在下面的行中使用时，if

$this->template->content = View::factory('about/about')->set('result', $result); 

初始化$result外if()外：

$result = ""; 
if(!empty($_POST['name'])){ 
    $name = Model::factory('index')->insert_names($_POST['name']);; 
    $result= $name; 
} 

或移动如下里面的if(){}整个块。

public function action_index() 
{ 
    if(!empty($_POST['name'])){ 
     $name = Model::factory('index')->insert_names($_POST['name']);; 
     $result= $name; 

     // move this inside the if() 
     $this->template->site_name = Kohana::$config->load('common')->get('site_name'); 
     $this->template->site_description = Kohana::$config->load('common')->get('site_description'); 
     $this->template->page_title = 'About'; 
     $this->template->content = View::factory('about/about')->set('result', $result); 
     $this->template->styles[] = 'index/index'; 
    } 
} 

Answer 2

您没有POST变量，称为name，因此$result从不设置。

Answer 3

方法属性添加到您的窗体：

<form action="" method="post"> 

变化：

if(!empty($_POST['name'])){ 

要：

$result = ''; 
if(!empty($_POST['name'])){ 

，并确保：

$this->template->content = View::factory('about/about')->set('result', $result); 

将在$result为空时工作。

Answer 4

你忘了实际运行查询：

public static function insert_names($name) 
{ 
    $query = DB::query(DATABASE::INSERT, 'INSERT INTO names (name) VALUES (:name)')->parameters(array(':name' => $name))->execute(); 
} 

但是它会是一个更好的方法来使用Kohana中的查询生成器：

public static function insert_names($name) 
{ 
    $query = DB::insert('names', array('name'))->values(array($name))->execute(); 
} 

或，从你的代码，我考虑可以判断你是初学者，可以直接在控制器中使用ORM并进一步简化它：

if(!empty($_POST['name'])) 
{ 
    $result = ORM::Factory('index')->set(array('name' => $_POST['name']))->save(); 
} 

但是，问题仍然存在，因为您的insert_names方法不会返回任何内容，所以您会将模板的结果变量设置为FALSE。

我相信你会想要做的是这样的：

public static function insert_names($name) 
{ 
    if(DB::insert('names', array('name'))->values(array($name))->execute()) 
    { 
     return $name; 
    } 
} 

（ORM与它不会是必要创建摆在首位此方法）

我看到了另一个错误你的控制器虽然 - 我想你不习惯E_NOTICE错误。而不是设置$结果为空字符串，它最好能够简单地重构你的代码一点点：

if(!empty($_POST['name'])) 
{ 
    $this->template->content = View::factory('about/about'); 

    if($name = Model::factory('index')->insert_names($_POST['name'])) 
    { 
     $this->template->content->set('result', $_POST['name']); 
    } 
    else 
    { 
     // some kind of error message 
    } 
} 

这可能是一个好主意，一群来自模板所有这些变量为一体，幸福的家庭：

class Controller_About extends Controller_Template{ 
    public function action_index() 
    { 
     $config = Kohana::$config->load('common'); 
     $this->template->set(array(
      'site_name' => $config->get('site_name'), 
      'site_description' => $config->get('site_description'), 
      'page_title' => 'About', 
      'styles' => 'index/index' 
     )); 

     $this->template->content = View::factory('about/about'); 

     if($name = Model::factory('index')->insert_names($_POST['name'])) 
     { 
      $this->template->content->set('result', $_POST['name']); 
     } 
     else 
     { 
      // some kind of error message 
     } 
    } 
} 

那里。是不是A LOT更清洁？ :)

虽然它仍然可以使用验证，但是这并不包括你的原始问题，所以我只是留下它的方式。