basenameを使いファイルを配列で取り出したい。

Question

前提・実現したいこと

fuelphpを使用

変数に配列で入ったファイルのパス名をforeachとbasename関数を使ってファイル名を配列で取り出したいです。

###ソースコード

else {
				$result = $this->__fileUp(null, null);
                $filename= [];
				$url= [];
				if (! $result) {throw new RuntimeException('upload failed', 500);}
				
				Log::debug($result, "Controller_Admin_Ticket::post_fileup() - __fileUp");
                    foreach($result as $results){
					$filename = basename($results);
				};
                $url = Components_S3::getUrl(sprintf('%d/ticket/_tmp/%s', Auth::get_contract_id(), $filename), false);
				Log::debug(['filename' => $filename]);
				return Response::forge(json_encode(['success' => true, 'url' => $url]), 200, ['Content-Type' => 'application/json']);
				
			}
		}

問題点

foreach($result as $results){
                    $filename = basename($results);

$resultには複数配列で入っているのですが、
$filenameをデバッグしてみても一つしか入っていません。なぜ配列で入っていないのでしょうか。
また配列で変数に入れる方法はないのでしょうか。

試したこと

pathinfo関数も使用してみたのですが結果は同じでした。

補足情報

初心者な為ソースコードが見ずらかったり、情報が足りないかもしれません。ご了承ください。

Comments

[m.ts10806]

「補足情報」にはフレームワークや言語のバージョンなど、環境に関する情報を書くようにしてください。

[T-Heisenberg]

詳しく教えていただきありがとうございます。
以後、気をつけます。

Answer 1

$filename = basename($results);

↓

$filename[] = basename($results);

Comments

[T-Heisenberg]

回答ありがとうございます。

$filename[] = basename($results);
でやってみたのですが、エラーがでてしまいました。
最初のほうに
$filename= [];
で定義してあるのですがこれではいけないのでしょうか。

[退会済みユーザー]

エラーメッセージを記述してください。

[T-Heisenberg]

ERROR - 2019-10-17 20:11:37 --> Controller_Admin_Ticket::post_fileup() - Array
(
[message] => exception 'Fuel\Core\PhpErrorException' with message 'Array to string conversion' in /opt/domains/app/chatplus/webfront/fuel/app/classes/controller/admin/ticket.php:1668
Stack trace:
#0 /opt/domains/app/chatplus/webfront/fuel/core/bootstrap.php(109): Fuel\Core\Error::error_handler(8, 'Array to string...', '/opt/domains/ap...', 1668)
#1 [internal function]: {closure}(8, 'Array to string...', '/opt/domains/ap...', 1668, Array)
#2 /opt/domains/app/chatplus/webfront/fuel/app/classes/controller/admin/ticket.php(1668): sprintf('%d/ticket/_tmp/...', 61, Array)
#3 [internal function]: Controller_Admin_Ticket->post_fileup()
#4 /opt/domains/app/chatplus/webfront/fuel/core/classes/request.php(443): ReflectionMethod->invokeArgs(Object(Controller_Admin_Ticket), Array)
#5 /opt/domains/app/chatplus/webfront/public/index.php(73): Fuel\Core\Request->execute()
#6 /opt/domains/app/chatplus/webfront/public/index.php(103): {closure}()
#7 {main}
[os] => Windows
[ua] => Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/77.0.3865.90 Safari/537.36
[file] => /opt/domains/app/chatplus/webfront/fuel/app/classes/controller/admin/ticket.php:L1681

[退会済みユーザー]

1668行目ってどこ？

[T-Heisenberg]

$url = から始まる文になります。

[退会済みユーザー]

$url = Components_S3::getUrl(sprintf('%d/ticket/_tmp/%s', Auth::get_contract_id(), $filename), false);

ここのことなら、質問内容に関係のないエラーですよ。

$filename は配列になっているんだから、sprinf() の引数にできませんよね？

[T-Heisenberg]

エラーがでてデバックがみれないのですが、
$filename[] = basename($results);

これで配列の形で入れることができているのでしょうか。

[T-Heisenberg]

なるほど。理解いたしました。ありがとうございます。