Thread: Dynamic Two-Dimensional Arrays

LloydUzari---If you have a debugger, step through it line by line. If not start throwing in lines like cout << "Here" << endl; to see how far your code gets before it bottoms out. Then you can start putting in lines to display a given variable to make sure it is being read/changed appropriately. This manual technique for debugging is a bit tedious, but it usually works, especially if you aren't comfortable with a debugger.

The code for creating a 2D array dynamically seems okay in your post. Sometimes the problem is as simple as making sure that you have the dimensions correctly. That is, are you sure data_length is the first size read in and data_width the second?

Right now, that wouldnt matter cause their both the same currently(10)

I have debugger in compiler, but the annoying thing itself is bugged and dont work I did the cout way of debuggin, and saw it freezes while declaring the array(I see the 'TEST' when I put it before it, but not after.)

I tried both ways of declaring the array, too(the way I have on code I posted and [length*width])

It also itsnt scanning the file right and is returning the values of length and height as 75x2406 instead of 10x10. Neither the file or the read mode is binary, got any clue why this is happening.

Note: by some fluke chance, it worked ONCE when I made the file read only, but now even read only doesnt work. When it did work, however, it's length and width were right, but its return from file was wrong... It wrote a whole bunch of random numbers instead of 20x20 grid of 0's(each slot had 00)

Gotta go right now, Ill try get() instead of >> later next to see if it works

Are you sure the file is opening at all? I don't see any error checking. Run this

Code:

std::ifstream s_size(dat_filename);
if(s_size) {
    int length, width;
    if(s_size >> width >> length) {
        std::cout << "file is " << width << " wide by " << length << " long";
        int count=0,n=0; 
        while(s_size>>n && n==0) ++count;
        std::cout << " I read " << count << " zeros and expected "<< (width*length) << std::endl;
        if(s_size.eof()) std::cout << "I read the whole file" << std::endl;
        else if(s_size) std::cout << "I stopped because I read a " << n << std::endl;
        else if(s_size.fail()) {
            s_size.clear();
            char ch;
            if(s_size.get(ch)) std::cout << "I choked on the character '" << ch << "'" << std::endl;
            else std::cout << "I cannot even read a single character now" << std::endl;
        } else {
            std::cout << "The stream has somehow become courrupt, but not fail()" << std::endl;
        }
    } else if(s_size.fail()) {
        s_size.clear();
        char ch;
        if(s_size.get(ch)) std::cout << "I choked on the character '" << ch << "'" << std::endl;
        else std::cout << "I cannot even read a single character now" << std::endl;
    } else {
        std::cout << "The stream has somehow become courrupt, but not fail()" << std::endl;
    }
} else {
    std::cout << "I could not open '" << dat_filename << "'" <<std::endl;
}

Accually, the file checking is outside of the functions. I did have file checking in the functions before, but they were slowing the program down like crazy, so basically how it works now is before it starts loading, it uses the checkfile func I made

Code:

int testclass::checkfile() {
  std::ofstream c_file(dat_filename);
  if (c_file) {
    c_file.close();
    return 1;
  }
  c_file.close();
  return 0;
}//testclass checkfile func

std::ofstream c_file(dat_filename); Did you really want to truncate the file? You have a zero length file living somewhere, but it aint where you think.

Thanks, that fixed the rest. I added seekg to get past that terminating thing