foreach variable scope

[Primera]

I am trying to use the below to return different values of an int variable,
but I seem to be running into scope issues as I cannot use the iSize variable
outside of the foreach block. Any help is appreciated.

public int CalcDiskSpace(s tring strComputer)
{
ManagementScope oMS = new ManagementScope ("\\\\" + strComputer);
ObjectQuery oQuery = new ObjectQuery("Se lect Size From
Win32_DiskDrive ");
ManagementObjec tSearcher oSearcher = new
ManagementObjec tSearcher(oMS, oQuery);
ManagementObjec tCollection colSearch = oSearcher.Get() ;

foreach ( ManagementObjec t oReturn in colSearch )
{
if ( (System.Int32.P arse(oReturn.To String())/100) < 1048576 )
{
int iSize = 1048576;

}
if ((System.Int32. Parse(oReturn.T oString()) / 100) > 4194304)
{
int iSize = 4194304;

}
else
{
int iSize = System.Int32.Pa rse(oReturn.ToS tring());

}
}
int iBytes = iSize + (iSize / 803);
DirectoryInfo odirInfo = new DirectoryInfo(@ "c:\");
int iFolders =
Int32.Parse(odi rInfo.GetFileSy stemInfos().Len gth.ToString()) * 1280;
int NeededSize = 196096 + iFolders + iBytes;
}

[Nicholas Paldino [.NET/C# MVP]]

Primera,

I'm a little confused by what you are trying to do here. First, there
is no way that this code can compile, as you arent returning anything.

What are you trying to do?

From what I can see, you have other problems. The iSize variable is
declared in the if block, which is nested in a foreach block. You should
declare the iSize variable outside of the foreach, and then you can access
it. The other problem is that iSize is only going to be equal to the last
result in this query (if you have multiple drives, then it is an issue).

This is why I asked what you are trying to do.
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"Primera" <Pr*****@discus sions.microsoft .com> wrote in message
news:B9******** *************** ***********@mic rosoft.com...

I am trying to use the below to return different values of an int variable,
but I seem to be running into scope issues as I cannot use the iSize
variable
outside of the foreach block. Any help is appreciated.

public int CalcDiskSpace(s tring strComputer)
{
ManagementScope oMS = new ManagementScope ("\\\\" +
strComputer);
ObjectQuery oQuery = new ObjectQuery("Se lect Size From
Win32_DiskDrive ");
ManagementObjec tSearcher oSearcher = new
ManagementObjec tSearcher(oMS, oQuery);
ManagementObjec tCollection colSearch = oSearcher.Get() ;

foreach ( ManagementObjec t oReturn in colSearch )
{
if ( (System.Int32.P arse(oReturn.To String())/100) <
1048576 )
{
int iSize = 1048576;

}
if ((System.Int32. Parse(oReturn.T oString()) / 100) >
4194304)
{
int iSize = 4194304;

}
else
{
int iSize = System.Int32.Pa rse(oReturn.ToS tring());

}
}
int iBytes = iSize + (iSize / 803);
DirectoryInfo odirInfo = new DirectoryInfo(@ "c:\");
int iFolders =
Int32.Parse(odi rInfo.GetFileSy stemInfos().Len gth.ToString()) * 1280;
int NeededSize = 196096 + iFolders + iBytes;
}

[Primera]

I apologize for the mess, but I'm just teaching myself c#. I'm trying to get
multiple pieces of information from a hard drive to compute space
requirements prior to running convert.exe to convert a FAT32 drive to NTFS.
This link will show the article that outlines how to compute the space
requirements ( http://support.microsoft.com/kb/156560/EN-US/ ). Thanks for
your help.

"Nicholas Paldino [.NET/C# MVP]" wrote:

Primera,

I'm a little confused by what you are trying to do here. First, there
is no way that this code can compile, as you arent returning anything.

What are you trying to do?

From what I can see, you have other problems. The iSize variable is
declared in the if block, which is nested in a foreach block. You should
declare the iSize variable outside of the foreach, and then you can access
it. The other problem is that iSize is only going to be equal to the last
result in this query (if you have multiple drives, then it is an issue).

This is why I asked what you are trying to do.
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"Primera" <Pr*****@discus sions.microsoft .com> wrote in message
news:B9******** *************** ***********@mic rosoft.com...

I am trying to use the below to return different values of an int variable,
but I seem to be running into scope issues as I cannot use the iSize
variable
outside of the foreach block. Any help is appreciated.

public int CalcDiskSpace(s tring strComputer)
{
ManagementScope oMS = new ManagementScope ("\\\\" +
strComputer);
ObjectQuery oQuery = new ObjectQuery("Se lect Size From
Win32_DiskDrive ");
ManagementObjec tSearcher oSearcher = new
ManagementObjec tSearcher(oMS, oQuery);
ManagementObjec tCollection colSearch = oSearcher.Get() ;

foreach ( ManagementObjec t oReturn in colSearch )
{
if ( (System.Int32.P arse(oReturn.To String())/100) <
1048576 )
{
int iSize = 1048576;

}
if ((System.Int32. Parse(oReturn.T oString()) / 100) >
4194304)
{
int iSize = 4194304;

}
else
{
int iSize = System.Int32.Pa rse(oReturn.ToS tring());

}
}
int iBytes = iSize + (iSize / 803);
DirectoryInfo odirInfo = new DirectoryInfo(@ "c:\");
int iFolders =
Int32.Parse(odi rInfo.GetFileSy stemInfos().Len gth.ToString()) * 1280;
int NeededSize = 196096 + iFolders + iBytes;
}

[Mike Hofer]

Primera wrote:

I am trying to use the below to return different values of an int variable,
but I seem to be running into scope issues as I cannot use the iSize variable
outside of the foreach block. Any help is appreciated.

public int CalcDiskSpace(s tring strComputer)
{
ManagementScope oMS = new ManagementScope ("\\\\" + strComputer);
ObjectQuery oQuery = new ObjectQuery("Se lect Size From
Win32_DiskDrive ");
ManagementObjec tSearcher oSearcher = new
ManagementObjec tSearcher(oMS, oQuery);
ManagementObjec tCollection colSearch = oSearcher.Get() ;

foreach ( ManagementObjec t oReturn in colSearch )
{
if ( (System.Int32.P arse(oReturn.To String())/100) < 1048576 )
{
int iSize = 1048576;

}
if ((System.Int32. Parse(oReturn.T oString()) / 100) > 4194304)
{
int iSize = 4194304;

}
else
{
int iSize = System.Int32.Pa rse(oReturn.ToS tring());

}
}
int iBytes = iSize + (iSize / 803);
DirectoryInfo odirInfo = new DirectoryInfo(@ "c:\");
int iFolders =
Int32.Parse(odi rInfo.GetFileSy stemInfos().Len gth.ToString()) * 1280;
int NeededSize = 196096 + iFolders + iBytes;
}

The scope of the iSize variable is limited to the curly braces in which
it is defined. You have three separate variables, with three separate
scopes, that just happen to have the same name.

Move the declaration of the variable out of the curly braces, and leave
the assignments within, like this:

int iSize = 0;
foreach ( ManagementObjec t oReturn in colSearch )
{
if ( (System.Int32.P arse(oReturn.To String())/100) < 1048576 )
iSize = 1048576;
if ((System.Int32. Parse(oReturn.T oString()) / 100) > 4194304)
iSize = 4194304;
else
iSize = System.Int32.Pa rse(oReturn.ToS tring());
}
}
int iBytes = iSize + (iSize / 803);

P.S. You might be missing an ELSE in there. If the first IF statement
executes, but then the second one enters the ELSE block, iSize will be
equal to Int32.Parse(oRe turn.ToString() ) instead of the fixed value of
1048576.