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

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

新規登録して質問してみよう
ただいま回答率
85.48%
PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

1回答

526閲覧

PHPの外部コマンド(exec)の実行でzipファイルを作成したい

nf10131999

総合スコア6

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

1グッド

1クリップ

投稿2022/06/21 02:04

編集2022/06/21 02:05

【作りたいもの】
選択したファイルを、パス付きでzip化する機能を実装したいです。
また、パスワードの設定は必須ではないのでパスワードが入力された場合のみ、パス付きでzip化させたいです。

【起きている問題】
上記によりパスワード入力は任意のため、exec()の実行を分岐させているのですが、パスワードを入力している状態でもパスなしでzip化されてしまいます。

【対象のコード】

index.php

1 <form method="post" enctype="multipart/form-data" action="./action.php"> 2 <label for="file" id="file-label" class="position-absolute-center"> 3 <input id="file" name="file[]" type="file" multiple> 4 <span id="select-button" class="button button-green">ファイルを選択</span> 5 </label> 6 </div> 7 <div class="flex mt-30"> 8 <label for="password" id="password-label" class="w-100 none"> 9 <input id="password" name="password" value="" class="mb-10" type="text"> 10 <p class="font-12 gray text" id="password-comment">※パスワードは8文字以上の半角で設定してください。</p> 11 </label> 12 </form>

action.php

1<?php 2$files = $_FILES['file']['tmp_name']; 3$fileName = $_FILES["file"]["name"]; 4$time = date("c"); 5$path = './'.$time.'/'; 6$psw = $_POST['password']; 7//保存用フォルダを作成 8mkdir("$time", 0777); 9//アップロードファイルを一時フォルダに保存 10for ($i = 0; $i < count($fileName); $i++){ 11 move_uploaded_file($files[$i], $path.$fileName[$i]); 12} 13//パスワードを付けてzip化 14if ($psw == '') { 15 exec("zip -r $time.zip $time"); //パスワードが入力されていない場合 16} 17else if ($psw != '') { 18 exec("zip -er -P='$psw' $time.zip $time"); //パスワードが入力されている場合 19 echo 0; 20} 21?>

【補足】

  • 分岐自体は正常に反映されていることが確認できています。
  • exec("zip -er -P='$psw' $time.zip $time");をif文の外に出して実行すると、パス付きのzipの生成に成功します。
aaabbbsss👍を押しています

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

mix-peach

2022/06/21 03:30

> 分岐自体は正常に反映されていることが確認できています。 とは、具体的に何を確認されたのですか? > exec("zip -er -P='$psw' $time.zip $time");をif文の外に出して実行すると、パス付きのzipの生成に成功します。 とのことですが、できたzipファイルを「入力したパスワード」を使って解凍する、というところまで確認されましたか? あと、index.php のform内のHTML構造が正しくない状態になっていますが、 実際にこのコードが使われているのでしょうか? それとも質問用に必要部分のみを切り出した経緯でこうなってしまっただけですか?
nf10131999

2022/06/21 03:38

> 分岐自体は正常に反映されていることが確認できています。 パスワードが入力されているか否かの分岐は、それぞれ違う数字を出力することで確認済みです。 > exec("zip -er -P='$psw' $time.zip $time");をif文の外に出して実行すると、パス付きのzipの生成に成功します。 「入力したパスワード」を使って解凍するところまで確認済みです。 > index.php のform内のHTML構造が正しくない状態 必要部分のみを切り出しています。
mix-peach

2022/06/21 07:28

ということは、現状は、 else if ($psw != '') { の分岐の中が処理されているのに、【作成されたzipファイルにパスワードがかかっていない】という状態なのでしょうか? 私の方で質問に提示されたコードを使って、 ・index.phpのHTMLを最低限調整(※) ・action.phpの中身はそのまま で、動作検証した限りでは、意図した通りに「パスありzip」と「パスなしzip」が作成できましたので、もしかすると、現状では提示していない部分に原因があるのかもしれません。 一度質問に掲載したコードを確認して、問題が再現できる状態で再提示いただければと思います (※)index.phpのHTMLを最低限調整・・した内容は、 </form>の直前に、<button type=submit>送信</button>を置き、 DTD/html/head/bodyなどの、indexページを文字化けせずに表示するための最低限必要なものを追加しただけです。
guest

回答1

0

var_dump($_POST); すれば、
パスワードが入力されていないときにどのように受信しているかわかるはず。

パスワード入力がない時、$_POST['password']も存在しないので、
カラ文字列と比較してはいけないです。
isset($_POST['password'])で存在確認するか、
array_key_exists('password', $_POST)で存在確認するかしないと。

投稿2022/06/21 05:03

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問