Линковщик студии суёт нос не туда куда надо

@Gepar · Регистрация: 01.07.2009

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

Хочу разбить код на .h и .cpp файл, но эта, самка собаки, линковщик студии начинает творить непотребства.
Пока весь код в .h файлах всё хорошо. Как есть сейчас:
Colors. h - пара функций для изменения цвета в консоли. Код заключён #ifndef COLORS_H #endif
List.h - класс-список с набором функций описанных прямо в классе List (те неявный инлайн у всего). Код заключён в #ifndef LIST_H #endif + в начале кода есть #include "Colors.h"
Main.cpp - обычный main, в начале есть подключение списка #include "List.h"
Студия линкует всё правильно.

Теперь я добавляю List.cpp файл и выношу туда одну функцию из класса List что описан в List.h и тут же студия мне говорит что она два раза сунула SetColor и не знает какой выбрать:
Ошибка 1 error LNK2005: "void __cdecl SetColor(enum ConsoleColor,enum ConsoleColor)" (?SetColor@@YAXW4ConsoleColor@@0@Z) єцх юяЁхфхыхэ т List.obj
Ошибка 2 error LNK2005: "void __cdecl SetDifColors(int)" (?SetDifColors@@YAXH@Z) єцх юяЁхфхыхэ т List.obj

Как она так умудрилась? Я там помню что есть вариант что если вынести все функции из .h в .cpp файл то якобы всё образумится, но мне это не надо делать, мне не нужно все функции выносить в cpp файл. Any ideas ?

@Kuzia domovenok · 09.12.2012, 23:58

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

Ошибка 1 error LNK2005: "void __cdecl SetColor(enum ConsoleColor,enum ConsoleColor)" (?SetColor@@YAXW4ConsoleColor@@0@Z) єцх юяЁхфхыхэ т List.obj

У тебя что? Студия русифицирована? Зря. Очень зря. Знатоком английского, чтоб разбираться в ней быть не надо. А подобные сообщения гуглить и вообще искать информацию о студии на порядок тяжелее!

@kravam · 10.12.2012, 00:18

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

Теперь я добавляю List.cpp файл и выношу туда одну функцию из класса List что описан в List.h

То есть тут перегрузкой не пахнет; но

Я конечно небольшой специалист во всём этом, но я все глаза просмотрел и не увидел подтверждения того, что ты функцию определил один раз, как ты сказал. То, что похоже на прототипы функции, похоже на РАЗНЫЕ прототипы функции.

Короче код нужен.

@Gepar · 10.12.2012, 00:34 **[ТС]**

Kuzia domovenok, а причём тут язык, ты думаешь что в gcc ошибки линковщика будут информативнее? Она же один фиг создаёт временные обжекты которые обзывает по своему (имя функции + какой-то псевдо случайный набор символов), ну а когда у неё потом что-то не сходится то выбрасывает имена этих уже "переработанных" функций в обжектах которые она и не смогла слинковать, gcc делает то же самое и ругается при ошибках линковки похоже. По обрывкам слов можно понять на какие функции она ругается что она два раза включила.

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

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

Содержимое Colors.h (если чем-то поможет). Содержимое остального не буду приводить так как там больно много всего, толку особо не будет, обычный class и обычный мейн, только большой

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
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
#ifndef COLORS_H
#define COLORS_H
 
#include<windows.h>
 
//коды цветов
enum ConsoleColor
{
        Black         = 0,
        Blue          = 1,
        Green         = 2,
        Cyan          = 3,
        Red           = 4,
        Magenta       = 5,
        Brown         = 6,
        LightGray     = 7,
        DarkGray      = 8,
        LightBlue     = 9,
        LightGreen    = 10,
        LightCyan     = 11,
        LightRed      = 12,
        LightMagenta  = 13,
        Yellow        = 14,
        White         = 15
};
 
//возможность установки цвета зная заранее его код
void SetColor(ConsoleColor text, ConsoleColor background = White)
{
    static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleTextAttribute(hStdOut, (WORD)((background << 4) | text));
}
 
//доп. возможность установкци цвета. Её отличие в том что функция
//выбирает цвета максимально отличающиеся для номеров что рядом (в отличии от распред. цветов в табличке)
void SetDifColors(int nColor)
{
    ConsoleColor color;
    switch(nColor)
    {
    case 0:
        color = Black;
    case 1:
        color = Red;
        break;
    case 2:
        color = Green;
        break;
    case 3:
        color = Blue;
        break;
    case 4:
        color = Yellow;
        break;
    case 5:
        color = LightGray;
        break;
    case 6:
        color = Black;
        break;
    case 7:
        color = Cyan;
        break;
    case 8:
        color = LightRed;
        break;
    case 9:
        color = LightCyan;
        break;
    case 10:
        color = Brown;
        break;
    case 11:
        color = LightGreen;
        break;
    case 12:
        color = LightBlue;
        break;
    case 13:
        color = DarkGray;
        break;
    case 14:
        color = Magenta;
        break;
    case 15:
        color = White;
        break;
    default:
        color = Black;
        break;
    }
    SetColor(color);
}
#endif

Меня больше удивляет почему она начинает два раза в list их включать. Это получается что у меня при выносе кода из list.h в list.cpp часть кода использующего colors.h находится в list.h, а часть в list.cpp после чего линковщик суёт нос не туда куда надо и подсовывает в list.obj определение colors.h для list.h, а потом засовывает определение colors.h для list.cpp. Другого объяснения я не нахожу. Вопрос в том КАКОГО ФИГА сунуть colors.h в list.cpp когда я не прошу этого (в list.cpp есть лишь include list.h), что за медвежья услуга

Добавлено через 4 минуты
Сделал глупое предположение что static элемент в colors.h сбивал линковщик, но нет, если убрать static у HANDLE hStdOut то ничего не поменяется

go · 10.12.2012, 00:34

Gepar, .h только прототип.

@Kuzia domovenok · 10.12.2012, 00:37

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

Меня больше удивляет почему она начинает два раза в list их включать. Это получается что у меня при выносе кода из list.h в list.cpp часть кода использующего colors.h находится в list.h, а часть в list.cpp после чего линковщик суёт нос не туда куда надо и подсовывает в list.obj определение colors.h для list.h, а потом засовывает определение colors.h для list.cpp. Другого объяснения я не нахожу. Вопрос в том КАКОГО ФИГА сунуть colors.h в list.cpp когда я не прошу этого (в list.cpp есть лишь include list.h), что за медвежья услуга

она включает всё это в каждый файл, где прописан этот инклуд. Если у тебя хотя бы два файла, в которых есть #include "colors.h", будет ошибка.

оставь лишь это

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
#ifndef COLORS_H
#define COLORS_H
//коды цветов
enum ConsoleColor
{
        Black         = 0,
        Blue          = 1,
        Green         = 2,
        Cyan          = 3,
        Red           = 4,
        Magenta       = 5,
        Brown         = 6,
        LightGray     = 7,
        DarkGray      = 8,
        LightBlue     = 9,
        LightGreen    = 10,
        LightCyan     = 11,
        LightRed      = 12,
        LightMagenta  = 13,
        Yellow        = 14,
        White         = 15
};
//возможность установки цвета зная заранее его код
void SetColor(ConsoleColor, ConsoleColor = White);
//доп. возможность установкци цвета. Её отличие в том что функция
//выбирает цвета максимально отличающиеся для номеров что рядом (в отличии от распред. цветов в табличке)
void SetDifColors(int nColor);
#endif

А реализацию функций вынеси в СРР файл. Сolors.cpp

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
58
59
60
61
62
63
64
65
66
67
68
69
 #include<windows.h>
#include "Colors.h"
//возможность установки цвета зная заранее его код
void SetColor(ConsoleColor text, ConsoleColor background = White)
{
    static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleTextAttribute(hStdOut, (WORD)((background << 4) | text));
}
 
//доп. возможность установкци цвета. Её отличие в том что функция
//выбирает цвета максимально отличающиеся для номеров что рядом (в отличии от распред. цветов в табличке)
void SetDifColors(int nColor)
{
    ConsoleColor color;
    switch(nColor)
    {
    case 0:
        color = Black;
    case 1:
        color = Red;
        break;
    case 2:
        color = Green;
        break;
    case 3:
        color = Blue;
        break;
    case 4:
        color = Yellow;
        break;
    case 5:
        color = LightGray;
        break;
    case 6:
        color = Black;
        break;
    case 7:
        color = Cyan;
        break;
    case 8:
        color = LightRed;
        break;
    case 9:
        color = LightCyan;
        break;
    case 10:
        color = Brown;
        break;
    case 11:
        color = LightGreen;
        break;
    case 12:
        color = LightBlue;
        break;
    case 13:
        color = DarkGray;
        break;
    case 14:
        color = Magenta;
        break;
    case 15:
        color = White;
        break;
    default:
        color = Black;
        break;
    }
    SetColor(color);
}

- ошибка исчезнет

@kravam · 10.12.2012, 00:41

Ничё не понимаю... А чё, так нельзя упростить код? Разве это упрощение повлияет сколько-нибудь на обсуждаемый вопрос?

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
#ifndef COLORS_H
#define COLORS_H
 
#include<windows.h>
 
//коды цветов
enum ConsoleColor
{
        Black         = 0,
};
 
//возможность установки цвета зная заранее его код
void SetColor(ConsoleColor text,ConsoleColor background = White)
{
    static HANDLE hStdOut;
    SetConsoleTextAttribute(hStdOut, (WORD) 1);
}
 
//доп. возможность установкци цвета. Её отличие в том что функция
//выбирает цвета максимально отличающиеся для номеров что рядом (в отличии от распред. цветов в табличке)
void SetDifColors(int nColor)
{
    ConsoleColor color;
    SetColor(color);
}
#endif

Прикинь, если бы ты весь код дал (а многие так и делают), я бы САМ весь его упрощал. А теперь уже не упрощаю, а работаю только с максимально упрощёнными кодами. Дашь упрощённый код?

@Gepar · 10.12.2012, 01:06 **[ТС]**

Сообщение от Kuzia domovenok

Если у тебя хотя бы два файла, в которых есть #include "colors.h"

Так он один, include "Colors.h" есть ТОЛЬКО в list.h, в list.cpp и main нет include "Colors.h" .
Плюс почему это

Сообщение от Kuzia domovenok

она включает всё это в каждый файл, где прописан этот инклуд.

У меня там ifndef endif макрос, если оно встретит во второй раз то оно не должно его включать.
Но я попробую сейчас воспользоваться твоим советом насчёт функций в Colors.h.

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

Прикинь, если бы ты весь код дал (а многие так и делают), я бы САМ весь его упрощал.

А толку упрощать, видно же что тупо две функции, а что они делают читать не надо же, пофиг ведь что там в них пописано.

@DU · 10.12.2012, 01:08

если функция реализована в h файле, то чтобы не было ругани линковщика, перед функцией нужно явно просталять ключевое слово inline. или же, как было сказано, выносить реализацию в отдельный cpp файл. это актуально для свободных функций. шаблонные функции и методы вроде как автоматом в этом случае инлайновые.

@Gepar · 10.12.2012, 01:09 **[ТС]**

Сообщение от Kuzia domovenok

А реализацию функций вынеси в СРР файл. Сolors.cpp

Таки помогло. Не понятно до конца только почему. Как же макросы и всё такое...
Получается я могу разнести код colors на .h и .cpp и оставить код list только в list.h, но я не могу оставить код colors в colors.h, а код list разнести на list.h и list.cpp. Неравноправие получается.

@Kuzia domovenok · 10.12.2012, 01:09

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

У меня там ifndef endif макрос, если оно встретит во второй раз то оно не должно его включать.

Вопрос в том, что за "ОНО" встретит его второй раз? Препроцессор, который эти дефайны обрабатывает? Ну да, но только если ты по ошибке два раза напишешь .
#include "colors.h"
#include "colors.h"
или вложишь один заголовок в другой.
Ведь препроцессор обрабатывает только ОДИН файл за раз. Он тупой как текстовый обработчик, ему интересен каждый раз один СРР файл + заголовки + пара твоих директив. О существовании других СРР файлов в момент обработки он не знает.
или что-то в этом роде. А к двум разным СРР файлам будет подключено по заголовку. Вот такие дела.

Croessmah · 10.12.2012, 01:10

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

У меня там ifndef endif макрос, если оно встретит во второй раз то оно не должно его включать.
Но я попробую сейчас воспользоваться твоим советом насчёт функций в Colors.h.

CPP файлы компилируются раздельно и ничего не знают друг о друге. Поэтому в каждом файле, где включен color.h он будет включаться. А потом у линковщика возникают вопросы - какую из копий функции вызывать.

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

Так он один, include "Colors.h" есть ТОЛЬКО в list.h, в list.cpp и main нет include "Colors.h" .

А list включается в main, да?

@DU · 10.12.2012, 01:16

пока лист.h c функциями, которые прям там и реализованы включается в один спп файл, все будет хорошо. как только это включение попадает в 2 и более файла - появится та же проблема. еще раз обращаю внимание на ключевое слово инлайн, с которым все должно быть ок.

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
#ifndef COLORS_H
#define COLORS_H
 
#include<windows.h>
 
//коды цветов
enum ConsoleColor
{
        Black         = 0,
};
 
//возможность установки цвета зная заранее его код
inline void SetColor(ConsoleColor text,ConsoleColor background = White)
{
    static HANDLE hStdOut;
    SetConsoleTextAttribute(hStdOut, (WORD) 1);
}
 
//доп. возможность установкци цвета. Её отличие в том что функция
//выбирает цвета максимально отличающиеся для номеров что рядом (в отличии от распред. цветов в табличке)
inline void SetDifColors(int nColor)
{
    ConsoleColor color;
    SetColor(color);
}
#endif

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! в-строка - входное арифметическое выражение в инфиксной(обычной). . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@Gepar 1186 / 543 / 78 Регистрация: 01.07.2009 Сообщений: 3,517

	Линковщик студии суёт нос не туда куда надо 09.12.2012, 23:55. Показов 2136. Ответов 12 Метки нет (Все метки) Хочу разбить код на .h и .cpp файл, но эта, самка собаки, линковщик студии начинает творить непотребства. Пока весь код в .h файлах всё хорошо. Как есть сейчас: Colors. h - пара функций для изменения цвета в консоли. Код заключён #ifndef COLORS_H #endif List.h - класс-список с набором функций описанных прямо в классе List (те неявный инлайн у всего). Код заключён в #ifndef LIST_H #endif + в начале кода есть #include "Colors.h" Main.cpp - обычный main, в начале есть подключение списка #include "List.h" Студия линкует всё правильно. Теперь я добавляю List.cpp файл и выношу туда одну функцию из класса List что описан в List.h и тут же студия мне говорит что она два раза сунула SetColor и не знает какой выбрать: Ошибка 1 error LNK2005: "void __cdecl SetColor(enum ConsoleColor,enum ConsoleColor)" (?SetColor@@YAXW4ConsoleColor@@0@Z) єцх юяЁхфхыхэ т List.obj Ошибка 2 error LNK2005: "void __cdecl SetDifColors(int)" (?SetDifColors@@YAXH@Z) єцх юяЁхфхыхэ т List.obj Как она так умудрилась? Я там помню что есть вариант что если вынести все функции из .h в .cpp файл то якобы всё образумится, но мне это не надо делать, мне не нужно все функции выносить в cpp файл. Any ideas ? 0

go 3646 / 1378 / 243 Регистрация: 16.04.2009 Сообщений: 4,526
	10.12.2012, 00:34
	Gepar, .h только прототип. 0

@DU 1500 / 1146 / 165 Регистрация: 05.12.2011 Сообщений: 2,279
	10.12.2012, 01:08
	если функция реализована в h файле, то чтобы не было ругани линковщика, перед функцией нужно явно просталять ключевое слово inline. или же, как было сказано, выносить реализацию в отдельный cpp файл. это актуально для свободных функций. шаблонные функции и методы вроде как автоматом в этом случае инлайновые. 0