Почему вылетает ошибка Call to a member function?

Question

[antonio0226]

Привет всем. Нужно сделать рассылку заявок. Использую PHPmailer

Всего три страницы с заявками. Решил сделать общий функционал в одном файле, к которому будет подключаться mail.php, в mail.php будет проверятся с какой формы нажата кнопка и далее будет вызываться нужная функция. Все бы хорошо, но сразу ошибка.
Вот код

<?php
require ('autoload.php');
?>
<?php

	use PHPMailer\PHPMailer\PHPMailer;
	use PHPMailer\PHPMailer\Exception;

	function osagoFunc(){
			if(isset($check)){
				mailFunc('Новая заявка осаго',$contact,$msg,'Страховка без ограничений количества водителей');
			}else{
				mailFunc('Новая заявка осаго',$contact,$msg,'нет');
			}
	}

	$contact = htmlspecialchars($_POST['contact']);
	$msg = htmlspecialchars($_POST['msg']);
	$check = $_POST['somecheck'];
	
	$mail = new PHPMailer(true);

	



	function mailFunc($subj,$cont,$ms,$chec){
		foreach ($_FILES["photo_file"]["error"] as $key => $error) {
		    if ($error == UPLOAD_ERR_OK) {
					
		        $tmp_name = $_FILES["photo_file"]["tmp_name"][$key];
		        $name = basename($_FILES["photo_file"]["name"][$key]);
		        move_uploaded_file($tmp_name, "photo/$name");
		        	$mail->setFrom('[e-mail]', 'Mailer');
					$mail->addAddress('[e-mail]', 'John Doe');
				    $mail->Subject = $subj;
				    $mail->msgHTML('Контактная информация : '.$cont.' Текст сообщения : '.$ms.' Дополнительная информация: '.$chec);
				    $mail->addAttachment('photo/'.$name);
		 	 }
		}
				    $mail->send();
		
	}
	
?>

Ошибка Uncaught Error: Call to a member function setFrom() 25 строчка.
Я так понимаю не правильная область видимости. В чем проблема?

Comments

[Станислав Б]

Ошибка Uncaught Error: Call to a member function setFrom() 25 строчка.

Uncaught Error: Call to a member function setFrom() on null
до конца ошибку бы дочитали

Answer 1

[Станислав Б]

проблема в области видимости, в mailFunc у вас нет $mail

Comments

[antonio0226]

Та же самая ошибка если добавляю $mail = new PHPMailer(true); в функцию.

[Станислав Б]

antonio0226, вы уверены что та же самая?

[antonio0226]

Станислав Б, Да.
При нажатии на кнопку отрабатывает mail.php, в гём

<?php
require_once ('func.php');
?>
<meta charset="utf-8">
<?php
	osagoFunc();
?>

В func.php написал:

<?php
require ('autoload.php');
?>
<?php

	use PHPMailer\PHPMailer\PHPMailer;
	use PHPMailer\PHPMailer\Exception;

	

	$contact = htmlspecialchars($_POST['contact']);
	$msg = htmlspecialchars($_POST['msg']);
	$check = $_POST['somecheck'];
	
	





	function mailFunc($subj,$cont,$ms,$chec){
		global $mail = new PHPMailer(true);
		foreach ($_FILES["photo_file"]["error"] as $key => $error) {
		    if ($error == UPLOAD_ERR_OK) {
					
		        $tmp_name = $_FILES["photo_file"]["tmp_name"][$key];
		        $name = basename($_FILES["photo_file"]["name"][$key]);
		        move_uploaded_file($tmp_name, "photo/$name");
		        	$mail->setFrom('[e-mail]', 'Mailer');
					$mail->addAddress('****@yandex.ru', 'John Doe');
				    $mail->Subject = $subj;
				    $mail->msgHTML('Контактная информация : '.$cont.' Текст сообщения : '.$ms.' Дополнительная информация: '.$chec);
				    $mail->addAttachment('photo/'.$name);
		 	 }
		}
				    $mail->send();
		
	}


	function osagoFunc(){
			if(isset($check)){
				mailFunc('Новая заявка осаго',$contact,$msg,'Страховка без ограничений количества водителей');
			}else{
				mailFunc('Новая заявка осаго',$contact,$msg,'нет');
			}
	}

	
?>

Ошибка та же.

[Станислав Б]

вообще у вас там жуткий код,

function osagoFunc(){
      if(isset($check)){
        mailFunc('Новая заявка осаго',$contact,$msg,'Страховка без ограничений количества водителей');
      }else{
        mailFunc('Новая заявка осаго',$contact,$msg,'нет');
      }
  }

это тоже работать не будет. и в целом идея использовать глобальные переменные, очень плохая

[antonio0226]

Станислав Б, Просто нужно вынести одинаковый код, в данном случае это код отправки сообщения, он на трех страницах одинаков. Думал что функция подойдёт для этого

[Станислав Б]

antonio0226,

global $mail = new PHPMailer(true);

так делать нельзя. нельзя определить глобальную переменную и присвоить ей значение.
должны получить PHP Parse error: syntax error, unexpected '=', expecting ',' or ';'

уберите глобал

[antonio0226]

Станислав Б, так будет лучше?

use PHPMailer\PHPMailer\PHPMailer;
	use PHPMailer\PHPMailer\Exception;

		function mailFunc($subj,$cont,$ms,$chec){
		
		$mail = new PHPMailer(true);
		foreach ($_FILES["photo_file"]["error"] as $key => $error) {
		    if ($error == UPLOAD_ERR_OK) {
					
		        $tmp_name = $_FILES["photo_file"]["tmp_name"][$key];
		        $name = basename($_FILES["photo_file"]["name"][$key]);
		        move_uploaded_file($tmp_name, "photo/$name");
		        	$mail->setFrom('[e-mail]', 'Mailer');
					$mail->addAddress('[e-mail]', 'John Doe');
				    $mail->Subject = $subj;
				    $mail->msgHTML('Контактная информация : '.$cont.' Текст сообщения : '.$ms.' Дополнительная информация: '.$chec);
				    $mail->addAttachment('photo/'.$name);
		 	 }
		}
				    $mail->send();
				    return;
		
	}
	function osagoForm($subj,$cont,$ms,$chec){
		mailFunc($subj,$cont,$ms,$chec);
	}

mail.php

if(isset($_POST['somecheck']))
	osagoForm('Новая заявка ОСАГО',htmlspecialchars($_POST['contact']),htmlspecialchars($_POST['msg']),'Страховка без ограничений количества водителей');
osagoForm('Новая заявка ОСАГО',htmlspecialchars($_POST['contact']),htmlspecialchars($_POST['msg']),'Нет');

[Станислав Б]

antonio0226, это все равно очень жутко, но хоть как-то работать должно

[antonio0226]

Станислав Б, а в чем жуткость например?)

[Станислав Б]

antonio0226, начиная с форматирования и имен переменных и заканчивая всем остальным.

например функция mailFunc, кажется что ее задача отправлять письмо, но в ней помимо этого происходить например move_uploaded_file. какое отношение это имеет к письму, почему там вообще происходит работа с загруженными файлами?