C#: Task を用いるようにした

[0918nobita]

C# において非同期処理を行う際に用いられる Task で、then then then 世を実現しました。

具体的には、

• 既存の RADWIMPS クラスに Task 型のプライベートフィールド task を追加し、(すでに完了した処理を表す) Task.CompletedTask で初期化
• then メソッドで、task フィールドに継続タスクを連結 (この時点では継続は実行されない) したうえで自分の this を返すようにした
• 世 メソッドで、task フィールドに継続タスクを連結したうえですべての継続を実行するようにした (これを呼び出したタイミングで「前前前世」が出力される)

のような変更を加えています。

[siketyan]

@0918nobita CI が落ちてるのはトークンの問題なので（PR Author のトークンでは当然リポジトリの操作ができない），気にしないでください 🙏
この問題については #122 で修正済みですが PR を作り直さないと反映されないので，そのままで大丈夫です！

[0918nobita]

@siketyan @kawaemon この (エントリポイントで Task を返却するようにする) 修正を加えると、最新版の mcs (mono 付属の C# コンパイラ) では C# 7.1 の言語機能「async main」がサポートされていないためコンパイルに失敗するようになりました。
OS は Ubuntu 20.04 LTS です。

$ mcs --version
Mono C# compiler version 6.8.0.105

$ mcs RADWIMPS.cs
RADWIMPS.cs(25,28): warning CS0028: `RADWIMPS.Program.Main(string[])' has the wrong signature to be an entry point
error CS5001: Program `RADWIMPS.exe' does not contain a static `Main' method suitable for an entry point
Compilation failed: 1 error(s), 1 warnings

参考 Issue：mono/mono#6854

[siketyan]

@0918nobita .NET Core 使いましょう……

[siketyan]

このプロジェクト全体として最新の言語機能を重視するのか互換性を重視するのかというのは重要な議論点だと思うのでこの機会に決めておいたほうがよさそうです
@approvers/all

[0918nobita]

まぁこのプロジェクトで互換性云々を重視してもあんまり意味がない気がするので、今回は .NET Core を想定する案に賛成です。この際 RADWIMPS.csproj を追加してちゃんと言語バージョン等を指定したほうがいいかもしれません

[siketyan]

@0918nobita 言語バージョンの指定は賛成ですが .csproj を Git 管理下に置くほどではないかな，と思いますね…
いくつかの言語ではすでにされていますがファイルの先頭にコメントで書く程度でいいかもしれません

[0918nobita]

@siketyan .csproj のない状態でクローンしてきて各自でなんとか動かしてくれ、というのはコメントに方法が追記されていたとしてもかなり面倒で、この話のように開発/動作環境に関する混乱を招く原因にもなり得ます。
#102 (comment) でも提案しているように、C# (または CSharp ？) ディレクトリを作成してそのなかに .cs, .csproj ファイルを入れてしまうのが、一番親切でそこまで面倒ではなくて良いと思います。

[siketyan]

@0918nobita たしかにそうですね…
私だけでは決めかねるので他のメンバーと相談します 🙇‍♂️
@approvers/all

[mirror-kt]

このPRの本筋から離れるため #123 にて検討します

[0918nobita]

ごちゃごちゃしてきたのでディレクトリ構造に関する議論が片付いてから PR 投げ直します