Một bài tập về con trỏ và xâu kí tự

[vuong_quoc_viet_PTCNN]

Đây là nội dung của bài tập:
Hãy định nghĩa một hàm để vào một danh sách tên và lưu trữ chúng vào vùng nhớ cấp phát động dưới dạng mảng các xâu kí tự, và một hàm để đưa chúng ra màn hình:

    void readnames(char *names[], const int size);
              void writenames(char *names[], const int size);

và viết một hàm nữa để sắp xếp danh sách trên bằng phương pháp nổi bọt:

    void bubblesort(char *names[], const int size);

Dưới đây là code của tôi, nó sai nhiều, mong các bạn sửa giùm.

#include <iostream.h>
#include <stdio.h>
#include <conio.h>
#include <string.h>

void input(char *names[], int size)
{
int i;
char *p=new char[20][20];
char s[20];
for (i=0; i<size; i++)
	{
   printf("\nEnter the name %i:", (i+1)); scanf("%s", s);
   strcpy((p+i), s);
   names[i]=(p+i);
   }
}

void bubblesort(char *names[], int size)
{
int i,j;
char temp[20];
for(i=0; i<(size-1); i++)
   {
   for(j=(i+1); j<size; j++)
   	if (strcmp(names[i], names[j])>0)
      	{
             strcpy(temp, names[i]);
             strcpy(names[i], names[j]);
             strcpy(names[j], temp);
             }
   }
}

void output(char *names[], int size)
{
int i;
cout<<"\n";
for (i=0; i<size; i++) printf("\n %s", names[i]);
}




void main(void)
{
int n;
char *thenames[20];
cout<<"Please enter the number of the students:"; cin>>n; cout<<"\n";
input(thenames, n);
bubblesort(thenames, n);
output(thenames, n);
getch();
}

Mong các bạn cũng chỉ rõ ra rằng làm cách nào tôi có thể khởi tạo một mảng của xâu kí tự (mảng 2 chiều kiểu char) trong vùng nhớ cấp phát động và gán địa chỉ của vùng nhớ đó cho con trỏ p kiểu char, và hơn thế nữa là cách sử dụng mảng 2 chiều đó thông qua con trỏ p.
Xin cảm ơn.

[rox_rook]

Strongly recommend : không bao h mix giữa scanf printf và cin, cout. Mix giữa C và C++ là poor coding style.
Đáng lý viết C++ nên dùng C++ string, nhưng tui hiểu mấy thầy cô muốn làm sv làm quen với cấp phát động nên mới ra đề kiểu quái đản thế này. Đây là cách cấp phát động cho mãng 2 chiều, và cậu quên 1 điều là phải giải phóng vùng nhớ, dùng new bao h cũng phải có delete.
Cậu pass const int size = vô nghĩa, int size là pass tham trị để const vào làm gì ?
- 1 là int size ( 4 byte không đáng kể )
- 2 là dùng const int& size
Còn phần cấp phát cậu viết thì tui hiểu là cậu chưa hiểu và chưa biết, đọc đoạn này còn thắc mắc gì thì hỏi tui sẽ giải thích. Phần sort tui dành cho cậu

#include <iostream>#include <cstring> char** read_name(int students){  int         index = 0;  //Cấp phát cho con trỏ đến con trỏ kiểu char với size là students  char**      ptr = new char*[ students ];  char        s_name[ 80 ];  for( index  = 0; index < students; ++index)  {    std::cout << "\n\nName : ";    std::cin.getline( s_name, 80 );    //Cho mỗi con trỏ, tui cấp phát tiếp vùng nhớ bên trong nó + 1 cho thằng '\0'    ptr[ index ] = new char[ strlen( s_name ) + 1 ];    strcpy( ptr[ index ], s_name );  }   return ptr; //trả về mãng con trỏ 2 chiều là đây.} void get_back_memory( char** ptr , int p_size ){  std::cout << "\n\n..I'm releasing my memory !";  int index;  for( index = 0; index <  p_size; ++index )  {    delete[ ] ptr[ index ];  }   delete[ ] ptr;} void print_name( char** student_name, const int& nums ){  int index;  for( index = 0; index < nums; ++index )  {    std::cout << student_name[ index ] << "\n";  }} int main( ){  int numbers_of_students;  std::cout << "-> Enter the number of students : ";  std::cin >> numbers_of_students;  std::cin.ignore( 80, '\n' );   char** student_name = read_name( numbers_of_students );   print_name( student_name, numbers_of_students );   get_back_memory( student_name, numbers_of_students );    return 0;}

[vuong_quoc_viet_PTCNN]

Cảm ơn bạn rox_rook rất nhiều về đoạn code rất độc đáo, rất hay mà tôi phải mất hàng tiếng đồng hồ mới có thể hiểu được. Tôi cũng muốn dùng thống nhất hàm ra/vào dữ liệu lắm chứ nhưng khổ nỗi là có một số chức năng mà printf dùng thì dễ hơn, cho nên tôi trộn cả hai thứ vào. . Đoạn code do tôi viết (mà tôi đã dẫn ở bài viết trước) tôi đã có một số chỉnh sửa qua tham khảo và đã chạy thử. Tôi xin viết ra đây để bạn xem và cho ý kiến:

#include <iostream.h>#include <stdio.h>#include <conio.h>#include <string.h> char *p=new char [20*20]; void input(char *names[], int size){int i;char s[20];for (i=0; i<size; i++)   {   printf("\nEnter the name %i:", (i+1)); gets(s);   strcpy((p+i*20), s);   names[i]=(p+i*20);   }} void bubblesort(char *names[], int size){int i,j;char temp[20];for(i=0; i<(size-1); i++)    {   for(j=(i+1); j<size; j++)    if (strcmp(names[i], names[j])>0)        {         strcpy(temp, names[i]);         strcpy(names[i], names[j]);         strcpy(names[j], temp);         }   }} void output(char *names[], int size){int i;cout<<"\n";for (i=0; i<size; i++) printf("\n %s", names[i]);}    void main(void){int n;char *thenames[20];cout<<"Please enter the number of the students:"; cin>>n; cout<<"\n";input(thenames, n);bubblesort(thenames, n);output(thenames, n);delete []p;getch();}

Công nhận là tôi cũng hơi sơ suất là thiếu câu lệnh delete. Nhưng tôi chạy vẫn thấy bình thường. Không biết quên lệnh delete có tác hại gì không nhỉ?
Tôi cũng có một thắc mắc là tại sao mỗi lần bạn dùng lệnh vào/ra dữ liệu bạn lại phải đặt trước chữ cout, cin là std::?

[Ada]

Không delete thì bộ nhớ sẽ không được giải phóng cho đến khi chương trình kết thúc (đến lúc này bộ nhớ cấp cho chương trình mới được giải phóng bởi hệ điều hành). Đối với chương trình đơn giản này thì không sao, nhưng đối với chương trình nói chung bạn sẽ sớm bị cạn bộ nhớ và gặp lỗi. Bởi thế, như rox_rook đã nói: luôn delete ngay đối tượng khi bạn không cần đến nó nữa.

[phallus113]

em cũng muốn hỏi 1 câu về con trỏ, em muốn khai báo mảng kí tự char từ A->Z hoặc từ 1-100, làm cách nào để khai báo 1 cách nhanh nhất hoặc tự động ?

[sieuphuong]

là sao nhỉ ?

#define max ???
char *s=new char[max];

[rox_rook]

em cũng muốn hỏi 1 câu về con trỏ, em muốn khai báo mảng kí tự char từ A->Z hoặc từ 1-100, làm cách nào để khai báo 1 cách nhanh nhất hoặc tự động ?

Example

#include <iostream>#include <cstdlib> char randchar(){  return "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ"[std::rand()%52];}int main(){  char* g = new char[10];  for(int x = 0; ++x < 10; g[x] = randchar())  delete[] g;}