有代码一百行，求简化

TheRebirth

Main.rar (722 Bytes, 下载次数: 495)

2015-7-10 15:31 上传

点击文件名下载附件

代码在上面，这个程序的目的是识别储存于文件中的命令并作出反应。
如你所见，这上面的代码实在有些臃肿，所用的方法简直可以用小儿科来形容。
我希望能用一种更简洁且执行效率更高的写法取代原来的代码，最好还能够放宽原有的限制条件（如“实际数值不管有多少位，始终只能占特定字符数，其他位没有就拿空格堆”这一问题）。

（我找到的这都什么破资料…）

drzzm32

额其实。。。字符串处理用.NET更方便呢。。。虽然我也不清楚你的定义是怎样的。。。
看看我的吧，从VB.NET翻译到C#的

1. public static Scene[] LoadScript(string Path)
   
2. {
   
3.         Scene[] Scenes;
   
4.         int SceneIndexTmp;
   
5.         int CGIndexTmp;
   
6.         int ChoiceAutoIndexTmp;
   
7.         string FullPath = Environment.CurrentDirectory + "\" + Path;
   
8.         IO.StreamReader Reader;
   
9.         try {
   
10.                 Reader = new IO.StreamReader(FullPath);
    
11.         } catch (Exception ex) {
    
12.                 Message MsgBox = new Message();
    
13.                 MsgBox.MsgShow("Error", "Config file contains error.");
    
14.                 MsgBox.Close();
    
15.                 return null;
    
16.         }
    
17. 
    
18.         string[] TmpString;
    
19.         int TmpHead;
    
20.          // ERROR: Not supported in C#: ReDimStatement
    
21. 
    
22. 
    
23.         while (!Reader.EndOfStream) {
    
24.                 TmpString = Reader.ReadLine.Split({
    
25.                         Convert.ToChar(Space(1)),
    
26.                         Convert.ToChar(vbCr),
    
27.                         Convert.ToChar(vbTab)
    
28.                 });
    
29.                 TmpHead = 0;
    
30.                 try {
    
31.                         while (TmpString(TmpHead) == "") {
    
32.                                 TmpHead += 1;
    
33.                         }
    
34.                 } catch (Exception ex) {
    
35.                         goto Bottom;
    
36.                 }
    
37. 
    
38.                 switch (TmpString(TmpHead)) {
    
39.                         case "开始定义场景":
    
40.                         case "defsc":
    
41.                         case "def.scene":
    
42.                                 SceneIndexTmp = Convert.ToInt32(TmpString(TmpHead + 1));
    
43.                                 Scenes(SceneIndexTmp) = new Scene();
    
44.                                 Scenes(SceneIndexTmp).SceneIndex = SceneIndexTmp;
    
45.                                 break;
    
46.                         case "结束定义场景":
    
47.                         case "endsc":
    
48.                         case "end.scene":
    
49.                                 SceneIndexTmp = -1;
    
50.                                 break;
    
51.                         case "背景":
    
52.                         case "backg":
    
53.                         case "set.back":
    
54.                                 Scenes(SceneIndexTmp).BG.Path = TmpString(TmpHead + 1);
    
55.                                 Scenes(SceneIndexTmp).BG.Index = SceneIndexTmp;
    
56.                                 break;
    
57.                         case "音乐":
    
58.                         case "music":
    
59.                         case "set.bgm":
    
60.                                 Scenes(SceneIndexTmp).BGM.Path = TmpString(TmpHead + 1);
    
61.                                 break;
    
62.                         case "开始定义立绘":
    
63.                         case "defcg":
    
64.                         case "def.cg":
    
65.                                 CGIndexTmp = Convert.ToInt32(TmpString(TmpHead + 1));
    
66.                                 Scenes(SceneIndexTmp).CG(CGIndexTmp).IMG.Index = CGIndexTmp;
    
67.                                 break;
    
68.                         case "结束定义立绘":
    
69.                         case "endcg":
    
70.                         case "end.cg":
    
71.                                 CGIndexTmp = -1;
    
72.                                 break;
    
73.                         case "立绘地址":
    
74.                         case "cgloc":
    
75.                         case "set.path":
    
76.                                 Scenes(SceneIndexTmp).CG(CGIndexTmp).IMG.Path = TmpString(TmpHead + 1);
    
77.                                 break;
    
78.                         case "立绘位置":
    
79.                         case "cgpos":
    
80.                         case "set.pos":
    
81.                                 Scenes(SceneIndexTmp).CG(CGIndexTmp).Location = Convert.ToSingle(TmpString(TmpHead + 1));
    
82.                                 break;
    
83.                         case "开始定义文字":
    
84.                         case "defwd":
    
85.                         case "def.word":
    
86.                                 ChoiceAutoIndexTmp = 0;
    
87.                                 switch (TmpString(TmpHead + 1)) {
    
88.                                         case "对话":
    
89.                                         case "comm":
    
90.                                         case "type.comm":
    
91.                                                 Scenes(SceneIndexTmp).WordType = WordType.Comment;
    
92.                                                 break;
    
93.                                         case "读档":
    
94.                                         case "read":
    
95.                                         case "type.read":
    
96.                                                 Scenes(SceneIndexTmp).WordType = WordType.Loading;
    
97.                                                 break;
    
98.                                         case "存档":
    
99.                                         case "save":
    
100.                                         case "type.save":
     
101.                                                 Scenes(SceneIndexTmp).WordType = WordType.Saving;
     
102.                                                 break;
     
103.                                         case "旁白":
     
104.                                         case "scpt":
     
105.                                         case "type.script":
     
106.                                                 Scenes(SceneIndexTmp).WordType = WordType.Script;
     
107.                                                 break;
     
108.                                         case "标题":
     
109.                                         case "titl":
     
110.                                         case "type.title":
     
111.                                                 Scenes(SceneIndexTmp).WordType = WordType.Title;
     
112.                                                 break;
     
113.                                         case "启动":
     
114.                                         case "stup":
     
115.                                         case "type.start":
     
116.                                                 Scenes(SceneIndexTmp).WordType = WordType.GameRun;
     
117.                                                 break;
     
118.                                 }
     
119.                                 break;
     
120.                         case "选择":
     
121.                         case "cword":
     
122.                         case "set.word":
     
123.                                 for (i = TmpHead + 1; i <= TmpString.Length - 1; i++) {
     
124.                                         if (i < TmpString.Length - 1) {
     
125.                                                 Scenes(SceneIndexTmp).Choices(ChoiceAutoIndexTmp).Context += TmpString(i) + " ";
     
126.                                         } else {
     
127.                                                 Scenes(SceneIndexTmp).Choices(ChoiceAutoIndexTmp).Context += TmpString(i);
     
128.                                         }
     
129.                                 }
     
130. 
     
131.                                 break;
     
132.                         case "跳转":
     
133.                         case "jmpto":
     
134.                         case "set.jump":
     
135.                                 Scenes(SceneIndexTmp).Choices(ChoiceAutoIndexTmp).JumpSceneIndex = Convert.ToInt32(TmpString(TmpHead + 1));
     
136.                                 ChoiceAutoIndexTmp = ChoiceAutoIndexTmp + 1;
     
137.                                 break;
     
138.                         case "结束定义文字":
     
139.                         case "endwd":
     
140.                         case "end.word":
     
141.                                 ChoiceAutoIndexTmp = -1;
     
142.                                 break;
     
143.                 }
     
144.                 Bottom:
     
145. 
     
146.         }
     
147. 
     
148.         Reader.Dispose();
     
149.         return Scenes;
     
150. }

复制代码

核心是String.Split，按指定分隔符切分字符串。如果真做还要去学习编译原理（

漆黑之翼

我记得ifstream跟cin一样可以直接读成对应类型的啊，何必搞这么麻烦呢

fancydz

我觉得还好...特别是在文件内容格式没有明显规律的情况下
如果要优化，那就是建立诸如读到空格即跳过，非空格则处理的策略了
另外不识别的指令直接用getline跳过去就好了，没必要一个个去偏移...