0

下位互換性の問題のため、一方が他方のエイリアスである2つの関数が必要です。

public function getMostPopularArticles(array $params = array())
{
    $this->getMostViewedArticles($params);
}

public function getMostViewedArticles(array $params = array())
{
    return $this->_response_request(".....");
}

getMostPopularArticles()古い関数のエイリアスである新しい関数です(そのうち削除されます)getMostViewedArticles()

私はそれを正しい方法でやっていますか?改善の余地はありますか?

getMostViewedArticles()(古い関数)が新しいパラメーターを使用して正しいデータを返すことを言及するgetMostPopularArticles()必要がありますが、ユーザーがコードを更新するために必要な時間を確保できるように、正しい名前()のエイリアスを作成する必要があります。古い名前の関数を完全に削除します。

4

2 に答える 2

2

やってみませんか:

public function getMostViewedArticles(array $params = array())
{
    return getMostPopularArticles($params);
}

このように、新しい変更はで行う必要があるだけですがgetMostPopularArticlesgetMostViewedArticlesレガシーコードで引き続き使用できます。

またgetMostPopularArticles、現在使用しているビジネスロジックはすべて必要です。これは、引き続き使用する新しい関数だからです。

public function getMostPopularArticles(array $params = array())
{
    return $this->_response_request(".....");
}
于 2012-08-22T15:05:42.447 に答える
0

非推奨の関数がある場合は、基本的にすべての新しいコードでそれを避ける必要があります。関数が削除されると、コードは(明らかに)壊れます。したがって、非推奨の関数を呼び出さないでください。代わりに、新しい呼び出しでコードをコピーして貼り付ける(または書き直す)必要があります。

于 2012-08-22T15:04:58.583 に答える