Načtení čísel z textového souboru do pole – C / C++ – Fórum – Programujte.com

V C++ jsem zatím nováček a potřeboval bych přečíst čísla (celá i desetinná) z textového souboru (pevný počet sloupců, proměnlivý počet řádků), uložit je do dvourozměrného pole (pokud není lepší možnost) a pak vypsat řádky, které splňují dané podmínky. Ztroskotal jsem ale už na načtení čísel do pole, když ho pak vypíšu, tak je prázdné (+ vypisuje se mi jen jeden řádek). Poradili byste mi prosím, kde mám chybu:

	int z, pocet=1,i,j;
	float x;
	FILE *soubor;
	soubor=fopen("soubor.txt", "r+");
	while ((z=getc(soubor)) != EOF)
		if (z=='\n') pocet++;
	cout << pocet;
	
	float pole[pocet][24];	
	for(i=0;i<=pocet;i++){
	for(j=0;j<=24;j++){
	pole[i][j]=0;
    }}
	for(i=0;i<=pocet;i++){
	for(j=0;j<=24;j++){
	fscanf(soubor, "%f",&x);
	pole[i][j]=x;
    }}
    	for(i=0;i<=pocet;i++){	
	for(j=0;j<24;j++){
	x=pole[i][j];
	fprintf(soubor,"%f \t",pole[i][j]);
    }}

#1 Metodej
Nejprve zkus místo všech "<=" použít "<". Tak jak to máš, lezeš mimo hranice pole.

Pokud nějaké pole plníš nějakými daty, nemusíš ho nulovat. Není to však chyba.

Nebude to tim, ze nejprve spocitas radky, ale uz se nevratis na zacatek souboru?

+ pole[pocet][24]  je blbej napad.

Taky bys mohl pouzivat ifstream, kdyz ses v C++.

Je jiste, ze mas vzdy 24 cisel na radku? Jinak se ti to mirne receno rozhodi.

Děkuji, s ifstream už to funguje.

Proč je pole[pocet][24] blbej nápad? Pomůže definovat předtím jinou konstantu o velikosti počtu řádků? Nebo to jde vyřešit i nějak jinak?

no jde to vyresit i jinak (pouzivam nektere veci ze -std=c++11 ale kdyz by se jich zbavilo, tak to pujde i bez toho), kazdopadne to zvladne libovolne mnozstvi cisel na radku a libovolny pocet radku (co se vejde do ram):

#include <iostream>
#include <fstream>
#include <sstream>
#include <iterator>
#include <string>
#include <vector>
#include <algorithm>

int main() {
  std::vector<std::vector<float>>  pole;

  std::ifstream soubor{"soubor.txt"}; // otevreme soubor
  while (soubor) { // soubor se da testovat jako boolean - vraci true dokud neni eof ci jina chyba
    std::string radek; // pripravime si retezec na ulozeni radku
    std::getline(soubor, radek); // nacteme cely radek

    std::istringstream vstup{radek}; // z radku si udelame vstupni stream (jde pouzivat >> float_val)

    std::vector<float> ln; // pripravime si pomocne pole
    // tady je to uz trosku zajimavejsi:
    std::copy( // copy ocekava iterator na zacatek a konec sekvence, ktera ma byt zkopirovana + zacatek sekvence kam ma byt zkopirovana (musi na to byt misto)
      std::istream_iterator<float>{vstup}, // specialni iterator pro nacitani ze streamu - dereferenci vrati nactenou hodnotu
      std::istream_iterator<float>{},      // a porovnava se s timto. Budou stejne, pokud se nepodarilo nacist dalsi polozku.
      std::back_insert_iterator<std::vector<float>>{ln} // nicmene existence mista lze obejit pomoci dalsiho iteratoru, ktery vlastne znamena:  ln.push_back(*iter)
      );
    if (! ln.empty()) {
      pole.push_back(move(ln)); // pokud se aspon neco nacetlo, muzeme to vlozit do vystupniho pole (move presune vnitrek vectoru bez kopirovani - setri cas) 
    }
  }

  // a letmo vypiseme
  for (auto & ln : pole) {
    for (float x : ln) {
      std::cout << x << ":";
    }
    std::cout << "\n";
  }
}

Jo takhle by to vypadalo bez noveho standardu:

#include <iostream>
#include <fstream>
#include <sstream>
#include <iterator>
#include <string>
#include <vector>
#include <algorithm>

int main() {
  std::vector<std::vector<float> >  pole;

  std::ifstream soubor("soubor.txt"); // otevreme soubor
  while (soubor) { // soubor se da testovat jako boolean - vraci true dokud neni eof ci jina chyba
    std::string radek; // pripravime si retezec na ulozeni radku
    std::getline(soubor, radek); // nacteme cely radek

    std::istringstream vstup(radek); // z radku si udelame vstupni stream (jde pouzivat >> float_val)

    std::vector<float> ln; // pripravime si pomocne pole
    // tady je to uz trosku zajimavejsi:
    std::copy( // copy ocekava iterator na zacatek a konec sekvence, ktera ma byt zkopirovana + zacatek sekvence kam ma byt zkopirovana (musi na to byt misto)
      std::istream_iterator<float>(vstup), // specialni iterator pro nacitani ze streamu - dereferenci vrati nactenou hodnotu
      std::istream_iterator<float>(),      // a porovnava se s timto. Budou stejne, pokud se nepodarilo nacist dalsi polozku.
      std::back_insert_iterator<std::vector<float> >(ln) // nicmene existence mista lze obejit pomoci dalsiho iteratoru, ktery vlastne znamena:  ln.push_back(*iter)
      );
    if (! ln.empty()) {
      pole.push_back(ln); // pokud se aspon neco nacetlo, muzeme to vlozit do vystupniho pole 
    }
  }

  // a letmo vypiseme
  std::vector<std::vector<float> >::iterator  it = pole.begin();
  for (;pole.end() != it; ++it) {
    std::copy(it->begin(), it->end(), std::ostream_iterator<float>(std::cout,":"));
    std::cout << "\n";
  }
}