Как мне вернуть эту функцию?

Question

Я скопировал-вставить функцию, которую я пытаюсь изменить. Я бы хотел, чтобы он правильно обрабатывал ошибки. Прямо сейчас, $smtpResponse не возвращает никаких значений. Когда я эхо значения внутри функции, они показывают, но $smtpResponse ничего не дает, и не отправляет истинные или ложные значения.

Я пробовал возвращать «Fail» и «Success» тоже, но они не отображаются. $response не показывает, но «Это сработало!» делает. Почему функция не возвращает ничего?

Я пытаюсь вывести следующим образом:

//Send e-mail 
$repsonse = authSendEmail($from, $namefrom, $sendto,"", $subject,$message); 

//Report success/failure 
echo $response; 

The culprit: 

function authSendEmail($from, $namefrom, $to, $nameto, $subject, $message) { 
    //Connect to the host on the specified port 
    $smtpConnect = fsockopen($smtpServer, $port, $errno, $errstr, 60); 
    $smtpResponse = fgets($smtpConnect, 515); 

    if(empty($smtpConnect)) { 
     $output = "Failed to connect: $smtpResponse"; 
     echo $output; 
     return "Fail"; 
     //return false; 
     //used to read 'return $output' 
    } 

    else { 
     $logArray['connection'] = "Connected: $smtpResponse"; 
    } 

    //Request Auth Login 
    fputs($smtpConnect,"AUTH LOGIN" . $newLine); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['authrequest'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    //Send username 
    fputs($smtpConnect, base64_encode($username) . $newLine); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['authusername'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    //Send password 
    fputs($smtpConnect, base64_encode($password) . $newLine); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['authpassword'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    //Say Hello to SMTP 
    fputs($smtpConnect, "HELO $localhost" . $newLine); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['heloresponse'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    //Email From 
    fputs($smtpConnect, "MAIL FROM: $from" . $newLine); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['mailfromresponse'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    //Email To 
    fputs($smtpConnect, "RCPT TO: $to" . $newLine); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['mailtoresponse'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    //The Email 
    fputs($smtpConnect, "DATA" . $newLine); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['data1response'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    //Construct Headers 
    $headers = "MIME-Version: 1.0" . $newLine; 
    $headers .= "Content-type: text/html; charset=iso-8859-1" . $newLine; 
    $headers .= "To: ".$nameto." <".$to.">" . $newLine; 
    $headers .= "From: ".$namefrom." <".$from.">" . $newLine; 
    fputs($smtpConnect, "Subject: $subject\n$headers\n\n $message \n.\n"); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['data2response'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    // Say Bye to SMTP 
    fputs($smtpConnect,"QUIT" . $newLine); 
    $smtpResponse = fgets($smtpConnect, 515); 
    $logArray['quitresponse'] = "$smtpResponse"; 

    if(!$smtpResponse) { 
     return "Fail"; 
    } 

    echo "It worked!"; 
    return "Success"; 
    //insert var_dump here -- uncomment out the next line for debug info 
    //var_dump($logArray); 
} 

Answer 1

:) у вас есть опечатка

//Send e-mail 
$repsonse = authSendEmail($from, $namefrom, $sendto,"", $subject,$message); 

//Report success/failure 
echo $response; 

вы возвращаетесь $ repsonse и вы эхо отклик

И для записи $ . Копирование/вставка какого-либо кода elses не приведет вас никуда. Если вы хотите научиться программировать, напишите код самостоятельно или если вам действительно нужно скопировать/вставить, попробуйте понять, что вы копируете/вставляете. Я не пытаюсь быть злым, просто давая вам совет. Удачи!

Answer 2

Может быть, это слишком упрощенно, но, действительно ли вы иметь такую ​​строку:

//Report success/failure echo $response; 

Это эхо является лишь частью комментария , вам нужно поместить его в новую строку.

Кроме того, вы должны закрыть это соединение сокета перед выходом из своей функции.

Кроме того, было бы лучше вернуть логическое значение true при успешном завершении и false при ошибке, IMHO.