¿Me puede decir qué ajustes puedo hacer para mi código o cualquier simplificación? ¿Qué no debería repetir, qué debería cambiar? Este código convierte cada palabra en mayúsculas, si encuentra algún problema, por favor escriba para solucionarlo))

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
int main(){
    FILE * fPtr, *fPtr1;
    int c; /*to store characters*/
    char filename[20];
    char filename2[20] = "temp.txt";
    printf("Enter name of file: ");
    scanf("%19s%*c",filename);
    fPtr = fopen(filename, "r");
    fPtr1 = fopen(filename2, "w");
    c = fgetc(fPtr);
    while(c!=EOF){
        if(c!='\n'){
            if(islower(c)){
                fputc(c-32,fPtr1);
            }else{
                fputc(c,fPtr1);
            }
        }else{
            fputc(c,fPtr1);
        }
        c = fgetc(fPtr);
    }
    fclose(fPtr);
    fclose(fPtr1);
    remove(filename);
    rename(filename2,filename);
    fPtr = fopen(filename, "r");
    c = fgetc(fPtr);
    while(c!=EOF){
        printf("%c",c);
        c = fgetc(fPtr);
    }
    fclose(fPtr);
}
0
Ugly Micha 3 may. 2020 a las 15:06

2 respuestas

La mejor respuesta

Este programa hace lo que tú dices. Pero le recomiendo algunos cambios que su yo futuro apreciará.

Primero, siempre inicialice sus variables; Este hábito ayudará a prevenir errores extraños en su código futuro. Establezca int s en un valor fuera de su rango esperado (por ejemplo, tal vez -1 en este caso); establecer punteros a NULL; establezca las matrices char en { '\0' } o "\0".

Luego, verifique los punteros de sus archivos (fPtr, fPtr1) para NULL después de fopen.

Finalmente, específico de este código, su verificación de nueva línea es innecesaria; islower devolverá 0 si el parámetro no es un carácter alfabético en minúsculas.

0
Shon 3 may. 2020 a las 12:52
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>

#define MAX 20

char *mygets(char *s, size_t sz) {
  int ch;
  size_t i = 0;

  while((ch = getchar()) != '\n' && i < sz)
    s[i++] = ch;

  s[i] = '\0';

  return s;
}

int main(void) {
  FILE *fPtr;
  char filename[MAX+1];
  int c, i;

  printf("Enter name of file: ");
  mygets(filename, MAX+1);

  if(!strstr(filename, ".txt"))
    strcat(filename, ".txt");

  if((fPtr = fopen(filename, "r+")) == NULL) {
    fprintf(stderr, "Could not open %s\n", filename);
    exit(1);
  }

  i = 0;
  while((c = fgetc(fPtr)) != EOF) {
    fseek(fPtr, i, SEEK_SET);
    fputc(toupper(c), fPtr);
    i++;
  }

  rewind(fPtr);

  while((c = fgetc(fPtr)) != EOF)
    putchar(c);

  fclose(fPtr);

  return 0;
}
0
Patrick Cardoso 3 may. 2020 a las 17:52