Array-related Functions

[brady]

i have a function called showArray(int ar[], int numElements, and int numPerLine)
i am having some issues with creating the function properly so that the caller can call it. so far i have this:

/******************************
Function Name: void showArray(int ar[], int numElements, int numPerLine)



******************************/
void showArray(int ar[], int numElements, int numPerLine)
{
int i;`


while(numElements < 200)
{
i = ar[0];
cout << i;
i++;
}
if(numPerLine *= 2)
cout << endl;


an array will be passed through which has 200 elements. in main i have a symbolic const for the number of elements but dont know what passes through the other two arguments. any suggestions?!?! i know the function is coded incorrectly also! Thanks

[brady]

By the way, the function needs to display the elements of the array, several item per line as specified by the third argument. The seconds argument tells the function how many items are in the array, so it knows where to stop. i need to output a newline character when the number of items slready printed is a multiple of numPerLines. maybe the % operator? Thanks

[Savage]

Hi brady,

You have not defined a good condition for your while loop.You cannot predict
that your array will have 200 elements.Why not use instead something like:

int i=0;
int element
 
while(i<numElements)
{
   element=ar[i];
   ar[i-1]='\0';
   cout<<element;
   i++;
}

Hmm,for third argument you could integrate inside loop a counter which will count up to numPerLine and then it must become counter=counter%numPerLine and cout \n character to force the start of a new line.


Savage

[brady]

Hello Savage,
Thanks for the reply! That makes a lot more sense then what i had. For the counter, can i just place that right after incrementing i? then maybe have a decision statement. im new with C++ so its a bit confusing.

Hi brady,

You have not defined a good condition for your while loop.You cannot predict
that your array will have 200 elements.Why not use instead something like:

int i=0;
int element
 
while(i<numElements)
{
   element=ar[i];
   ar[i-1]='\0';
   cout<<element;
   i++;
}

Hmm,for third argument you could integrate inside loop a counter which will count up to numPerLine and then it must become counter=counter%numPerLine and cout \n character to force the start of a new line.


Savage

[Savage]

Hello Savage,
Thanks for the reply! That makes a lot more sense then what i had. For the counter, can i just place that right after incrementing i? then maybe have a decision statement. im new with C++ so its a bit confusing.

Yes,that should do it!!!

Savage

[brady]

Another question, is the '\0' a null value?

Hi brady,

You have not defined a good condition for your while loop.You cannot predict
that your array will have 200 elements.Why not use instead something like:

int i=0;
int element
 
while(i<numElements)
{
   element=ar[i];
   ar[i-1]='\0';
   cout<<element;
   i++;
}

Hmm,for third argument you could integrate inside loop a counter which will count up to numPerLine and then it must become counter=counter%numPerLine and cout \n character to force the start of a new line.


Savage

[Savage]

No,it's a terminating character which will ensure that you don't acsidently
cout previos element.


Savage

[brady]

Yes thats right! Thanks for the refresher and thanks for your help! This function was driving me crazy! i'll post the final here in a minute and if you're around and happen to view the post and see a mistake could you let me know? i cant test it yet because im having issues passing arguments into the function. im getting the invalid conversion to int*, int or something like that

No,it's a terminating character which will ensure that you don't acsidently
cout previos element.


Savage

[Savage]

Yes thats right! Thanks for the refresher and thanks for your help! This function was driving me crazy! i'll post the final here in a minute and if you're around and happen to view the post and see a mistake could you let me know? i cant test it yet because im having issues passing arguments into the function. im getting the invalid conversion to int*, int or something like that

Please post just the troubleshooting part because if you post full code it will
be snipped by the moderators(see posting guidelines)

Savage

[brady]

I didnt know that. and yes i will follow those guidlines. thanks for the fore warning

Please post just the troubleshooting part because if you post full code it will
be snipped by the moderators(see posting guidelines)

Savage

[Savage]

I didnt know that. and yes i will follow those guidlines. thanks for the fore warning

Always a pleasure..


Savage

[brady]

Well i received a successful build after implementing the counter. just hope it works. here is what i coded after i was incremented:
while....
//body of loop previously posted
counter++;
if(counter = counter%numPerLine)
cout << "/n";

Always a pleasure..


Savage

[Savage]

Well i received a successful build after implementing the counter. just hope it works. here is what i coded after i was incremented:
while....
//body of loop previously posted
counter++;
if(counter = counter%numPerLine)
cout << "/n";

Here I can find 2 errors.

FIRST:

if(counter = counter%numPerLine)

is invalid.This condition will never become TRUE.

try using instead

if(counter==numPerLine) counter=counter%numPerLine,cout <<"\n";

SECOUND:

You might noticed that i used == when declaring a condition.Without these it would submit message:

"Possibly incorrect assigment ........"

Savage

[Savage]

Sorry for double post but i maked a mistake .The condition might become TRUE but I think that is not the best solution,anyway you would need to
"reset" a counter.


Savage

[brady]

Yea that makes sense because im not assigning values. Can i code it like this:
if(counter==numPerLine)
counter=counter%numPerLine;
cout <<"\n";
is this the same as how you had it on one line?

Here I can find 2 errors.

FIRST:

if(counter = counter%numPerLine)

is invalid.This condition will never become TRUE.

try using instead

if(counter==numPerLine) counter=counter%numPerLine,cout <<"\n";

SECOUND:

You might noticed that i used == when declaring a condition.Without these it would submit message:

"Possibly incorrect assigment ........"

Savage

[brady]

Not a prob. i can reset the counter like so, right?


if(counter==numPerLine)
counter=counter%numPerLine;
cout <<"\n";
counter = 0;

Sorry for double post but i maked a mistake .The condition might become TRUE but I think that is not the best solution,anyway you would need to
"reset" a counter.


Savage

[Savage]

Yea that makes sense because im not assigning values. Can i code it like this:
if(counter==numPerLine)
counter=counter%numPerLine;
cout <<"\n";
is this the same as how you had it on one line?

Yes,it's same.Operator ',' serves to declare actions that are correlated and it makes your code smaller.

Note:Use this operator only when actions are correlated or u might get lost

in code!!!

Savage

[brady]

so instead of having lines of code like i just did it, the ',' operator lets you code, in our example, an entire loop on one line? very interesting. i didnt know that!

Yes,it's same.Operator ',' serves to declare actions that are correlated and it makes your code smaller.

Note:Use this operator only when actions are correlated or uou might get lost

in code!!!

Savage

[Savage]

Well,no one is borned with knowledge.

We exist to learn.


Savage

[brady]

Another question for you, if you dont mind. my caller for that function is as followed:

showArray(storescores, 50, 8);

storescores is my array to hold scores from a file with 200 spaces available
50 and 8 are numbers i just made up to see if they would pass through


but im receiving a invalid conversion from int* to int. why would that be?

[brady]

ha ha that is true. i like that

Well,no one is borned with knowledge.

We exist to learn.


Savage

[Savage]

Another question for you, if you dont mind. my caller for that function is as followed:

showArray(storescores, 50, 8);

storescores is my array to hold scores from a file with 200 spaces available
50 and 8 are numbers i just made up to see if they would pass through


but im receiving a invalid conversion from int* to int. why would that be?

Hmm,how have you declared storescores?

Savage

[brady]

Yup, like so:
int storescores[200];

i had a symbolic constant of scores set to 200 and then i had the array like so:
storescores[scores];
but that was also giving me the same error

Hmm,how have you declared storescores?

Savage

[Ganon11]

Just a message that I've renamed the thread to be a bit more descriptive and not so verbose. :) Continue on!

MODERATOR

[brady]

Thank you moderator, i was panicing and got a little keyboard happy. :)

Just a message that I've renamed the thread to be a bit more descriptive and not so verbose. :) Continue on!

MODERATOR

[Savage]

Yup, like so:
int storescores[200];

i had a symbolic constant of scores set to 200 and then i had the array like so:
storescores[scores];
but that was also giving me the same error

try declaring function as following:

void showArray(int *ar[],int numElements,int numPerLine);

Savage

[brady]

That would be before main() right?

try declaring function as following:

void showArray(int *ar[],int numElements,int numPerLine);

Savage

[Savage]

That would be before main() right?

Yes,before main() and after it when defining what is function supposed to do.

(Thats,the part u first posted)


Savage

[brady]

Got a successful build. i declared it before main as:
void showArray(int *ar[],int numElements,int numPerLine)
and i also changed the caller of the function to that also. is that correct?

Yes,before main() and after it when defining what is function supposed to do.

(Thats,the part u first posted)


Savage

[Savage]

Got a successful build. i declared it before main as:
void showArray(int *ar[],int numElements,int numPerLine)
and i also changed the caller of the function to that also. is that correct?

Yes it is.Bravo!!!


Savage

[brady]

Now correct me if im wrong but shouldnt the caller of the function (in main) be something different from what i declared the function as (before main)?

Yes it is.Bravo!!!


Savage

[brady]

Savage,
Thanks for all your help. greatly appreciated!

Now correct me if im wrong but shouldnt the caller of the function (in main) be something different from what i declared the function as (before main)?

[Savage]

Now correct me if im wrong but shouldnt the caller of the function (in main) be something different from what i declared the function as (before main)?

Just a litlle.The main differance is that when calling a function you don't

type like:

void showarray(int *ar[],int numOfElements,int numPerLine);

You type it like:

showarray(scores,200,50);

for e.g


Savage

[Savage]

Savage,
Thanks for all your help. greatly appreciated!

Just as I said:

Always a pleasure to help...


Savage