サンプルコード：

2019/02/05 04:06

コードはマークダウンのcode機能を利用してご提示ください（質問は編集できます）

2019/02/05 04:06

>画面は真っ白エラー表示をOnにしてご確認ください。 https://qiita.com/shotets/items/3c95aef631b2c5eadae5

2019/02/05 04:08

PHPマニュアル確認されたほうがいいですね。例えば下記。 http://php.net/manual/ja/mysqli.query.php mysql関数とは引数が違います。 http://php.net/manual/ja/function.mysql-query.php

2019/02/05 07:26

コメント2つ目の対応でエラーを確認願います。「真っ白」とはコード実行の継続に差し支えのあるエラーが出て中断しているだけ、ということがほとんどなので、エラー表示すれば前に進むことがよくあります。要は「どこで止まってるか」の確認が必要です。

2019/02/05 07:34

コード編集は結構なのですが、現在の提示の仕方だと質問者さんの実際のコードと違いますし構文エラーにもなりますので、例えば下記のようにコメントで追記してください。 $rs = mysqli_query($sql); //{実際に表示されたエラーメッセージを書く}

2019/02/05 07:34

エラー個所は_**で示したところのようですが、直し方に苦戦しています。

2019/02/05 07:34

というかエラーメッセージをそのままコピペで別途マークダウンのcode機能で提示いただければそれで良いのですが・・

2019/02/05 07:36

あと何度もすみませんが、 get_sql_data()という関数が呼ばれているのはどこからでしょうか。

2019/02/05 07:54

エラー個所は mysqli_query(CHARDB."SET NAMES "); $rs = mysqli_query($sql); while($row = mysqli_fetch_array($rs)){ array_push($ary,$row); } です。

SET NAMES あるいは SET CHARACTER SET などで文字コードを指定している

2019/02/05 07:56

エラー箇所はエラーメッセージに　.php line XXと含まれますのでそのままエラーメッセージを追記していただければ良いです。 ※ただ、回答はしました。この問題だけを解決するのは簡単ですが、「移行」としたときにはこれだけで済まない可能性も高いので、よりクリーンな形へ移行するようにしてください。

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

回答2件

下記でエラーがでているということで、

php
1mysqli_query(CHARDB."SET NAMES ");

CHARDBという定数がどこでどう定義されているのか提示内容では分からないですが、
既にコメントで提示したようにmysqli_query()の第1引数は手続き型のみ: mysqli_connect() あるいは mysqli_init() が返すリンク IDである必要があります。
つまり上記の書き方では単にSQLをセットしているだけになっています。
他で下記のような形でちゃんとその情報を渡しているので同じようにするだけ

php
1$result = mysqli_query($dbHandle, "SELECT DATABASE()")

2つ目のmysqli_fetch_array()のところに出ているエラーはmysqli_query()が失敗していて返り値がfalseであるがゆえに、mysqli_fetch_array()どころじゃないといった感じですね。

ではありますが、
現在の作りはかなり非効率に思います。
charsetがSJISなのはともかくとして（今の主流は確かutf8mb4）

文字化け対応にSET NAMESを実行するのは推奨されていません。

PHPでデータベースに接続するときのまとめ

初心者がやりがちなミス

確かにmysql→mysqliへの移行をしたいのは分かりますが、
PDOとmysqliはどっちを使うべき？の質問の回答にあるように、移植性が非常に良くなく、質問者さんのように問題を潰したと思ったらまた問題が・・・と問題対応しているだけで疲弊してしまいます。

できれば上記記事にあるようなPDOへの移行を強くすすめます。

サンプルコード：

答えは「1つ目で出ているエラーではmysqli_connect()の結果linkを第1引数に渡していない」
これは何度もコメントしています。
そこを入れればとりあえず1つ進むはず。

けど、せっかくPHP7ですし、無駄な記述も多いのでPHP7っぽい要素を入れて書き換えてみた。
※文字コードはUTF8前提。もし違うならそこは書き加えてください。その場合の動作の保障はしませんのでエラーが出たら自身で追ってみてください。

php
1<?php
2function get_sql_data(String $sql,Bool $record_count = true):array{
3    $dbh = connect_db();
4    try{
5        $rs = mysqli_query($dbh,$sql);
6
7        //データを返す
8        if($record_count){
9            //レコード数取得
10            return ["count"=>mysqli_num_rows($rs)];
11        } else {
12            //レコード返却
13           //蛇足コメント：whileでfetchしても結局全行取り出すならmysqli_fetch_allで第2引数オプション指定するだけでいい
14            return mysqli_fetch_all($rs,MYSQLI_ASSOC);
15        }
16
17    }catch(mysqli_sql_exception $e){
18        var_dump($e);
19        die();
20    }
21    //dbの解放は余程でかい処理をする以外は自動でされるのでなくてもいいと思う。
22}
23
24function connect_db(){
25    try{
26        //蛇足コメント：元のコードではSERVER_NAMEとかで分岐させているけどかなり古いやり方。
27       //それなら設定ファイルを別にもってそこから読み出すようにした方がいい。定数定義のファイルとか。「環境毎に変わらない情報」なので。コードを何度も書き換えることになるのでメンテナンス性が悪い
28        $dbh = mysqli_connect("***", "***", "***", "***");     //MySQL接続
29
30       //蛇足コメント：SET NAMESは基本使わない。それならmysqli()クラスでnewしてset_charset()した方が自然
31        /* 例：PHPマニュアルより引用
32    $mysqli = new mysqli("***", "***", "***", "***");
33    $mysqli->set_charset('utf8');
34        */
35        if (mysqli_connect_errno()) {
36            printf("Connect failed: %s\n", mysqli_connect_error());
37            die();
38        }
39    //蛇足コメント：これで捕捉できるかは未確認。でもtry-catchは必須。PDOExceptionのほうが正直扱いやすい
40    }catch(mysqli_sql_exception $e){
41        var_dump($e);
42        die();
43    }
44    return $dbh;
45}
46
47var_dump(get_sql_data($sql,false));
48

蛇足コメントも読んでください。

蛇足コード

ちなみにPDO版。結果は同じ。

php
1<?php
2function get_sql_data(String $sql,Bool $record_count = true):array{
3    $dbh = connect_db();
4
5    try{
6        $stmt = $dbh->query($sql);
7
8        //データを返す
9        if($record_count){
10            //レコード数取得
11            return ["count"=>$stmt->rowCount()];
12        } else {
13            //レコード返却
14            return $stmt->fetchAll();
15        }
16
17    }catch(PDOException $e){
18        var_dump($e);
19        die();
20    }
21}
22
23function connect_db(){
24    try{
25        return new PDO("mysql:dbname=***;host=***;charset=utf8mb4"
26            ,"***"
27            ,"***",
28            [
29                PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
30                PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
31            ]
32        );
33    } catch (PDOException $e) {
34        var_dump($e);
35        die();
36    }
37}
38
39var_dump(get_sql_data($sql,false));

mysqli版でもpdo版でもですが、本来はSQLには色々あって、
例えばSELECTだとwhere句がついてきたりします。
その場合だとSQLインジェクションの対策もきちんとやってるか？
というのが大事になります。
prepare→bindValueの流れは必須ですね。
今回はそこまで求められていないので書いていませんが。

あと、この記事は読んでいただきたい。

エラーメッセージの読み方と対処, 検索や質問の原則

投稿2019/02/05 07:52

編集2019/02/06 03:14

総合スコア80850

2019/02/05 09:09

pdoに変えていきたいのですが、技術が無いのでまずはmysqliに移行と考えたのです。SET NAMESも使わずmysql_set_charsetに変えようとしたのですが、いずれにせよ無理みたいです。エラーが消えません。

2019/02/05 09:25

別途コメントでお願いしていますが、エラーをそのまま質問本文に追記してください。 pdoよりもmysqliへの移行の方が技術が必要ですよ。私ですらお断りしたいくらいです。やることが多すぎます。

2019/02/05 09:31

あと、現在のコードですね。当初より変わってますよね。正しく再現確認できるコードを提示されないとこちらもアドバイスができません。

2019/02/05 09:54

質問文を直しました。よろしくお願いいたします。

2019/02/05 10:18

あれ？mysqli_queryの第一引数やSET NAMES直されたのではないのですか？

2019/02/06 01:22

マニュアルのように入れてみてもエラーが消えないのでそのままにしてあります。

2019/02/06 01:26

それでは話が合わなくなってきます。エラーの内容を確認してください。そのままにしているということは全く進んでいないということです。マニュアルのように入れてエラーがでたということは少し進んだということです。進んで戻っていては全く進歩がないですよ。どのように入れたかわからないとこちらもアドバイスしようがありません。

2019/02/06 01:28 編集

「マニュアルを参考にしたらエラーが消えるよ」という話ではなく、あくまで「プログラムの仕様（マニュアル）通りに組むのが前提」という話です。そこでエラーがでたということは仕様に近づいてはいるけど、何かしら入れる情報がおかしいとか構文が間違っているとか、整合性がとれていないとかそのあたりになるわけですが、今の何も進んでいない状態より100倍マシです。エラーは問題解決のヒントです。それを嫌がっていては何もできあがりません。

2019/02/06 01:34

質問に提示されているエラーは「引数が最低２つ必須だが１つしか与えられていない」です。マニュアル参考に組んだのでしたら２つ目の引数を与えたのでしょうけどそれがマニュアルの通りかどうかというのが一点目ですね。というか、回答に書いてます。質問者さんのコードでは別の箇所できちんと正しく引数を与えています。まずは同じようにするだけですよ？

2019/02/06 01:57 編集

ところで実際のDBはSJISなんですが？あとプログラム自体の文字コードは？無駄なエンコーディングが沢山走っているような気がしてなりません。 SQL文に対するエンコーディングは正直意味がないように思いますし。 get_sql_data() に渡されてる$charset　どこにも使われてませんし。

2019/02/06 02:44

勝手に憶測でサンプルコード作りました。 pdo版も提示してあります。わからないことはPHPマニュアル読んでください。というか一通り理解してから作らないとセキュリティリスクたんまり作りこんだ大事故コードになるので（そこから出る損失は計り知れない）私が回答に提示した「PHPでデータベースに接続するときのまとめ」の記事などはきちんと読んで理解を進めてください。素人であろうと初心者であろうと上級者であろうと何であろうとプログラムは書いた通りにしか動かないので、もしそのコードにセキュリティリスクがあれば容赦なく侵されます。

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

今出ているエラーはこいつですか？これが起きる原因は３パターン考えられます。
ちなみに起きる原因は、「定義されていない関数が呼び出されている」というものです。

Fatal error: Call to undefined function get_sql_data() in C:\xampp\htdocs\html\output\c.php on line 30

たしかにget_sql_data()は３０行目に記述されていますが、その呼び出し先を確認してください。

ケース１：関数の名前が違っている

function get_sql_bata(){…}　とかなっている場合です。

ケース２：外部ファイルに置かれている場合で、それを呼び出す手段が記述されていない

問題のプログラムを見ると、二種類PHPファイルがあるようですが、それを
require_once(外部ファイル.php);で呼びに行くようにしていますか？

PHPはあくまで問い合わせ型の言語であり、個々のサーバーに対して各ファイルが独立しているのでそのファイルの外に関数が置かれている場合、require_once()やinclude_once()などでそのファイルを呼び出さないといけません。

ケース３：サーバールートの外側に置かれているファイルを参照しようとしている

PHPプログラムはサーバのどこに置かれても参照できるものではなく、サーバールート配下でないとPHPを動かすことができません。よくあるミスとしてサーバコンフィグファイル(xxx.conf)のサーバールート設定がhtmlになっているのに、参照しようとしているPHPファイルが同じ階層、あるいは上位に置かれている場合はPHPプログラムを実行することができず、ただの文字列が書かれたテキストファイルとして読み取ってしまいます。

投稿2019/02/14 06:42

編集2019/02/14 09:03

総合スコア3635

2019/02/14 08:01

ありがとうございます。include("ａ．ｐｈｐとｂ．ｐｈｐ.php");してます。require_onceに変えましたが同じエラーでした。元のa.phpファイルには$sql=の文があるのですが関係しているのでしょうか。

2019/02/14 08:13 編集

require_once('a.php')という風に文字列にしていますか？あと、ファイルのパス（要はファイルが置かれている場所）が合っていますか？あとinclude_once()にすると、呼び出せなかった場合でも普通にエラーを吐かずに通ってしまうので、最初のうちはrequire_once()を使うように心がけた方がいいです。

2019/02/14 08:13

include("readdata.php");となっていて、readdata.phpの中がinclude_once("a.php"); include_once("b.php");となっています。

2019/02/14 08:17

c.phpからはget_sql_data()が記述されているファイルへのアクセスはきちんとできていますか？

2019/02/14 08:30

c｡phpは変更していないのでアクセスできていると思っています。

2019/02/14 08:34

だとすると、その関数名が本当に同じか確認してみてください。それか呼び出す方の関数名を定義されている関数名に上書きしてみてください。それで通れば、何かしら全角文字などが混じっていた可能性が高いです。

2019/02/14 08:38

あと、もう一つ考えられるケースとしてはサーバーの外側に置かれている場合です。php.iniのserverrootはだいたいhtml配下、あるいはhtdocs配下になっていますが、それより上に置かれたファイルはそもそもPHPで呼び出すことができないので、上記のエラーが起きます。

2019/02/14 08:41

ありがとうございます。確認します。

2019/02/14 09:34

試したことに示したようにa.phpを変更しました。 Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\html\output\schedule.php on line 50　今度は50行目、foreach($ary_imm as $key => $value){?> です。

2019/02/14 10:05 編集

一旦は、この質問に対する問題はクリアできていますね。そこは別の部分のエラーになります。 invalid argument supplied for foreach というのは foreach文においてsupply for（渡して）いるinvalid(不正な) argument(引数）、つまり引数に置いている変数に間違いがあるということです。よくあるパターンとしては配列変数を置くべき場所に通常の変数を置いていた、または$ary_immの宣言を別の場面で、配列以外で宣言済みだった、などが考えられます。この辺は少し調べたらすぐ出てくるので、都度都度エラーを追っていけば、だいたい糸口は見えるようになります。自分もあと少ししかお相手できないので、後は自力で頑張ってみてください。そもそも、Mysql5.6の時点でしっかりと動いていたプログラムかどうかも気になるところではあるのですが…。

2019/02/18 01:48

お世話になります。現状は問題なく、こちらの要求通りの表示になっていますが、エラーを表示させてないのでエラー表示させたら？です。

2019/02/19 05:25

問題は解決ということですか？

2019/02/23 02:50

別の仕事のため離れていました。まだ解決していません。兆しも見えない。。。