• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 663
  • Last Modified:

Doing calculations with arrays - invalid operands to binary

I'm new to C programming but feel i'm pretty good at perl so I feel like I really should be able to do this.
I've looked at C tutorials and in several text books but I can't find anything which helps.

All I want to do is do some calculations with arrays of data.
for (i=1; i<=d; i++)
a[i] = b[i] / c[i];
Why does this give me the error message: invalid operands to binary /. I'm using Dev-C++.
The division seems to be causing a problem.
I've tried all data types. I want to use double or float.
I'm adding to an existing C program and earlier in the program there is this code:
for (i=1; i<=x; i++)
y[i] = z[i]  /  t;
This works but mine doesn't.

This type of thing is easy in perl but i've spent ages on this.

Can anyone tell me how to work through arrays and perform calculations on each element or point me to a useful website. Thanks

1 Solution
Hello csnell,

What you have written is fine and should work ... can you post complete code ...

Or you can try this small example.

#include <stdio.h>

int main ()
    float a[] = { 1.1, 2.2, 3.3 };
    float b[] = { .1, .2, .3 };
    float c[3];
    int i;

    for (i=0; i<3; i++)
        c[i] = a[i]/b[i];
    for (i=0; i<3; i++)
        printf ("%f\t",c[i]);

First, array indexing in C starts at 0, not 1, so most likely :

        for (i=1; i<=d; i++)

has to be :

        for (i=0; i<d; i++)

where d is the length of the array.

About the division not working : can you show how you defined the arrays a, b and c ?
evilrixSenior Software Engineer (Avast)Commented:
It should be more or less the same as Perl... try this...
#include <iostream>
int main()
	int a[] = {1,2,3,4,5};
	int b[] = {1,2,3,4,5};
	for(size_t i = 0 ; i < 5 ; ++i)
		std::cout << a[i] << "/" << b[i] << "=" << a[i]/b[i] << std::endl;
		std::cout << a[i] << "*" << b[i] << "=" << a[i]*b[i] << std::endl;
		std::cout << a[i] << "+" << b[i] << "=" << a[i]+b[i] << std::endl;
		std::cout << a[i] << "-" << b[i] << "=" << a[i]-b[i] << std::endl;

Open in new window

csnellAuthor Commented:
Thanks for your help.
It wasn't working because when I had initialized the arrays I had used *. I had looked at what had been done previously in the program for what I thought were arrays. I now know they are pointers!. I have only just begun to understand the basics of pointers so I didn't notice my error.
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

Join & Write a Comment

Featured Post

Get your problem seen by more experts

Be seen. Boost your question’s priority for more expert views and faster solutions

Tackle projects and never again get stuck behind a technical roadblock.
Join Now