fscanf() doesn't want to read values after the first two are read

Question

Basically, I'm reading a bunch of values from a text file, which has them in the following layout:

4 1 1 2 3 4

But the following block of code doesn't want to read the double type value following the first two int type values:

int matrix_read(struct matrep *mat, const char *filename)
{
    FILE *fptr;
    unsigned m, n;
    double *ptr = NULL;
    int i, j;
    double x;
    if ((fptr = fopen(filename, "r")) == NULL)
    {
        fprintf(stderr, "Cannot Open File %s\n", filename);
        return -1;
    }
    if(fscanf(fptr, "%u", &m) != 1)
    {
        fprintf(stderr, "Failed to read number of rows\n");
        return -1;
    }
    if(fscanf(fptr, "%u", &n) != 1)
    {
         fprintf(stderr, "Failed to read number of columns\n");
        return -1;
    }

    mat->matrix = (double *)malloc(sizeof(double) * m * n);
    if (mat->matrix == 0)
    {
        fprintf(stderr, "Failed to allocate %d*%d matrix\n", m, n);
        return -1;
    }
    ptr = mat->matrix;

    for (i = 0; i < m; i++)
    {
        for (j = 0; j < n; j++)
        {
            double x;
            if (fscanf(fptr, "%f", &x) != 1)
            {
                fprintf(stderr, "Failed to read element matrix[%d,%d]\n", i, j);
                free(mat->matrix);
                mat->matrix = 0;
                mat->cols = 0;
                mat->rows = 0;
                return -1;
            }
            *ptr++ = x;//Here it reads nothing, it just gives me: -9.2559604281615349e+061
        }
    }
    fclose(fptr);
    mat->cols = m;
    mat->rows = n;

    return 0;  // Success   
}

What am I doing wrong?

On an unrelated note, you declare `x` twice. – Some programmer dude Nov 12 '12 at 11:29 — Some programmer dude, Nov 12 '12 at 11:29

score 5 · Accepted Answer · answered Nov 12 '12 at 11:28

5

fscanf(fptr, "%f", &x)

For scanning doubles, you need the %lf format. %f scans a float. Using the wrong format invokes undefined behaviour, what probably happens is that the scanned value is converted to a float and then stored in the first four bytes of the pointed-to double.

answered Nov 12 '12 at 11:28

Daniel Fischer

181,706
17
308
431

Wrong format specifiers with scanf and printf - http://stackoverflow.com/questions/12830052/wrong-format-specifiers-in-scanf-or-printf – CCoder Nov 12 '12 at 11:34

fscanf() doesn't want to read values after the first two are read

1 Answers1