Ese if no es correcto, no puedes intercalar condiciones con una ,.

Puedes crearte un array con todas las letras del alfabeto romano y recorrerlo.

gracias amchacon seria algo como esto???:

#include <stdio.h>
#include <stdlib.h>
#define TAM 7
 
int main()
{
    char romano[TAM],letras[]={'I','V','X','L','C','M'},*ptr,*p;
    printf("Dame el primer numero: ");
    fgets(romano,TAM,stdin);
    ptr = romano;
    p = letras;
    while(*ptr++ != '\0')
    {
        if(*ptr++ != *p++)
            printf("Error!! no es numero romano\n");
    }
    return 0;
}

y que significa k&r?

No, seria un bucle anidado.

Te lo hago con fors que me gustan más:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define TAM 7
 
int main()
{
    int i,j;
    char romano[TAM],letras[]={"IVXLCM"};
    printf("Dame el primer numero: ");
    fgets(romano,TAM,stdin);
 
    for (i = 0; romano[i] != '\n';i++)
    {
        for (j = 0;letras[j];j++)
        {
            if (romano[i] == letras[j]) break;
        }
 
        if (!letras[j]) { printf("Error! No es un numero romano: %c \n",romano[i],i);}
    }
 
    return 0;
}

aqui tengo algunas mejoras que le hice al codigo, aunque cuando le ingreso un numero romano cualquiera como: XVI, me imprime error!no es numero romano y no tengo idea de por que, ya van varias veces que lo checo

#include <stdio.h>
#include <stdlib.h>
#define TAM 7
 
int main()
{
    int i,j,bandera;
    char romano[TAM],letras[]={'I','V','X','L','C','M','\0'},*ptr,*p;
    printf("Dame el primer numero: ");
    fgets(romano,TAM,stdin);
    ptr = romano;
    p = letras;
    for(i=*ptr;*ptr!='\0';*ptr++)
    {
        bandera = 0;
        for(j=*p;((*p!='\0')&&(bandera==0));*p++)
        {
            if(*ptr == *p)
               bandera = 1;
        }
        if(bandera!=1)
        {
            printf("Error!No es numero romano\n");
            exit(1);
        }
    }
    return 0;
}

Ya veo, no sabía que hubieran otras opiniones respecto a su uso. En mi facultad los profesores nos prohibían estas instrucciones debido a que utilizar un break/continue es equivalante a la instrucción goto: estás "saltando" desde un punto del bucle a otro modificando el curso de ejecución del programa. Teniendo en cuenta que los paradigmas heredan unos de otros: El paradigma orientado a objetos hereda del estructurado que a su vez hereda del secuencial, C++ es OO y Secuencial a la vez, se deben ejecutar las instrucciones sin saltos de línea.

Tú código queda claro porque hay un único break, pero si estuvieramos hablando de una aplicación un poco más compleja en la que, al permitirse el uso de break/continue, cada bucle tuviera 3 o 4, haría casi imposible su comprensión.

Cualquier opinión o debate es bienvenido. Un saludo.