这两个程序哪个写的好，请大家发表意见。

[zeliangzhang19801124]

1.半角转换为全角

CString CTestExcelChange091230Dlg::DbcToSbc(CString stringText)
{
CString str=L"";

int nLength=stringText.GetLength();
ASSERT(nLength!=0);
TCHAR *c=new TCHAR[sizeof(TCHAR)*nLength+1];
memset(c,0,sizeof(TCHAR)*nLength+1);
wcscpy(c,stringText);
for (int i=0;i<nLength;i++)
{
if (c[i]==32)
{
c[i]=(TCHAR)12288;
continue;
}
if (c[i]<127)
{
c[i]=(TCHAR)(c[i]+65248);
}

}

str.Format(L"%s",c);

delete c;
c=NULL;
return str;
}


2.这段程序今天看来有许多不好的地方，现将第一个程序其改为如下：

void CTestDlg2008Dlg::DbcToSbc(const CString stringText,CString &str)
{
int nLength=stringText.GetLength();
ASSERT(nLength!=0);
TCHAR *c=new TCHAR[sizeof(TCHAR)*nLength+1];
memset(c,0,sizeof(TCHAR)*nLength+1);
wcscpy(c,stringText);
for (int i=0;i<nLength;i++)
{
if (c[i]==32)
{
c[i]=(TCHAR)12288;
continue;
}
if (c[i]<127)
{
c[i]=(TCHAR)(c[i]+65248);
}

}

str.Format(L"%s",c);

delete []c;
c=NULL;
}
2010.7.3

[lazy_2010]

首先，可以看出是 vs2008；

其次，可以看出 TCHAR 实际对应 wchar_t；

第三，不如定义一个映射表 (ASCII, MBCS)，使用查找表格的方式更合理；比如 ! @ 这样的符号不是直接加一个整数能映射的；

[buyong]

delete 对于内部的数据类型 例如 int float double 等的动态数组p delet p和delete[]p是等价的 对于非内部数据类型的动态数组objects 而言 delete objects 和delete []objects是不一样的
如果 Obj *objects=new Objects[100] 用delete objects相当于delete objects[0]，漏掉了 另外99个对象

[buyong]

delete 对于内部的数据类型 例如 int float double 等的动态数组p delet p和delete[]p是等价的 对于非内部数据类型的动态数组objects 而言 delete objects 和delete []objects是不一样的
如果 Obj *objects=new Objects[100] 用delete objects相当于delete objects[0]，漏掉了 另外99个对象

[zeliangzhang19801124]

顶一下。

[zdcju]

看半天才发现，原来就这么点不同啊。。。。。。
貌似上面说的 第一个比第二个要多做一次深拷贝 外应该没什么影响
还有用应该都用 delete [] 才对

[康斯坦汀]

都不好，应该按标准C++写，这样可以更容易移植。
TestExcelChange091230Dlg 、CTestDlg2008Dlg 这种命名方式很不好。
不符合开发规范。
if (c[i]==32)
{
c[i]=(TCHAR)12288;
continue;
}
if (c[i]<127)
{
c[i]=(TCHAR)(c[i]+65248);
}

直接在程序中使用具体数值，叫幻数吧，也是很不好的设计风格。

[c271525290]

[Quote=引用 14 楼 r3000 的回复:]
都不好，应该按标准C++写，这样可以更容易移植。
TestExcelChange091230Dlg 、CTestDlg2008Dlg 这种命名方式很不好。
不符合开发规范。
if (c[i]==32)
{
c[i]=(TCHAR)12288;
continue;
}
if (c[i]<127)
{
c[i]=(TCHAR)(c[i]+65248……
[/Quote]
虽然我只是个菜鸟，但是我也觉得应该要按照标准C++来写……

[hahaking119]

看不懂怎么转的

[sweetygirl]

不可能

[zeliangzhang19801124]

受益匪浅，大家继续讨论

[ssy0691]

刚来看不懂

[jogger007]

虚心学习来的

[zeliangzhang19801124]

23楼说的好

[用户 昵称]

俺习惯查字典方法。

[搬了20多年的砖]

两段代码都有Bug,转换方式差不多.没有看出有什么改进.

[zeliangzhang19801124]

大家都来讨论啊。
自己顶一下。

[BoYueJiang]

不用讨论吧，
首先你的编码风格应该控制控制，

其次，如果你非要用第二种，可以试着让他有返回值类型，返回BOOL之类的。另
外，你还发现了吗？ 你的new 出来的c 没有做NULL检查，纵使这个机率很小，但依然会存在。
还有就是楼上几位说的，直接使用了幻数。 哪怕你#define一下也会好很多。

还有像这种以WINDOWS界面无关的函数，你最好能够不要依赖WINDOWS平台的东西。 最好的方法就是先在控制台下测试，测试的时候，不要包含WINDOWS。H等头文件！

[jackson35296]

两段程序都有问题
TCHAR *c=new TCHAR[sizeof(TCHAR)*nLength+1];
memset(c,0,sizeof(TCHAR)*nLength+1);
c分配的空间太多了，造成浪费。new TCHAR[nLength+1];就行了，你那样写，如果是unicode的话，c分配的大小是2*(2*len+1)而你只需要2*len+2就够了。new的单元大小会考虑TCHAR的大小的，你这样是浪费内存。而且你的memset也只是对2*len+1清0,这个也是错误的，应该是memset(c,0,sizeof(TCHAR)*(nLength+1));应该是对2*len+2清0。因为unicode的0是以两个\0结尾。你这样写如何遇到字符串长度刚好是nLength的时候就暴露了bug。

[cdsnpeter]

编码风格不是很好，你的代码命名还能改进。

[qq_heilongjiang]

－＿－