PHP-форма отправляет письма различным получателям на основе да или нет переменных

Question

У меня есть форма, которая содержит 4 вопроса, да или нет может быть выбран для каждого вопроса. Когда форма отправлена, я хочу отправить один адрес электронной почты с помощью PHPMailer. Есть 7 человек, которые, возможно, должны быть cc'd, но только, если да, для ответа на вопрос, который относится к ним.

Вопрос 1: Если да, куб.см recipient1

Вопрос 2: Если да, куб.см recipient2, recipient3

Вопрос 3: Если да, куб.см recipient4, recipient5, recipient6, recipient7

Вопрос 4: Если да, cc recipient6, recipient7

В настоящее время я использую инструкцию switch, которая работает, но я имеют в общей сложности 16 случаев. Есть ли более простой способ сделать это, о котором я не думаю?

switch (true) { 
    case ($Question1 === 'Yes' and $Question2 === 'Yes' and $Question3 === 'Yes' and $Question4 === 'Yes'): 
    sendmail('recipient1@example.edu', 'recipient2@example.edu', 'recipient2@example.edu', 'recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'Yes' and $Question2 === 'Yes' and $Question3 === 'Yes' and $Question4 === 'No'): 
    sendmail('recipient1@example.edu', 'recipient2@example.edu', 'recipient2@example.edu', 'recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'Yes' and $Question2 === 'Yes' and $Question3 === 'No' and $Question4 === 'Yes'): 
    sendmail('recipient1@example.edu', 'recipient2@example.edu', 'recipient2@example.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'Yes' and $Question2 === 'Yes' and $Question3 === 'No' and $Question4 === 'No'): 
    sendmail('recipient1@example.edu', 'recipient2@example.edu', 'recipient2@example.edu'); 
    break; 

    case ($Question1 === 'Yes' and $Question2 === 'No' and $Question3 === 'Yes' and $Question4 === 'Yes'): 
    sendmail('recipient1@example.edu', 'recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'Yes' and $Question2 === 'No' and $Question3 === 'Yes' and $Question4 === 'No'): 
    sendmail('recipient1@example.edu', 'recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'Yes' and $Question2 === 'No' and $Question3 === 'No' and $Question4 === 'Yes'): 
    sendmail('recipient1@example.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'Yes' and $Question2 === 'No' and $Question3 === 'No' and $Question4 === 'No'): 
    sendmail('recipient1@example.edu'); 
    break; 

    case ($Question1 === 'No' and $Question2 === 'Yes' and $Question3 === 'Yes' and $Question4 === 'No'): 
    sendmail('recipient2@example.edu', 'recipient2@example.edu', 'recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'No' and $Question2 === 'Yes' and $Question3 === 'No' and $Question4 === 'Yes'): 
    sendmail('recipient2@example.edu', 'recipient2@example.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'No' and $Question2 === 'Yes' and $Question3 === 'No' and $Question4 === 'No'): 
    sendmail('recipient2@example.edu', 'recipient2@example.edu'); 
    break; 

    case ($Question1 === 'No' and $Question2 === 'Yes' and $Question3 === 'Yes' and $Question4 === 'Yes'): 
    sendmail('recipient2@example.edu', 'recipient2@example.edu', 'recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'No' and $Question2 === 'No' and $Question3 === 'Yes' and $Question4 === 'Yes'): 
    sendmail('recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'No' and $Question2 === 'No' and $Question3 === 'Yes' and $Question4 === 'No'): 
    sendmail('recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'No' and $Question2 === 'No' and $Question3 === 'No' and $Question4 === 'Yes'): 
    sendmail('recipient6@example.edu', 'recipient7@example.edu'); 
    break; 

    case ($Question1 === 'No' and $Question2 === 'No' and $Question3 === 'No' and $Question4 === 'No'): 
    sendmail(); 
    break; 

    default: 
sendmail(); 

    } 



function sendmail($cc, $cc2, $cc3, $cc4, $cc5, $cc6, $cc7){ 
$mail    = new PHPMailer; 
more PHPMailer code here 
} 

Answer 1

Я не включил все ваши случаи, но вы можете видеть здесь, что, добавив каждого получателя электронной почты в массив, упомянутый @Alejandro C в комментариях, вы можете создать свой список рассылки. При необходимости измените условия и дополнения к электронной почте.

$mailer = phpmailer(); 
$emailList = array(); 

if($Question1 == "Yes") { 
    $emailList[] = "recipient1@example.edu"; 
    $emailList[] = "recipientY@example.edu"; 
} 
if($Question2 == "Yes") { 
    $emailList[] = "recipientX@example.edu"; 
    $emailList[] = "recipientXY@example.edu"; 
} 
if($Question3 == "Yes") { 
    $emailList[] = "recipientXX@example.edu"; 
} 
if($Question4 == "Yes") { 
    $emailList[] = "recipientXXX@example.edu"; 
    $emailList[] = "recipientYXY@example.edu"; 
} 
$emailList = array_unique($emailList); 

foreach($emailList as $val) { 
    $mailer->addCC($val); 
} 

// make sure you have added all the other necessary info to $mailer 
if(!$mailer->send()) { 
    echo 'Message could not be sent.'; 
    echo 'Mailer Error: ' . $mail->ErrorInfo; 
} else { 
    echo 'Message has been sent'; 
} 

Answer 2

Мне лично это не нравится switch(true) подход. Clean Code (хорошая книга, кстати) всегда предложить нам использовать читаемый код, так что вы можете перевести так:

switch (true) { 
    case ($Question1 === 'Yes' and $Question2 === 'Yes' and $Question3 === 'Yes' and $Question4 === 'Yes'): 
    sendmail('recipient1@example.edu', 'recipient2@example.edu', 'recipient2@example.edu', 'recipient4@example.edu', 'recipient5@example.edu@goodwin.edu', 'recipient6@example.edu', 'recipient7@example.edu'); 
    break; 
... 

Для этого:

class AnswersEnum 
{ 
    const SCENARIO_1 = ['Yes', 'Yes', 'Yes', 'Yes']; //use meaningful names here 
    const SCENARIO_2 = ['Yes', 'Yes', 'Yes', 'No']; 
    const SCENARIO_3 = ['Yes', 'Yes', 'No', 'No']; 
    ... 
} 

OtherClass.php

.... 
$answers = [$Question1, $Question2, $Question3, $Question4]; 

switch($answers) { 
    case AnswersEnum::SCENARIO_1: //use a meaningful name here 
     sendMailToScenario1recipients(); 
     break; 
    case AnswersEnum::SCENARIO_2: //use a meaningful name here 
     sendMailToScenario2recipients(); 
     break; 
    .... 

private function sendMailToScenario1recipients() 
{ 
    sendmail(...); 
} 

Это один из подходов. Поскольку я не вижу весь ваш проект, нелегко сказать, какое решение лучше.

Answer 3

Вот другой подход:

// Define answers as an array 
$answers = [ 
    'Q1' => 'Yes', 
    'Q2' => 'No', 
    'Q3' => 'No', 
    'Q4' => 'Yes', 
]; 

// Define subscribers (this could be database driven) 
$subscribers = [ 
    'Q1' => ['recipient1@example.edu'], 
    'Q2' => ['recipient2@example.edu', 'recipient3@example.edu'], 
    'Q3' => ['recipient4@example.edu', 'recipient5@example.edu', 'recipient6@example.edu', 'recipient7@example.edu'], 
    'Q4' => ['recipient6@example.edu', 'recipient7@example.edu'] 
]; 

// Determine list of recipients 
$recipients = []; 
foreach ($answers as $question => $answer) { 
    if ($answer == 'Yes') { 
     $recipients = array_unique(array_merge($recipients, $subscribers[$question])); 
    } 
} 

// Send mail however you planned on sending it 
sendmail($recipients); 

Answer 4

Многое, как ответ Джереми, но с добавлением формы и ответ фильтра.

<?php 

$questions = [ 
    'Q1' => 'Do you like pigs?', 
    'Q2' => 'Is Percy a good name for a pig?', 
    'Q3' => 'Would you eat a pig?' 
]; 

$yes_people = [ 
    'Q1' => ['foo@example.com'], 
    'Q2' => ['bar@example.com', 'baz@example.com'], 
    'Q3' => ['big@example.com', 'fat@example.com', 'baz@example.com'], 
]; 

if($_SERVER['REQUEST_METHOD'] == 'POST') { 

    $answers = []; 

    foreach(array_keys($questions) as $key) 
     $answers[$key] = isset($_POST[$key]) ? $_POST[$key] : null; 

    function answer_filter($item) { 
     return in_array($item, ['Yes', 'No']) ? $item : 'Invalid'; 
    } 

    $answers = array_map('answer_filter', $answers); 

    // Build message content here. 

    $recipients = []; 
    foreach($answers as $key => $value) 
     if($value == 'Yes') 
      $recipients = array_unique(array_merge($recipients, $yes_people[$key])); 

    var_dump($recipients); 
} 
?> 
<form method="POST" action=""> 
    <?php foreach($questions as $key => $question) { ?> 
     <p><?= $question ?></p> 
     <input type="radio" name="<?= $key ?>" value="Yes">Yes 
     <input type="radio" name="<?= $key ?>" value="No">No 
    <?php } ?> 
    <br/> 
    <input type="submit" value="Go"> 
</form>