Thread: Need solution for assigning char string to malloc

I'm new in C programming, and this is my first assignment. I have completed all the coding, but there's one part doesn't seems to work.

I actually require to build a conceptual washing machine, where I just need to inform the user about the washing mode and choice. To do this, I have created a structure for storing information about the washing details, and I assign a pointer of the structure with malloc(), so that I can free the pointer to save some power during off mode. Here's part of the code:

Code:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

typedef struct LogInfo                        //create new name for structure
{
	char DateInfo[10];
	char TimeInfo[10];
	char Load[25];
	char WashMode[10];
	char Wash[4];
	char Rinse[4];
	char Spin[4];
	char Soak[4];
	int SoakTime;
	char Temperature[14];
	int EstimatedTime;
}structure;

typedef structure *WashInfoPointer;           //create new type (pointer to structure)

WashInfoPointer WashInfo;                     //define a pointer to structure

main()
{
	WashInfo = malloc( sizeof(structure));        //malloc and initialize values
	
	WashInfo = 0,0,0,0,0,0,0,0,0,0,0; 

	strcpy(WashInfo -> Load, "Small");            //Update washing details

	return;
}

The problem is here, when it come to the last line, when it try to assign "Small" to the character array "Load".

Are there any good solution for assigning "Small" to "Load" without disabling the "malloc"? Or are there any better solution? I have search the web for 2 days, but don't find a solution for this...

Congratulations on writing your first program. The washing machine idea is very original .

Now for your code :

1) WashInfo = 0,0,0,0,0,0,0,0,0,0,0; ---- not really sure what this is supposed to do. In fact, remove it!

2) I don't see any problem with the strcpy call.

You are clobbering your memory at the following line

Code:

WashInfo = 0,0,0,0,0,0,0,0,0,0,0;

Now here is what I get when I remove this line and run your program.

Code:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

typedef struct LogInfo                        //create new name for structure
{
  char DateInfo[10];
  char TimeInfo[10];
  char Load[25];
  char WashMode[10];
  char Wash[4];
  char Rinse[4];
  char Spin[4];
  char Soak[4];
  int SoakTime;
  char Temperature[14];
  int EstimatedTime;
}structure;

typedef structure *WashInfoPointer;           //create new type (pointer to stru
cture)

WashInfoPointer WashInfo;                     //define a pointer to structure

main()
{
  WashInfo = malloc( sizeof(structure));        //malloc and initialize values
  
  /*WashInfo = 0,0,0,0,0,0,0,0,0,0,0; */

  strcpy(WashInfo -> Load, "Small");            //Update washing details

  printf("%s\n", WashInfo->Load);
  return;
}

[cd@localhost oakland]$ ./killua
Small

It's not the cause of your error, but always make sure to free() allocated memory at the end of your program.

And in some indirect way, I pointed this out a few minutes ago.

Oh, thanks, the memset really helps in initializes the structure.

So now, I have another array pointer to act as the washing operation control, as shown below:

Code:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

main()
{
	int *WashControl;                          //define array pointer for control propose

	WashControl = (int*) malloc( sizeof(int[7]) );   //get 7 slots for storing interger

	memset(WashControl,0,sizeof(*WashControl));     //store "load type", "wash mode", "soak"...etc


	//after some choices were made, the back button were pressed
	//all wash control details should be reset to 0, except "load", the first element


	//will this work?

	*WashControl[1] = 0,0,0,0,0,0;

	//or how will memset work without touching the first element of the array?

	free(WashControl);

	return;
}

So lets say I have two menu, 1st: Load Menu (select load type), 2nd: Wash Menu (select wash mode etc...).

When I push the "Back" button somewhere after "Wash Menu", it should back to "Wash Menu" and reset only related settings, without changing the load setting.

If I were to use memset, how would I type in order for it to re-initialize the array, without bothering the first element, and without touching other element it not suppose to touch?

lets say:

memset( WashControl, 0, sizeof(*WashContro)l) ; //Will initialize the first element, that's not what i want

memset( WashControl + 1, 0, sizeof(*WashContro)l) ; //Will touch 1 more element that's not belong to that array


So how should I solve this?

If I were to use memset, how would I type in order for it to re-initialize the array, without bothering the first element, and without touching other element it not suppose to touch?

One way is to track the position. Here is an example that I came up with that doesn't use memset.

Code:

#include <stdio.h>

#define MAX 10

int main(void)
{
  int i;
  char previous[MAX];
  char current[MAX];

  /*set certain elements*/
  previous[0] = 'a';
  previous[7] = 'b';

  current[0] = previous[0];
  current[7] = previous[7];

  for(i = 0; i < MAX; i++) {
    current[i] = 'c';
  }

  printf("The current 0 and 7 positions are: %c, %c\n", current[0], current[7]);

  printf("The previous 0 and 7 positions are: %c, %c\n", 
         previous[0], previous[7]);


  return 0;
}

[cd@localhost oakland]$ gcc -Wall -Wextra reinit.c -o reinit
[cd@localhost oakland]$ ./reinit
The current 0 and 7 positions are: c, c
The previous 0 and 7 positions are: a, b
[cd@localhost oakland]$

And I little bit more to the point

Code:

#include <stdio.h>

#define MAX 10

int main(void)
{
  int i;
  char previous[MAX];
  char current[MAX];

  /*set certain elements*/
  previous[0] = 'a';
  previous[7] = 'b';

  current[0] = previous[0];
  current[7] = previous[7];

  printf("The current 0 and 7 positions before the loop are: %c, %c\n",
         current[0], current[7]);

  for(i = 0; i < MAX; i++) {
    current[i] = 'c';
  }

  printf("The current 0 and 7 positions are now: %c, %c\n", 
         current[0], current[7]);

  printf("The previous 0 and 7 positions are: %c, %c\n", 
         previous[0], previous[7]);


  return 0;
}

[cd@localhost oakland]$ gcc -Wall -Wextra reinit.c -o reinit
[cd@localhost oakland]$ ./reinit
The current 0 and 7 positions before the loop are: a, b
The current 0 and 7 positions are now: c, c
The previous 0 and 7 positions are: a, b
[cd@localhost oakland]$

Omg, i can't belief I forget such a simple solution my lecturer teach, even before exam...I'm doomed =="

By the way, it works. Thanks guys, I couldn't complete my assignment without help from you guys. ^^