Lỗi: Không nhập được họ tên trong c++ trong màn hình console?

[linhcodervn]

Hi cả nhà ! Hôm nay ngồi rảnh. không biết làm chi cả. nên tập tọe học đối tượng C++ như thế nào. Lâu giờ code C quen tay rồi. cái này lạ quá ^^

Em có đoạn code thế này
Đề bài :

Xây dựng lớp cơ sở HOCSINH có các thông tin: Họ tên, Lớp, Điểm toán, lý, hoá và các phương thức nhập, xuất dữ liệu, tính điểm trung bình.
Viết chương trình chính thực hiện nhập vào một danh sách n học sinh, sau đó hiển thị danh sách những học sinh có điểm trung bình >=5 .

Code :

#include<iostream.h>#include<conio.h>#include<stdio.h>class hocsinh{char hoten[25];int lop;float dtoan,dly,dhoa,tb; public:    void nhap()    {        cout<<"Ho ten:";gets(hoten);        cout<<"lop:";cin>>lop;        cout<<"diemtoan:";cin>>dtoan;        cout<<"diem ly:";cin>>dly;        cout<<"diem hoa:";cin>>dhoa;            tb=(dtoan+dly+dhoa)/3;    }    void xuat()    {         cout<<"\nhoten:"<<hoten;        cout<<"\nlop:"<<lop;        cout<<"\ndiem toan:"<<dtoan;        cout<<"\ndiem ly:"<<dly;        cout<<"\ndiem hoa:"<<dhoa;        cout<<"\ndiemtrungbinh:"<<tb;        }        void sapxep()        {        if(tb>=5)         {          cout<<"\n  "<<hoten;        cout<<"      "<<lop;        cout<<"         "<<dtoan;        cout<<"             "<<dly;        cout<<"                "<<dhoa;        cout<<"                   "<<tb;        }        }    };hocsinh sinhvien[100];main(){int i,n;cout<<"nhap so sinh vien: ";cin>>n;for(i=0;i<n;i++){    sinhvien[i].nhap();     }     for(i=0;i<n;i++)     {      sinhvien[i].xuat();      cout<<"\n";      }    cout<<"\nnhung sinh vien co diem trung binh >=5";     cout<<"\nHoten      Lop     Diem toan    Diem ly     Diem hoa      Diem tb";      for(i=0;i<n;i++)         {         sinhvien[i].sapxep();         } getch();}

Khi mình chạy. Nhập số học sinh xong. Nó hiện luôn là

Ho ten: lop :

luôn. nên mình không hiểu. Ai giải thích hộ cái hè :P

P/s : Devc++ 4 chấm

[bookhe]

Bạn thêm

fflush(stdin)

trước mỗi dòng lệnh nhập chuỗi thử xem.

[xuyenit55]

@linhcodervn : với hướng đối tượng khi viết thì với những hàm có chứa ít câu lệnh ( chứa 1 , 2 câu lệnh thôi) thì nên viết trong hàm public vì nó sử dụng phương thức inline và chương trình sẽ chạy nhanh hơn còn với những đoạn nhập và xuất , sắp xếp như thế kia thì nên định nghĩa ở ngoài lớp thì nhìn nó dễ và syte cũng đẹp hơn !
thêm nữa cái đoạn này có thể viết lại như sau :

        void sapxep()        {        if(tb>=5)         {          cout<<"\n  "<<hoten;        cout<<"      "<<lop;        cout<<"         "<<dtoan;        cout<<"             "<<dly;        cout<<"                "<<dhoa;        cout<<"                   "<<tb;

Nhìn nó sẽ gọn hơn hoặc dùng setw(int n) trong thư viện < iomanip.h> để định dạng khi in ra

void sapxep(){    if ( tb >= 5 )    {        cout <<"\n" << hoten << "\t\t " << lop << "\t\t" << dtoan << "\t\t" << dly << "\t\t" << dhoa << "\t\t" <<tb ;    }}

[linhcodervn]

Bạn thêm

fflush(stdin)

trước mỗi dòng lệnh nhập chuỗi thử xem.

fflush(stdin) là dùng đằng sau hàm scanf thì phải chứ bạn. dùng để chống trôi stdin. cái này ko dùng được

[linhcodervn]

@linhcodervn : với hướng đối tượng khi viết thì với những hàm có chứa ít câu lệnh ( chứa 1 , 2 câu lệnh thôi) thì nên viết trong hàm public vì nó sử dụng phương thức inline và chương trình sẽ chạy nhanh hơn còn với những đoạn nhập và xuất , sắp xếp như thế kia thì nên định nghĩa ở ngoài lớp thì nhìn nó dễ và syte cũng đẹp hơn !
thêm nữa cái đoạn này có thể viết lại như sau :

        void sapxep()        {        if(tb>=5)         {          cout<<"\n  "<<hoten;        cout<<"      "<<lop;        cout<<"         "<<dtoan;        cout<<"             "<<dly;        cout<<"                "<<dhoa;        cout<<"                   "<<tb;

Nhìn nó sẽ gọn hơn hoặc dùng setw(int n) trong thư viện < iomanip.h> để định dạng khi in ra

void sapxep(){    if ( tb >= 5 )    {        cout <<"\n" << hoten << "\t\t " << lop << "\t\t" << dtoan << "\t\t" << dly << "\t\t" << dhoa << "\t\t" <<tb ;    }}

ok ! thank xuyên. Cái nội dung mình cần hỏi chưa giải quyết đc :P
p/s : dạo ni thấy đọc truyền liên tục hầy

[nguyenquanghung]

Trong c++ thì dùng cin.ignore(1); để bỏ qua ký tự chuyển dòng.Bạn thử thêm vào xem sao

[xuyenit55]

ok ! thank xuyên. Cái nội dung mình cần hỏi chưa giải quyết đc :P
p/s : dạo ni thấy đọc truyền liên tục hầy

muốn định dạng khi in ra đẹp và chuẩn nên dùng sewt(n) nhé , dùng tab vẫn chưa đc chuẩn và đẹp!

#include<iostream.h>#include<conio.h>#include<stdio.h>class hocsinh{char hoten[25];int lop;float dtoan,dly,dhoa,tb; public:    void nhap();        void xuat();    void sapxep()    {        if ( tb >= 5 )    {        cout <<"\n" << hoten << "\t" << lop << "\t" << dtoan << "\t" << dly << "\t\t" << dhoa << "\t" <<tb << endl;    }  }       }; // phuong thuc nhap sinh vien   void hocsinh::nhap(){        cout<<"Ho ten:";        fflush(stdin);        gets(hoten);        cout<<"lop:";       cin>>lop;        cout<<"diemtoan:";  cin>>dtoan;        cout<<"diem ly:";   cin>>dly;        cout<<"diem hoa:";  cin>>dhoa;            tb=(dtoan+dly+dhoa)/3;} // phuong thuc suat sinh vien  void hocsinh::xuat()  {         cout << " \nhoten : " << hoten;        cout << " \nlop : " << lop;        cout << " \ndiem toan: "<< dtoan;        cout << " \ndiem ly: " << dly;        cout << " \ndiem hoa: "<< dhoa;        cout  << "\ndiemtrungbinh: "<<tb;}          hocsinh sinhvien[100];main(){int i,n;cout<<"nhap so sinh vien: ";cin>>n;for(i=0;i<n;i++){    sinhvien[i].nhap();     }     for(i=0;i<n;i++)     {      sinhvien[i].xuat();      cout<<"\n";      }    cout<<"\nnhung sinh vien co diem trung binh >=5";     cout<<"\nHoten      Lop     Diem toan    Diem ly     Diem hoa      Diem tb";      for(i=0;i<n;i++)         {         sinhvien[i].sapxep();         } getch();}

[vantrong]

Thêm vào câu lệnh cin.ignore(); ở trước lệnh nhập họ tên.
Và nhập họ tên dùng phường thức getline(hoten, 25); là okie.
Cái này khi code với char* là hay dính lắm. Nếu dùng string thì khỏe hơn

Góp ý:Ở chỗ sapxep(), if(...) sau đó xuất ra hócinh đó, thì gọi hàm xuất lun, đỡ phải viết lại code.

[kedocthan2009]

Dùng fflush(stdin) đúng mà bạn.

cout<<"Ho ten: ";
fflush(stdin);
gets(hoten);

mình không rành dùng cout cin, nhưng mình xài hàm printf(); cũng tương tự.

printf("Ho ten: ");
fflush(stdin);
gets(hoten);

[kitti]

Trong C++ thì mình nghĩ lên dùng như thế này:

cin.ignore();cin.getline(HoTen,25);