c error

[mcorn]

Hi all,
I have a program that I'm trying to write. I get an error that I don't know how to correct. Can anyone tell me where my error is? I'm using Borland 4.5 and the error states "cannot covert char* to char in sales function().
This is the portion of the program and the error comes up on the line....trans_descript[trans_count]=descript
This may be an easy error for most of you out there, however, I can't spot it because I'm new to programming. Thanks, M


#include<stdio.h>
#include<stdlib.h>
#include<conio.h>
//#define MAX 50
//#define MAX1 5
float bal=30000;
char trans_type[50];
float trans_amount[50];
char trans_descript[50];
float cur_bal[50];
int trans_count=0;
char type;
char descript[50];
float amount;

void sales(void){
//printf("\nThis is Sales.\n\n");
printf("\nEnter transaction amount: ");
scanf("%f",&amount);
fflush(stdin);
printf("\nEnter transaction decription: ");
gets(descript);
fflush(stdin);
type='S';
bal=bal-amount;
trans_amount[trans_count]=amount;
trans_descript[trans_count]=descript; <--here is where it is
trans_type[trans_count]=type;
cur_bal[trans_count]=bal;
trans_count +=1;
}

[ygfperson]

what are you trying to do at that point in your code?
trans_descript is a pointer to an array of char. descript is simply a char. i can't tell from your program what your purpose is. either you meant to specifiy a char of the array trans_descript, or you meant descript to contain the string trans_descript points to.

[Prelude]

>trans_descript[trans_count]=descript;
trans_descript is an array of 50 char, descript is an array of 50 char. You're trying to assign an array of 50 char to a single char. To copy descript to trans_descript, use

strcpy ( trans_descript, descript );

And include string.h as well.

float bal=30000;
char trans_type[50];
float trans_amount[50];
char trans_descript[50];
float cur_bal[50];
int trans_count=0;
char type;
char descript[50];
float amount;

Avoid global variables wherever possible.

>fflush(stdin);
Using fflush on an input stream results in undefined behavior, don't do it.

>gets(descript);
gets is very unsafe, use fgets instead:

fgets ( descript, sizeof descript, stdin );

-Prelude

[red_baron]

i think he's trying to take descript's content and give it to trans_descript.
you can't do that like that, you could do it like this:

strcpy(trans_descript, descript);

[Unregistered]

Yes that's exactly what I'm attempting to do. I tried this and if then gave me and error ..."Must have Lvalue...." I can't remember exactly but that is the gist of it. I'm basically trying to keep an running history in an inventory program. I used the string.h header file. Is that the correct one? Mark PS. Thanks for trying to help!

[Prelude]

Could you post the most current code you have which displays the error?

-Prelude