質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

解決済

2回答

2132閲覧

PHP関数のメソッドを指定したいのに、自作Classのメソッドとして処理される場合の対処法

junkboy

総合スコア45

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

0クリップ

投稿2015/12/16 08:57

下記コードを実行すると

Fatal error: Call to undefined method app\Model\Scraping::query()

が発生します。

DOMXPath::queryを指定しなければいけないところを、自作クラスのScraping::queryとして実行しようとしているため、エラーが出力されているのだと考えています。

DOMXPath::queryを実行したい場合、どのように記述すればよいのでしょうか。

ご教授いただけますと、幸いです。

lang

1namespace app\Model; 2 3class Scraping { 4 private $xpath; 5 6 public function __construct($url) { 7 // 文字化け回避のため自作関数を呼び出し 8 $html = $this->mb_c($url); 9 10 $dom = new \DOMDocument; 11 @$dom->loadHTML($html); 12 13 $this->xpath = new \DOMXPath($dom); 14 } 15 16 private function mb_c($url) { 17 $html = file_get_contents($url); 18 $html = mb_convert_encoding($html, 'HTML-ENTITIES', 'UTF-8'); 19 return $html; 20 } 21} 22

lang

1require_once(__DIR__ . '/app/Model/Scraping.php'); 2 3$url = HOGE_URL; 4$xpath = new app\Model\Scraping($url); 5 6// この処理でエラー 7foreach ($xpath->query('//ul[@class="clearfix"]/li/div') as $node) { 8}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

グローバル変数$xpathと、クラスのフィールド$xpathが同名のため混乱しているようですね。

php

1require_once(__DIR__ . '/app/Model/Scraping.php'); 2 3$url = HOGE_URL; 4$globalXpath = new app\Model\Scraping($url); 5 6// この処理でエラー 7foreach ($globalXpath->query('//ul[@class="clearfix"]/li/div') as $node) { 8}

このように表現すると、$globalXpathは上記処理の中で宣言された変数であることがはっきりするかと思います。
便宜上もともとこういうコードだったことにして話を進めます。

$globalXpath = new app\Model\Scraping($url); となっているため、$globalXpathapp\Model\Scrapingクラスのインスタンスです。
もちろんこれは\DOMDocumentクラスとは無関係な別クラスなので、queryメソッドは定義されていません。

内部のxpathフィールドにアクセスする方法は$globalXpath->xpath->query(...)となります。
入れ子状のイメージです。

private宣言されていますのでまた別のエラーが発生しますが。そちらはすぐに解決できるでしょう。

というのが直感的な解決策なのですが、 maisumakun さんの方法は更に進んだ綺麗な方法です。
できない理由だけ理解して、実装は maisumakun さん方式がおすすめです。

投稿2015/12/16 09:42

tozjp

総合スコア790

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

junkboy

2015/12/17 06:16

ご丁寧に補足していただき、ありがとうございます。 オブジェクト指向、MVCを勉強しつつ、実践しているところなので、わかっているような、わかってないような、という状態でした。 理屈としてものすごく分かりやすかったです。 助かりました:)
guest

0

Scrapingの中の$xpathはprivateで外から呼べませんので、$xpath -> query()を呼ぶScraping::queryを用意する必要があります。

php

1class Scraping { 2 private $xpath; 3 //中略 4 public function query($str) { 5 return $this -> xpath -> query($str); 6 } 7 // 後略 8}

投稿2015/12/16 09:06

maisumakun

総合スコア145121

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

junkboy

2015/12/17 06:13

じっくり理解しながら頂いたコードを実装させていただきます。 的確なご回答ありがとうございます:)
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問