CMS falsch gesetzt

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • CMS falsch gesetzt

    Das kann im Moment noch warten ich habe ein anderes Problem mit einem CMS. I-was habe ich falsch gestezt etc. Ist nur ein Punkt oder so was, denn ch nicht finde.

    Code:

    Quellcode

    1. class modernCMS {
    3. var $host;
    4. var $username;
    5. var $password;
    6. var $db;
    8. function connect() {
    9. $con = mysql_connect($this->host, $this->username, $this->password) or die(mysql_error());
    10. mysql_select_db($this->db, $con) or die(mysql_error());
    11. }
    13. function get_content($id = '') {
    15. if($id != ""):
    16. $id = mysql_real_escape_string($id);
    17. $sql = "SELECT * FROM modernCMS WHERE id = '$id'";
    19. $return = '<p><a href="index.php">Go Back to content</a></p>';
    20. else:
    21. $sql = "SELECT * FROM modernCMS ORDER BY id DESC";
    22. endif;
    24. $res = mysql_query($sql) or die(mysql_error());
    26. if(mysql_num_rows($res) != 0):
    27. while($row = mysql_fetch_assoc($res)) {
    28. echo '<h1><a href="index.php?id=' . $row['id'] . '">' . $row['title'] . '</a></h1>';
    29. echo '<p>' . $row['body'] . '</p>';
    30. }
    31. else:
    32. echo '<p>Uh oh, this doesn\'t exist!</p>';
    33. endif;
    35. echo $return;
    36. }
    38. function add_content($p) {
    39. $title = mysql_real_escape_string($p['title']);
    40. $body = mysql_real_escape_string($p['body']);
    42. if(!$title || !$body):
    44. if(!title):
    45. echo "<p>The Title is Required</p>";
    46. endif;
    47. if(!body):
    48. echo "<p>The Body is Required</p>";
    49. endif;
    51. echo '<p><a href="add-content.php">Try Again!</a></p>';
    52. else:
    53. $sql = "INSERT INTO modernCMS VALUES (null, '$title', '$body')";
    54. $res = mysql_query($sql) or die(mysql_error());
    55. echo "Added Successfully!";
    56. endif;
    57. }
    59. function manage_content {
    60. echo '<div id="manage">';
    61. $sql = "SELECT * FROM modernCMS";
    62. $res = mysql_query($sql) or die(mysql_error());
    63. while($row = mysql_fetch_assoc($res)) :
    64. ?>
    65. <div>
    66. <h2 class="title"><?=$row['title']?></h2>
    67. <span class="actions"><a href="">Edit</a> | <a href="?delete=<?=$row['id'];?>">Delete</a></span>
    68. </div>
    70. <?php
    71. endwhile;
    72. echo '</div>'; // Closes Manage Div
    73. }
    75. function delete_content($id) {
    76. if(!$id) {
    77. return false;
    78. }else {
    79. $id = mysql_real_escape_string($id);
    80. $sql = "DELETE FROM modernCMS WHERE id = '$id'";
    81. $res = mysql_query($sql) or die(mysql_error());
    82. echo "Content Deleted Succesfully!";
    83. }
    84. }
    86. } // End the Class
    Alles anzeigen

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von Torben Brodt () aus folgendem Grund: syntax

  • Was ist den Dein eigentliches Problem bei der Klasse?

    Ich würde Dir jedoch gerne einen Vorschlag bei deinem Konzept geben. Du öffnest deine Datenbankverbindung direkt in der Klasse. Ich würde das ganze in eine eigene Klasse auslagern. Beispiel:

    Quellcode

    1. <?php
    2. class DatabaseException extends Exception {
    4. /**
    5. * constructor
    6. *
    7. * @param string $message[optional]
    8. * @param int $code[optional]
    9. */
    10. public function __construct($message='', $code=0) {
    11. parent::__construct($message, $code);
    13. die("<pre>".(string) $this."</pre>");
    14. }
    16. }
    18. class DatabaseConnectionException extends DatabaseException { }
    20. class DatabaseQueryException extends DatabaseException { }
    22. class Database {
    24. /**
    25. * database object
    26. *
    27. * @var Database $singleton
    28. */
    29. private static $singleton;
    31. /**
    32. * singleton pattern
    33. *
    34. * @return Database
    35. */
    36. public static function getInstance() {
    37. if(!is_object(self::$singleton) || !(self::$singleton instanceof Database)) {
    38. self::$singleton = new Database('localhost', 'root', '', 'test');
    39. }
    41. return self::$singleton;
    42. }
    44. /**
    45. * database connection configuration
    46. *
    47. * @var array $config
    48. */
    49. protected $config = array();
    51. /**
    52. * mysql connection
    53. *
    54. * @var ressource $connection
    55. */
    56. protected $connection;
    58. /**
    59. * constructor
    60. *
    61. * @param string $host
    62. * @param string $username
    63. * @param string $password
    64. * @param string $database
    65. * @param bool $autoConnect[optional]
    66. */
    67. public function __construct($host, $username, $password, $database, $autoConnect=true) {
    68. $this->config['host'] = $host;
    69. $this->config['username'] = $username;
    70. $this->config['password'] = $password;
    71. $this->config['database'] = $database;
    73. // connect automaticly
    74. if($autoConnect) {
    75. $this->connect($host, $username, $password);
    76. $this->selectDatabase($database);
    77. }
    78. }
    80. /**
    81. * destructor
    82. *
    83. */
    84. public function __destruct() {
    85. $this->close();
    86. }
    89. /**
    90. * mysql connect
    91. *
    92. * @param string $host
    93. * @param string $username
    94. * @param string $password
    95. * @return resource
    96. */
    97. public function connect($host, $username, $password) {
    98. $this->connection = mysql_connect($host, $username, $password);
    100. if(!$this->connection) {
    101. throw new DatabaseConnectionException;
    102. }
    103. else return $this->connection;
    104. }
    106. /**
    107. * mysql close
    108. *
    109. */
    110. public function close() {
    111. if($this->connection) {
    112. mysql_close($this->connection);
    113. }
    114. }
    116. /**
    117. * mysql select database
    118. *
    119. * @param string $database
    120. * @return bool
    121. */
    122. public function selectDatabase($database) {
    123. if(!$this->connection) {
    124. throw new DatabaseConnectionException;
    125. }
    127. if(mysql_select_db($database, $this->connection)) {
    128. return true;
    129. }
    130. else throw new DatabaseException;
    131. }
    133. /**
    134. * mysql query
    135. *
    136. * @param string $query
    137. * @return resource
    138. */
    139. public function query($query) {
    140. if(!$this->connection) {
    141. throw new DatabaseConnectionException;
    142. }
    144. $result = mysql_query($query, $this->connection);
    146. if($result) {
    147. return $result;
    148. }
    149. else throw new DatabaseQueryException;
    150. }
    152. /**
    153. * mysql fetch
    154. *
    155. * @param resource $result
    156. * @return array
    157. */
    158. public function fetch($result) {
    159. if(!is_resource($result)) {
    160. throw new DatabaseQueryConnection;
    161. }
    163. return mysql_fetch_assoc($result);
    164. }
    166. }
    167. ?>
    Alles anzeigen


    Beispiel

    Quellcode

    1. $result = Database::getInstance()->query("SELECT * FROM test");
    2. while($row = Database::getInstace()->fetch($result)) { }


    Natürlich kann man das ganze noch schöner machen und auch um einiges erweitern, habs jetzt aber nur mal schnell als Gedankenanreiz runtergetippt.