[C++] Lỗi Access violation....

[gialoui]

struct Point
{
    char name[5];
    float x, y;
};

struct MyArray
{
    Point *a;
    int num;
};

void readStationsFromFile(char *fileName, MyArray &a, MyArray &b)
{
    FILE *fp;
    int m, n;
    m = 0;
    n = 0;
    countColRow(fileName, m, n);
    a.num = n;

    fopen_s(&fp, fileName, "rt");
    if (!fp)
    {
        printf("Cannot open file...\n");
        return;
    }

    if (a.num <= 0)
    {
        printf("Your Stations.txt is empty...\n");
        return;
    }
    a.a = new Point[a.num];

    if (a.a == NULL)
    {
        return;
    }

    int i = 0;

    while (!feof(fp))
    {
        fscanf_s(fp, "%s %f %f\n", &a.a[i].name, &a.a[i].x, &a.a[i].y);  // Dòng bị lỗi
        i++;
    }

    fclose(fp); 
} 

Debug thì thấy bị lỗi ngay chỗ fscanf_s cuối, nó báo lỗi Access violation writing location.... Giờ em vẫn chưa biết là nên fix chỗ nào, phiền anh chị nào chỉ giúp em. Thanks.

Em nghĩ là bị sai khi khai báo và cấp phát cho chuỗi.

[tara95]

Bạn có thể ghi thêm đề bài và up đầy đủ project của bạn lên được không?

[11520214]

Bạn cấp phát bộ nhớ cho con trỏ chưa?

[gialoui]

Bạn có thể ghi thêm đề bài và up đầy đủ project của bạn lên được không?

Đề này khá dài, mình chỉ post 1 đoạn phần mình đang làm trước nha:
Bài tập của mình yêu cầu đọc từ file text có nội dung theo mẫu sau (số dòng không biết trước), cấu trúc của mỗi dòng gồm có tên ga chứa tối đa 5 kí tự (A->Z, 0->9), và 2 tọa độ số thực. Đọc vào mảng và phải xài con trỏ để cấp phát:
A34 -1.5 2.4
B 2 4.5
C21 4.2 4

[gialoui]

Bạn cấp phát bộ nhớ cho con trỏ chưa?

Mình có cấp phát bằng new trong đoạn code rồi mà

[gialoui]

Ai giúp với

[10520100]

Chúng tôi vô cùng quan ngại trước việc bạn không chịu up nguyên project. Rất khó để debug bạn à.

[gialoui]

Chúng tôi vô cùng quan ngại trước việc bạn không chịu up nguyên project. Rất khó để debug bạn à.

Project của mình:Assignment03.rar

Gồm 1 file .h và 2 file .cpp

[10520100]

Rồi, bạn sai ở đây nè.

struct SArray    // Mảng lưu trữ tên ga
{
    char[5] array;
    int num;
}; 

Đây là một struct bạn khai báo đề lưu trữ tất cả tên ga. Trong ví dụ của bạn sẽ là 5 tên ga : "A45", "B", "C", "D", "E2" tương ứng với 5 chuỗi !
Tuy nhiên bạn lại chỉ dùng 1 mảng, có 5 ký tự để lưu trữ.
Khi bạn gọi hàm fscanf_s, bạn lại muốn gán chuỗi "A45" cho kí tự SArray.a[1], chuỗi "B" cho kí tự SArray.a[2], .... vì vậy bạn vi phạm vùng nhớ.

Gới ý cách sửa, bạn sửa mảng các kí tự thành mảng các chuỗi hoặc đại loại như thế

struct SArray    // Mảng lưu trữ tên ga
{
    string* array;
    int num;
}; 

Hi vọng giúp được bạn.

[gialoui]

Rồi, bạn sai ở đây nè.

struct SArray    // Mảng lưu trữ tên ga
{
    char[5] array;
    int num;
}; 

Đây là một struct bạn khai báo đề lưu trữ tất cả tên ga. Trong ví dụ của bạn sẽ là 5 tên ga : "A45", "B", "C", "D", "E2" tương ứng với 5 chuỗi !
Tuy nhiên bạn lại chỉ dùng 1 mảng, có 5 ký tự để lưu trữ.
Khi bạn gọi hàm fscanf_s, bạn lại muốn gán chuỗi "A45" cho kí tự SArray.a[1], chuỗi "B" cho kí tự SArray.a[2], .... vì vậy bạn vi phạm vùng nhớ.

Gới ý cách sửa, bạn sửa mảng các kí tự thành mảng các chuỗi hoặc đại loại như thế

struct SArray    // Mảng lưu trữ tên ga
{
    string* array;
    int num;
}; 

Hi vọng giúp được bạn.

Sao mình sửa lại thành string rồi mà nó vẫn bị lỗi đó, bạn thử sửa trên project của mình rồi chạy xem được ko? Còn cái chỗ fscanf_s vậy có cần chỉnh sửa gì ko bạn?

[gialoui]

Rồi, bạn sai ở đây nè.

struct SArray    // Mảng lưu trữ tên ga
{
    char[5] array;
    int num;
}; 

Đây là một struct bạn khai báo đề lưu trữ tất cả tên ga. Trong ví dụ của bạn sẽ là 5 tên ga : "A45", "B", "C", "D", "E2" tương ứng với 5 chuỗi !
Tuy nhiên bạn lại chỉ dùng 1 mảng, có 5 ký tự để lưu trữ.
Khi bạn gọi hàm fscanf_s, bạn lại muốn gán chuỗi "A45" cho kí tự SArray.a[1], chuỗi "B" cho kí tự SArray.a[2], .... vì vậy bạn vi phạm vùng nhớ.

Gới ý cách sửa, bạn sửa mảng các kí tự thành mảng các chuỗi hoặc đại loại như thế

struct SArray    // Mảng lưu trữ tên ga
{
    string* array;
    int num;
}; 

Hi vọng giúp được bạn.

Bạn ơi, mình sửa thành string rồi mà sao vẫn bị lỗi Xem lại giúp mình với, mình đang cần gấp để hoàn thành bài tập. Thanks

[sinhvien.uit]

Sao mình sửa lại thành string rồi mà nó vẫn bị lỗi đó, bạn thử sửa trên project của mình rồi chạy xem được ko? Còn cái chỗ fscanf_s vậy có cần chỉnh sửa gì ko bạn?

string là một class của C++ thì phải nhập bằng iostream chứ không có scanf nhá

[gialoui]

string là một class của C++ thì phải nhập bằng iostream chứ không có scanf nhá

Đọc file bằng iostream sử dụng như thế nào vậy bạn?

[09520109]

struct Point
{
    char name[5];
    float x, y;
};

struct MyArray
{
    Point *a;
    int num;
};

void readStationsFromFile(char *fileName, MyArray &a, MyArray &b)
{
    FILE *fp;
    int m, n;
    m = 0;
    n = 0;
    countColRow(fileName, m, n);
    a.num = n;

    fopen_s(&fp, fileName, "rt");
    if (!fp)
    {
        printf("Cannot open file...\n");
        return;
    }

    if (a.num <= 0)
    {
        printf("Your Stations.txt is empty...\n");
        return;
    }
    a.a = new Point[a.num];

    if (a.a == NULL)
    {
        return;
    }

    int i = 0;

    while (!feof(fp))
    {
        fscanf_s(fp, "%s %f %f\n", &a.a[i].name, &a.a[i].x, &a.a[i].y);  // Dòng bị lỗi
        i++;
    }

    fclose(fp); 
} 

Debug thì thấy bị lỗi ngay chỗ fscanf_s cuối, nó báo lỗi Access violation writing location.... Giờ em vẫn chưa biết là nên fix chỗ nào, phiền anh chị nào chỉ giúp em. Thanks.

Em nghĩ là bị sai khi khai báo và cấp phát cho chuỗi.

bạn thay dòng lỗi đó thành thế này xem sao:

 fscanf_s(fp, "%s %f %f\n", &(a.a+i)->name, &(a.a+i)->x, &(a.a+i)->y) 

[gialoui]

bạn thay dòng lỗi đó thành thế này xem sao:

 fscanf_s(fp, "%s %f %f\n", &(a.a+i)->name, &(a.a+i)->x, &(a.a+i)->y) 

Mình sửa lại vậy thì nó xuất hiện thêm cái lỗi ở chỗ: &(a.a+i)->name ngay dấu ngoặc đầu tiên: Expression must have pointer-to-class type. Bạn có thể download cái project ở page 1 về xem thử giúp mình với Mình cũng nhờ sửa lỗi này nhiều lắm mà vô phương.