El programa funcionara correctamente siempre y cuando ninguna palabra tenga una longitud mayor a 19 caracteres pero, si existe una, el programa puede terminar reventando.

Para evitarlo dentro del bucle debes verificar el valor del contador "n_char", si este tiene un valor mayor al permitido le asignas el máximo, de esta forma:

while ((c = getchar()) != EOF)
   if (c != ' ' && c != '\n' && c != '\t')
      ++n_char;
   else if (n_char > 0){
      if (n_char > 19)
         n_char = 19;
 
      index[n_char]++;
      n_char = 0;
   }

Un saludo

Por partes.

El programa que presentas es uno de los ejemplos del primer capitulo del libro "The C Programming Language", una versión sencilla del comando wc (abreviación de word count). En el los autores enseñan como detectar la transición fuera-->dentro, cuando esta sucede se incrementa el contador de palabras en una unidad.

Paginas mas adelante uno de los ejercicios a resolver es el que comentas: imprimir las frecuencias de las palabras obtenidas de la entrada estándar. Para esto se debe detectar la transición opuesta (dentro-->fuera) y cuando esto suceda incrementar el contador de frecuencia correspondiente.

El primer error importante ya te lo indico reich: utilizar el operador de asignación "=" cuando deberías utilizar el de comparación "==". Otros son no colocar a cero el contador de longitud cuando se sale de una palabra y la falta de los paréntesis en la expresión que controla al bucle:

while (c = getchar() != EOF) ...

Hay que cambiarla a:

while ((c = getchar()) != EOF) ...

Otros mas son imprimir el primer elemento del array cuando este no se utiliza y no tomar previsiones en caso de una palabra de mas de diez caracteres (lo cual llevaría a utilizar un indice invalido al acceder al array).

Por ultimo no es necesario la bandera de estado, en su lugar se utiliza al contador de longitud para conocer este (cero es fuera de una palabra, mayor que cero es dentro). El bucle en pseudocódigo:

Un saludo

Ya que solo se trata de un contador con la salida NN:NN:NN se puede reducir el numero de bucles a dos y el numero de expresiones mediante un centinela. Un programa para explicarlo mejor (C90):

#include <stdio.h>
#include <stdlib.h>
 
int main(void)
{
   int num[4] = {0}; /* num[0] es un centinela */
   int i;
 
   while (num[0] == 0){
      printf("%02d:%02d:%02d\n", num[1], num[2], num[3]);
 
      for (i = 3; ++num[i] == 100; --i)
         num[i] = 0;
   }
 
   return EXIT_SUCCESS;
}

Un saludo

No veo cual sea el problema ya que puedes copiar las estructuras mediante el operador de asignación y en cuanto a comparar los campos (tipo string) "Naame" de dos estructuras puedes utilizar los operadores relacionales.

Para darte una idea si se quiere verificar si los dos primeros elementos están desordenados e intercambiarlos si es el caso el condicional seria:

if (emp[0].Naame > emp[1].Naame){
   Employees aux;
 
   aux = emp[0];
   emp[0] = emp[1];
   emp[1] = aux;
}

Donde "emp" es el nombre del array de estructuras. Por ultimo si no se desea realizar el intercambio manualmente se puede utilizar la función swap.

Un saludo

El problema principal se debe a que en la función "subMenu" abres el archivo en modo "a+t" y con ello todas las operaciones de escritura se realizaran al final del archivo sin importar el uso de funciones de posicionamiento como fseek y rewind. En su lugar debes abrir el archivo en el modo "r+b" cuidando que antes de cualquier operación de escritura te posiciones al principio o final del archivo (dependiendo si vas a escribir el encabezado o agregar un registro) mediante la función fseek.

Ademas se debe cambiar/corregir una parte significativa del programa:

* Si declaras una macro no debes darle un nombre que inicie con un guion bajo ya que esos nombres se reservan para la implementación (algunos ejemplos son __DATE__, __FILE__, __LINE__, etc.).

* Cuando se declara una función con paréntesis vacíos:

int readInt();
 
/* ... */
 
void menu();

Ello indica un numero no determinado de argumentos, para indicar que no recibe argumentos debes utilizar "(void)".

* Si no utilizas los argumentos pasados mediante la linea de comandos cambia la definición de la función principal a "int main(void) ..." o "int main() ...".

* La biblioteca estándar de C no incluye una función con el nombre "fgetchar", en su lugar utiliza getchar o mejor todavía elimina por completo esa llamada.

* En el bloque de prototipos falta la declaración de la función "seeAll".

* En la función "subMenu" en la llamada a "readLine":

readLine(&buffer.firstName, TAM_CADENA);

El operador "dirección de" (el '&') esta de mas en su primer argumento, eliminalo.

* En la función "readInt" no se especifica el valor a retornar si ocurre un error y si la linea es demasiado larga el restante se quedara en el bufer de la entrada estándar (para el caso mejor utiliza scanf para leer el entero y a continuación descarta el resto de la linea con un bucle.

* En la función "readLine" si el ultimo carácter no es el avance de linea no es necesario sobrescribir este con un '\0' porque fgets lo agrega automáticamente. Y el tipo de retorno de getchar es int, la función hay que cambiarla a:

void readLine (char cadena[], int tam)
{
   if (fgets(cadena, tam, stdin) != NULL){
      size_t i = strlen(cadena) - 1;
 
      if (cadena[i] == '\n')
         cadena[i] = '\0';
      else {
         int ch;
 
         while ((ch = getchar()) != EOF && ch != '\n')
            ;
      }
   }
}

* En la funcion "seeAll" el bucle para imprimir todos los registros:

while (fread (&buffer, 1, header.recordLength, fd) == 1){
   /* ... */
}

No funcionara correctamente, las razones de ello te las explique en tu tema Funcion que no puede retornar un struct [C].

Un saludo

Aparte de lo ya comentado por leosansan otro cambio a realizar son eliminar las declaraciones:

int nonul=0;
 
// ...
 
int i,j;

Ya que no las utilizas en tu programa.

Y no es necesario utilizar "<= N - 1" cuando se puede utilizar con el mismo efecto "< N", con ese cambio los bucles terminan así:

for (int i = 0; i < TAM - 1; i++)
   for (int j = i + 1; j < TAM; j++)
      if (M[i][j] == 0)
         c++;
cout << "nulos:" << c << endl;

Un saludo

Ademas (y respondiendo a tu otro tema sobre el mismo programa, deberías haber respondido en este) para obtener la diagonal principal solo necesitas de un bucle, la funcion "Diag" hay que cambiarla a:

void Diag(int a[TAM][TAM], int n, int d[TAM])
{
   cout << "Te voy a dar la diagonal principal:" << endl;
 
   for (int i = 0; i < n; i++)
      d[i] = a[i][i];
}

Un saludo

Otra aproximación se basa en considerar la salida esperada (la alineación solo es para facilitar la lectura):

Como el resultado del absoluto mas uno de las series:

La implementación en C++ consiste en dos bucles anidados (antes de utilizar la función abs se debe incluir el encabezado <cstdlib>):

for (int i = 0; i != 5; ++i){
   for (int j = -i; j <= i; ++j)
      cout << abs(j) + 1;
 
   cout << endl;
}

Un saludo

La salida que mencionas sucede porque al detectar la división por cero imprimes el mensaje de error pero realizas de todos modos la operación cuando no deberías.

Para solucionarlo debes utilizar una sentencia "if ... else ..." donde si el divisor es cero presentas el mensaje y en caso contrario (divisor diferente de cero) realizas la operación. En el caso de la primera operación el condicional debe cambiarse a:

if (operation == "/" || operation == "div"){
   if (seconnum == 0)
      cout << "cero is impossible to operate" << endl;
   else
      cout << "The first div is: " << (firstnum / seconnum) << endl;
}

El cambio en la segunda operación es similar.

Por ultimo puedes eliminar las variables donde almacenas el resultado (plus, minus, etc.) ya que no son necesarias, puedes imprimir el resultado directamente.

Un saludo

Pides la entrada al usuario y lees esta como una linea mediante la función getline, para conocer el numero de caracteres utilizas la función miembro size y para verificar si un carácter es alfanumérico utilizas la función isalpha isalnum.

Editado: agregue la corrección indicada por ivancea96. Gracias.

Un saludo