Jesli ten kod "jednolinijkowy", bedziesz wywolywal tylko raz, to nie ma sensu z niego funkcję robic. Nawet jesli kilka razy, to również nie ma sensu. Co innego gdyby kod w funkcji byl parametryzowalny i mial ciut wiecej linijek, to wowczas funkcja jak najbardziej.

tak naprawde to ci to nie zadziala
nigdzie w funckji nie widzę deklaracji zmiennych $mail, $temat, $tresc. nie są też przekazywane jako paramter do funckji.

Jak juz mowilem, jesli ty to tylko raz masz zamiar uzyc, to nie ma sensu funkcji z tego robic.

Poczytaj sobie: Zasięg zmiennych
Bo podstaw Ci prakuje.

To że masz wczesniej to nic nie znaczy, w funkcji niczego nie zobaczysz.