Một số vấn đề thắc mắc về char*?

[khoadangnguyen]

Đề bài yêu cầu:chuyển một chuỗi thường cho trước về kiểu in hoa (nhất định phải sử dụng con trỏ).

#include <iostream.h>#include <string.h> char* upper_case(char *s){    char* sPtr;    sPtr = new char [strlen(s)];    char* p = sPtr;    for(;*s!='\0';++s)    {        *p++ = *s - 32;    }    *p = '\0';    return(sPtr);} int main(){    char* str = "congdongcviet";    char* str1;    str1 = upper_case(str);    cout <<str1<<endl;    delete [] str1; //  error why?    return(0);}

--Phải chăng dạng bài về chuỗi nào cũng phải xin cấp phát thêm một khối nhớ char* mới?Giả sử mình muốn xử lý trực tiếp lên chuỗi nguồn luôn đc ko?(vẫn phải sử dụng con trỏ nha).Nếu đc bạn có thể cho mình một ví dụ về cách đó nhá!!Ngay đề bài đổi thường -> hoa này luôn.

--Còn nữa tại sao khi mình giải phóng bằng toán tử delete lại bị lỗi nhỉ?Các bạn giải tích dùm mình ?ngay dòng gần cuối cTrình đó.

Cám ơn nhiều!

[rox_rook]

#include <iostream>#include <cstring> char* upper_case(char *s){    char* sPtr = new char[strlen(s) + 1];    char* p = sPtr;    for(;*s!='\0'; ++s){        *p++ = *s - 32;    }    *p = '\0';    return sPtr;} int main(){    char* str = "congdongcviet";    char* str1;    str1 = upper_case(str);    std::cout << str1 << "\n";        delete [] str1; //  error why?        return 0;}

Sai đâu mà sai cậu ?
trong các header của C++ không có iostream.h không có string.h
Nếu thực sự cậu viết C++ :
- #include <iostream>
- #include <cstring>
và phải dùng using namespace std;
hoặc :
using std::cout;
using std::cin;
hoặc dùng thẳng trong câu lệnh.
Thêm nữa mấy cái dấu ngoặc của cậu thêm vào return () để làm gì, nó chẳng có 1 ý nghĩa gì, nếu không có thì đừng add vào.
Còn về bài trên, thao tác trực tiếp vào chuỗi cũng được, vấn đề là size không của chuỗi không thay đổi nên chẳng cần cấp phát động làm gì !

[khoadangnguyen]

- Đúng rồi! ko có lỗi syntax khi mình compile.Nhưng khi chạy thì bị lỗi chỗ này nè:mặc dù vẫn ra kết quả nhưng sau đó là 1 bảng thông báo lỗi run-time: ở chỗ cái delete đó!
Mình xài VC++ 6.0!

[rox_rook]

Tui dùng g++ ở nhà, h tui ở trường xài visual C++ 2008 có run-time gì đâu cậu, code đó làm sao có lỗi chỗ delete được. Cậu check kĩ lại xem.

[rox_rook]

Thêm nữa vấn đề cậu dùng char* str = " ", đây là 1 dạng chuỗi const, cho nên cấp phát động sẽ là bắt buộc, vì cậu không thể chỉnh sữa dữ liệu của 1 chuỗi dạng const. const char* là style của C. Nếu thực sự cậu học C++ thì cậu nên dùng #include <string>. Và đây tui sẽ cho cậu vài examples về cách giải bài trên của cậu.

#include <string>#include <iostream>using std::size_t; void upper_case(std::string& s){  for(size_t x = 0; x < s.length(); ++x){    if(islower(s[x]))      s[x] = toupper(s[x]);  }} int main(){  std::string str = "congdongcviet";  upper_case(str);  std::cout << str << "\n";    return 0;}

Dùng STL :

#include <iostream>#include <algorithm>#include <string> int main(){  std::string s("cong dong c viet");  std::cout << s << "\n";    std::transform(s.begin(), s.end(), s.begin(), toupper);    std::cout << s;  return 0;}