我有一个我正在研究的项目,我刚刚切换到PHPStorm,因为它开始变大,我需要一些重构工具。我注意到,由于变量未被声明为类型(就像它们在Java中一样),PHPStorm有时不知道在哪里查找变量名的方法调用。这是我的代码。
<?php
require_once "autoloader.php";
class User {
private $userID;
private $email;
private $encryptedPassword;
private $userDBWriter;
private $company;
private $companyInfoChangeRequest;
private $admin;
private $accountIsActive;
private $dealTracker;
private $changeRequestPending;
function __construct($email, $encryptedPassword) {
$this->email = $email;
$this->encryptedPassword = $encryptedPassword;
$this->userDBWriter = new UserDBWriter();
$this->admin = false;
$this->dealTracker = new DealTracker($this);
}
public function addUserToDB() {
$this->userDBWriter->addUserToDB($this);
}
public function setUserAsAdmin($adminStatus) {
$this->admin = (bool) $adminStatus;
}
public function userAccountActiveStatus($accountStatus) {
$this->accountIsActive = (bool) $accountStatus;
}
public function setUserID($userID) {
$this->userID = $userID;
}
public function getUserID() {
return $this->userID;
}
public function getEmail() {
return $this->email;
}
public function isAdmin() {
return $this->admin;
}
public function isAccountActive() {
return $this->accountIsActive;
}
public function getEncryptedPassword() {
return $this->encryptedPassword;
}
public function toArray() {
$userArray = array(
"id"=>$this->userID,
"email"=>$this->email,
"company_name"=>$this->getCompanyName(),
"business_type"=>$this->getBusinessType(),
"company_phone"=>$this->getCompanyPhone(),
"company_street"=>$this->getCompanyStreet(),
"company_city"=>$this->getCompanyCity(),
"company_zip"=>$this->getCompanyZip(),
"monday_hours"=>$this->getMondayHours(),
"tuesday_hours"=>$this->getTuesdayHours(),
"wednesday_hours"=>$this->getWednesdayHours(),
"thursday_hours"=>$this->getThursdayHours(),
"friday_hours"=>$this->getFridayHours(),
"saturday_hours"=>$this->getSaturdayHours(),
"sunday_hours"=>$this->getSundayHours(),
"store_image_path"=>$this->getStoreImagePath(),
"shop_description"=>$this->getShopDescription(),
"deals"=>$this->dealTracker->dealsToArray()
);
return $userArray;
}
public function addCompany(Company $company) {
$this->company = $company;
}
public function getCompanyName() {
return $this->company->getCompanyName();
}
public function getBusinessType() {
return $this->company->getBusinessType();
}
public function getCompanyPhone() {
return $this->company->getCompanyPhone();
}
public function getCompanyStreet() {
return $this->company->getCompanyStreet();
}
public function getCompanyCity() {
return $this->company->getCompanyCity();
}
public function getCompanyState() {
return $this->company->getCompanyState();
}
public function getCompanyZip() {
return $this->company->getCompanyZip();
}
public function getMondayHours() {
return $this->company->getMondayHours();
}
public function getTuesdayHours() {
return $this->company->getTuesdayHours();
}
public function getWednesdayHours() {
return $this->company->getWednesdayHours();
}
public function getThursdayHours() {
return $this->company->getThursdayHours();
}
public function getFridayHours() {
return $this->company->getFridayHours();
}
public function getSaturdayHours() {
return $this->company->getSaturdayHours();
}
public function getSundayHours() {
return $this->company->getSundayHours();
}
public function getStoreImagePath() {
return $this->company->getStoreImagePath();
}
public function getShopDescription() {
return $this->company->getShopDescription();
}
public function isBusinessVerified() {
return $this->company->getVerifiedBusiness();
}
public function setCompanyInfoChangeRequest($company) {
$this->companyInfoChangeRequest = $company;
}
public function submitCompanyInfoChangeRequest($company) {
$this->companyInfoChangeRequest = new CompanyInfoChangeRequest($this, $company);
$this->companyInfoChangeRequest->submitCompanyInfoChangeRequest();
}
public function cancelCompanyInfoChangeRequest() {
if ($this->changeRequestPending) {
$this->companyInfoChangeRequest->cancelRequest();
}
}
public function isCompanyChangeRequestPending() {
return $this->changeRequestPending;
}
public function approveCompanyInfoChangeRequest($company) {
$this->companyInfoChangeRequest->approveCompanyInfoChangeRequest($this);
}
public function setChangeRequestPending($isPending) {
$this->changeRequestPending = (bool) $isPending;
}
public function getChangeRequestCompanyName() {
return $this->companyInfoChangeRequest->getCompanyName();
}
public function getChangeRequestBusinessType() {
return $this->companyInfoChangeRequest->getBusinessType();
}
public function getChangeRequestCompanyPhone() {
return $this->companyInfoChangeRequest->getCompanyPhone();
}
public function getChangeRequestCompanyStreet() {
return $this->companyInfoChangeRequest->getCompanyStreet();
}
public function getChangeRequestCompanyCity() {
return $this->companyInfoChangeRequest->getCompanyCity();
}
public function getChangeRequestCompanyState() {
return $this->companyInfoChangeRequest->getCompanyState();
}
public function getChangeRequestCompanyZip() {
return $this->companyInfoChangeRequest->getCompanyZip();
}
public function getChangeRequestMondayHours() {
return $this->companyInfoChangeRequest->getMondayHours();
}
public function getChangeRequestTuesdayHours() {
return $this->companyInfoChangeRequest->getTuesdayHours();
}
public function getChangeRequestWednesdayHours() {
return $this->companyInfoChangeRequest->getWednesdayHours();
}
public function getChangeRequestThursdayHours() {
return $this->companyInfoChangeRequest->getThursdayHours();
}
public function getChangeRequestFridayHours() {
return $this->companyInfoChangeRequest->getFridayHours();
}
public function getChangeRequestSaturdayHours() {
return $this->companyInfoChangeRequest->getSaturdayHours();
}
public function getChangeRequestSundayHours() {
return $this->companyInfoChangeRequest->getSundayHours();
}
public function getChangeRequestStoreImagePath() {
return $this->companyInfoChangeRequest->getStoreImagePath();
}
public function getChangeRequestShopDescription() {
return $this->companyInfoChangeRequest->getShopDescription();
}
}
警告"Method not found in class..."
的一些方法是getCompanyName()
getBusinessType()
getCompanyPhone()
以及在$this->company
或{{1}上调用方法的任何其他方法}。
我知道为什么会这样。这是因为PHPStorm不知道变量是什么的实例。我注意到Method not found in class他们使用PHPDoc注释键入提示变量类型,以便PHPStorm知道在哪里查看。 我想要一个“非评论”解决方案,因为如果我以后发现评论,我可能会删除它。抱歉长班。我正在将项目导入PHPStorm,因此我可以开始进行一些重构。感谢。
答案 0 :(得分:27)
为了完整起见,并为SO用户提供可行的替代方案,我更喜欢使用PHPdocs执行此操作的基于注释的解决方案,因此在您的实例中我会这样做:
/**
* @var Company
*/
private $company.
当您编辑使用相关类的代码时,您可以获得记录良好的类的好处,并获得有意义的信息。我经常将phpdoc放在变量,构造方法和大多数公共业务方法上。系统地进行这项工作是一个很小的代价,可以通过大量的代码库和PHPStorm为您带来的好处付出代价。
找到详细信息答案 1 :(得分:4)
将$this->company = new Company;
添加到您的__construct
功能中。现在,phpStorm确切地知道$this->company
具有哪些函数。
这可能会使您的应用程序变得非常复杂,因为它会在Company::__construct
内调用User::__construct
- 有效地将其处理和内存负载加倍,因为从我所看到的,{{1}的实例{1}}已经存在。也许您不再需要在Company
类之外创建公司实例?
此外,由于您已经在内存中找到了User
个对象,因此规范化您的类并在“公司”类中放置(并使用)所有“公司”功能似乎更为实际,并限制将“用户”类添加到“用户”功能,只需使用$company
获取该信息,而不是使用循环$company->getCompanyName()
功能 - 仅仅调用$user->getCompanyName()
。
试一试,看看你的想法?
答案 2 :(得分:0)
遇到此问题的用户,如果文件长度过大(>5000行代码),2021.1版本之前的代码分析将停止工作。
如果您使用的是以前的版本,那么似乎只有重构代码以减少代码长度的唯一解决方案。