質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.06%

PHPで数字のデータを文字列変換させるには

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 405

score 7

実現したいこと

PHPで登録したレシピを一覧表示するシステムを作っています。
DBに登録した材料(Integer型)をblade上に文字列で表示しなおす処理を実現したいです。

アドバイスをいただけると幸いです。
よろしくお願いいたします。

前提

値/材料名
1 => 'たまねぎ',  
2 => 'にんじん',  
4 => 'たまご',  
8 => 'きゃべつ',
が材料の変換データです。

登録済のDBに'1'が登録されていたら「たまねぎ」を、
'3'が登録されていたら「たまねぎ、にんじん」、
'7'が登録されていたら「たまねぎ、にんじん、たまご」
のように、値の合計に該当する材料の文字列をblade上に出力したいです。

該当のソースコード

RecipesController.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

use App\Recipe;

class RecipesController extends Controller
{
    /**
     * index()のみ記載
     */

    public function index()
    {
        $recipes = Recipe::all();
        $ingredients = new Ingredients($recipes);

        return view('recipes.index', [
            'recipes' => $recipes,   
            'ingredients' => $ingredients,    //材料クラスのインスタンスを渡す
        ]);
    }
<?php

namespace App;

  //材料の文字列変換をさせるための配列定数(定義の仕方に誤りがあればすいません)
    define('INGREDIENTS', [
              1 => 'たまねぎ',  
              2 => 'にんじん',  
              4 => 'たまご',  
              8 => 'きゃべつ',  
    ]);

//材料クラス
class Ingredients
{
    public $id;//レシピテーブルに登録されているID
    public $ingredients;////レシピテーブルに登録されている材料(integer型)

    //コンストラクタ
    function __construct($recipes)
    {

     foreach($recipes as $recipe){
           $this->id = $recipe->$id;
      $this->ingredients = $recipe->$ingredients;//ここを数字から文字列に変換させたい
        }
    }

    function getAllRecipes()//ここでコントローラーにすべてのレシピデータを渡したい
    {
   
   foreach($recipes as $recipe){
        return $this->$recipe;
     }

  }
}


index.blade.php

@foreach($recipes as $recipe)
   @if(count($recipes) > 0)
    <p>{{ $recipe->id}}</p>//レシピID
    <p>{{ $recipe->ingredients</p>//最終的に材料の文字列を表示させたい(3であれば「たまねぎ、にんじん」と表示させたい)
   @endif
@endforeach

試したこと

配列定数の定義の仕方は
https://www.php.net/manual/ja/language.constants.syntax.php

を参考にさせていただきました。

補足情報(FW/ツールのバージョンなど)

php 7.2.22
Laravel 5.5

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2019/12/28 14:49

    現在のコードにおける課題は何でしょうか。

    下記が書き方間違ってる、くらいしかコードからは読み取れませんけども。
    ><p>{{ $recipe->ingredients</p>

    キャンセル

回答 2

checkベストアンサー

+3

そもそもこの設計が根本的におかしい。やめたほうがいい。

登録済のDBに'1'が登録されていたら「たまねぎ」を、
'3'が登録されていたら「たまねぎ、にんじん」、
'7'が登録されていたら「たまねぎ、にんじん、たまご」

材料のデータはマスターデータとして別テーブルに定義。
RecipeとIngredientは多対多のリレーションで繋げる。
それだけで解決すること。

Laravelでdefine()使うことはない。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/12/28 15:39

    その前に define に配列登録できるようになったの?

    キャンセル

  • 2019/12/28 15:43 編集

    @asahina1979
    php7から配列定義できます。

    @kyontama
    kawax さんと同意見です。
    レシピにおける材料って、DBに登録すべきだと思います。
    レシピを追加するたびに材料を追加する必要があるときに、わざわざファイルを編集するんですか?
    そんな設計はしませんよ。

    キャンセル

  • 2019/12/30 14:33

    >kawaxさん

    ご回答ありがとうございます。
    多対多でやり直します。

    >Kosuke_Shibuyaさん
    材料をマスタデータにしなおすことにしました。
    ありがとうございます。

    キャンセル

+1

    define('INGREDIENTS', [
1 => 'たまねぎ',  
2 => 'にんじん',  
4 => 'たまご',  
8 => 'きゃべつ',  
]);

これくらいの内容であればconfigで定義して持ってきたほうが良いと思います。
defineで定義するのはフレームワークの利点ほとんど活かせてません。
というか、望ましいのはDBにマスタ作ってIDから名称引くやり方ですけど。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/12/30 14:34

    >m.ts10806さん

    ありがとうございます。
    マスタデータに材料テーブルを作ることにしました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.06%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る