PHP OOP MVC Project Structure












0














I'm a beginner in OOP and PHP frameworks. I used slim skeleton to create a project and designed the project in this way.



Project structure is given by the slim skeleton. Object Mappers are added by my self. I want to know whether this is a good practice. What I'm doing wrong here. Is there any better way to do this?



src
Controllers
PostController.php
CommentController.php
Models
Post.php
Comment.php
ObjectMappers
PostMapper.php
CommentMapper.php
Dependencies.php
Middleware.php
Routes.php
Settings.php


Model files are used to include queries for relevant class.
Mappers are the real classes that used to map query results to necessary class.
Controllers are for necessary operations before sending it to the user.



For example



Post.php



namespace AppModels;

class Post {

public function getPostById($id){

$sql = "SELECT `post_id`, `post` FROM `posts` WHERE `post_id` = :id";

try {
$stmt = $this->db->prepare($sql);
$stmt->execute(['id' => $id]);
$post = $stmt->fetchObject('AppObjectMappersPostMapper');
if($post){
return $post;
}
return false;
} catch (PDOException $e){
return false;
}
}

// Other queries
}


PostMapper.php



namespace AppObjectMappers;

class PostObject {

public $post_id;
public $post;

public function __construct(){}

public function getPost_id()
{
return $this->post_id;
}

public function setPost_id($post_id)
{
$this->post_id= $post_id;
return $this;
}

public function getPost()
{
return $this->post;
}

public function setPost($post)
{
$this->post= $post;
return $this;
}
}


PostController.php



namespace AppControllers;

class PostController {

public function ($request, $response, $args){

$postId = $request->getAttribute("post_id");

if(!$postId){
return $response->withJSON("post id not found");
}
$post = $this->Post->getPostById($postId);

if(!$post ){
return $response->withJSON("post not found");
}

return $response->withJSON($post);
}
}


This is only a sample code. There are lot of functionalities. I'm glad if you can help me. I'm trying to avoid ORMs here.









share







New contributor




tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.

























    0














    I'm a beginner in OOP and PHP frameworks. I used slim skeleton to create a project and designed the project in this way.



    Project structure is given by the slim skeleton. Object Mappers are added by my self. I want to know whether this is a good practice. What I'm doing wrong here. Is there any better way to do this?



    src
    Controllers
    PostController.php
    CommentController.php
    Models
    Post.php
    Comment.php
    ObjectMappers
    PostMapper.php
    CommentMapper.php
    Dependencies.php
    Middleware.php
    Routes.php
    Settings.php


    Model files are used to include queries for relevant class.
    Mappers are the real classes that used to map query results to necessary class.
    Controllers are for necessary operations before sending it to the user.



    For example



    Post.php



    namespace AppModels;

    class Post {

    public function getPostById($id){

    $sql = "SELECT `post_id`, `post` FROM `posts` WHERE `post_id` = :id";

    try {
    $stmt = $this->db->prepare($sql);
    $stmt->execute(['id' => $id]);
    $post = $stmt->fetchObject('AppObjectMappersPostMapper');
    if($post){
    return $post;
    }
    return false;
    } catch (PDOException $e){
    return false;
    }
    }

    // Other queries
    }


    PostMapper.php



    namespace AppObjectMappers;

    class PostObject {

    public $post_id;
    public $post;

    public function __construct(){}

    public function getPost_id()
    {
    return $this->post_id;
    }

    public function setPost_id($post_id)
    {
    $this->post_id= $post_id;
    return $this;
    }

    public function getPost()
    {
    return $this->post;
    }

    public function setPost($post)
    {
    $this->post= $post;
    return $this;
    }
    }


    PostController.php



    namespace AppControllers;

    class PostController {

    public function ($request, $response, $args){

    $postId = $request->getAttribute("post_id");

    if(!$postId){
    return $response->withJSON("post id not found");
    }
    $post = $this->Post->getPostById($postId);

    if(!$post ){
    return $response->withJSON("post not found");
    }

    return $response->withJSON($post);
    }
    }


    This is only a sample code. There are lot of functionalities. I'm glad if you can help me. I'm trying to avoid ORMs here.









    share







    New contributor




    tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
    Check out our Code of Conduct.























      0












      0








      0







      I'm a beginner in OOP and PHP frameworks. I used slim skeleton to create a project and designed the project in this way.



      Project structure is given by the slim skeleton. Object Mappers are added by my self. I want to know whether this is a good practice. What I'm doing wrong here. Is there any better way to do this?



      src
      Controllers
      PostController.php
      CommentController.php
      Models
      Post.php
      Comment.php
      ObjectMappers
      PostMapper.php
      CommentMapper.php
      Dependencies.php
      Middleware.php
      Routes.php
      Settings.php


      Model files are used to include queries for relevant class.
      Mappers are the real classes that used to map query results to necessary class.
      Controllers are for necessary operations before sending it to the user.



      For example



      Post.php



      namespace AppModels;

      class Post {

      public function getPostById($id){

      $sql = "SELECT `post_id`, `post` FROM `posts` WHERE `post_id` = :id";

      try {
      $stmt = $this->db->prepare($sql);
      $stmt->execute(['id' => $id]);
      $post = $stmt->fetchObject('AppObjectMappersPostMapper');
      if($post){
      return $post;
      }
      return false;
      } catch (PDOException $e){
      return false;
      }
      }

      // Other queries
      }


      PostMapper.php



      namespace AppObjectMappers;

      class PostObject {

      public $post_id;
      public $post;

      public function __construct(){}

      public function getPost_id()
      {
      return $this->post_id;
      }

      public function setPost_id($post_id)
      {
      $this->post_id= $post_id;
      return $this;
      }

      public function getPost()
      {
      return $this->post;
      }

      public function setPost($post)
      {
      $this->post= $post;
      return $this;
      }
      }


      PostController.php



      namespace AppControllers;

      class PostController {

      public function ($request, $response, $args){

      $postId = $request->getAttribute("post_id");

      if(!$postId){
      return $response->withJSON("post id not found");
      }
      $post = $this->Post->getPostById($postId);

      if(!$post ){
      return $response->withJSON("post not found");
      }

      return $response->withJSON($post);
      }
      }


      This is only a sample code. There are lot of functionalities. I'm glad if you can help me. I'm trying to avoid ORMs here.









      share







      New contributor




      tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.











      I'm a beginner in OOP and PHP frameworks. I used slim skeleton to create a project and designed the project in this way.



      Project structure is given by the slim skeleton. Object Mappers are added by my self. I want to know whether this is a good practice. What I'm doing wrong here. Is there any better way to do this?



      src
      Controllers
      PostController.php
      CommentController.php
      Models
      Post.php
      Comment.php
      ObjectMappers
      PostMapper.php
      CommentMapper.php
      Dependencies.php
      Middleware.php
      Routes.php
      Settings.php


      Model files are used to include queries for relevant class.
      Mappers are the real classes that used to map query results to necessary class.
      Controllers are for necessary operations before sending it to the user.



      For example



      Post.php



      namespace AppModels;

      class Post {

      public function getPostById($id){

      $sql = "SELECT `post_id`, `post` FROM `posts` WHERE `post_id` = :id";

      try {
      $stmt = $this->db->prepare($sql);
      $stmt->execute(['id' => $id]);
      $post = $stmt->fetchObject('AppObjectMappersPostMapper');
      if($post){
      return $post;
      }
      return false;
      } catch (PDOException $e){
      return false;
      }
      }

      // Other queries
      }


      PostMapper.php



      namespace AppObjectMappers;

      class PostObject {

      public $post_id;
      public $post;

      public function __construct(){}

      public function getPost_id()
      {
      return $this->post_id;
      }

      public function setPost_id($post_id)
      {
      $this->post_id= $post_id;
      return $this;
      }

      public function getPost()
      {
      return $this->post;
      }

      public function setPost($post)
      {
      $this->post= $post;
      return $this;
      }
      }


      PostController.php



      namespace AppControllers;

      class PostController {

      public function ($request, $response, $args){

      $postId = $request->getAttribute("post_id");

      if(!$postId){
      return $response->withJSON("post id not found");
      }
      $post = $this->Post->getPostById($postId);

      if(!$post ){
      return $response->withJSON("post not found");
      }

      return $response->withJSON($post);
      }
      }


      This is only a sample code. There are lot of functionalities. I'm glad if you can help me. I'm trying to avoid ORMs here.







      php object-oriented mvc slim





      share







      New contributor




      tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.










      share







      New contributor




      tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.








      share



      share






      New contributor




      tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      asked 6 mins ago









      tha07

      11




      11




      New contributor




      tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.





      New contributor





      tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






      tha07 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.



























          active

          oldest

          votes











          Your Answer





          StackExchange.ifUsing("editor", function () {
          return StackExchange.using("mathjaxEditing", function () {
          StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
          StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
          });
          });
          }, "mathjax-editing");

          StackExchange.ifUsing("editor", function () {
          StackExchange.using("externalEditor", function () {
          StackExchange.using("snippets", function () {
          StackExchange.snippets.init();
          });
          });
          }, "code-snippets");

          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "196"
          };
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function() {
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled) {
          StackExchange.using("snippets", function() {
          createEditor();
          });
          }
          else {
          createEditor();
          }
          });

          function createEditor() {
          StackExchange.prepareEditor({
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader: {
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          },
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          });


          }
          });






          tha07 is a new contributor. Be nice, and check out our Code of Conduct.










          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f210473%2fphp-oop-mvc-project-structure%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown






























          active

          oldest

          votes













          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes








          tha07 is a new contributor. Be nice, and check out our Code of Conduct.










          draft saved

          draft discarded


















          tha07 is a new contributor. Be nice, and check out our Code of Conduct.













          tha07 is a new contributor. Be nice, and check out our Code of Conduct.












          tha07 is a new contributor. Be nice, and check out our Code of Conduct.
















          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.





          Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


          Please pay close attention to the following guidance:


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f210473%2fphp-oop-mvc-project-structure%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          Morgemoulin

          Scott Moir

          Souastre