C# foreach string array

Question

class DM_Matrix {
     public string[] DMInput_Name = new string[] {}; 
     public string[] DMOutput_Name = new string [] {};
     public int[] DMInput = new int[] { 99, 1, 2, 3, 4 };
     public int[] DMOutput = new int[] { 99, 1, 2, 3, 4 };              
}
        
public void Initialize() {
     foreach (var i in DM_Matrix.DMInput_Name) {
          CrestronConsole.PrintLine("[DM.Module.Input_Name" 
               + DM_Matrix.DMInput_Name[i]);
     }
}

Compiler error on "i":

"(local variable) string i

ERROR cannot implicitly convert type 'string' to 'int'"

I'm trying to print every the whole DM.Module.Input_Name array

I've tried to set use "int i" instead of "var i" or cast i from string to integer buy no joy. Not sure why "i" is recognised as string. In my understanding it should be recognised as "int" for the array.

You don't need to do DM_Matrix.DMInput_Name[i]. i itself is the string item coming from DM_Matrix.DMInput_Name collection. So you can simply do CrestronConsole.PrintLine("[DM.Module.Input_Name" + i); — Chetan
– Chetan, Commented Aug 11, 2019 at 2:38

Sweeper · Accepted Answer · 2019-08-11 02:43:14Z

3

You are using a foreach loop, not a for loop. If this were a for loop like this:

foreach (var i = 0 ; i < DM_Matrix.DMInput_Name.Length ; i++)
{
    CrestronConsole.PrintLine("[DM.Module.Input_Name" +DM_Matrix.DMInput_Name[i]);
}

Then yes you can use i as an array index.

But in a foreach loop, the loop variable represents an item of the array, so your loop should be written like this:

foreach (var name in DM_Matrix.DMInput_Name) {
    CrestronConsole.PrintLine("[DM.Module.Input_Name" + name);
}

answered Aug 11, 2019 at 2:43

Sweeper

292k23 gold badges260 silver badges437 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Prasad Telkikar · Accepted Answer · 2021-06-19 10:12:47Z

3

Here you need to understand difference between for and foreach loop.

for loop : If you want to access element from an array by index, then use for loop.

foreach loop : If you want to iterate through each object, then use foreach loop

To solve your problem, use for loop or foreach loop, you are mixing both loops into one.

Solution using for loop,

//Use index to get value from an array
for (int i = 0; i < DM_Matrix.DMInput_Name.Length; i++)
     {
         CrestronConsole.PrintLine("[DM.Module.Input_Name" +DM_Matrix.DMInput_Name[i]);
     }

Solution using foreach loop

//Iterate over elements instead of index
foreach (var item in DM_Matrix.DMInput_Name)
     {
         CrestronConsole.PrintLine("[DM.Module.Input_Name" + item);
     }

edited Jun 19, 2021 at 10:12

answered Aug 11, 2019 at 3:32

Prasad Telkikar

16.1k7 gold badges26 silver badges49 bronze badges

Collectives™ on Stack Overflow

C# foreach string array

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related