Danh sách liên kết ???

[new_bie]

Mình có một DSLK được khai báo như sau:

struct number
{
	int value;
	number* next;
};

struct number *head=NULL,*t,*p,*q;

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:

//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;
	}

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.

[rox_rook]

Post cả hết cả bài lên luôn đi cậu !

[new_bie]

#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;
	}
		
}

Mong các bạn giúp đỡ

[rox_rook]

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.

#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 saivoid 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){    std::cout << "Empty list detected\n";  }   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 dcvoid traverse(){  if(head == NULL){    std::cout << "Empty []\n";    return ;  }  node *_iter;      _iter = new node;  _iter = head;  while(_iter->next != NULL){    std::cout << _iter->data << '\n';    _iter = _iter->next;  }  std::cout << _iter->data << '\n';} //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;}

Đâ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.
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.

[atuan1989]

rox_rook có thể giải thích cho mình hiểu về đoạn code :

 while(prev->data > x || foll->data < x){
      prev = prev->next;
      foll = prev->next;
    }

Với lại đoạn này nữa

head = head->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 nha

int main(){
  
  add(1);
  add(2);
  add(4);
  add(6);
  add(7);
  add(8);

  int value = 5;

  insert_sort(value);
  traverse();
  return 0;
}

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è

[rox_rook]

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

[atuan1989]

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.