aoisupersix / bve5_parsing Goto Github PK

ParseTreeNodeの型を判別する方法は、現在各ノードの名前で判別していたが、名前を変更した際の保守性を考慮してtypeof、もしくはGetType()による判別方法に変更する。

現在は構文名や引数名がハードコードになってる

一番気になっていた部分なので早めに修正する

構文名をEnumにするか、構文ごとにクラスを定義するか

IList、Dictionaryをpublicで使うのは良くないらしいので修正する

便利そうだけどよく分からないのでとりあえず起票だけしておく。

恐らくScriptExceptionのstackTraceがnullだからなので、stackTraceの修正？

ASTノードでTrack.Cant.InterpolateとTrack.Cant.Setgaugeを継承しているのが原因。本来存在しないはずのSubElementNameが親クラスで存在するので結果がおかしくなる。

OUTER_CHAR定義は不要のため。

現在マップ要素と関数しかないので、構文の種類が特定しづらい。

関数名にドットが入る関数(Track[].X.Interpolateみたいな)はSubElement:XとFunction:Interpolateに分解する

Includeディレクティブは、その構文位置に指定されたマップ構文ファイル（ヘッダを除く）を挿入する機能らしい。
つまり、Includeディレクティブの動作としては、

1. 指定されたマップ構文ファイルを読み込み
2. ファイルヘッダを削除
3. Includeディレクティブを読み込んだマップ構文で置き換え
   ということになる。

これを実装するためには、

1. デベロッパーがBve5_Parsingを利用する前にIncludeディレクティブを構文で置き換え
2. Bve5_Parsingのパース前にIncludeディレクティブを構文で置き換え
   という２つの方法がいいかな。

デフォルトでは2のBve5_Parsing内で置き換えを行って、オプションとして置き換えを行わずにパース可能にする

Scrapboxの草案を参考に画像を交えつつドキュメントを作る。

#42 対応のため。
現在引数を辞書形で無理やりやっているところを、クラスのプロパティに改修する。
また、EvaluteするかASTをそのまま返すか、デベロッパーが選択できるようにする（これは #39 で対応）

パースごとにVariableStoreのインスタンスを作る方が正しい気がする。
ついでにVariableStoreをパース間で引き継げるようにする。

非推奨な構文にはDeprecated属性が付与されているのでそれを使って警告を出す

#82 をマージしたらテストが走らなくなってしまったので修正する

以下エラー：

Starting test execution, please wait...
Unable to find /home/travis/build/aoisupersix/Bve5_Parsing/Bve5_ParsingTests/bin/Debug/netcoreapp2.0/testhost.dll. Please publish your test project and retry.
Test run in progress.Test Run Aborted.

デバッグでテストを動かしたときに何故か例外が発生しなくて調べてみたら、例外が全て握りつぶされていたので修正する。

どうやら本家ソフトは

• 空白（スペース）は文字間に必ず一つ
• ファイルヘッダに空白は不可
• 大文字小文字は無視でOK
  らしいのでそれ準拠に変更する

マルチターゲット機能を利用する。
参考：
https://stackoverflow.com/questions/42747977/how-do-you-multi-target-a-net-core-class-library-with-csproj/42748264

今まで面倒で作っていなかったので

b5gのbracesが原因。0.0.3で解決させたと思うのでパッケージを更新する

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Datasource	Name	Replacement PR?
npm	tslint

Rate-Limited

These updates are currently rate-limited. Click on a checkbox below to force their creation now.

• Update dependency b5g to v0.1.2
• Update dependency linq to v3.2.4
• Update dependency mustache to v4.2.0
• Update dependency typescript to v3.9.10
• Update xunit-dotnet monorepo (xunit, xunit.runner.visualstudio)
• Update dependency @types/node to v20
• Update dependency Microsoft.NET.Test.Sdk to v17
• Update dependency linq to v4
• Update dependency tslint to v6
• Update dependency typescript to v5
• 🔐 Create all rate-limited PRs at once 🔐

Edited/Blocked

These updates have been manually edited so Renovate will no longer make changes. To discard all commits and start over, click on a checkbox.

• Update dependency Microsoft.CSharp to 4.7.0
• Update dependency Microsoft.NET.Test.Sdk to 16.11.0
• Update dependency System.Text.Encoding.CodePages to 4.7.1
• Update dependency ts-node to v8.10.2
• Update dependency tsutils to v3.21.0
• Update dependency strip-bom to v5

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• Update dependency @types/node to v12.20.55
• Update dependency docfx.console to 2.59.4
• Update dependency System.Text.Encoding.CodePages to v8
• Update dependency ts-node to v10
• Click on this checkbox to rebase all open PRs at once

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

nullableExprはnullが入力された場合0を返すが、一部引数がnullの場合は特定の意味をもつ構文が存在する。(ex.Interpolateなど)

現在ステートメントはSyntax1~3に分けて解析を行っているが、一つに統一できそうなので統一させる。

もっとわかりやすいドキュメントにする。

追加予定

• 構文解析の評価(戻り値)の詳細
• ファイルの構造など
• もっとそれらしい画像

適当な文字列でテストしたい

現在GrammarTest.csでやっているコンソールによるパーサの動作確認を別プロジェクトで行いたい。

とりあえずTODOということで

• Legacy構文の引数名startがParserRuleContextとかぶってる
• Legacy構文のASTノードを作る
• Include構文でディレクトリの大文字小文字を無視できるようにする

MapGrammarのパース処理に関する例外処理やエラーメッセージの追加。

取り敢えず現状の問題点↓

• MapGrammarの第一ステートメントに予期しない構文が入力された場合、その後のステートメントがパースされない。
• 数式の項に文字列が含まれていた際の演算で、'+'以外の演算子を利用していた場合に例外が発生する。
• パーサのエラーメッセージが単純に分かりづらい。

他の用途でも活用できそうなのでもっと真面目に実装する。

npmでパッケージを公開したのでそっちを使うように改修する。

MapGrammarと同様の機能を実装する。

構文解析の結果、構文が正しくない場合はScriptExceptionを返すが、現時点では例外の中にファイル名が含まれていないので、どのファイルの例外か判別できない。

改善案

• ScriptException内のinnerExceptionにファイル名を含む例外を入れておく
• 独自の例外を追加する

参考

• http://ufcpp.net/study/csharp/oo_exception.html
• https://github.com/Alxandr/Irony/blob/master/Irony/Utilities/LogMessage.cs
• https://github.com/Alxandr/Irony/blob/master/Irony.Interpreter/SriptApplication/ScriptThread.cs