Error al leer el fichero txt

[Hetzfield]

Hola a todos:

Estoy realizando un proyecto y tengo que leer de cada línea de un fichero "txt" dos bloques distintos.

La estructura de la línea es la siguiente, los bloques están separados por un espacio en blanco:
FECHA " " HORA " " PRODUCCIÓN " " MÓDULO
Ejemplo:
2008-01-02 12:00 273 82.04

Tengo que leer la producción y el módulo, con lo que la fecha y la hora al ser constantes, empiezo a leer a partir de la posición 17. El problema me da al leer caracter a caracter, si muestro la línea entera, la enseña sin problemas, en cambio, sin voy carácter a carácter, me muestra muchísima basura. Por más vueltas que le he dado no hay forma de encontrar el fallo, espero que me podáis echar una mano.

Os adjunto el código a continuación:

Código C++:

#include<iostream.h>
#include<fstream.h>
#include<stdio.h>
#include<string.h>
#include<conio.h>
#include<stdio.h>
#include<stdlib.h>
 
 
void main()
{
  int i,p,m;
  float modulo;
  bool espacio,finlinea=false;
  char c,linea[30],prod[5],mod[5];
  ifstream MiArchivo;
 
  //ABRIMOS EL FICHERO
  MiArchivo.open("c:\\datos.txt");
 
  if (!MiArchivo.fail())
  {
    while (!MiArchivo.eof() )
    {
      strcpy(linea,"");
      strcpy(prod,"");
      strcpy(mod,"");
 
      //LEEMOS LA LINEA
      MiArchivo.getline(linea,sizeof(linea));
 
      i=17;
      p=0;
      m=0;
      finlinea=false;
 
      //ESTE IF ESTÁ PARA SALTAR LINEAS QUE NO QUEREMOS PARA NADA
      if(strlen(linea)>17)
      {
       while(i<30 && !finlinea)
       {
            c=linea[i];
                if(c==' ')
                         espacio=true;
            if(c=='/n')
                finlinea=true;
             else
                { 
                         //SI TODAVIA NO HEMOS ENCONTRADO EL ESPACIO,ES LA PRODUCCION
                         if(!espacio)
                         {
                                prod[p]=linea[i];
                                p++;
                          }
 
                          else
                          {
                                mod[m]=linea[i];
                                m++;
                          }
                 }
         i++;
         }
         cout<<"Produccion: "<<prod<<endl;
         cout<<"Módulo: "<<mod<<endl;
      }
      else
        cout<<linea<<endl;
    }
    MiArchivo.close();
  }
  else
   printf ("No se pudo abrir el archivo.");
  getch();
}
 

Muchas Gracias

Un saludo

[paracoccidio]

Pues vi tu codigo y me llamo la atencion, la verdad no supe que es lo que querias hacer pero pude corregir muchas cosas que te complicaban te dejo el codigo:

Código C++:

#include<iostream>
#include<fstream>
#include<cstring>
 
 
using namespace std; 
 
int main()
{
    int i,p,m;
    float modulo;
    bool espacio,finlinea=false;
    char c,linea[30],prod[5],mod[5];
    ifstream MiArchivo;
    
    //ABRIMOS EL FICHERO
    MiArchivo.open("datos.txt",ifstream::in);
    
    if (!MiArchivo.fail())
    {
        while (!MiArchivo.eof() )
        {
            strcpy(linea,"");
            strcpy(prod,"");
            strcpy(mod,"");
        
            //LEEMOS LA LINEA
            MiArchivo.getline(linea,sizeof(linea));
        
            i=17;
            p=0;
            m=0;
            finlinea=false;
        
            //ESTE IF ESTÁ PARA SALTAR LINEAS QUE NO QUEREMOS PARA NADA
            if(strlen(linea)>=17)
            {
                while((i < 30) && (!finlinea))
                {
                    c=linea[i];
                    if(c==32)//32 espacio en codigo ascii
                        espacio=true;
                    if(c==10)// salto de linea          
                        finlinea=true;
                    else
                    { 
                        //SI TODAVIA NO HEMOS ENCONTRADO EL ESPACIO,ES LA PRODUCCION
                        if(!espacio)
                        {
                            prod[p]=linea[i];
                            p++;
                        }
                        else
                        {
                            mod[m]=linea[i];
                            m++;
                        }
                    }
                    i++;
                }
                cout<<"Produccion: "<<prod<<endl;
                cout<<"Módulo: "<<mod<<endl;
            }
            else
                cout<<linea<<endl;
        }
        MiArchivo.close();
    }
    else
        printf ("No se pudo abrir el archivo.");
    cin.get();
    return 0;
}
 

[sam90]

Dos cosas: paracoccidio que corregiste???

Segunda tu problema recide en que no le agregas el fin de cadena a prod y mod. Despues a demas tenias otro problemitas. Asi quedaria el codigo:

Código C++:

#include<iostream>
#include<fstream>
#include<stdio.h>
#include<string.h>
#include<stdio.h>
#include<stdlib.h>
 
using namespace std; 
 
int main()
{
  int i,p,m;
  float modulo;
  bool espacio,finlinea=false;
  char c,linea[30],prod[15],mod[15];
  ifstream MiArchivo;
 
  //ABRIMOS EL FICHERO
  MiArchivo.open("datos.txt");
 
  if (!MiArchivo.fail())
  {
    while (!MiArchivo.eof() )
    {
      strcpy(linea,"");
      strcpy(prod,"");
      strcpy(mod,"");
 
      //LEEMOS LA LINEA
      MiArchivo.getline(linea,sizeof(linea));
 
      i=17;
      p=0;
      m=0;
      finlinea= espacio  = false;
 
      //ESTE IF ESTÁ PARA SALTAR LINEAS QUE NO QUEREMOS PARA NADA
      if(strlen(linea)>17)
      {
       while(i<30 && !finlinea)
       {
            c=linea[i];
            //cout << c << endl;
        if(c==' ')
                         espacio=true;
            else  if(c=='\n' || c == '\0')
                finlinea=true;
             else
                {
                         //SI TODAVIA NO HEMOS ENCONTRADO EL ESPACIO,ES LA PRODUCCION
                         if(!espacio)
                         {
                                prod[p]=linea[i];
                                //cout << p << " " <<prod[p] << endl;
                p++;
                          }
 
                          else
                          {
                                mod[m]=linea[i];
                                //cout << m  << " " << mod[m] << endl;
                m++;
                          }
                 }
         i++;
         }
     prod[p] = 0;
         mod[m] = 0;
         cout<<"Produccion: "<<prod<<endl;
         cout<<"Módulo: "<<mod<<endl;
      }
      else
        cout<<linea<<endl;
    }
    MiArchivo.close();
  }
  else
   printf ("No se pudo abrir el archivo.");
 
}
 

Por ejemplo, tenias la barra en el caractar nuevalinea al revez. Es esta \.
la variable espacio no estaba seteada a false para comenzar.
Buscabas el fin de linea pero nunca el '\0'