Не могу понять в чем проблема.

[Демик]

Есть код программы, программа считывает данные из файла data.dat и заносит их в массив. В файле data.dat по заданию находятся длины сторон треугольников, нужно считать их из файла и найти периметры этих треугольников, далее нужно найти максимальный периметр и вывести его на экран. У меня массив состоит из 10-ти строк 3-х столбцов, то есть 10 треугольников, далее циклом находятся их периметры и заносятся в одномерный массив. После этого я нахожу максимальный массив "методом коробочки" и вывожу его на экран. Что удивительно но программа компилируется без ошибок и предупреждений, но результат выводится неверный. я думаю ошибка в считывании файла, так как если в файле data.dat отсутствуют какие либо данные программа все равно выводит неизвестно откуда взявшийся результат. Вот код программы:

#include<stdio.h>
#include<conio.h>
#include<iostream>
  int main()
    {
       int mass[10][3];
       int P[10];
       int PP;
          FILE*file_1=fopen("data.dat","r");
              for (int i=0;i<10;i++)
                for(int j=0;j<3;j++)
                     {
                        fseek(file_1,j,SEEK_SET);
                        fread(&mass[i][j],sizeof(int),1,file_1);
                        for (int g=0;g<10;g++)
                           {
                              P[g]=mass[i][0]+mass[i][1]+mass[i][2];
                              std::cout<<"P["<<g<<"]:"<<P[g]<<std::endl;
                              PP=P[0];
                              if (P[g]>PP)
                              PP=P[g];
                           }   
                     }
               std::cout<<"Pmax:"<<PP<<std::endl;
             {
                getch();
             }   
      return 0; 
    }

[NiCola999]

я бы сделал так:

#include <iostream>
#include <fstream>
using namespace std;

class Triangle {
      private: 
              double a, b, c, _P;
      public:
              Triangle(){ a = b = c = _P = 0 ; }
              Triangle(double _a, double _b, double _c){
                     a = _a; b = _b; c = _c;
                     _P = a + b + c;
              }
              double P(){ return _P; }
};

int main(void){
    const int MAX_SIZE = 100;
    int i, j;
    double a,b,c, maxP, cP;
    Triangle triangles[MAX_SIZE];    
    
    ifstream input;
    input.open("data.dat");
    
    
    for(i=0; !input.eof(); i++){
            input >> a >> b >> c;
            triangles[i] = Triangle(a, b, c);
    }
    
    input.close();    
    
    maxP = triangles[0].P();
    
    for(j=0; j < i; j++){
             cP = triangles[i].P();    
             if(maxP < cP ) maxP = cP;
    }
    cout << "Max perimeter: "<<maxP<<"" << endl;    
    cin.get();
    return 0;
}

только почему-то 2 раза последнюю строку читает из файла... я не понял

[Демик]

Я как понял вы создали новый класс? Объявили переменные класса и сказали что они равны нулю. так? а вот что означает это:

  Triangle(double _a, double _b, double _c){
                     a = _a; b = _b; c = _c;
                     _P = a + b + c;
              }
              double P(){ return _P; }
} ;

я не понял.

[NiCola999]

              Triangle(){ a = b = c = _P = 0 ; }              
              Triangle(double _a, double _b, double _c){
                     a = _a; b = _b; c = _c;
                     _P = a + b + c;
              }
              double P(){ return _P; }

2 конструктора

[NiCola999]

вообще говоря можно проще:

#include <iostream>
#include <fstream>
using namespace std;

int main(void){    
    double a,b,c, maxa, maxb, maxc, maxP = 0, P = 0;    
    ifstream input;
    input.open("data.dat");    
    input >> a >> b >> c;                    
    for(; !input.eof(); ){
            input >> a >> b >> c;            
            P = a + b + c;
            if(maxP < P) {
                    maxP = P;
                    maxa = a; maxb = b; maxc = c;
            }
    }        
    input.close();            
    cout << "Max perimeter: " << maxP << endl
         << "a = " << maxa << endl        
         << "b = " << maxb << endl
         << "c = " << maxc << endl;
    cin.get();
    return 0;
}

[Демик]

Согласен, так проще, но не могли бы вы объяснит как пользоваться fstream'ом?

[NiCola999]

http://server.179.ru/tasks/cpp/files.html

[Демик]

спасибо за помощь