[Mauzen]

strcmp returns 0 if the string match each other. But you added a not (!) to it.
It should be either
if(strcmp(...) == 0)
or
if(!strcmp(...))

but not both at the same time.