search

S-H-GAMELINKS / Novel.Game.Engine.LINKS

ノベルゲームエンジン「LINKS」
Other
13 stars 1 forks source link

Refactor/basic refactor2 #4

Closed yumetodo closed 7 years ago

yumetodo commented 7 years ago

2 の続き

C++14 規格書によれば

https://github.com/cplusplus/draft/blob/master/papers/n4140.pdf § 2.3 lex.charset In both the source and execution basic character sets, the value of each character after 0 in the above list of decimal digits shall be one greater than the value of the previous.

とあり、 '0'から '9' の文字コードは連続することが保証されていているため、これを利用して文字列を整数にパースすることで大幅なコード削減を実現した。(0fdbf402c619397fa0ba7a24aaf27cf71cdc30e2, 9bd0def3631f68e4c3d71adbe908b2f92d29e772)

ref

使用しているC++11の機能(core)

使用しているC++11の機能(library)

S-H-GAMELINKS commented 7 years ago

今回も一通りチェックさせていただきました。マージさせていただきます。

各種素材の描画関連は以前から短縮したいと思っていました。本当にありがとうございます。

yumetodo commented 7 years ago

@S-H-GAMELINKS チェックお疲れ様です。自分だけだとどうしても確認しきれないので(単体テストないし)確認助かります。

S-H-GAMELINKS commented 7 years ago

いえいえ、こちらこそありがとうございます。

S-H-GAMELINKS commented 7 years ago

あと、テストしてみたところ呼び出すスクリプトファイルが一つずれていたようです。

とりあえず、以下のように修正して、コミットしておきました。

if (0 < EndFlag && EndFlag <= countof(ScriptFileNames)) {

// スクリプトファイルを開く

const int ScriptFile = FileRead_open(ScriptFileNames[EndFlag - 1]);

SCRIPT_TO_ARRAY(ScriptFile);

FileRead_close(ScriptFile);

}

yumetodo commented 7 years ago

31d0fb74bd9b8c2fef91adb32355ffc6ec357730 確認しました。確かにずれてますね・・・見落としていました

S-H-GAMELINKS commented 7 years ago

いえいえ、大丈夫です。すぐに気づけましたし。

少し実際のテストファイル使いつつ、検証を進めていこうとは思います