web-dev-qa-db-ja.com

PHP：致命的なエラー：非オブジェクトのメンバー関数の呼び出し

ここで非常に奇妙なエラーが発生しました。フラットファイルデータベースクラスを作成していますが、更新するまでこれはすべて正常に機能していましたが、次のメッセージが常に表示されます。

致命的なエラー：/ home/reithg/public_html/test/engine/class.databaseの非オブジェクトに対するメンバー関数name（）の呼び出し.php行50

私はクラスを次のように呼んでいます：

_<?php
ini_set('display_errors', '1');

require('engine/class.database.php');

$config = new Config("lessons", array('first', 'second', 'third', 'fourth'), "data/");
$db = new Database($config, true);

print("Querying DB for 'theta' no exclusions: <br />");
print_r($db->query('theta', NULL, NULL));

print("<p /> Querying DB for 'theta' in column 'second': <br />");
print_r($db->query('theta', 'second', NULL));

print("<p /> Querying DB for first two rows: <br />");
print_r($db->getRows(2));

print("<p /> Querying DB for last three rows: <br />");
print_r($db->getRows(3, true));

print("<p /> Cleaning data for safe DB input: <br />");
$testInput = array('escape|these||delimiters','and\these\\slashes','and\0these\0nulls',"don't, forget quotes");
print("input: ");
print_r($testInput);
echo("<br />output: ");
print($db->addRow($testInput));
?>
_

これが私のclass.database.phpです

_<?php
require('class.config.php');
require('class.column.php');

    class Database {
        private
            $_config,
            $_pointer;

        public function __construct(Config $config)  {
            $this->_config = $config;
            return true;
        }

        private function connect($method) {
            if (!($this->_pointer = @fopen($this->_config->db(), $method)))
            echo("Unable to connect to database");
        }

        private function disconnect() {
            fclose($this->_pointer);
        }

        private function lock($method) {
            if(flock($this->_pointer, $method))
                return true;
            return false;
        }

        private function unlock() {
            flock($this->_pointer, LOCK_UN);
        }

        private function cleanInput($input) {   
            $data = array_map(array($this, 'escapeData'), $input);
            $output = implode($this->_config->delimiter(), $data)."\r\n";
            return $output;
        }

        private function escapeData($data) 
        {
            $search = array('\\', '"', "'", '\\0', '\n', $this->_config->delimiter());
            $replace = array('\\\\', '\"', "\'", '\\0', '\\n', '\\'.$this->_config->delimiter());
            $output = str_replace(array_unique($search), array_unique($replace), $data);
            return $output;
        }

        private function formatRow($data) {
            foreach($data as $key => $value) {
                $row[$this->_config->columns($key, "position")->name()] = $value;
            }
            return $row;
        }

        public function dumpToArray() {
            $arrayDump;
            foreach(file($this->_config->db(), FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES) as $row => $content)
                $arrayDump[$row] = formatRow(explode($this->_config->delimiter(),$content));
            return $arrayDump;
        }

        public function addRow(array $data) {
            $this->connect('ab');
            if($this->lock(LOCK_EX)) {
                // fwrite($this->_pointer, $this->cleanInput($data));
                echo($this->cleanInput($data));
                $this->unlock();
                $this->disconnect();
                return true;
            } else {
                $this->disconnect();
                return false;
            }
        }

        public function query($value, $column = NULL, $limit = NULL) {
            $this->connect('rb');
            $results = array();
            while ((is_null($limit) || (count($results) < $limit)) && !feof($this->_pointer)) {
                $data = explode($this->_config->delimiter(), fgets($this->_pointer, 1024));
                if(!is_null($column)) {
                    if ($data[$this->_config->columns($column, "string")->index()] == $value)
                        array_Push($results, $this->formatRow($data));
                } else {
                    if (in_array($value, $data))
                        array_Push($results, $this->formatRow($data));
                }
            }
            $this->disconnect();
            switch (count($results)) {
                case 0;
                    return false;
                case 1;
                    return $results[0];
                default;
                    return $results;
            }
        }

        public function getRows($limit = 1, $reverse = false) {
            $this->connect('rb');
            $offset = 0;
            $results = array();
            if ($reverse) {
                while(count($results) < $limit && fseek($this->_pointer, $offset, SEEK_END) >= 0) {
                    $char = fgetc($this->_pointer);
                    if($char == "\n" || $char == "\r"){
                        $offset --;
                        $data = explode($this->_config->delimiter(), fgets($this->_pointer, 1024));
                        array_Push($results, $this->formatRow($data));
                    }
                    $offset--;
                }
                $results = array_reverse($results);
            } else {
                while ((($limit === NULL) || (count($results) < $limit)) && !feof($this->_pointer)) {
                    $data = explode($this->_config->delimiter(), fgets($this->_pointer, 1024));
                    array_Push($results, $this->formatRow($data));
                }
            }
            $this->disconnect();
            return $results;
        }
    }
?>
_

class.config.php

_<?php
    class Config {
        private
            $_db,
            $_file,
            $_columns = array(),
            $_directory,
            $_delimiter;

        public function __construct($file, array $columns, $directory = NULL, $delimiter = "|")  {
            $this->_db = $directory.$file.".db";
            $this->defineColumns($columns);
            $this->_directory = $directory;
            $this->_delimiter = $delimiter;
        } 

        public function db() {
            return $this->_db;
        }

        public function delimiter() {
            return $this->_delimiter;
        }       

        private function defineColumns($constants) {
            for ($i=0;$i<count($constants);$i++) {
                if(in_array($constants[$i], $this->_columns))
                    die("Column names must be unique");
                $column = new Column($constants[$i], $i);
                $this->_columns[$column->name()] = $column;
            }
        }

        public function columns($index, $search = "string") {
            switch ($search) {
                case "string";
                    return $this->_columns[$index];
                    break;
                case "position";
                    $keys = array_keys($this->_columns);
                    return $this->_columns[$keys[$index]];
                    break;
                default;
                    return false;
            }   
        }
    }
?>
_

class.column.php

_<?php
    class Column { 
        const
            ALL = "0",
            STRING = "1",
            NUMBER = "2",
            INT = "3",
            AUTO_INCREMENT = "4",
            CURRENT_TIME = "5";

        private
            $_type = ALL,
            $_name,
            $_index,
            $_maxChars = "256";

        public function __construct($name, $index, $type = NULL, $maxChars = NULL)  {
            $this->_name = $name;
            $this->_index = $index;
            if(!is_null($type))
                setDataType($type);
            if(!is_null($maxChars))
                setMaxChars($maxChars);
            return $this;
        }

        public function setDataType($type) {
            switch ($type) {
                case ALL;
                case STRING;
                case NUMBER;
                case INT;
                case AUTO_INCREMENT;
                case CURRENT_TIME;
                    $this->_type = $type;
                    break;
                default;
                    return false;
            }
        }

        public function auditData($data) {
            switch ($this->_type) {
                case ALL;
                    $output = $data;
                    break;
                case STRING;
                    $output = (string) $data;
                    break;
                case NUMBER;
                    $output = (float) $data;
                    break;
                case INT;
                    $output = (int) $data;
                    break;
                case AUTO_INCREMENT;
                    $output = (int) $data;
                    break;
                case CURRENT_TIME;
                    $output = time();
                    break;
                default;
                    return false;
            }
            return $output;
        }

        public function setMaxChars($maxChars) {
            if(is_int($maxChars)) {
                $this->_maxChars = $maxChars;
            }
        }

        public function name() {
            return $this->_name;
        }

        public function index() {
            return $this->_index;
        }
    }
?>
_

たくさんのコードがあることはわかっていますが、コードに変更を加えずに、文字どおり1回の更新でこれが突然発生する理由を理解できません。以前に動作した以前のバージョンにバックトレースしても、これは起こっています。

私がしようとすると：

_print($this->_config->columns($key, "position"));
_

それは返します：

キャッチ可能な致命的エラー：クラスColumnのオブジェクトを/ home/reithg/public_html/test/engine/class.database.phpの文字列に変換できませんでした*50行目*

これは、name()というパブリックメソッドを持つColumnクラスのメンバーでname()を実行していることを示しています

私がする時：

_print($this->_config->columns($key, "position")->name());
_

返されます（foreachループの場合と同じように、一度に1ワード）。

最初2番目3番目4番目1番目2番目3番目4番目

そのため、その前の1行が明らかに機能しています。

phpfatal-error
12
George Reith

答えはlessons.dbファイルにある隠し文字によるものでした。

表示されたエラーはこれとは関係がありませんでした。2ペンスを払うのに時間を割いてくださった皆さんに感謝します。

7
George Reith

私はあなたのために2つのことを持っています：

  1. あなたが言うとき：「私はそれがたくさんのコードであることを知っていますが、なぜこれが突然、文字通り1回の更新でコードに変更を加えずに起こっているのか理解できません。これを機能させた以前のバージョンにバックトレースしても、ハプニング。"ブラウザでPHPコードをプログラミングしているかどうかを確認したいのですが。その場合、キャッシュが混乱することがあります。PHPコマンドラインインターフェイスを使用してPHPモデルを開発する方がはるかに簡単です。これが適切でない場合は、開発時にブラウザーのキャッシュを完全にオフにすることを検討してください。
  2. あなたが経験している問題は、Configクラスのパブリック関数列の構文エラーに起因すると思います。私はあなたがswitch-cases http://php.net/manual/en/control-structures.switch.php の構文を読む必要があると思いますここであなたは後にセミコロンを使用したことに気づくでしょうケースガード。単にコロンを使用する必要があります。また、中断する必要はありません。あなたが戻った後、あなたのコードはとにかくこの時点に達することはないので...

私がこれが役に立てば幸い-幸運...

0
kraenhansen