QUản lí học sinh với Linked List

[goroshi]

Mình đọc sách đến phần này, mình thử làm 1 bài tập quản lí sinh viên dùng linked list nhưng gặp vấn đề

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
typedef struct pp
{
    char ht[40];
    char diem[10];
    struct pp*tiep;    
} person;
person *pdau=NULL,*p,*last,*tam;
void khoitao();
void nhap();
main()
{
    int i;
    do
    {
        printf("Hay lua chon\n");
        printf("1.Nhap them nguoi\n");
        printf("2.Xoa 1 nguoi khoi danh sach\n");
        printf("3.Tim kiem 1 nguoi trong danh sach\n");
        printf("4.Liet ke danh sach\n");
        printf("5.Thoat ra\n");
        printf("Ban chon:\n");
        scanf("&d",&i);
        if(i==1) nhap();
        if(i==2) xoa();
        if(i==3) timkiem();
        if(i==4) lietke();
        if(i==4) thoat();
        else printf("Moi ban nhap lai\n");    
    }while(i!=5);
}
//Ở đây mình chỉ đưa hàm nhập dữ liệu lên thôi, các hàm khác cũng gặp tình trạng tương tự
void khoitao()
{
    if(pdau==NULL)
    {
        pdau=malloc(sizeof(person));
        printf("\nNhap ho va ten\n");
        scanf("%s",pdau->ht);
        printf("Nhap diem\n");
        scanf("%s",pdau->diem);
        pdau->tiep;
        }
    return;
}
void nhap()
{
    p=pdau;
    if(p==NULL) khoitao();
    else
    {
        last=malloc(sizeof(person));
        if(last==NULL) printf("\nKhong du bo nho\n");
        while(p->tiep!=NULL) p=p->tiep;
        printf("Nhap ho va ten\n");
        scanf("%s",&p->ht);
        printf("Nhap diem\n");
        scanf("%s",&p->diem);
        last->tiep=NULL;
        p->tiep=last;
    }
    return;
} 

Vấn đề ở đây là khi mình ấn số 1 để nhập danh sách thì thấy các dòng chữ chạy liên tục như trong phim ý, mà mình thấy hàm của mình có vòng lặp nào đâu :((
Tương tự với các hàm khác, mình ấn số nào thì nó cũng chạy liên tục như thế
(ở đây mình chỉ post hàm nhập thôi, các hàm khác chắc cũng bị lỗi giống hàm này). Nhờ mọi người giúp

[6220119]

        printf("5.Thoat ra\n");
        printf("Ban chon:\n");
        scanf("%d",&i);

của bạn là scanf("&d",&i);

[rox_rook]

- Mấy hàm kia đâu ?
- Tránh dùng global variable !

[trunkthai]

scanf("%s",&p->ht);

scanf("%s",&p->diem);

Cho em hỏi cái &p-> là sao ???
&(*p) sao trỏ được

[rox_rook]

Đọc cú pháp C/C++ từ phải qua trái :
p->ht = 1 giá trị nào đó.
lấy địa chỉ của ( p -> ht ) tức là &( p->ht )
Nếu tui viết tui cũng sẽ viết như cú pháp trên, khỏi phải cần nhớ thứ tự ưu tiên và cũng dễ dàng hơn cho người đọc. Nhưng ở trên mãng pure char có thể nhận hoặc là địa chỉ hoặc là cái tên thôi cũng được, vì tên mãng cũng là con trỏ trỏ vào địa chỉ đó so either way.

#include <cstdio> struct Sthing {  char data[ 80 ];};  int main() {  Sthing  my_struct1;  Sthing  my_struct2;   Sthing* p1 = &my_struct1;  Sthing* p2 = &my_struct2;   scanf( "%s", &( p1->data ) );  printf( "%s \n", p1->data );    scanf( "%s", p2->data );  printf( "%s \n", p2->data );   return 0;}

Thử là biết liền !

[goroshi]

        printf("5.Thoat ra\n");
        printf("Ban chon:\n");
        scanf("%d",&i);

của bạn là scanf("&d",&i);

Đúng rồi, vấn đề là ở chỗ này. do mình không để ý. Thanks, có gì mình lại hỏi tiếp
@rox:cho hỏi global variable là gì thế? Tại sao không nên dùng nó?

[rox_rook]

Global variable là biến toàn cục, không nên xài biến toàn cục là vì :
- Giả nếu cả chục hàm xài biến đó, mà vô tình 1 hàm sử dụng phép gán mà ta không mong đợi thì coi như xong !!! Debug là bằng niềm tin.
- Thậm chí ở các trường College, hay Uni ở bên tui nếu xài global mà không dùng const sẽ bị 0 ngay khỏi chấm bài nữa, vì global hầu như bị banned hoàn toàn rồi.
- Còn thắc mắc gì thì post lên, tui dạo này cũng biết C 1 chút rùi !

[trunkthai]

Cám ơn anh R_R(<-cái này chắc đọc là anh 2 rờ )