How I can modify the code to make work all statements?

Question

I have the next code :

private void Install_Click(object sender, EventArgs e)
{

    string configfilename = path + "config.ini";
    string installerfilename = path + "installer.ini";

    string configtext = File.ReadAllText(configfilename);
    string installertext = File.ReadAllText(installerfilename);

    var link = File.ReadLines(path + "config.ini").ToArray();
    var lin = File.ReadLines(path + "installer.ini").ToArray();

    foreach (var txt in link)
    {

        if (txt.Contains("PLP="))
        {
            var PLPPath = txt.Split('=')[1];
            installertext = installertext.Replace("fileInstallationKey=", "fileInstallationKey=" + PLPPath);
        }
        else if (txt.Contains("Output="))
        {
            var outputPath = txt.Split('=')[1];
            installertext = installertext.Replace("outlog=", "outlog=" + outputPath);
        }
        else
        {

            var license = lin.Select(line => Regex.Replace(line, @"license=.*", "license=yes"));
            File.WriteAllLines(installerfilename, license);

            var lmgr_files = lin.Select(line => Regex.Replace(line, @"lmgr_files=.*", "lmgr_files=false"));
            File.WriteAllLines(installerfilename, lmgr_files);

            var lmgr_service = lin.Select(line => Regex.Replace(line, @"lmgr_service=.*", "lmgr_service=false"));
            File.WriteAllLines(installerfilename, lmgr_service);
        }
        File.WriteAllText(installerfilename, installertext);
    }

But the problem is this code copies the data from config.ini in installer.ini, from the PLP row, Output row. And in installer.ini the rows :license, lmgr_files, lmgr_service are not completed with data : yes, false, false. With other words who is after else doesn't work . How I can modify the code to make all statements execute?

In installer.ini I have many things but the most important are :

license=false //I want to have license=yes
lmgr_files=   //I want to have lmgr_files= 
lmgr_service=  
fileInstallationKey=0000000  //the number from config.ini
outlog="jhaoif"              //the code from config.ini

remove the else brace for "license, lmgr_files, lmgr_service" and for output remove "else if" use "if:. — Chandrasekar Kesavan, Sep 23 '15 at 06:08
That's what I'm saying remove " else {" "}" from your code, then it's will work fine. — Chandrasekar Kesavan, Sep 23 '15 at 06:25
Don't work fine what? replace is not working or something else — Chandrasekar Kesavan, Sep 23 '15 at 06:39
"license, lmgr_files, lmgr_service" this line is not depends on foreach cut and past after the foreach block. — Chandrasekar Kesavan, Sep 23 '15 at 07:12
In "installerfilename" -- license=yes is not replaced, is that your problem? — Chandrasekar Kesavan, Sep 23 '15 at 07:29
is that possible for you to provide some sample config.ini and installer.ini — Chandrasekar Kesavan, Sep 23 '15 at 07:45
var license = Regex.Replace(lin, @"license=.*", "license=yes"); var lmgr_files = Regex.Replace(lin, @"lmgr_files=.*", "lmgr_files=false"); Try this by removing else. — Chandrasekar Kesavan, Sep 23 '15 at 08:08
I think the problem is here File.WriteAllLines(installerfilename, license); — ben, Sep 23 '15 at 11:11
I have posted the answer, try and let me know if you have any issues. — Chandrasekar Kesavan, Sep 23 '15 at 12:02

score 1 · Answer 1 · edited May 23 '17 at 12:22

1

There is a mechanism to read/write to ini file. There is no need to re-invent the wheel here. Take a look at Reading/writing an INI file

edited May 23 '17 at 12:22

Community

1
1

answered Sep 23 '15 at 06:04

Viet Nguyen

406
3
12

Chandrasekar Kesavan · Accepted Answer · 2015-09-23T12:35:22.810

Try this code.

string configfilename = path + "config.ini";
string installerfilename = path + "installer.ini";

var link = File.ReadLines(path + "config.ini").ToArray();
var lin = File.ReadLines(path + "installer.ini").ToArray();
IEnumerable<string> newInstaller = lin;
foreach (var txt in link)
 {

     if (txt.Contains("PLP="))
     {
        var PLPPath = txt.Split('=')[1];
        newInstaller = newInstaller.Select(line => Regex.Replace(line, @"fileInstallationKey=.*", "fileInstallationKey=" + PLPPath));                   
     }
     if (txt.Contains("Output="))
     {
         var outputPath = txt.Split('=')[1];
         newInstaller = newInstaller.Select(line => Regex.Replace(line, @"outlog=.*", "outlog=" + outputPath));  
     }
 }
newInstaller = newInstaller.Select(line => Regex.Replace(line, @"license=.*", "license=yes"));
newInstaller = newInstaller.Select(line => Regex.Replace(line, @"lmgr_files=.*", "lmgr_files=false"));
newInstaller = newInstaller.Select(line => Regex.Replace(line, @"lmgr_service=.*", "lmgr_service=true"));

string strWrite = string.Join(Environment.NewLine, newInstaller.ToArray());

File.WriteAllText(installerfilename,strWrite);

@ben I have updated the code by adding new Environment.NewLine and convert the IEnumerable to Array. — Chandrasekar Kesavan, Sep 23 '15 at 12:46

How I can modify the code to make work all statements?

2 Answers2

Linked