0

我目前正在尝试将我们的页面模板转换为 OOP，并且我感觉我为导航类提出的内容从根本上来说并不完全正确。

  • 这些方法中的一些真的属于扩展类drawNav吗？
  • getMenuBar-> generateMenuBar->结构是否generateMenuItems过多分解？它应该只是getMenuBar，并把所有的内容从generateMenuBar()和generateMenuItems()到getMenuBar()？

我调用类和方法的方式：

$drawNav = new drawNav();

$breadcrumbTrail = $drawNav->getBreadcrumbTrail();
$menuBar = $drawNav->getMenuBar();

编码：

class drawNav { 

            public function __construct() {
                //I’ve not nothing to put here…                                
            }

            public function getMenuBar()
            {
               return $this->generateMenuBar();                          
            }

            public function getBreadcrumbTrail()
            {
                return $this->generateBreadcrumbTrail();                           
            }

            public function getSocialMediaButtons()
            {
                return $this->generateSocialMediaButtons();    
            }

            private function generateSocialMediaButtons()
            {
               //return the HTML code with the social media buttons
            }

            private function generateMenuBar()
            {
               //Generate the HTML containing the menu and social media buttons
               $this->generateMenuItems();
               $this->getSocialMediaButtons();
               //Generate the HTML closing tags for the container for the menu and social media buttons
            }

            private function generateMenuItems()
            {
                //Call to the database and generate each individual menu item and its dropdown
            }

            private function generateBreadcrumbTrail()
            {
                //Generate the HTML containing the breadcrumb trail
                $this->generateBreadcrumbs();
                //Generate the HTML closing tags for the container for the breadcrumbtrail
               }

            private function generateBreadcrumbs()
            {
                //Call to the database and generate the pieces of the breadcrumb trail
            }
}
4

3 回答 3

6

getMenuBar-> generateMenuBar->结构是否generateMenuItems过多分解？

是的。绝对没有理由对包含私有方法的单行公共方法进行一对一映射。这不在任何人的最佳 OOP 实践列表中。

而不是这种奇怪：

        public function getSocialMediaButtons()
        {
            return $this->generateSocialMediaButtons();    
        }
        // ...
        private function generateSocialMediaButtons()
        {
           //return the HTML code with the social media buttons
        }

你应该这样做：

        public function getSocialMediaButtons()
        {
            //return the HTML code with the social media buttons 
        }

如果您担心能够在公共接口中混合和匹配私有方法，这在以后很容易重构。但是编写单行公共方法，其唯一目的是调用具有几乎完全相同名称的私有方法，这是一种巨大的代码气味。

否则，您的代码很好，但有一个警告：我希望您的“返回带有社交媒体按钮的 HTML 代码”正在呈现一些外部 HTML 模板文件，并且您没有在类内编写 HTML 内联。后端/前端逻辑的良好分离比代码部分的结构更重要；我宁愿看到清晰地分离业务/视图逻辑的过程代码，也不愿看到精心设计的将它们混合在一起的面向对象的代码。

于 2012-08-29T18:16:36.537 回答
1

我真的不认为你如何分离你的方法有什么大问题。我个人宁愿让类方法处理一些特定的操作，然后使用其他方法将“构建块”方法组合成更复杂的操作。例如，如果您需要更改面包屑数据库逻辑，则只需在该方法中更改它，并且该方法已从其他方法中抽象出来，足以使其不需要更改。

于 2012-08-29T18:19:08.877 回答
0

你所拥有的似乎很好。

我会质疑你是否有一个 Navigation 类，因为这都可以添加到其中，或者作为 Navigation 类的扩展，以保持清洁。

于 2012-08-29T18:22:22.247 回答