Jak poprawnie uzyc funkcji wewnątrz if do wysyłania wiadomości

pytanie zadane 14 grudnia 2015 w PHP przez kaminiak23 Użytkownik (640 p.)

Jestem początkujący.
Szukam w bazie userów i do nich wysyłam wiadomość. Normalnie wiadomość ustawiłbym jako zmienną ale w wiadomości potrzebuję wczytać dane w zależności od usera więc jak gdyby musi ona być w pętli. To działa ale wiadomość jest długa, w pętli mam kilka if i po wklejeniu wiadomości w każdy if jest to bardzo długi pliki i chyba niepotrzebnie powtarzam kod ale nie umiem inaczej.
Próbowałem z funkcjami ale nie udaje mi się - dociera wiadomość bez treści.

Najlepiej będzie widać to na przykładzie kodu:

function message ()
{
	$message = 'Abonament kończy się: '.$dataKonca;
}

foreach (...) 
{
	message();
	// - jak zrobić aby tu dodała się zawartość funkcji i została ona pobrana do funkcji mail?
	// potrzebuję w zmiennych w treści wiadomości podać wartości z bazy
	mail($email, $temat, $message, $headers); // wysłanie wiadomosci
}

Mogę jakoś tę treść wiadomości umieścić w funkcji i wywołać jak tak jak tu ?

1 odpowiedź

odpowiedź 14 grudnia 2015 przez efiku Szeryf (75,160 p.)
edycja 14 grudnia 2015 przez efiku

1. Poczytać w manualu o funkcjach i argumentach

Na przykład:
2. Napisać sobie funkcję która będzie się zajmować pobieraniem danych użytkowników , tutaj maila i daty abonamentu zwróci kolekcję użytkowników.

3 Napisać sobie funkcję która będzie zajmować się obliczaniem różnicy między aktualną datą a tamtą w bazie ( DateTime klasa lub Carbon ) i będzie zwracać string z pozostałymi dniami.

4. Skorzystać z biblioteki http://swiftmailer.org/ do wysyłania maili a nie mail().

komentarz 14 grudnia 2015 przez efiku Szeryf (75,160 p.)

<?php
 
function getUsersData( \PDO $pdo) {
    $query =  $pdo->query( "SELECT name,mail FROM users" ); 
    return  $query->fetchAll( PDO::FETCH_OBJ ) ;
}

function calculateDate( $date ) {

  $currentDate = new DateTime();
  $diff = $currentDate->diff( new DateTime($date) ) ;
 return $diff;
}

$users = getUsersData($connection);

foreach( $users as $user ){
    
    $dateEnd = calculateDate( $user->date );
   
   $message =  sprintf("Twoj abonamnet konczy sie za %s dni" , $dateEnd->format("%d"));
    // mailer 
   
}

PS: sprawdzaj sobie czy diff nie zwrócił czasem invert ;)

https://3v4l.org/ajLm2

komentarz 14 grudnia 2015 przez kaminiak23 Użytkownik (640 p.)

Dzięki za odpowiedzi ale wydaje mi się, że chyba nie do końca one są na moje pytanie. Najpewniej dokładnie przeczytaliście czego potrzebuję ale z tego co czytam to chyba nie (nie denerwujcie się jeśli jest inaczej).

Jako początkujący może robię to nie do końca poprawnie jednak to mi działa jak trzeba tylko nie umiem zrobić tak jak opisałem. Umiem powyciągać wszystkie potrzebne mi wartości tylko nie wiem czy można w funkcji zrobić zmienną z treścią i później w if umieścić taką funkcję w taki sposób aby zmienna ta była dostępna w tym if i aby ta właśnie zmienna została użyta do funkcji mail();

dzięki za inne wskazówki, poczytam o tym.

komentarz 14 grudnia 2015 przez efiku Szeryf (75,160 p.)

Odpowiedź jest rozwiązaniem na Twoje pytanie :P Normalny gotowiec :P

Jako początkujący może robię to nie do końca poprawnie jednak to mi działa jak trzeba tylko nie umiem zrobić tak jak opisałem. Umiem powyciągać wszystkie potrzebne mi wartości tylko nie wiem czy można w funkcji zrobić zmienną z treścią i później w if umieścić taką funkcję w taki sposób aby zmienna ta była dostępna w tym if i aby ta właśnie zmienna została użyta do funkcji mail();

Dlatego 1 link kieruje do manuala, abyś poczytał o argumentach funkcji i zwracaniu wartości.

<?php

function wiadomosc( $datakonca ){
   return "Abonament konczy sie .. " . $datakonca ;
}

if( costam) {
     $data = "20-12-2015";
    mail ( cos, cos , cos , wiadomosc($data) );
}