Godot: GDScriptの配列スライシング

これは、 rangeと＃4716の内包表記を使用して簡単に実行できることに注意してください。

var result = array[from:to:step]
# Is the same as:
var result = (array[i] for i in range(from,to,step))

また、ここでもCoffeeScript構文をスローするには：

x = array[from...to]

ステップサイズをサポートするためにこれを適応させるかもしれません（サポートしていないため）：

x = array[from..step..to]
# or
x = array[from...to:step]

：を使用したimho python構文は、より便利です-文字数が少なく、コンパクトです

：を使用したimho python構文は、より便利です-文字数が少なく、コンパクトです

+1。 そして、それほど混乱しません。 私たちはPythonに似ているので、世界中の他のすべての言語から動作をインポートしないようにしましょう。そうしないと、GDScriptが実際に一貫性を失うことになります。

D構文（$は配列のサイズです）：

array[2..4]
array[2..$]
array[$-1..$]

参考までに。 私はPythonのものがGDScriptによく合うことに同意します：P

私はこれに取り組みたいと思っています。

最初のステップとして、負のインデックスを追加するPRを準備しています。 （ array[:-1]を許可する場合は、$＃ array[-1]も許可する必要があります。）

配列スライスへの割り当てはどうですか？ ユースケースはありますか？

私はそれをサポートするコードを入手できると確信していますが、ユースケースがなければ、私はそれを気にしないかもしれません。

私たちがしなければならない制限の1つは、置換値が同じサイズでなければならないことだと思います（Pythonは、バイトなどの固定サイズの配列のような構造にも適用します）

つまり、次のように機能するはずです。

var a = range(10)
a[2:5] = [10, 11, 12]
print(a) #[0, 1, 10, 11, 12, 3, 4, 5, 6, 7, 8, 9]

次の可能性があります。

a = range(10)
a[1::2] = [10, 11, 12, 13, 14]
print(a) # [0, 10, 2, 11, 4, 12, 6, 13, 8, 14]

しかし、これはおそらくありません：

a = range(10)
a[2:5] = [10]
print(a) # [0, 1, 10, 5, 6, 7, 8, 9]

意見？

念のため、まだ作業中です。 来週にはPRの準備ができるかもしれませんが、もっと時間がかかる場合があります。

@brakhane誰も急いでいません-時間をかけてください:)

＃5701および＃5765を参照してください
単純な2引数のスライス操作をDVectorおよびByteArrayバインドに追加しました。これは、将来のバインドで、GDSript低レベル配列タイプ（IntArray、StringArrayなど）のいずれにも適用できます。 -1を処理するコードはまだありません。 私はちょうど今その便利さを省いたことに気づきました。

これは3.0に実装する必要があると思います。現在、PoolByteArrayにはsubarray(from, to)メソッドが実装されていますが、他の組み込み型には実装されていないため、非常に一貫性がありません。 3.0で一貫性のあるものを実装しない場合は、後で置き換えられる可能性のあるAPIの導入を避けるために、＃5879（安定版リリースではまだ出荷されていません）を元に戻すことを提案します。

私の実装は98％完了しています（動作しますが、デバッグする時間がなかったメモリリークやランダムなクラッシュがいくつかあります）が、実際の時間の制約のため、実装を完了できません。

誰かが（私を助けて）それを終わらせることができるように、私は自分のWIPをブランチに入れても構わないと思っています。 実装に関する質問をお手伝いできます。 私の努力が無駄になるのは残念です（私はおそらくこれまでに約40時間を費やしています）

実装はかなり完全な機能です（そして、実際にそれがどれほど完全であるかを誇りに思っています）。たとえば、スライスは、割り当てを含め、ほとんどの配列タイプでPythonのように機能します（たとえば、 a[1:4] = [1,2,3]は機能します。 a[1:3] = b[10:13:2]およびその逆）。

ただし、いくつかの注意点があります。

• 実装はやや複雑であり、Variant.cppとvariant_op、およびそれを理解するためのバイトコードインタープリターについてある程度理解している必要があります。
• 正気を保ち、デバッグを支援するために、variant_opの一部のマクロを同等のテンプレートに置き換える必要がありました。IMOを使用すると、コードが読みやすくなります（また、以前と同じように最適化が有効になります）が、コードは2.1に基づいているため、 3.0でコンパイルするには、おそらくリファクタリングが必要です。 また、コードはWIPであるため、個別のコミットに分割する時間がありませんでした。そのため、マクロがテンプレートに置き換えられると同時に、バリアントロジックも追加されることがあります。 2つを区別するのが少し難しくなります（ただしそれほど難しくはありません）
• バイトコードインタープリターが古いバージョンと互換性がなくなるのを避けるために、新しいデータ型sliceを導入する必要がありました。これは、エディターに公開する必要があります。 この作業は完了しましたが、いくつかの荒削りな部分があります。
• すでに述べたように、残っている「唯一の」問題は、私が理解できなかったメモリの破損です。 これをデバッグしたい人は誰でも、Godotバイトコードインタープリターがどのように機能するかを知っている必要があります

コードをどこかに置く必要があるかどうか教えてください。 godotスクリプトの内部に精通している人なら、10時間以内にねじれが解消されると思います。

@brakhaneすでに作業が完了しているので、コードをフォークに入れてプルリクエストを開くと便利です。 このようにして、他の人があなたのコードをレビューしてコメントすることができます。

調子はどう？
これを楽しみにしています。 :)

@ bojidar-bgは私たちの新しいgdscript開発者です、試してみませんか？ ：P

まあ、3.1をプッシュします

サブアレイメソッドは包括的範囲を使用しますが、これは（うまくいけば）Pythonスタイルのスライス範囲を使用することに注意してください。

サブアレイメソッドを使用するのは少し面倒なので、幅がゼロのスライスを作成しようとしないように特別なケーシングを作成する必要があるためです（たとえば、バッファの一部を2回スライスして消費しようとしている場合は、 '残りの'スライスの長さはゼロにすることができます）、スライスを要求するときにすべての長さに-1を追加します。

編集：実際、一定数のバイトをスライスして-1を実行できる場合を除いて、最後のスライスに-1を追加する必要がない状況を頭の中で考えることはできません。それを書くとき。

これで何か進歩はありますか？
GDscriptでこの機能を実際に使用できることがわかりました

3.1は機能が凍結されているため、次のマイルストーンに移行します（3.2ではさまざまなGDScriptの改善が計画されています）。

こんにちは私はちょうどこのアイテムに出くわし、私の2セントを投入したかったです。

スマートスライスが何らかの形で個々の配列インデックスのパフォーマンスに影響を与える場合は、おそらくそれを避ける必要があります。 GDScriptでオブジェクトを反復処理することは、ちょっとした構文上の問題のためにパフォーマンスのペナルティを受けるにはあまりにも一般的なユースケースです。

私はすべてシンタックスシュガーが好きですが、平均的なゲーム開発者は、角かっこ、中括弧、括弧などに関連するものがGDScriptに実装されている場合、原子性と速度に関連する合理的な期待を持っていると思います。 その契約が破られた場合、GDNativeに何かをいつプッシュする必要があるかという問題ははるかに曖昧になります。 良い「sub_array（）」または「slice（）」関数が最善の方法だと思います。そうすれば（私は）既存の配列インデックスコードをそのままにしておくことができるからです。

これについて何か進展はありますか？ そのような方法を追加したPRを提出することができます。

それでもこれを実装したい。 連絡あった？

同じように、配列からサブセットを取得したいのですが、リスト内包表記とジェネリック配列をスライスする関数の両方が存在しないようです。 代替案は少し冗長なので、これらの1つをおそらく追加する必要があります。

@ bojidar-bg

これは、 rangeと＃4716の内包表記を使用して簡単に実行できることに注意してください。

var result = array[from:to:step]
# Is the same as:
var result = (array[i] for i in range(from,to,step))

内包表記がまだ含まれていないため、これは現在正しくありません。 現在の構文は次のとおりです。

var result = []
for i in range(from, to, step):
    result.append(array[i])

奇妙なことに、開発者は読みやすさの理由でこの機能（＃15222）のPRを拒否しました。

これについてはIRCで多くのコア開発者と話し合っており、リスト内包表記の構文はコードを非常に読みにくくし、GDScriptの設計原則に反して、単純で読みやすい構文にするという一般的な感覚があります。

個人的には、1行は3行よりも読みにくいと思います。 他の誰かがこの会話を再開すべきだと思いますか？

@SnailBones組み込みのArray.slice()メソッドがあることが、ここで最も理にかなっていると思います。 JavaScriptのような言語にも同様の機能があり、そこではうまく機能しているようです：slightly_smiling_face：

リスト内包表記はいいと思いますが、 Array.slice()を別にしておくと便利です。

現在、 Array.sliceメソッドに取り組んでいます

そのスライスオブジェクトが演算子として配列に供給される場合、または配列内のスライス関数の呼び出しがコンパイラでハードコーディングされる場合に、Pythonがどのように行うかなど、新しいVariantスライスタイプを作成する方が好ましいでしょうか？ sliceメソッドを実行し、 start:end:delta構文をgdscriptインデックスに追加することを検討しています。

スライスオブジェクトはより「Pythonic」になります;）ただし、ハードコードされたメソッド
ほぼ確実に速くなります。 いずれにせよ、
start：end ：delta構文
とても参考になります。

2019年7月20日土曜日午後6時5分[email protected]
書きました：

どのように新しいバリアントスライスタイプを作成する方が好ましいでしょうか
Pythonは、そのスライスオブジェクトが配列にフィードされる場所でそれを行います
演算子、または配列内のスライス関数の呼び出しが
パーサーにハードコーディングされていますか？ スライスメソッドを実行して、探しています
start：end ：delta構文をgdscriptインデックスに追加します。

—
コメントしたのでこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/godotengine/godot/issues/4715?email_source=notifications&email_token=ACJUE6P4A4CDQWUS4PSQQH3QAOK2JA5CNFSM4CEI4JPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/ACJUE6LOTSF243FOYXXR2SDQAOK2JANCNFSM4CEI4JPA
。

新しいスライスの種類のバリアントを使用することだけを考えています。なぜなら、スライスはパーサーのオペレーターであり、うまくいく可能性があるからです。しかし、私はその分野であまり経験がないので、gdscriptの1つがあれば素晴らしいと思います。開発者が加重します。

あなたはそれについて正しいです。 また、構文を再利用する方がはるかに簡単です。
後で他のもののための構造。 私にとって最良の選択肢のように思えます。

2019年7月20日土曜日午後11時28分[email protected]
書きました：

スライスするので、新しいスライスの種類のバリアントを使用することだけを考えています
パーサーのオペレーターになる可能性があり、それはよりうまくいくでしょう

—
コメントしたのでこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/godotengine/godot/issues/4715?email_source=notifications&email_token=ACJUE6IMHLNTYTVL7DITY2TQAPQX3A5CNFSM4CEI4JPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/ACJUE6P5Q55EXAWD6IB4ARDQAPQX3ANCNFSM4CEI4JPA
。

まだどこかにある古い実装を見つけることができるかどうかを確認します。 配列にインデックスを付けるために使用できるスライスオブジェクトがあるという点で、Pythonと同じように、90％完成し、動作しました。 arr [2：4] = [1,2,3]のようなものを実行することもできます。

時間の制約で作業をやめました

@brakhaneそれは非常に役に立ちます。 現在、新しいスライスオブジェクトバリアントタイプを追加しているところです。これにより、スライスの動作を操作と見なして、パーサーの変更をはるかに簡単にすることができます。

@creikeyそれは私のアプローチとかなり似ているように聞こえます。 godot GitHubリポジトリを削除しましたが、コードが他の場所に残っている可能性があります。 更新を投稿します

@creikey完全に最近ではないWIPを見つけました（このバージョンにはいくつかの機能が欠けている可能性があります）。 ここで見つけることができます： https ：//github.com/brakhane/godot/compare/4c4ab14..8a258c0

まだいくつかのバグがあります。たとえば、 https： //github.com/brakhane/godot/compare/4c4ab14..8a258c0#diff-7d521a4f767fb1ae3c908a20616084a4R1446「.start」ではなく「.end」と表示されていることに気づきました。

ご不明な点がございましたら、お気軽にお問い合わせください。 また、より新しいバージョンを見つけようとします。

私は今、包括的上限を持つ関数ではなくpythonic配列スライシングを追加すると、メソッドを追加するだけの場合と比較してほとんどメリットがなく、新しいバリアントを作成する必要があるため、不必要な複雑さが追加されると考えています。