[Skript] このコードを短縮できませんか？

[りくとん2002 ID:f2f4ac0c]

コード: 全て選択

options:
   default : §d§lSOM §7§l≫ §a§l[N] §e§l%player%
   toprank : §d§lSOM §7§l≫ §b§l[TR] §e§l%player%

on join:
   if player has permission "default.chat":
      set the join message to " §d§lSOM §7§l≫ §a§l[N] §e§l%player%§a§lさんが§b§lログイン§a§lしました！"
      stop
   if player has permission "toprank.chat":
      set the join message to " §d§lSOM §7§l≫ §b§l[TP] §e§l%player%§a§lさんが§b§lログイン§a§lしました！"
      stop
on quit:
   if player has permission "default.chat":
      set the quit message to " §d§lSOM §7§l≫ §a§l[N] §e§l%player%§a§lさんが§c§lログアウト§a§lしました..."
      stop
   if player has permission "default.chat":
      set the quit message to " §d§lSOM §7§l≫ §b§l[TR] §e§l%player%§a§lさんが§c§lログアウト§a§lしました..."
      stop

これの§a§l[N],§a§l[TR]とこだけを変数化？したいのですが可能でしょうか？
具体的には
if player has permission "default.chat":
　A = §a§l[N]
if player has permission "toprank.chat":
　A = §b§l[TR]
　　　set the join message to " §d§lSOM §7§l≫ {@A} §e§l%player%§a§lさんが§b§lログイン§a§lしました！"

見ないな感じにしたいのですが、一部だけpermissionによって変わるようにしたいのですが可能でしょうか？
最悪permissionではなくても構いません

[りくとん2002 ID:f2f4ac0c]

追記: spigot1.8 skript 最新？(verわかんなくてすいません)

[mcgo ID:2a37f7a7]

これの§a§l[N],§a§l[TR]とこだけを変数化？したいのですが可能でしょうか？

コード: 全て選択

on join:
   if player has permission "default.chat":
      set {_temp} to "&a&l[N]"
   else if player has permission "toprank.chat":
      set {_temp} to "&b&l[TP]"
   set the join message to "&d&lSOM &7&l≫ %{_temp}% &e&l%player%&a&lさんが&b&lログイン&a&lしました！"

on quit:
   if player has permission "default.chat":
      set {_temp} to "&a&l[N]"
   else if player has permission "toprank.chat":
      set {_temp} to "&b&l[TP]"
   set the quit message to "&d&lSOM &7&l≫ %{_temp}% &e&l%player%&a&lさんが&c&lログアウト&a&lしました！"

りくとん2002さんのコードを省略すると上記のように出来ます。
ログイン・ログアウトした際に、{_temp}というローカル変数にプレイヤーのパーミッション毎に文字をセットして
それを最終的にメッセージにするというものです。
ローカル変数は一連の処理を終えると変数の中身が削除されるため、他に干渉しないという特徴があります。

一部だけpermissionによって変わるようにしたいのですが可能でしょうか？

一部だけというのはどの部分のことでしょうか？
具体的なものを答えていただけると回答が可能です。

[sympholy ID:1134fc5e]

以下の書き方が自分的にはいいと思います

コード: 全て選択

on join:
   if player has permission "toprank.chat":
      set {rank.%player%} to "&a&l[TR]"      
   else if player has permission "default.chat":
      set {rank.%player%} to "&a&l[N]"

   set the join message to "&d&lSOM &7&l≫ %{rank.%player%}% &e&l%player%&a&lさんが&b&lログイン&a&lしました！"
   
on quit:
   set the quit message to "&d&lSOM &7&l≫ %{rank.%player%}% &e&l%player%&a&lさんが&c&lログアウト&a&lしました..."
   clear {rank.%player%}


変数{rank.%player%}にパーミッションごとに変えたい文字列を格納しています.
その文字列をjoin と quit で使いまわすことで，パーミッションを増やした時に編集する箇所を減らしています．
最後にclear {rank.%player%}で変数の削除を行なっています.(特に保存する必要がない変数なので)

mcgoさんは{_変数名}とすることで，そのイベント内のみで動く変数にすることで，variables.csvへの書き込みを減らすように工夫をしています．(このskriptに限定すると，書き込みはjoinとquitのみなので，負荷はほぼ皆無ではありますが)負荷が少ないという面で，有効なコードです．

一方で，自分の提案するコードは，よりミスを少なくすることを意識して作っています．
全く同じ条件分岐を複数箇所で使うと，修正時にミスを起こしやすい・可読性の低下などの問題が起こります．
また，書き込みによる負荷も考えられないことからこのようなコードにしています．

パーミッションを追加したい場合

コード: 全て選択

on join:
   if player has permission "toprank.chat":
      set {rank.%player%} to "&a&l[N]"
   else if player has permission "middle.chat":
      set {rank.%player%} to "&a&l[MD]"      
   else if player has permission "default.chat":
      set {rank.%player%} to "&a&l[TR]"

   set the join message to "&d&lSOM &7&l≫ %{rank.%player%}% &e&l%player%&a&lさんが&b&lログイン&a&lしました！"
   
on quit:
   set the quit message to "&d&lSOM &7&l≫ %{rank.%player%}% &e&l%player%&a&lさんが&c&lログアウト&a&lしました..."
   clear {rank.%player%}

8/23 15:00 編集（勘違いが起こりそうな部分について加筆）

[mcgo ID:3214aafb]

コード: 全て選択

on join:
   if player has permission "toprank.chat":
      set {rank.%player%} to "&a&l[TR]"     
   else if player has permission "default.chat":
      set {rank.%player%} to "&a&l[N]"

   set the join message to "&d&lSOM &7&l≫ %{rank.%player%}% &e&l%player%&a&lさんが&b&lログイン&a&lしました！"
   
on quit:
   set the quit message to "&d&lSOM &7&l≫ %{rank.%player%}% &e&l%player%&a&lさんが&c&lログアウト&a&lしました..."
   clear {rank.%player%}

確かに僕のよりコードが省略されていて良いコードだと思いますが、通常の変数に%player%を使うと
例えclearをしていてもプレイヤーの数だけ変数が作られ、その数だけvariable.csvファイルに変数が書き込みされてしまいます。
しかし、ローカル変数を使うことでファイルへの書き込みがなくなり、さらに１つの変数で済むということです。
つまり、ローカル変数の方がわずかながら軽いということです。
これはりくとん2002さん次第ですが、これからSkriptで多くの通常の変数を扱う場合は
なるべく%player%の入った変数を使わないようにしたほうがいいので、少ししかSkriptを使わないのであれば
どちらのコードを使っても問題ありませんが、Skriptをかなり使う場合は僕の提案したコードをりくとん2002さんにおすすめします。

[りくとん2002 ID:f2f4ac0c]

sympholyさんのコードがかなりイメージしていたのと近いので使わさせていただきます!ありがとうございました