複数の変数の受け取りについて(mb_send_mailについても教えてください)

それぞれの商品欄にチェックボックスがあり、複数の商品の受け取りをしても各店舗に1回しかメールが届かないようにしたいのですが上手くいきません。ご教授のほどよろしくお願い致します。

product_list

(id)     (product)     (give_shop)
 1        product1        shop1
 2        product2        shop1
 3        product3        shop2
 4        product4        shop2
 5        product5        shop2

user_list

(name)      (email)
 shop1       email1
 shop2       email2

list1.php

<form method="post" action="list2.php">
<input type="checkbox" name="ids[]" >
<input type="submit"  value="チェックした商品を受け取る">';
</form>

list2.php

<?php
if($_SERVER["REQUEST_METHOD"] ==  'POST' ) {
$ids==$_POST['ids'];

for($i=0; $i<count($ids); $i++) {

$pdo = new PDO("mysql:host=xxx; dbname=xxx; charset=utf8", "xxx", "xxx", array( PDO::ATTR_EMULATE_PREPARES => false ) );

$sql = "select * from product_list where id = '".$_POST['ids'][ $i]."'";

$statement = $pdo->prepare( $sql );
    $statement->execute( );
	$results = $statement->fetchAll();
	foreach( $results as $result ){

$name=$result['give_shop'];

try
{

$dsn='mysql:dbname=xxx;host=xxx;charset=utf8';
$user='xxx';
$password='xxx';
$dbh=new PDO($dsn,$user,$password);
$dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$sql="SELECT * FROM user_list where name='".$name."'";
$stmt=$dbh->prepare($sql);
$stmt->execute();

$rec=$stmt->fetch(PDO::FETCH_ASSOC);
$email=$rec['email'];
$dbh=null;

}
catch(Exception $e)
{
	print'ただいま障害により大変ご迷惑をお掛けしております1。';
	exit();
}


$mail_sub = '登録した商品の引き取り先が見つかりました';
$mail_body = $result["name"]."様\n\n発送準備が完了したら、以下のURLに飛んでください。";
$mail_body = html_entity_decode($mail_body,ENT_QUOTES,"UTF-8");
$mail_head = 'From:xxx';
mb_language('Japanese');
mb_internal_encoding("UTF-8");
mb_send_mail($email, $mail_sub, $mail_body,$mail_head);

}
}
?>

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

回答1件

ベストアンサー

ループの中でメールを送信するのをやめて、次のようにしたらどうですか？

php
1<?php
2if($_SERVER["REQUEST_METHOD"] ==  'POST' ) {
3// 正しくは$ids=$_POST['ids'];？これでは$idsと$_POST['ids']の比較になっている
4$ids==$_POST['ids'];
5
6// 送信先を入れておく配列
7$send_shop = array();
8// 相手の名前
9$email_name = "";
10
11for($i=0; $i<count($ids); $i++) {
12    $pdo = new PDO("mysql:host=xxx; dbname=xxx; charset=utf8", "xxx", "xxx", array( PDO::ATTR_EMULATE_PREPARES => false ) );
13
14
15    // プレースホルダを使う
16    $sql = "select * from product_list where id = :id";
17
18    $statement = $pdo->prepare( $sql );
19    $statement->execute(array(':id' => $_POST['ids'][ $i]));
20    $results = $statement->fetchAll();
21    foreach( $results as $result ){
22        $name=$result['give_shop'];
23        try
24        {
25
26            $dsn='mysql:dbname=xxx;host=xxx;charset=utf8';
27            $user='xxx';
28            $password='xxx';
29            $dbh=new PDO($dsn,$user,$password);
30            $dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
31            // プレースホルダを使う
32            $sql="SELECT * FROM user_list where name=:name";
33            $stmt=$dbh->prepare($sql);
34            $stmt->execute(array(':name' => $name));
35
36            $rec=$stmt->fetch(PDO::FETCH_ASSOC);
37            $email=$rec['email'];
38            $dbh=null;
39
40        }
41        catch(Exception $e)
42        {
43            print'ただいま障害により大変ご迷惑をお掛けしております1。';
44            exit();
45        }
46
47        // 送信先に登録
48        $send_shop[$email] = true;
49
50        // 相手の名前を登録
51        $email_name = $result["name"];
52    }
53}
54
55foreach($send_shop as $email => $val){
56    $mail_sub = '登録した商品の引き取り先が見つかりました';
57    $mail_body = $email_name."様\n\n発送準備が完了したら、以下のURLに飛んでください。";
58    $mail_body = html_entity_decode($mail_body,ENT_QUOTES,"UTF-8");
59    $mail_head = 'From:xxx';
60    mb_language('Japanese');
61    mb_internal_encoding("UTF-8");
62    mb_send_mail($email, $mail_sub, $mail_body,$mail_head);
63}
64?>