お世話になります。

ECサイトのユーザーのログイン状態に応じてナビボタンの表示を切り替えるために、PHPのsession変数の有無によって、HTML内の表示内容を切り替えようとしているのですが、現状では画像の通りクラスファイルのクラスメソッド内に<nav>タグを書いてしまっており、MVC的に好ましくありません。

理想的には、HTML内に直接<nav>を書いて、session変数の有無（ユーザーのログイン状態）に応じて2種類の<nav>分岐処理させたいのですが、そのためにはどういった方法が考えられますか？

●現状
sessionに関するクラスファイル内にcreateNavButtonというクラスメソッドを用意して、その中でsession変数の有無によって２種類の<nav>タグをif文で分岐し、メソッドの呼び出し元のphpファイル内に表示させている。

PHP
1<?php
2
3class Session
4{
5    public function createNavButtonForController()
6    {
7        session_start();
8        session_regenerate_id(true);
9        if(isset($_SESSION['member_login'])==false)
10        {
11	        print '<nav>';
12            print   '<ul>';
13            print     '<li><a href="./shopTop.php">Top</a></li>';
14            print     '<li><a href="../view/about.html">About</a></li>';
15	        print     '<li><a href="../view/shop_cartlook_view.php">Cart</a></li>';
16	        print     '<li><a href="../view/member_login.html">Login</a></li>';
17            print   '</ul>';
18            print '</nav><br><br>';
19	        print '<a href="../view/member_login.html">会員様はこちらからログインできます</a><br>';
20	        print '<br>';
21        }
22        else
23        {
24	        print '<nav>';
25            print   '<ul>';
26            print     '<li><a href="./shopTop.php">Top</a></li>';
27            print     '<li><a href="../view/about_logout.html">About</a></li>';
28	        print     '<li><a href="../view/shop_cartlook_view.php">Cart</a></li>';
29	        print     '<li><a href="../view/member_logout_view.php">Logout</a></li>';
30            print   '</ul>';
31            print '</nav><br><br>';
32	        print 'いらっしゃいませ  ';
33	        print $_SESSION['member_name'];
34	        print ' 様  ';
35	        print '<a href="../view/member_logout_view.php">ログアウト</a><br>';
36	        print '<br>';
37        }
38    }
39
40以下、略

php
1<?php 
2
3try
4{
5
6require_once dirname(__FILE__).'/../model/Session.class.php'; 
7require_once dirname(__FILE__).'/../model/connectMySQL.class.php'; 
8require_once dirname(__FILE__).'/../model/Item.class.php'; 
9
10// ログイン状態の確認
11$sessionCheck = new Session();
12$sessionCheck->createNavButtonForController();
13
14// データベースの接続
15$connect = new connectMySQL();
16$stmt = $connect->prepareAndExcuteSqlForItemList();
17
18$item = new Item();
19$item->dispItemlist($stmt);
20
21}
22catch (Exception $e)
23{
24	 print 'ただいま障害により大変ご迷惑をお掛けしております';
25	 exit();
26}
27
28?>
29
30<!DOCTYPE html>
31<html>
32<head>
33	<meta charset="UTF-8">
34	<title>ArmShop</title>
35	<link rel="stylesheet" href="../css/normalize.css">
36	<link rel="stylesheet" href="../css/shop_list.css?20190901">
37</head>
38<body>
39<div id = "wrapper">
40
41</div>
42</body>
43</html>

m.ts10806

2019/09/09 18:56

うーん。何を優先させたいですか？「MVC的に好ましくない」を追求すると丸っと作り直ししたほうが良さそうに思います。設計も含めて。

masa9216

2019/09/09 19:50

コメントありがとうございます。フレームワーク無しで既存のECサイトをMVC化しようとしており、現在作り直している最中なのですが、まずはタイトルの通り、「PHPのsession変数の有無によって、HTML内の表示内容を切り替えたい」を優先させたいです。

退会済みユーザー

2019/09/09 22:48

フレームワークを使わずに実装した末路 https://github.com/EC-CUBE/eccube-2_13

m.ts10806

2019/09/10 00:09

MVCしようと思ったらオレオレでも良いのでフレームワーク化はマストだと思います。オブジェクト指向の本質は役割分担にあるので、どれだけ細分化できるかがキーになってきます。このあたりは読んでおいたほうが良いです。 https://qiita.com/tadsan/items/fb496e450fc27c8c4494 https://qiita.com/rana_kualu/items/95f0c8be51e8665015d5

行動規範の内容に同意します

回答1件

ベストアンサー

そもそもの php の基礎から学習し直してください。
EC サイト構築に十分なレベルのスキルがありません。

ざっと気になったところを記述しておきます。
・意味なくゆるい比較を使用している
・php の変数を html として出力する時には必ずエスケープ
・session_regenerate_id の使用箇所がおかしい
・try-catch のくくる範囲がおかしい
・class Session に createNavButtonForController() はありえないと思う。設計からやり直しが必要かと。