Apologies in advance, but that's not the worst practice in your example. My vote would go to this gem:

for(loop = 0; loop < strlen(string); loop = loop + 1 )

The inner loop exhibits the same problem, which is calling strlen() in the condition of a loop. The length of the string doesn't change throughout the program, so repeatedly calling strlen() is unnecessary overhead. In fact, it doubles the time complexity of your loops. That's not a big deal here, but given how simple it is to avoid the problem, there's really no excuse not to.

You also forgot to include conio.h for the use of getch(). I didn't dock any points for a silly use of getch() because it was also in the OP's code. ;)

On a side note, I'd probably solve the problem with printf() format specifiers: