Is there a better way to get user status on a terminal server in C#?

I need to get user status for a specific user on a terminal server in c#.

I can do this by calling query.exe via System.Diagnostics.Process.
And then read/split the last line of the output stream.
As i do in the code example.

I need to use this in a web service, which is running on a different server, and i don't think this is the proper way to do this.

So is there a better way to do this.

public class TSTools
{
    public static TSInfo GetTSInfo(string server, string user)
    {
        Process process = new Process();
 
        process.StartInfo.FileName = "query.exe";
        process.StartInfo.Arguments = "user " + user + " /server:" + server;
        process.StartInfo.UseShellExecute = false;
        process.StartInfo.RedirectStandardOutput = true;
        process.Start();
 
        string lastLine = null;
        while (!process.StandardOutput.EndOfStream)
        {
            lastLine = process.StandardOutput.ReadLine();
        }
 
        TSInfo tsInfo = new TSInfo();
        if (!string.IsNullOrEmpty(lastLine))
        {
            tsInfo.UserName = lastLine.Substring(0, 22).Trim();
            tsInfo.SessionName = lastLine.Substring(22, 17).Trim();
            tsInfo.SessionID = lastLine.Substring(39, 6).Trim();
            tsInfo.Status = lastLine.Substring(45, 9).Trim();
            tsInfo.IdleTime = lastLine.Substring(54, 10).Trim();
            tsInfo.LogonTime = lastLine.Substring(64).Trim();
        }
        else
        {
            tsInfo.UserName = user;
            tsInfo.SessionName = "";
            tsInfo.SessionID = "";
            tsInfo.Status = "Offline";
            tsInfo.IdleTime = "";
            tsInfo.LogonTime = "";
        }
 
        return tsInfo;
    }
}
 
public class TSInfo
{
    public string UserName { get; set; }
    public string SessionName { get; set; }
    public string SessionID { get; set; }
    public string Status { get; set; }
    public string IdleTime { get; set; }
    public string LogonTime { get; set; }
}

Open in new window

thomas_egAsked:
Who is Participating?
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

Gary DavisDir Internet SvcsCommented:
This is the "Unix" way of doing things. Many Unix commands are written to have their output easily parsed and used by other programs.
I think this idea is fine. Windows is not as fast as Unix in creating processes but if your process does not call the web service too quickly, perfomance should be OK.
The advantage is that the Query.exe is already written and debugged. You could write your parse routine to be a little more flexible so blank delimiters are used in stead of the actual column locations.
Gary Davis
0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
C#

From novice to tech pro — start learning today.