---
title: "Встраивание метода"
type: "refactoring-technique"
slug: "inline-method"
url: "http://localhost:3000/ru/inline-method.md"
category: "Составление методов"
description: "Проблема: Стоит использовать в том случае, когда тело метода очевиднее самого метода. Решение: Замените вызовы метода его содержимым и удалите сам метод."
languages: ["java", "csharp", "php", "python", "typescript"]
---
# Встраивание метода

> Проблема: Стоит использовать в том случае, когда тело метода очевиднее самого метода. Решение: Замените вызовы метода его содержимым и удалите сам метод.

## Problem

Стоит использовать в том случае, когда тело метода очевиднее самого метода.

## Solution

Замените вызовы метода его содержимым и удалите сам метод.

## Why Refactor

Основаная причина — тело метода состоит из простого делегирования к другому методу. Само по себе такое делегирование — не проблема. Но если таких методов довольно много, становится очень легко в них запутаться.

Зачастую методы не бывают слишком короткими _изначально_, а становятся такими в результате изменений в программе. Поэтому не стоит бояться избавляться от ставших ненужными методов.

## Benefits

* Минимизируя количество бесполезных методов, мы уменьшаем общую сложность кода.

## How to Refactor

1. Убедитесь, что метод не переопределяется в подклассах. Если он переопределяется, воздержитесь от рефакторинга.
2. Найдите все вызовы метода. Замените эти вызовы содержимым метода.
3. Удалите метод.
## Relations

**Opposite refactorings**

- [Извлечение метода](/ru/extract-method.md)

**Eliminates smells**

- [Теоретическая общность](/ru/smells/speculative-generality.md)

## Code Examples

### java

```java
// Before
class PizzaDelivery {
  // ...
  int getRating() {
    return moreThanFiveLateDeliveries() ? 2 : 1;
  }
  boolean moreThanFiveLateDeliveries() {
    return numberOfLateDeliveries > 5;
  }
}

// After
class PizzaDelivery {
  // ...
  int getRating() {
    return numberOfLateDeliveries > 5 ? 2 : 1;
  }
}
```

### csharp

```csharp
// Before
class PizzaDelivery 
{
  // ...
  int GetRating() 
  {
    return MoreThanFiveLateDeliveries() ? 2 : 1;
  }
  bool MoreThanFiveLateDeliveries() 
  {
    return numberOfLateDeliveries > 5;
  }
}

// After
class PizzaDelivery 
{
  // ...
  int GetRating() 
  {
    return numberOfLateDeliveries > 5 ? 2 : 1;
  }
}
```

### php

```php
// Before
function getRating() {
  return ($this->moreThanFiveLateDeliveries()) ? 2 : 1;
}
function moreThanFiveLateDeliveries() {
  return $this->numberOfLateDeliveries > 5;
}

// After
function getRating() {
  return ($this->numberOfLateDeliveries > 5) ? 2 : 1;
}
```

### python

```python
// Before
class PizzaDelivery:
    # ...
    def getRating(self):
        return 2 if self.moreThanFiveLateDeliveries() else 1
  
    def moreThanFiveLateDeliveries(self):
        return self.numberOfLateDeliveries > 5

// After
class PizzaDelivery:
  # ...
  def getRating(self):
    return 2 if self.numberOfLateDeliveries > 5 else 1
```

### typescript

```typescript
// Before
class PizzaDelivery {
  // ...
  getRating(): number {
    return moreThanFiveLateDeliveries() ? 2 : 1;
  }
  moreThanFiveLateDeliveries(): boolean {
    return numberOfLateDeliveries > 5;
  }
}

// After
class PizzaDelivery {
  // ...
  getRating(): number {
    return numberOfLateDeliveries > 5 ? 2 : 1;
  }
}
```