Access the same method by two different worker threads

ankit goel 746

I am working on a C# project where I am trying to remove the duplicacy as i have declared the three methods named GetObjArr , copydata and Filter at least two times which are doing the same thing. i have got success in removing the duplicacy for the method GetObjArr but failed in removing the duplicacy for the method copydata and filter . Since i cannot upload the full code , i am sharing the github link for my code and attaching the neccessary files here .

https://github.com/Ankitgoel2009/projectforratlamandoffice/tree/complete_collections

harshitjinames.txt

ankitjinames.txt

I cannot upload the excel here i am pasting the screenshot of the relevant files .
Harshitstatewisenames.xlsx
User's image

statewisenames.xlsx
User's image

Minxin Yu 10,031 Reputation points Microsoft Vendor

2023-03-13T09:13:55.3066667+00:00

Can you explain the txt? What is the result you expect? And can you provide steps to reproduce the problem?
I encountered Cannot convert type 'string' to 'decimal' when uploading excel. You can put excel in Github repos.
ankit goel 746 Reputation points

2023-03-15T11:25:28.3766667+00:00

@Minxin Yu , i have uploaded the file in github . Name is outside15.xlsx . it must be used as a input file . . Now the text file contains the names which i want to put in my source excel .
Minxin Yu 10,031 Reputation points Microsoft Vendor

2023-03-16T09:08:58.76+00:00

I ran the program but still got errors. The project requires a lot of time to read the code and it is difficult to understand what the functions do. Could you provide a minimal example?

ankit goel 746

Ok Here , i will try to minimise the things . But first some explaination .

I have used threading (in my startingmethod()) , so that method3 should not have to wait for method2 to finish . and this is what causing problems . The reason for threading is that my initilal data is huge and requires some time for processing so i decided to use threading so that some processing should took place already before the final output .

For this sake please concentrate on the below code only .

// starting point of my code

public void startingmethod()
{
 Excel.Application excel = null;
        Excel.Workbook wkb = null;
        Excel._Worksheet sheet = null;
        Excel.Range range1 = null;
        int rowindex, columnindex;     
        void dowork()
        {
            excel = new Excel.Application();
            excel.Visible = true;
            // return instance of excel file by sending its path 
            wkb = Open(excel, selectedFile);
            sheet = wkb.Sheets[1];

range1 = sheet.UsedRange; // returns range till grandtotal 
            rowindex = range1.Rows.Count;
            columnindex = range1.Columns.Count;
           // below I have used threading  so that method3 should not have to wait for method2 to finish . and this is what causing problems 
            Thread t2 = new Thread(Method2);
            Thread t3 = new Thread(method3);
            t2.Start();
            t3.Start();


            // wait for both threads to finish
              t2.Join();
             t3.Join();

              wkb.Close(true);
            excel.Quit();
            // CLEAN UP.
            System.Runtime.InteropServices.Marshal.ReleaseComObject(excel);
            System.Runtime.InteropServices.Marshal.ReleaseComObject(wkb);
            System.Runtime.InteropServices.Marshal.ReleaseComObject(sheet);
            System.Windows.Forms.Application.Exit();

 public void Method2()
        {
// ignore rest code for sake of simplicity 
copydata();

  // filter from only index which conatins  names and add the balance 
            Filter(listnew, list); 
var arr2 = GetObjArr(@"C:\ratlamfile\statewisenames.xlsx"); 
// ignore rest code for sake of simplicity 
}

public void method3()
        {

copydata1(); // different method but exact same working as copydata 
            filter1(initialoutputlistforharshit); // different method but exact same working as filter  method 
GetObjArr1(@"C:\ratlamfile\harshitstatewisenames.xlsx"); // this too , same as  GetObjArr
}

now the problem is since both methods executes at same time due to threading and tries to do the same thing , I want that each thread should have its own copy of the methods(copydata,filter and getobjarr) and the output generated from these methods on which it has to work

blasco air 0 Reputation points

2023-03-20T09:21:00.18+00:00

Tesed the repos. The program crashed at range.select.