HTMLからPHPの外部ファイルにある関数を呼び出すとエラーが発生する件

前提・実現したいこと

HTMLにPHPの外部ファイルをから関数を呼び出そうとするとエラーメッセージがでます。
結果出来には、sample.phpからtest.phpを呼び出し、sheet_display()を正常に動かしたいです。

発生している問題・エラーメッセージ

GET http://localhost/php_th/composer_project/test.php 500 (Internal Server Error)

該当のソースコード

php
1//sample.php
2<?php
3include('./vendor/autoload.php');
4
5use PhpOffice\PhpSpreadsheet\Reader\Xlsx as XlsxReader;
6
7$reader = new XlsxReader();
8$spreadsheet = $reader->load('XXXXXXXXXXXX.xlsx'); // ファイル名を指定
9$sheet = $spreadsheet->getSheet(1); // 読み込むシートを指定
10
11$data = $sheet->rangeToArray('A1:A1'); // 配列で取得したい範囲を指定
12
13$data3 = $sheet->getCell('A1')->getValue(); // 通常はこちら
14
15// foreach($sheet->getRowIterator() as $row){
16//     foreach($sheet->getColumnIterator() as $column){
17//         echo $sheet->getCell($column->getColumnIndex() . $row->getRowIndex())->getValue();
18//     }
19//     echo'<br>';
20// }
21
22function sheet_display() {
23    foreach($sheet->getRowIterator() as $row){
24        foreach($sheet->getColumnIterator() as $column){
25            echo $sheet->getCell($column->getColumnIndex() . $row->getRowIndex())->getValue();
26        }
27        echo'<br>';
28    }
29}
30
31function for_test() {
32    foreach($sheet->getRowIterator() as $row){
33        echo $row;
34    }
35}
36
37
38function aiueo() {
39    echo 'aiueo';
40}
41?>

php
1//test.php
2<!DOCTYPE html>
3<html lang="ja">
4<link rel="stylesheet">
5
6<head>
7    <meta charset="UTF-8">
8    <title>Hello!</title>
9</head>
10
11<body>
12    <h1>Hello,GitHub!</h1>
13    <?php include_once('./sample.php');
14
15        aiueo();
16        echo $data3;
17        for_test();
18        sheet_display();
19
20    ?>
21</body>
22
23</html>

試したこと

関数’aiueo’は正常に動くので、sample.phpの

php
1foreach($sheet->getRowIterator() as $row)

部分がエラーの原因ではないかと考えましたが、解決方法はわかリませんでした。

結果的には、sheet_display()を正常に動かしたいです。

行動規範の内容に同意します

回答1件

ベストアンサー

関数(function)の中と外は別の空間。
sheet_display()やfor_test()で$sheetを使ってるけど関数内から外の$sheetは見えてない。
定義してない$sheetをいきなり使おうとしてるからエラー。

解決方法は引数で渡すとか

function sheet_display($sheet) {

test.phpでは

include_once('./sample.php');
sheet_display($sheet);

global使うよりはさっさとclassに進んだほうがいい。

投稿2020/12/27 12:08

kawax

総合スコア10377

あなたの回答

tips

プレビュー

行動規範の内容に同意します

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

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

ただいまの回答率
85.46%

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

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

質問する

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

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