Длина метода составляет 27 линий (20 разрешено) ⇐ Php

[Anonymous]

В то время как я писал метод внутри класса, используя Netbeans 25, я заметил предупреждение, которое предупредило меня «длина метода - 27 строк (20 разрешено)», я исследовал причину этого предупреждения, и кажется, что он был разработан, чтобы сделать методы более мелкими и читаемыми, но у меня есть большие сомнения, причина, по которой мой метод - это 27 строк, потому что я более четко и понятно, что я могу получить познание, и понятно, что я могу получить познание. Весь код с использованием меньшего количества строк, приводящих к более компактному, но менее читаемому, тогда это правило уменьшенного числа строк кода будет контрпродуктивным, а также с 20 строками кода невозможно написать функцию со сложными алгоритмами, не делая ее нечитаемым. Несколько упорядоченных строк.

Код: Выделить всё

public function SetData($TypeData, $DataContent){
$FunSuc = 1;
if (strtolower(trim($TypeData) == "file name")){
//File name will be filled
$this->FileName = $DataContent;
$FunSuc = 0;
}
if (strtolower(trim($TypeData) == "file type")){
$this->FileType = $DataContent;
//File Type will be filled
$FunSuc = 0;
}
if (strtolower(trim($TypeData) == "file path")){
//File Path will be set
$this->FilePath = $DataContent;
$FunSuc = 0;
}
if (strtolower(trim($TypeData) == "file content")){
//File content will be filled
$this->FileContent = $DataContent;
$FunSuc = 0;
}
if ($FunSuc == 0){
//will check if some of the type are found
if($this->Verbouse > 0){
echo "Data Storing Success";
}

}else{
if($this->Verbouse > 0){
echo "Data Storing Failed, unrecognized type data";
}
}
return $FunSuc;
}

и это компактный образец кода

Код: Выделить всё

public function SetData($TypeData, $DataContent) {
$FunSuc = 1;
$TypeData = strtolower(trim($TypeData));
if ($TypeData == "file name") $this->FileName = $DataContent;
elseif ($TypeData == "file type") $this->FileType = $DataContent;
elseif ($TypeData == "file path") $this->FilePath = $DataContent;
elseif ($TypeData == "file content") $this->FileContent = $DataContent;
$FunSuc = ($TypeData == "file name" || $TypeData == "file type" || $TypeData == "file path" || $TypeData == "file content") ? 0 : $FunSuc;
if ($this->Verbouse > 0) echo $FunSuc == 0 ? "Data Storing Success" : "Data Storing Failed, unrecognized type data";
return $FunSuc;
}

Как вы можете видеть из двух примеров, компактный код менее читабелен, но уважает запрос NetBeans, чтобы иметь уменьшенное количество строк, напротив, код, написанный с большим количеством строк с различными комментариями, нарушает правило максимального числа строк.>

Подробнее здесь: https://stackoverflow.com/questions/795 ... 20-allowed