PHPのメッセージ

質問

PHP学習者です。
機械の部品表（Bill of materials) のテーブルであるpartsmaster から、条件によって　レコードを検索して画面表示させるプログラムselect.php で、以下のようなメッセージが、何行も出てきます。
Notice: Undefined offset: 1 in C:\xampp\htdocs\BOMdatabase\select.php on line 60

使っているサーバーはXAMPP　V.3.2.4 からの物です。
Undefined offset (未定義のオフセット)とは何でしょうか。
どうすればこのメッセージが出ないようになるのでしょうか？
詳しい方どうぞご教授ください。

例えば条件を　between 5 and 10 とすると、結果としては希望のレコードが表示されるのですが、上のようなメッセージが出てきます。

このselect.php のソースコードを下に張り付けてあります。また　selectQuery.php 　と　select.php　の画面スクリーンショットを添付します。selectQuery.phpのコードを入れようとしましたが、文字制限を超えてしまったので、出来ませんでした。

select.php　のソースコード
<?php

// C:\xampp\htdocs\OOP-PHP\BOMdatabase\select.php
require_once "DBtable.php";

$table= $_POST['tablename'];
$obj = new DBtable($table);

$column2typelist = $obj->getCol2TypeList();// pairs of field name & data type
$column2commentlist = $obj->getCol2CommentList(); // pairs of field name & comment

$fieldOnly = array();
$typeOnly = array();
$commentOnly = array();

foreach($column2typelist as $col=>$type){
$fieldOnly[] = $col;
$typeOnly[] =$type;
}

foreach($column2commentlist as $col=>$comment){
$commentOnly[] = $comment;
}

$cols = array();// field names in search input table
$opes = array();// operator marks in search input table
$vals1 = array();// values 1 in search input table
$vals2 = array();// values 2 in search input table

foreach($_POST['cols'] as $item){
$cols[]=$item;
}
foreach($_POST['opes'] as $item){
$opes[]=$item;
}
foreach($_POST['vals1'] as $item){
$vals1[]=$item;
}
foreach($_POST['vals2'] as $item){
$vals2[]=$item;
}

$count = count($cols);
for($i=0; $i<$count; $i++){
// remove irrelevant field
//because we need only cols.opes.$vals combination
// for earch condition
if( empty($vals1[$i]) ){
unset($cols[$i]);
unset($opes[$i]);
unset($vals2[$i]);
unset($column2typelist[$i]);
unset($column2commentlist[$i]);
}
}

$condition = "";
for($i=0; $i<$count; $i++){
if( $cols[$i]=="" OR empty($cols[$i]) ){ //
continue; // this column has been unset( ) previously
}

if($opes[$i] == "contains"){
$condition .= $cols[$i]." LIKE '%".$vals1[$i]."%' AND ";
continue;
}

if( $opes[$i] == "between" ){

if( $typeOnly[$i] == "date"){
$condition .= $cols[$i]." >='".$vals1[$i]."' AND ".$cols[$i]." <='".$vals2[$i]."' AND ";
}else{
$condition .= $cols[$i]." >=".$vals1[$i]." AND ".$cols[$i]." <=".$vals2[$i]." AND ";
}
continue;
}

if( $typeOnly[$i]=="int"
OR $typeOnly[$i]=="smallint"
OR $typeOnly[$i]=="float"
OR $typeOnly[$i]=="decimal"){
$condition .= $cols[$i]." ".$opes[$i]." ".$vals1[$i]." AND ";
}else{
$condition .= $cols[$i]." ".$opes[$i]." '".$vals1[$i]."' AND ";
}
} // for

$condition = rtrim($condition,"AND ");

if( empty($condition) ){
die( "No search condition is specified. Try again.");
}

$query = "SELECT * FROM ".$table." WHERE ".$condition;
echo "Query = ".$query."</br>";//////////////////////
echo "________________________________________</br>";
global $connection;
$result = mysqli_query($connection, $query);
if($result){

$obj->tabulate($result);
}else{
die("No record found that matches the serch condition.");
}

unset($obj);
unset($temp);

?>

どうぞよろしくお願いします。

回答No.5

superside0 · ベストアンサー

まずは、　""とNULLは異なります。

$cols[$i] = "" は　長さが０の文字列という値です。
$cols[$i] = NULLは、　要素はあるけど、それに値がないこと
unset($cols[$i]) は、　要素自体をなくすこと
で、それぞれ違います。

> $cols[$i] =="" と empty($cols[$i]) はどう違うのでしょうか？　

empty($cols[$i])は
$cols[$i]の中身が　長さが０の文字列や　0や　"0"　やfalseやNULL　のときや
　$cols[$i]という要素自体がないときに
真(TRUE)となります。

しかし、$cols[$i]===""は、 長さが０の文字列のときだけ真となり
unsetされたり、もともと代入されていなければ Undefined offsetのNoticeが発生します。

ちなみに、上の通り、長さ０の文字列ときは　 empty($cols[$i] 　は真になってくれるので
　empty($cols[$i] OR $cols[$i]===""
と書かなくても
　empty($cols[$i] 　
だけ書いても、同じことですね。

変数や配列要素の　未定義や　空判定に
!isset ( )
empty ( )
is_null( )
のなにを使うべきなのかは、　結構ややこしいのですが、以下が参考になるかと
https://taisablog.com/archives/1324

お礼

詳しい説明、本当に有難うございます。
今まであまりよく知らずにempty( )などを使っていたのが恥かしいです。
教えて頂いたサイトを徹底的に勉強してみようと思います。

回答No.4

okwavey6

>早速にご回答を頂きまして、有難うございます。

どういたしまして。

>私はNotepad ++ をエディターとして使っています。

私も使っていたことがあります。
Windowsでは良いんですが、Macで同じ環境を構築出来なかったので乗り換えてしまいました。

>もちろん行番号も出ますが、ソースコードをコピーするときに行番号を入れることが出来ませんでした。それで「問題となっている６０行とはこれです、」と記入するのを忘れてしまい、見てもあまり意味のないものになってしまいました。

そうなんですよね。
プログラミング用の質問掲示板だと、始めから必要なものが揃っていたりします。

>教えて頂いたClimbi を少し調べてみましたが、ドメインネームが売りに出されていて、オファー金額を入力してくださいのような画面が出てきて、自分のプログラムソースコードをほかの人とシェアー出来るURLが出てくるようなものには出会えませんでした。

失礼しました。確認しておりませんでした。

>引き続き調べてみます。

検索スキルのアップがプログラミング学習ではプラスに働くと思います。

学習意欲があるのなら、環境構築も自分で出来るようになるとステップアップに繋がります。

自宅のPCにXAMPPをインストールするのも良いですし、VM Player で仮想PCを立て、そこにCentOSなどのLinux環境を構築してテスト用のサーバーとしても良いと思います。前者の方が簡単です。

お礼

色々とためになるご指摘を頂き、本当に有難うございます。プログラミングや環境設定などについては、まだまだ勉強の足りない未熟者ですので、励みになります。

回答No.3

superside0

60行目とは、
if( $cols[$i]=="" OR empty($cols[$i]) ){
ですね。
このOR順だと、先にORの左側の　$cols[$i]=="" の評価を行うので
$cols[$i]がunsetで削除されていたとき、
存在しない配列の要素を参照しているので,Undefined offsetとなります。
評価順を逆にすることで、ORの左側の式が真のとき右側の式は実行しなくなるので、Undefined offsetのNoticeは出なくなります。
つまり、
if ( empty($cols[$i]) OR $cols[$i]=="" ) {
ですね。

お礼

早速のご回答、有難うございます。
なるほど、わかったような気がします。早速やりたいところですが、今は家にいて、明日会社に行ってみるまでは試してみることが出来ません。
ちなみに、$cols[$i] =="" と empty($cols[$i]) はどう違うのでしょうか？　配列の中である要素をunset( ) すると、その要素があった場所が空になるということですよね。この時には　empty($cols[$i])　ではあるけれど、$cols[$i] =="" 　ではない、ということなのでしょうか？　つまり"" は empty ではなく、NULL が入っていると考えて良いのでしょうか？　もしご迷惑でなければ教えてください。

回答No.2

noname#263248

60行目で配列内にないものを指定している
例：配列が(1,2,3)だとして4番目の要素を指定した等

お礼

早速にご回答いただき、有難うございます。
配列　$cols[] は存在しますし、forループ中で添え字として使う$i は　for($i=0; $i<$count; $i++) として０から配列にある要素の数だけを参照するようにしてます。
ちなみに、このプログラムは自分の家にある古いXAMPP Version 1.6.4. の上では何のメッセージもなく期待通りに動きますが、会社で使っている新しいXAMPP Version 3.2.4 で走らせると質問したようなメッセージが出てくるのです。しかし、家のXAMPPも新しくしようと考えているので、あえてこのことは言わずに質問しました。
何かバージョンによるPHP言語の仕様変化のようなものが関係しているのでしょうか？

回答No.1

okwavey6

学習者と言うことで、本当は技術者向けなんですが、簡単にアドバイスです。

ソースコードをそのままコピペしても、見辛いので、例えばこのようなものを使いましょう。

https://nelog.jp/climbi

エラーメッセージでは、エラーが発生した行数も表示されていますので、行番号が表示されるようなエディタを使うと良いですね。

好みですか、VScodeと言うのを調べて使ってみると良いかもしれません。

エラーの意味としてよくあるのは、定義されていない配列のインデックスを指定して参照しているときに出ることが多いです。

また、学習者の質問も対象としている場で質問した方が、為になる回答が多いと思います。

お礼

早速にご回答を頂きまして、有難うございます。
私はNotepad ++ をエディターとして使っています。もちろん行番号も出ますが、ソースコードをコピーするときに行番号を入れることが出来ませんでした。それで「問題となっている６０行とはこれです、」と記入するのを忘れてしまい、見てもあまり意味のないものになってしまいました。
教えて頂いたClimbi を少し調べてみましたが、ドメインネームが売りに出されていて、オファー金額を入力してくださいのような画面が出てきて、自分のプログラムソースコードをほかの人とシェアー出来るURLが出てくるようなものには出会えませんでした。　引き続き調べてみます。