Thread: CD Collector of sorts...

The "%s" format of scanf only reads up to the next whitespace character (after skipping initial whitespace). So if you entered " hello world ", it would just read "hello" and leave the stream pointer pointing to the first space after "hello" (that's where the next input function will start reading).

To read an entire line you need to read up to the newline character which can be done with the "%[^\n]" format. To skip initial whitespace (which is a good idea), put a space before it " %[^\n]".

The reason that the program isn't looping is that the newline is left in the stream as the next character to read after the last previous scanf, and since you are reading a single character without skipping whitespace, "answer" gets set to the newline character, and since that is not 'Y', the loop stops.

Just put a space before the format, like " %c", to skip the newline.

Also note that you set your string sizes to 30 and 20, but tell scanf_s that they are all 50. You should put the actual size there (possibly using sizeof).