If/Else with C#

Question

Okay, so I'm working with some if/else statements now. But, I'm having some trouble.

Here's the full code that is depending on the version clicked.

private void button_Click(object sender, EventArgs e)
{
    using (OpenFileDialog file = new OpenFileDialog())
    {
        file.Filter = "File(*.file)|*.jar|All Files (*.*)|*.*";
        file.Title = "Open File...";
        if (file.ShowDialog() == DialogResult.OK)
        {
            string fullFileName = item.FileName;
            FileInfo userSelected = new FileInfo(fullFileName);
            string fileNameWithExt = Path.GetFileName(fullFileName);
            string destPath = Path.Combine(Application.UserAppDataPath, fileNameWithExt);
            string mcAD = Environment.ExpandEnvironmentVariables("%AppData%");
            File.Copy(item.FileName, mcAD, true);
        }
    }

But what I'm having trouble is with this. Below is the code, but here's how the program is lain out. There's a menu at the bottom of the program. It's named "Version" you click and you can choose version 1.0, 2.0, and 3.0. I have it set so there's text beside it telling it which version you chose. Now, the issue is I need an if/else statement for all the version for the above code cause all files for each version go to a different location.

Here's the other code...

private void Version_1_0_Click(object sender, EventArgs e)
{
    string Version_1_0_Selected = VersionText.Text = "1.0 Selected";
}

private void Version_1_6_1_Click(object sender, EventArgs e)
{
    string Version_2_0_Selected = VersionText.Text = "2.0 Selected";
}

private void Version_3_0_Click(object sender, EventArgs e)
{
    string Version_3_0_Selected = VersionText.Text = "3.0 Selected";
}

Answer 1

You can use Control.Tag for storing version index. For example:

private void Version_1_0_Click(object sender, EventArgs e)
{
  VersionText.Text = "1.0 Selected";
  VersionText.Tag= 1;
}

Then, you can define your target paths:

string[] paths = {@"c:\path1.txt", @"c:\path2.txt", @"c:\path3.txt"};

Finally, when you writing your files you can lookup the path like this:

File.Copy(item.FileName, paths[VersionText.Tag], true);

You might need to modify this code if the target file name is based on the source file name, but that should not be difficult.

Answer 2

Abstract the FileDialog code to a separate method and pass in your version string so that you can then perform the checks.

public void OpenVersionDialog(string version)
{
     string mcAD = GetCopyPath(version);
     if(!String.IsNullOrEmpty(mcAD))
     {
        using (OpenFileDialog file = new OpenFileDialog())
        {
            file.Filter = "File(*.file)|*.jar|All Files (*.*)|*.*";
            file.Title = "Open File...";
            if (file.ShowDialog() == DialogResult.OK)
            {
               string fullFileName = item.FileName;
               FileInfo userSelected = new FileInfo(fullFileName);
               string fileNameWithExt = Path.GetFileName(fullFileName);
               string destPath = Path.Combine(Application.UserAppDataPath, fileNameWithExt);


               File.Copy(item.FileName, mcAD, true);
            }
        }
     }
     else
     {
        //invalid version selected
     } 
}


public string GetCopyPath(string versionInput)
{
   //these are case-insensitive checks but you can change that if you want case-sensitive
   if(string.Equals(versionInput, "1.0 Selected", StringComparison.OrdinalIgnoreCase))
      return "YOUR_PATH_FOR 1.0";

  if(string.Equals(versionInput, "2.0 Selected", StringComparison.OrdinalIgnoreCase))
      return "YOUR_PATH_FOR 2.0";

   if(string.Equals(versionInput, "3.0 Selected", StringComparison.OrdinalIgnoreCase))
      return "YOUR_PATH_FOR 3.0";

   return String.Empty;
}

If I understand correctly that should be what you want. If you have more versions, you could store them in a dictionary where the key is the version and the value is the path that the file should be copied to.

I'm not sure what the difference between mcAD and destPath is but I assume mcAD is the variable that changes based on the version as that's being used in File.Copy.