PHPでOOPat PHPPHPでOOP - 暇つぶし2ch■コピペモード□スレを通常表示□オプションモード□このスレッドのURL■項目テキスト495:nobodyさん 08/02/23 21:52:46 function GetNextData(){ if( $line = fgets($this->m_file_hd, 1024) ){ $line2 = split($this->m_pause_chr, $line); $ans = new Line(); $ans->SetData($line2[0], $line2[1]); }else{ $ans = ""; } return $ans; } これは下記に修正した方がわかりやすいよ function GetNextData(){ $ans = ""; if( $line = fgets($this->m_file_hd, 1024) ){ list($name,$msg) = split($this->m_pause_chr, $line); $ans = new Line(); $ans->SetData($name, $msg); } return $ans; } 496:nobodyさん 08/02/23 21:55:49 変数にオブジェクトが入ってくるなら 初期化はこうだった function GetNextData(){ $ans = null; if( $line = fgets($this->m_file_hd, 1024) ){ list($name,$msg) = split($this->m_pause_chr, $line); $ans = new Line(); $ans->SetData($name, $msg); } return $ans; } 497:nobodyさん 08/02/23 22:04:10 else{ $ans = ""; } これ全部 $ans = null; に初期化に変えて elseとっぱらった方がいいよ 返り値はオブジェクトが入ってるか入ってないかという処理なのに 空文字を返すのよくないよ! 次ページ最新レス表示レスジャンプ類似スレ一覧スレッドの検索話題のニュースおまかせリストオプションしおりを挟むスレッドに書込スレッドの一覧暇つぶし2ch