Pig-Latin | Pig Latin Translator | Translation library

Remember that strtok modifies the buffer.

The caller to your functions would have to pass a temporary buffer that has been copied from the original before each call.

In other words, the call to ModifiedStringSize trashes inputString so that when you call ManipulateString, the updated value for inputString is [effectively] garbage.

The usual here is to parse a buffer only once and retain a char ** array that is built up from the strtok loop.

UPDATE:

Sorry, I'm still learning C and I'm not really that good with pointers. How exactly do I implement this?

There are basically two ways to do this:

1. Caller passes a pointer to a fixed size char ** array that the subfunction fills in with tokens [for a given line]
2. The subfunction uses malloc/realloc to create and maintain a dynamically increasing list of tokens. It returns a pointer to that array.

Option (1) is easier/simpler and is suitable for line-at-a-time processing. That is, all necessary processing can be done without having to combine multiple lines of input (e.g. this is probably suitable for your use case).

Option (2) is used if we have to read an entire multiline file and store all lines before doing any processing. Or, we can not predict how many tokens we'll need before attempting to parse. It is the caller's responsibility to call free on the array elements and the array pointer itself.

Note that I had to refactor your code a bit.

Here's the first method:

You return the word arkShay before captalized ...Please return pigged_input only at function end.

I have modified your code