How can I send an email when a function returns false?

Question

Im trying to make this code work, but i still receive an email when the function is equal to false. Can anyone assist me in finding the issue? What the program does is look for servers that has a set limit (in GB) on specific drives.
#This Program goes through all the servers and informs any level below the limit
$ErrorActionPreference = "SilentlyContinue";
$scriptpath = $MyInvocation.MyCommand.Definition
$dir = Split-Path $scriptpath
#=================Function============
Function isLow($server, $drive, $limit)
$disk = Get-WmiObject -ComputerName $server -Class Win32_LogicalDisk -Filter "DriveType = 3 and DeviceID = '$drive'";
[float]$size = $disk.Capacity;
[float]$freespace = $disk.FreeSpace;
$sizeGB = [Math]::Round($size / 1073741824, 2);
$freeSpaceGB = [Math]::Round($freespace / 1073741824, 2);
if($freeSpaceGB -lt $limit){
return $true;
else{
return $false;
#================Servers==============
#------------------###server1--------------
$server = "###server1";
$drive = "C:";
$lim = 25;
if(isLow $server $drive $lim)
$Alert += $server + " || " + $drive + " is low<br>"
$server = "###server1";
$drive = "D:";
$lim = 35;
if(isLow $server $drive $lim)
$Alert += $server + " || " + $drive + " is low<br>"
#-----------------###(more servers ect.)--------------
#================EMAIL===============
$smtpServer = "192.168.x.x"
$ReportSender = "[email protected]"
$users = "[email protected]"
$MailSubject = "ALERT!!! Low DiskSpace"
foreach($user in $users){
if($true){
Write-Host "Sending Email notification to $user"
$smtp = New-Object Net.Mail.SmtpClient($smtpServer)
$msg = New-Object Net.Mail.MailMessage
$msg.To.Add($user)
$msg.From = $ReportSender
$msg.Subject = $MailSubject
$msg.IsBodyHTML = $True
$msg.Body = $Alert
$smtp.Send($msg)
else($false){
Write-Host "No one is pass the limit"

Answer 1

Using a CSV is good.  Just load the "$servers" hash array from a CSV and the rest will work the same. The overall structure would work better if it was more like this.
Function isLow($server, $drive, $limit){
$disk = Get-WmiObject -ComputerName $server -Class Win32_LogicalDisk -Filter "DriveType = 3 and DeviceID = '$drive'"
if($disk.FreeSSpace -lt $limit){
$true
}else{
$false
$servers=@()
$servers=@{
Server="###server1"
Drive='C:'
Limit=25Gb
$servers=@{
Server="###server2"
Drive='D:'
Limit=15Gb
$servers=@{
Server="###server3"
Drive='C:'
Limit=50Gb
$results=foreach($servers in $servers){
if(isLow $server.Server $server.Drive $server.Limit){
'{0} || {1} is low<br>' -f $servers.Server,$server.Drive
if($results){
Write-Host 'Sending Email notification' -fore green
$mailprops=@{
SmtpServer='192.168.x.x'
From='[email protected]'
To=$users
Subject='ALERT!!! Low DiskSpace'
Body=$results
BodyAsHtml=$true
Send-MailMessage @mailprops
}else{
Write-Host 'No one is past the limit' -ForegroundColor green
The biggest issue is to realize that this is a computer.  If you type the same thing more than once then consider that the computer can do it for you.  Once you learn to think like a computer all of this becomes easier.
¯\_(ツ)_/¯