Dlaczego pole w klasie nie jest modyfikowanie, porawnie? Zgłupiałem Opcje

[Aztech]

Zwątpiłem! Nie wiem dlaczego poniższy kod nie działa.

klasa Forum

[PHP] pobierz, plaintext 
<?php
 
/* ... wycięty zbędny kod ... */
 
  /**
   * Informuje o aktywowaniu (zatwierdzeniu przez administratora) forum
   * @var boolean
   * @access private
   */
  private $active;
 
/* ... wycięty zbędny kod ... */
 
  /**
   * Aktywacja forum
   * 
   * @access public   
   */
  public function activate(){
	$this->active = true;
  }
 
  /**
   * Deaktywowanie forum
   * 
   * @access public
   *
   */
  public function deactivate() {
	$this->active = false;
  }
 
  /**
   * Sprawdzenie stanu forum, (czy zostało aktywowane)
   *
   * @return boolean
   * @access public
   */
  public function isActive(){
	return $this->active;
  } 
 
 
?>
[PHP] pobierz, plaintext 

kod w php, który nie działa - w komentarzach wyjaśnienei problemu

[PHP] pobierz, plaintext 
<?php
 
/ * kod */
case 'deact': //deaktywowanie forum
if (isset($arr[3])) {
	try {
	  $act_forum = new Forum($arr[3]);
	  #odczytaj dane z bazy							  
	  $act_forum->readDataFromSQLById();
	  echo "<BR>AKTYWNY? = ".$act_forum->isActive()."<BR>"; #tutaj wyświetla mi się pobrana z bazy wartość 1(true)
	  #deaktywuj
	  $act_forum->deactivate();
	  echo "<BR>AKTYWNY? = ".$act_forum->isActive()."<BR>"; # a tutaj nie wyrzuca mi żadnej wartości, nawet null
	  #uaktualnij dane
	  $act_forum->updateSQL();
	  #zakończenie							  
	  $arr[2]==null;
	}
	catch (PDOException $e) {
	  echo $e;
	}
}
break;
/* kod */
 
?>
[PHP] pobierz, plaintext 

Konia z rzędem dla tego ktro mi wytłumaczy dlaczego to nie działa?

Ten post edytował Aztech 1.04.2006, 18:59:45

[dr_bonzo]

[PHP] pobierz, plaintext 
<?php
$x = true;
echo "[" . $x . "]<br />";
var_dump( $x );
 
echo "<br />";
 
$x = false;
echo "[" . $x . "]<br />"; // <<<<<<<<<<<<<<
var_dump( $x );// <<<<<<<<<<<<<<
?>
[PHP] pobierz, plaintext 

powinno wystarczyc

[Aztech]

var_dump() zwróciło:

Kod

wartość po odczytaniu z bazy: string(1) "1"
wartość po deaktywowaniu: bool(false)

jak widać, potem jest funkcja updateSQL, która zapisuje wartość z pola active i zapisuje za pomocą setSql do obiektu PDOStatement

[PHP] pobierz, plaintext 
<?php
	if (isset($update_id)) {
	  $this->setSql("UPDATE rbx_forums"
	  ." SET"
	  ." forum_id = ".$update_id.","
	  ." app_order = ".$this->getAppearanceOrder().","
	  ." subforums = ".$this->getSubforumNum().","
	  ." topics = ".$this->getTopicNum().","
	  ." title = '".$this->getTitle()."',"
	  ." last_post_id = ".$this->getLastPostId().","
	  ." last_topic_id = ".$this->getLastTopicId().","
	  ." active = ".$this->isActive().","
	  ." posts = ".$this->getPostNum().","
	  ." description = '".$this->getDescription()."'"
	  ." WHERE forum_id = ".$update_id
	  ." LIMIT 1");
 
?>
[PHP] pobierz, plaintext 

no i print_r() zwraca mi:

Kod

PDO Object ( ) [result:private] => [sql:private] => UPDATE rbx_forums SET forum_id = 4, app_order = 5, subforums = 0, topics = 0, title = 'algebra', last_post_id = 0, last_topic_id = 0, active = , posts = 0, description = 'Zadania oraz materiały z wykładu' WHERE forum_id = 4 LIMIT 1

a PDO wyrzuca wyjątek:

Kod

exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' posts = 0, description = 'Zadania oraz materiały z wykładu' WHERE forum_id = 4 '

[dr_bonzo]

1. $this->isActive() wyswietla ci pusty string, wiec zapytanie wyglada:

[SQL] pobierz, plaintext 
WHERE active = , posts = 3 ...
[SQL] pobierz, plaintext 

w powodu wyzej wymienionego: jesli w bazie zapisujesz stan active:
active: 1
non active: 0

to najlepiej tak samo zapisywac w obiekcie

[PHP] pobierz, plaintext 
<?php
public function activate(){
	$this->active = 1;
  }
 
  public function deactivate() {
	$this->active = 0;
  }
 
  public function isActive(){
	return $this->active !== 0;
  } 
?>
[PHP] pobierz, plaintext 

wtedy zapytanie bedzie prawidlowe.

Lub przy zapytaniu:

[PHP] pobierz, plaintext 
<?php
active = " . ( $this->isActive() ? 1 : 0 ) . ",  posts = ". ....
?>
[PHP] pobierz, plaintext 

[Aztech]

Dzieki, pomogło. Wszystko Już zrozumiałem, gdzie popełniałem błąd.