Changeset 7720 for trunk/src/inc/FileHandler.class.php

Timestamp:

Apr 12, 2010, 5:56:17 PM (15 years ago)

Author:

Pieter Naber

Message:

Adding a lot of checks for the variables in the status and the location file. Added even more comments everywhere

File:

• trunk/src/inc/FileHandler.class.php (modified) (1 diff)

trunk/src/inc/FileHandler.class.php

@@ -7 +7 @@
 
 class FileHandler {
-        private $file;
+        private $filename;
+        private $handle;
+        
+        /*
+         * Function: __construct (constructor)
+         * Description: Handling with a node file 
+         * Parameters: string $filename, string $mode
+         * Returns: true is we've succesfully opened the file, otherwise false
+         */
+        public function __construct($filename, $mode) {
+                $this->filename = $filename;
+
+                try {
+                        trigger_log(SYSLOG_DEBUG, 'Opening the file "' . $this->filename . '"', __FILE__, __LINE__);
+                        $this->handle = fopen($this->filename, $mode);
+                } catch (Exception $err) {
+                        trigger_log(SYSLOG_ERR, 'Reading from file "' . $this->filename . '" failed', __FILE__, __LINE__);
+                        return false;
+                }
+
+                return true;
+        }
 
         /*
-         * Function: __construct (constructor)
-         * Parameters: string $filename
-         * Function: Handling with a node file 
+         * Function: read
+         * Description: Try to read the file we've opened in the constructor
+         * Parameters: -
+         * Returns: The content of the file if we can read from the file, otherwise false
          */
-        public function __construct($filename) {
+        public function read() {
+                $file = '';
+
                 try {
-                        trigger_log(SYSLOG_DEBUG, 'Opening the file "' . $filename . '"', __FILE__, __LINE__);
-                        $handle = fopen($filename, 'r');
+                        trigger_log(SYSLOG_DEBUG, 'Reading from file "' . $this->filename . '"', __FILE__, __LINE__);
+                        $file = stream_get_contents($this->handle);
+                } catch (Exception $err) {
+                        trigger_log(SYSLOG_ERR, 'Reading from file "' . $this->filename . '" failed', __FILE__, __LINE__);
+                        return false;
+                }
 
-                        trigger_log(SYSLOG_DEBUG, 'Reading from file "' . $filename . '"', __FILE__, __LINE__);
-                        $this->file = '';
-                        while (!feof($handle)) {
-                                $this->file .= fread($handle, 8192);
-                        }
-
-                        trigger_log(SYSLOG_DEBUG, 'Closing file "' . $filename . '"', __FILE__, __LINE__);
-                        fclose($handle);
-
-                        trigger_log(SYSLOG_INFO, 'Reading from file "' . $filename . '" is done', __FILE__, __LINE__);
-                } catch (Exception $err) {
-                        trigger_log(SYSLOG_ERR, 'Reading from file "' . $filename . '" failed', __FILE__, __LINE__);
-                }
+                return $file;
         }
 
-        public function getFile() {
-                return $this->file;
+        /*
+         * Function: write
+         * Description: Try to write to the file we've opened in the constructor 
+         * Parameters: string $data
+         * Returns: true if we can write to the file, otherwise false
+         */
+        public function write($data) {
+                // TODO David: Make write function
+
+                return false;
+        }
+
+        public function __destruct() {
+                trigger_log(SYSLOG_DEBUG, 'Closing file "' . $this->filename . '"', __FILE__, __LINE__);
+                fclose($this->handle);
         }
 }