グループに登録したユーザーの一覧を表示するプログラム

Question

今回の実装はhamlを用いています。
登録されたユーザーは"@group.users"で受け取っています。
left-header__membersの中にグループに登録したユーザーの名前が一覧で表示されるように実装を行うプログラムになります。
現在理解ができる範囲の中で実装を行いました。
登録されるユーザーは複数になるためにeachを使いましたが正確な実装方法がこの先わからずご教授頂けないでしょうか。

もしグループに登録されているユーザーがTanaka, Sato, Suzukiの3人の場合はブラウザ上に下記の画像のように表示がされるようになっております。

もし情報に不備などがありましたら教えてください。

#現在の実装コード

        .left-header__members
          Member：
          %ul.
          @group.users.each
          %li= user.name

Answer 1

何か困っていることがあるのでしょうか？
特に動作に問題がないかのような印象を受けました。

もしそうではなく、何か実現できなくて困っているようであれば、その旨を記載しないと、どなたからも回答が来ないのではないかと思います。

---

あくまで、実装コードをより一般的に修正する方法だけ説明すると、 ul を使うのであれば次のようにするのが一般的でしょう

.left-header__members
  Member：
  %ul
    - @group.users.each do |user|
      %li= user.name

こうすることで、liタグがulタグの中に納まるので、本来のul, liの動きが活かされると思います。

要注意ポイント

• hamlでは、インデントの数に特に注意しましょう
• hamlでrubyのメソッドを使用し、しかし画面上には表示しない時は、'-' と半角スペースを行の先頭に付けましょう

参考になる記事

Hamlの書き方

---

おまけ

画像で貼っていただいた見た目を維持するのであれば、次のようになるかと思います。

.left-header__members
  Member：
  - @group.users.each_with_index do |user, i|
    - unless i.zero?
      ,
    = user.name

','の後ろに半角スペースを入れられるか正直自信がないので、それは他の方の回答を待ちたいと思います...
helperとかを使えばできると思いますが、まだそのレベルには達していないように感じたので、記載しません。

Comments

[_chii]

この度は有難うございました。