How am I doing this wrong. It seems to be replacing my whole string with either \' or spaces.
My struct looks like thisCode:for (i = 0; i < sizeof(id3v1.artist); i++) { if (strcmp(id3v1.artist + i, "\"") == TRUE) { memcpy(id3v1.artist + i, "\'", sizeof(char)); } }
Code:struct id3v1 { char title[30]; char artist[30]; char album[30]; char year[4]; char comment[30]; BYTE genre; } id3v1;
should be:memcpy(id3v1.artist + i, "\'", sizeof(char));
memcpy(id3v1.artist + i, "\"", sizeof(char));
Intel Core 2 Quad Q6600 @ 2.40 GHz
3072 MB PC2-5300 DDR2
2 x 320 GB SATA (640 GB)
NVIDIA GeForce 8400GS 256 MB PCI-E
strcmp does not returns TRUE or FALSE
strcmp used to compare strings not chars
something like that should work better for replacing 1 charCode:if (id3v1.artist[i] == '\"') { id3v1.artist[i] = '\''; }
All problems in computer science can be solved by another level of indirection,
except for the problem of too many layers of indirection.
– David J. Wheeler
> for (i = 0; i < sizeof(id3v1.artist); i++)
In addition to the other comments, strlen() might be more appropriate here:
Code:size_t len = strlen(id3v1.artist); for (i = 0; i < len; i++)
