13
public static void launchProcess(string processName, string arguments, out string output)
{
    Process p = new Process
    {
        StartInfo = { UseShellExecute = false, RedirectStandardOutput = true, FileName = processName, Arguments = arguments }
    };

    p.Start();
    output = p.StandardOutput.ReadToEnd();
    p.WaitForExit();

}

And if my arguments contains the file names like:

D:\Visual Studio Projects\ProjectOnTFS\ProjectOnTFS

Then I get the error:

Avshalom
  • 8,657
  • 1
  • 25
  • 43
vakas
  • 1,799
  • 5
  • 23
  • 40

3 Answers3

19

It'll need doubles quotes, but will also likely need an @ to treat the string word-for-word (verbatim string) i.e. the "\" has a special meaning in string e.g. \t means a tab, so we want to ignore the \

So not only the double quotes, but also @

string myArgument = @"D:\Visual Studio Projects\ProjectOnTFS\ProjectOnTFS";
Alex KeySmith
  • 16,657
  • 11
  • 74
  • 152
6

I use the following in most of my apps (if required) to add double quotes at the start and end of a string if there are white spaces.

public string AddQuotesIfRequired(string path)
{
    return !string.IsNullOrWhiteSpace(path) ? 
        path.Contains(" ") && (!path.StartsWith("\"") && !path.EndsWith("\"")) ? 
            "\"" + path + "\"" : path : 
            string.Empty;
}

Examples..

AddQuotesIfRequired(@"D:\Visual Studio Projects\ProjectOnTFS\ProjectOnTFS");

Returns "D:\Visual Studio Projects\ProjectOnTFS\ProjectOnTFS"

AddQuotesIfRequired(@"C:\Test");

Returns C:\Test

AddQuotesIfRequired(@"""C:\Test Test\Wrap""");

Returns "C:\Test Test\Wrap"

AddQuotesIfRequired(" ");

Returns empty string

AddQuotesIfRequired(null);

Returns empty string

EDIT

As per suggestion, changed the name of the function and also added a null reference check.

Added check to see if double quotes already exist at the start and end of string so not to duplicate.

Changed the string check function to IsNullOrWhiteSpace to check for white space as well as empty or null, which if so, will return an empty string.

Magic Mick
  • 1,475
  • 1
  • 21
  • 32
  • Good, but I would call it AddQuotesIfRequired and add a null check in begin. – StefanG Nov 14 '17 at 17:11
  • Also, shouldn't it also check if the string is already wrapped in quotes? Calling this multiple times on it's own result should produce the same string each time. – Rufus L Sep 23 '18 at 01:23
3

I realize this is an old thread but for people who see this after me, you can also do: 

string myArgument="D:\\Visual Studio Projects\\ProjectOnTFS\\ProjectOnTFS"

By escaping the back slashes you do not have to use the @ symbol. Just another alternative.

Greg H-R
  • 83
  • 7
  • 1
    Although this is true as I have used this in the past, its much simpler and less time consuming to add a `@` at the start of the string, which indicates a true string. Otherwise you will find yourself wasting time adding back slashes all day. – Magic Mick Nov 20 '17 at 03:37