Post cả hết cả bài lên luôn đi cậu!
Mình có một DSLK được khai báo như sau:
DSLK này được nhập theo đúng thứ tự tăng dần. Giờ mình muốn chèn một phần tử vào đúng vị trí của nó, mình dùng code sau:Code:struct number { int value; number* next; }; struct number *head=NULL,*t,*p,*q;
Mình cho dịch chương trình thì ko báo lỗi nhưng khi chạy thì cứ đến chỗ "Nhap gia tri can chen vao: " thì đơ luôn. Bạn nào chỉ cho mình sai ở chỗ nào mới. Cảm ơn.Code://Chen mot phan tu t=head; cout<<"Nhap gia tri can chen vao: "; cin>>x; while (t->next!=NULL) { q=t->next; if((t->value<=x)&&(q->value>=x)) { p=new number; p->value=x; p->next=q; t->next=p; } t=t->next; }
Post cả hết cả bài lên luôn đi cậu!
Mong các bạn giúp đỡCode:#include "iostream.h" struct number { int value; number* next; }; void main() { int n,x,i; struct number *head=NULL,*t,*p,*q; // Nhap cout<<"Nhap n= "; cin>>n; for (i=0;i<n;i++) { cout<<"Phan tu thu "<<i+1<<" la "; cin>>x; t=new number; t->value=x; t->next=head; head=t; } // Xuat ra man hinh t=head; cout<<"Cac phan tu vua nhap la "; while (t!=NULL) { x=t->value; cout<<" "<<x; t=t->next; } //Chen mot phan tu t=head; cout<<"Nhap gia tri can chen vao: "; cin>>x; while (t->next!=NULL) { q=t->next; if((t->value<=x)&&(q->value>=x)) { p=new number; p->value=x; p->next=q; t->next=p; } t=t->next; } }
Thành thật sorry cậu, tui làm nãy h gần cả tiếng mới ra, lại bận đi qua nhà bà dì nên không trả lời liền cho cậu được.
- Đúng là tui quá chủ quan, do tui viết link list dạng class quen rùi mà tui quên mất khai báo kiểu đó là thì cái link là global scope, debug cả tiếng đồng hồ T_T. Gà không chịu được.
- Góp ý cho cậu :
+ viết con trỏ không bao h đặt tên biến 1 chữ cái (q,t,v,a,...), vì debug con trỏ phải comment và đặt tên kiểu này thì có nc debug bằng niềm tin cậu à, chịu khó đặt tên cho nó dài 1 chút, không tốn hơn bao nhiêu thời gian đâu mà còn tốt hơn cho việc debug.
Tui đã viết lại + 1 số comment cho cậu thông suốt đã, còn code của cậu tui sẽ tìm bug rùi trả lời cho cậu sau vì nói thật tui không quen đọc code kiểu struct C cho lắm.
Đây chắc là code tồi nhất mà tui từng viết T_T, quen viết kiểu class mà coi thường procedural cũng thật là T_T, tui sẽ nghĩ cách tốt hơn.C++ Code:
#include <iostream> #include <cstdlib> void insert_sort(const int& x); void traverse(); void add(const int& x); struct node{ int data; node *next; }; node *head = NULL; //Hàm này là hàm cậu sai void insert_sort(const int& x){ //1 node giữ thằng đầu, và thằng kế tiếp nó node *prev, *foll; //nếu list rỗgn thì ok -> báo rỗng if(head == NULL){ } else{ //thằng prev ( previous ) giữ cái head //foll ( following giữ thằng kế tiếp nó ) prev = head; foll = prev->next; //Khi vòng lặp while này kết thúc thì thằng //prev sẽ trỏ ngay vị trí cần chèn vào while(prev->data > x || foll->data < x){ prev = prev->next; foll = prev->next; } //tạo 1 node mới tên new_node node* new_node = new node; //set dữ liệu cho nó new_node->data = x; //liên kết 2 link tạo thành 1 link list prev->next = new_node; new_node->next = foll; } } //Hàm in ra có lẽ cậu cũng dễ dàng hiểu dc void traverse(){ if(head == NULL){ return ; } node *_iter; _iter = new node; _iter = head; while(_iter->next != NULL){ _iter = _iter->next; } } //Chỗ này tui pass const ref giống như pass bằng trị thôi, nhưng tui //quen pass const ref vì nó tạo cho tui thói quen không tạo //ra 1 copy của biến khi không cần thiết, tuy đây chỉ là 1 biến int //nhưng nếu là 1 struct hay 1 class thì sẽ là quá lãng phí void add(const int& x){ node *iss, *_iter; iss = new(node); iss->data = x; iss->next = NULL; if(head == NULL){ head = iss; return; } else{ _iter = head; while(_iter->next != NULL) _iter = _iter->next; _iter->next = iss; } } //Giải phóng vùng nhớ void release(){ while(head->next != NULL){ head = head->next; delete head; } head = 0; } int main(){ add(1); add(2); add(4); add(6); add(7); add(8); int value = 5; insert_sort(value); traverse(); return 0; }
Nhưng nói chung làm kiểu này rất không an toàn về nhiều mặt :
- Tạo 1 node global -> very poor design.
- Node chỉ nên là 1 type, và 1 class Link sẽ gồm các node sẽ rõ hơn vấn đề
- cout và cin là của C++, nếu viết C thì nên dùng printf và scanf.
- Nếu viết C++ nên dùng class.
rox_rook có thể giải thích cho mình hiểu về đoạn code :
Với lại đoạn này nữaCode:while(prev->data > x || foll->data < x){ prev = prev->next; foll = prev->next; }
Thầy giạy đoạn đó nhanh quá làm em kô hiểu em mới năm 1 àh thông cảm nhaCode:head = head->next;
Còn thêm cái hàm int main() lần đầu tiên gặp nữa. Hic sao em ngu wá vậy nèCode:int main(){ add(1); add(2); add(4); add(6); add(7); add(8); int value = 5; insert_sort(value); traverse(); return 0; }
Đã được chỉnh sửa lần cuối bởi atuan1989 : 27-03-2008 lúc 07:54 AM.
Ví dụ 1 danh sách gồm :
[A] [B] [C] [D]
Và prev đang trỏ tới [B]
-> prev = prev->next nghĩa là nó trỏ tới thằng kế [B] là [C]
prev là previous ( thằng trước thằng hiện tại )
foll là following ( thằng sau thằng hiện tại )
1 thằng giữa cái sau lưng và 1 thằng giữ cái trước, sthing look like
[prev]->[hiện tại]->[foll]
Tui muốn insert vào vị trị mà đảm bảo nó vừa lớn hơn prev và nhỏ hơn foll.
Vậy thì...suy nghĩ tí đi
Chắc lục lại những cái cơ bản rồi coi lại quá.Điệu này chắc rớt môn cấu trúc dữ liệu rùi hic.
Đã được chỉnh sửa lần cuối bởi atuan1989 : 27-03-2008 lúc 07:48 PM.