Mensajes

Las funciones fscanf, scanf y sscanf retornan el numero de conversiones realizadas con éxito. La condición del bucle:

Código (c) [Seleccionar] Expandir

while (fscanf(ptrBdd, " %[^:]:%d:%s", campoUno, &campoDos, campoTres) == 3)
   printf( "%s %d %s\n", campoUno, campoDos, campoTres);
fclose (ptrBdd);
Compara el resultado de fscanf con el valor tres porque este es el numero de conversiones que debe realizar la función al leer cada registro. En otras palabras "Mientras se lean con éxito los tres campos que componen un registro ellos se imprimen en la salida estándar".

Si no tienes una referencia de la biblioteca estándar de C dos (en ingles) que puedes utilizar son:
Plauger C90 Reference
C Plus Plus: scanf

Un saludo

antes de este while, ¿es necesaria una lectura previa como con el feof?

No es necesario, esa es una de las razones por las cuales no se recomienda feof: te obliga a duplicar las operaciones de lectura. En el caso que mencionas basta con una sola llamada fread la cual sirve como condición del bucle.

Si te interesa ahondar sobre las razones para no usar feof te recomiendo utilizar el motor de búsqueda de los foros, encontraras una media docena de mensajes sobre ello.

Código (c) [Seleccionar] Expandir

while (fread(&dato,sizeof(dato),1,arch) == 1)

Por otro lado, no entiendo una cosa, la condicion dice que mientras fread() == 1, ¿Cómo es eso que el fread puede ser igual o distinto a 1? Porque te refieres explicitamente a la función fread, y no a la variable donde almaceno el valor (dato).

Explico: la función fread trata de leer del stream indicado (cuarto argumento) el numero de elementos (tercer argumento) del tamaño indicado (en bytes, segundo argumento) y los almacena a partir de la dirección indicada (primer argumento).

Su valor de retorno es el numero de elementos leídos con éxito y que en tu caso si es uno indica una lectura exitosa lo que nos lleva a olvidarnos de feof (si la función retorna cero ello indica que se activo el indicador de error o fin de archivo).

Un saludo

[avanza]

me gustaría tener una explicación de porque se solucionó con ese fseek, el de la línea 17, ya que no me queda muy claro.

No debes utilizar feof para controlar el bucle de lectura de datos, en su lugar puedes utilizar el valor de retorno de la función fread con lo cual el bucle termina así:

Código (c) [Seleccionar] Expandir

while (fread(&dato,sizeof(dato),1,arch) == 1)
   if (dato > prom){
      fseek(arch, -1 * (int) sizeof(dato), SEEK_CUR);
      fwrite(&menosuno, sizeof(menosuno), 1, arch);
   }

Es necesario utilizar fseek porque cada vez que lees/escribes en un archivo el indicador de posición de este avanza a la siguiente posición, si no fuera así y, por ejemplo, tratas de leer diez valores en las diez ocasiones leerías el mismo valor.

En resumen con fread lees un valor, el indicador avanza a la siguiente posición, con fseek regresas a la posición del valor leído y finalmente con fwrite lo sobrescribes con un menos uno.

Por ultimo la llamada a función:

Código (c) [Seleccionar] Expandir

fseek(arch,0,SEEK_CUR);
Es inútil porque indica que a partir de la posición actual (indicada por SEEK_CUR) se desplace ... cero bytes (segundo argumento).

Un saludo

¿es correcto poner el cierre del archivo dentro del while()? ¿ o tengo que cerrarlo con el condicional if?

El programa tal como esta funciona correctamente:
A) Si la apertura del archivo falla no es necesario cerrarlo.
B) Si la apertura del archivo es exitosa este se cierra después del bucle while.

Un detalle es la especificación de rangos en la forma "a-z" ya que estos dependen del compilador utilizado, otro mas es la falta de validaciones mas el uso de feof para controlar el bucle. Para el caso es mejor:

Código (c) [Seleccionar] Expandir

if ( ptrBdd == NULL) {
   perror("datos.txt");
}else {
   while (fscanf(ptrBdd, " %[^:]:%d:%s", campoUno, &campoDos, campoTres) == 3)
      printf( "%s %d %s\n", campoUno, campoDos, campoTres);
   fclose (ptrBdd);
}

Un saludo

alguien me puede decir si esta es una manera correcta de duplicar un archivo

El programa debe funcionar correctamente, solo un detalle: si alguna de las aperturas de archivo falla terminaras llamando a la función fclose con su argumento igual a NULL y eso no lo define el lenguaje (comportamiento no definido).

Para solucionarlo antes de llamar a fclose verifica el valor de su argumento mediante una sentencia condicional:

Código (c) [Seleccionar] Expandir

if (ptrOrigen != NULL)
   fclose (ptrOrigen);
if (ptrCopia != NULL)
   fclose (ptrCopia);

Un saludo

hay alguna forma de guardar en primer lugar siempre un numero? me explico, guardar un numero y despues al guardar otro se ponga el primero, no a continuacion del primero

Depende de la forma en que abras el archivo y guardes los datos en el, algo que debes explicar a detalle.

A grandes rasgos es posible en dos escenarios, si se trata de un archivo:
A) de texto plano y el numero en realidad es un dígito (o bien se imprimió indicando una anchura de campo suficiente para el valor máximo del tipo).
B) binario (apertura en modo "b" y escritura con fwrite).

En ambos casos basta con abrir el archivo en modo "r+" (mas "b" dependiendo del caso), colocarse al principio del archivo con fseek y por ultimo utilizar la misma función para escribir los datos en el.

Un saludo

Formas para extraer los números de la linea de texto hay varias, por ejemplo utilizando un objeto de la clase istringstream para tratar la linea como un stream:

Código (cpp) [Seleccionar] Expandir

#include <sstream>
using std::istringstream;

// ...

string linea = "1 2 3 4";
int num;

// ...

istringstream in(linea);
while (in >> num)
   cout << num << endl;

Un saludo

segundo error.... todos (o casi todos) tus scanf están mal declarados...

Código (cpp) [Seleccionar] Expandir

scanf(" %c", &tipo);
espera literalmente lo que está ahí... que escribas un espacio y luego una letra... sin embargo lo agarra bien, aunque debes quitarle los espacios

No es un error sino un forma común de utilizar esa función. Explico:

scanf con todos los especificadores de formato (excepto dos) lo primero que hace es descartar el espacio blanco (espacio, tabulador, etc.), solo entonces consume los caracteres validos para la conversión indicada. Por ejemplo si tratamos de leer un entero y una palabra con "%d%s" una entrada como esta funcionara correctamente:

Código [Seleccionar] Expandir

      123


fulano

Las dos excepciones son "%c" y "%[]", con estas el descarte no sucede (la función se "traga" los caracteres que pueda), por ejemplo si tratamos de leer un entero y un carácter con "%d%c" la siguiente entrada no dará el resultado esperado:

Código [Seleccionar] Expandir

123{ENTER}
x{ENTER}
Ello porque la función consumirá los caracteres '1', '2' y '3' para generar el entero 123 y el avance de linea que sigue sera almacenado en la dirección indicada.

Para evitarlo se suele utilizar un espacio antes del especificador "%c" ya que este (el espacio) se interpreta como "descartar todo carácter para el cual la función isspace resulte en verdadero".

Por ultimo con la bendita función los caracteres de espacio blanco no se toman literalmente, en su lugar cualquier secuencia de estos se interpreta de la forma indicada (lectura y descarte de espacio blanco).

Un saludo

tengo una duda a la hora de hacer un menú que devuelva la opción elegida por el usuario: el problema es que tengo que hacer un do-while para que vaya verificando si la opción es correcta y hay opciones numéricas (1,2,3,etc) y una letra(q-para salir-).

Solo tienes que leer la opción como un carácter, continuación realizas la operación correspondiente de acuerdo con el enunciado de tu programa, mas o menos así:

Código (cpp) [Seleccionar] Expandir

#include <iostream>
using std::cin;
using std::cout;
using std::endl;

int main()
{
   char opcion;
   bool salir;
   
   salir = false;
   do {
      cin >> opcion;
     
      switch (opcion){
      case '0':
         cout << "La opcion es 0" << endl;
         break;
      case '1':
         cout << "La opcion es 1" << endl;
         break;
      case '2':
         cout << "La opcion es 2" << endl;
         break;
      case 'q':
         cout << "La opcion es q" << endl;
         salir = true;
         break;
      default:
         cout << "Opcion no valida" << endl;
         break;
      }
   }while (!salir);
   
   return 0;
}

Un saludo

¿me podrías decir si mi programa tiene que ver con el enunciado?

Tu programa realiza la operación indicada de forma correcta, solo una recomendación: no utilices la función printf para imprimir una cadena vacía. En su lugar puedes utilizar un condicional para imprimir el espacio o bien ya que se trata de imprimir un solo carácter puedes usar la función putchar y el operador condicional, de esta forma:

Código (c) [Seleccionar] Expandir

while (bits){
   putchar(v & bits ? '1' : '0');
   bits >>= 1;
   
   sep++;
   if (sep % CHAR_BIT == 0)
      putchar(sep == sizeof(unsigned) * CHAR_BIT ? '\n' : ' ');
}

Un saludo