Как избежать дублирования кода в Си

WoodHead · Регистрация: 15.01.2011

Студворк — интернет-сервис помощи студентам

Допустим, есть такие структуры данных:

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
enum Color{
    Red,
    Blue,
    Black,
    White
};
 
struct Car{
    char* model;
    enum Color color;
    double engineCapacity;
};
 
struct Cars{
    struct Car *garage;
    size_t count;
};

Чтобы найти элемент с соответствующим полем color и engineCapacity в списке типа Cars есть две функции:

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
struct Car *findByColor(struct Cars cars, enum Color color){
    while(cars.count-- > 0){
        if (cars.garage->color == color){
            return cars.garage;
        }
        cars.garage++;
    }
    return NULL;
}
 
struct Car *findByEngineCapacity(struct Cars cars, double engineCapacity){
    while(cars.count-- > 0){
        if (cars.garage->engineCapacity == engineCapacity){
            return cars.garage;
        }
        cars.garage++;
    }
    return NULL;
}

Видно, что функции практически повторяют друг друга. Подскажите, пожалуйста, как избежать дублирования кода в этом случае?

@politoto · 23.09.2018, 15:42

Функции маленькие, в данном случае дублирования лучше не избегать. Исходник при желании можно сократить, используя макросы.

Можно было бы передавать в findXXX() указатель на функцию, сравнивающую Cars с искомым образцом.

WoodHead · 23.09.2018, 17:27 **[ТС]**

politoto, спасибо.
Вот так сделал:

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
#include <stdio.h>
#include <stdlib.h>
 
enum Color{
    Red,
    Blue,
    Black,
    White
};
 
struct Car{
    char* model;
    enum Color color;
    double engineCapacity;
};
 
struct Cars{
    struct Car *garage;
    size_t count;
};
 
int equalColor(struct Car *car, uintptr_t color){
    return car->color == (enum Color) color;
}
 
int equalEngineCapacity(struct Car *car, uintptr_t engineCapacity){
    return car->engineCapacity == *((double *)engineCapacity);
}
 
struct Car *findByField(struct Cars cars, int (*equal)(struct Car *, uintptr_t), uintptr_t value){
    while(cars.count-- > 0){
        if ((*equal)(cars.garage, value)){
            return cars.garage;
        }
        cars.garage++;
    }
    return NULL;
}
 
int main() {
    struct Car car1 = {"ford", Red, 2.5};
    struct Car car2 = {"reno", Black, 1.8};
    struct Cars cars;
    cars.count = 2;
    cars.garage = (struct Car*) malloc(sizeof(struct Car) * cars.count);
    *cars.garage = car1;
    *(cars.garage + 1) = car2;
 
    double engineCapacity = 2.5;
    struct Car *car3 = findByField(cars, &equalEngineCapacity, (uintptr_t) &engineCapacity);
 
    enum Color color = Black;
    struct Car *car4 = findByField(cars, &equalColor, (uintptr_t) color);
 
    free(cars.garage);
    return 0;
}

Evg · 23.09.2018, 20:20

Сообщение от WoodHead

Подскажите, пожалуйста, как избежать дублирования кода в этом случае?

Присоединяюсь к мнению коллеги:

Сообщение от politoto

Функции маленькие, в данном случае дублирования лучше не избегать

Избегать дублирования кода надо тогда, когда это приносит очевидный профит. В твоём случае на одной чаше весов лежит как бы непродублированный код, на другой чаше весов - простота чтения кода. Конкретно в данном случае я бы выбрал именно вариант с простотой чтения

Хотя тот вариант, который сделал ты, он по своей сути правильный. В более сложном случае часто придётся применять подобную технику. Но более грамотно будет в findByField последним параметром всегда передавать указатель. Чтобы для всех типов данных работало симметрично. Т.е. тебе на свою реализацию следует смотреть в первую очередь как на тренировку для будущего, чем на правильное решение конкретно в данном коде

Новые блоги и статьи Все статьи Все блоги /
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .	Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .
Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .

@politoto Модератор 2131 / 1000 / 170 Регистрация: 23.07.2018 Сообщений: 3,349 Записей в блоге: 3
	23.09.2018, 15:42
	Функции маленькие, в данном случае дублирования лучше не избегать. Исходник при желании можно сократить, используя макросы. Можно было бы передавать в findXXX() указатель на функцию, сравнивающую Cars с искомым образцом. 1