Solved

Problems calculating total directory size inclusive of files and sub directories

Posted on 2007-04-11
11
271 Views
Last Modified: 2010-04-23
I am using the following function to calculate TOTAL directory size, inclusive of sub directories and files.
I works for all of the files within the main directory, but then resets when it goes to the first sub directory.

Can anyone see where it is going wrong and how it can be fixed.

Thanks,
Michael

    Function GetFolderSize(ByVal DirPath As String, Optional ByVal IncludeSubFolders As Boolean = True) As Long

        Dim lngDirSize As Long
        Dim objFileInfo As FileInfo
        Dim objDir As DirectoryInfo = New DirectoryInfo(DirPath)
        Dim objSubFolder As DirectoryInfo

        Try
            'add length of each file
            For Each objFileInfo In objDir.GetFiles()
                lngDirSize += objFileInfo.Length
            Next
            'call recursively to get sub folders
            'if you don't want this set optional
            'parameter to false
            If IncludeSubFolders = True Then
                For Each objSubFolder In objDir.GetDirectories()
                    lngDirSize += GetFolderSize(objSubFolder.FullName)
                Next
            End If
        Catch Ex As Exception
        End Try
        If lngDirSize > 0 Then
            Return BytesToMegabytes(lngDirSize)
        Else
            Throw New Exception("Invalid or unreadable directory")
            Exit Function
        End If
    End Function

    Private Function BytesToMegabytes(ByVal Bytes As Long) _
    As Long
        Dim dblAns As Double
        dblAns = (Bytes / 1024) / 1024
        BytesToMegabytes = Format(dblAns, "###,###,##0.00")
    End Function

It is being called with:

TotalData = GetFolderSize("c:\temp\", True)
0
Comment
Question by:webkiwi1
  • 4
  • 4
  • 3
11 Comments
 
LVL 62

Expert Comment

by:Fernando Soto
ID: 18892513
This value of Dim lngDirSize As Long is getting rest to 0 each time the function GetFolderSize is called. Pass in the value of lngDirSize to the function and the first time you call it pass in 0.
0
 

Author Comment

by:webkiwi1
ID: 18892541
I understand what you are saying, how would I pass in the value?
0
 
LVL 11

Expert Comment

by:Babycorn-Starfish
ID: 18892589
Would the call to BytesToMegaBytes towards the bottom of your function affect things too if you're calling it recursively and adding the results together? You're essentially working with two different scales, if this is right call this on the overal result of GetFolderSize
0
 

Author Comment

by:webkiwi1
ID: 18892680
Another good point Babycorn-Starfish (?!?!?!)

Perhaps I should move the call to MytesToMegaBytes from the GetFolderSize function to the page load sub.
0
 
LVL 11

Accepted Solution

by:
Babycorn-Starfish earned 250 total points
ID: 18892683
hi try this,


    Sub Main()
        Dim totalSize As Long = GetFolderSize("d:\New Folder", True)
        Console.Write("" + BytesToMegabytes(totalSize).ToString())
        Console.Read()
    End Sub

    Function GetFolderSize(ByVal DirPath As String, Optional ByVal IncludeSubFolders As Boolean = True) As Long
        Dim lngDirSize As Long
        Dim objFileInfo As IO.FileInfo
        Dim objDir As IO.DirectoryInfo = New IO.DirectoryInfo(DirPath)
        Dim objSubFolder As IO.DirectoryInfo

        Try
            'add length of each file
            For Each objFileInfo In objDir.GetFiles()
                lngDirSize += objFileInfo.Length
            Next
            'call recursively to get sub folders
            'if you don't want this set optional
            'parameter to false
            If IncludeSubFolders = True Then
                For Each objSubFolder In objDir.GetDirectories()
                    lngDirSize += GetFolderSize(objSubFolder.FullName)
                Next
            End If
        Catch Ex As Exception
        End Try
        GetFolderSize = lngDirSize
    End Function

    Private Function BytesToMegabytes(ByVal Bytes As Long) As Double
        Dim dblAns As Double
        dblAns = (Bytes / 1024) / 1024
        BytesToMegabytes = dblAns 'Format(dblAns, "###,###,##0.00")
    End Function
0
Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

 
LVL 11

Expert Comment

by:Babycorn-Starfish
ID: 18892700
comment or uncomment the Format bit depending on what you need. You may want to round it up or down depending on how accurate you need it too
0
 
LVL 11

Expert Comment

by:Babycorn-Starfish
ID: 18892713
Sorry, left my file path in there !
0
 
LVL 62

Assisted Solution

by:Fernando Soto
Fernando Soto earned 250 total points
ID: 18892831
   Private Sub Button1_Click(ByVal sender As System.Object, _
        ByVal e As System.EventArgs) Handles Button1.Click

        Dim TotalData As Long = GetFolderSize("C:\Temp\MSPress\Crystal_Reports_10\", 0, True)

    End Sub

    Function GetFolderSize(ByVal DirPath As String, ByVal dirSize As Long, _
        Optional ByVal IncludeSubFolders As Boolean = True) As Long

        Dim lngDirSize As Long = dirSize
        Dim objFileInfo As FileInfo
        Dim objDir As DirectoryInfo = New DirectoryInfo(DirPath)
        Dim objSubFolder As DirectoryInfo

        Try
            'add length of each file
            For Each objFileInfo In objDir.GetFiles()
                lngDirSize += objFileInfo.Length
            Next
            'call recursively to get sub folders
            'if you don't want this set optional
            'parameter to false
            If IncludeSubFolders = True Then
                For Each objSubFolder In objDir.GetDirectories()
                    lngDirSize += GetFolderSize(objSubFolder.FullName, lngDirSize)
                Next
            End If
        Catch Ex As Exception
        End Try

        Return lngDirSize

        ' You do not want to do this here, you want to do it once ater the initial call
        'If lngDirSize > 0 Then
        '    Return BytesToMegabytes(lngDirSize)
        'Else
        '    Throw New Exception("Invalid or unreadable directory")
        '    Exit Function
        'End If
    End Function

    Private Function BytesToMegabytes(ByVal Bytes As Long) _
    As Long
        Dim dblAns As Double
        dblAns = (Bytes / 1024) / 1024
        BytesToMegabytes = Format(dblAns, "###,###,##0.00")
    End Function
0
 
LVL 62

Expert Comment

by:Fernando Soto
ID: 18892847
Change my file path in my last post
0
 
LVL 62

Expert Comment

by:Fernando Soto
ID: 18892922
Hi webkiwi1;

As I stated in my post above you want to convert the bytes to megabytes at the very end and not after each directory is completed being tallied up. Also the exception you are throwing is not really true, "Invalid or unreadable directory", because directory info only returns valid names. Also a directory may have no files in it and will have zero bytes returned which does not make it a invalid directory.

       If lngDirSize > 0 Then
           Return BytesToMegabytes(lngDirSize)
       Else
           Throw New Exception("Invalid or unreadable directory")
           Exit Function
       End If

Fernando
0
 

Author Comment

by:webkiwi1
ID: 18893279
Got both to work. For some reason the calculated value using Fernado's was off somewhere, but function still worked. Thanks guys for the fast and accurate responses.
0

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Does the idea of dealing with bits scare or confuse you? Does it seem like a waste of time in an age where we all have terabytes of storage? If so, you're missing out on one of the core tools in every professional programmer's toolbox. Learn how to …
This is about my first experience with programming Arduino.
In this fourth video of the Xpdf series, we discuss and demonstrate the PDFinfo utility, which retrieves the contents of a PDF's Info Dictionary, as well as some other information, including the page count. We show how to isolate the page count in a…
In this fifth video of the Xpdf series, we discuss and demonstrate the PDFdetach utility, which is able to list and, more importantly, extract attachments that are embedded in PDF files. It does this via a command line interface, making it suitable …

910 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

25 Experts available now in Live!

Get 1:1 Help Now