Пораздельная компиляция и повоторное переопределением в gcc

@xtorne21st · Регистрация: 25.09.2010

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

Доброго времени суток. Пытаюсь разобрать пример с учебника. Теоритически вроде всё делаю правильно...

Bash
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
ilyuha21st@coldshoot:~/Projects/calc$ g++ calc.cpp -c -o calc.o
ilyuha21st@coldshoot:~/Projects/calc$ g++ main.cpp -c -o main.o
ilyuha21st@coldshoot:~/Projects/calc$ g++ main.o calc.o -o prog
calc.o:(.bss+0x0): multiple definition of `Driver::no_of_errors'
main.o:(.bss+0x0): first defined here
calc.o:(.bss+0x8): multiple definition of `Lexer::string_value'
main.o:(.bss+0x8): first defined here
calc.o:(.bss+0x10): multiple definition of `Lexer::number_value'
main.o:(.bss+0x10): first defined here
calc.o:(.data+0x0): multiple definition of `Lexer::curr_token'
main.o:(.data+0x0): first defined here
calc.o:(.bss+0x20): multiple definition of `Driver::table'
main.o:(.bss+0x20): first defined here
calc.o:(.bss+0x50): multiple definition of `Driver::input'
main.o:(.bss+0x50): first defined here
collect2: error: ld returned 1 exit status

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
// calc.h
#ifndef _CALCULATOR_HEADER_
#define _CALCULATOR_HEADER_
 
#include <string>
#include <iostream>
#include <map>
 
namespace Driver
{
    int no_of_errors = 0;
}
 
namespace Error
{
    struct Zero_divide
    {
        Zero_divide() { Driver::no_of_errors++; }
    };
 
    struct Syntax_error
    {
        Syntax_error(const char* p) { err = p; Driver::no_of_errors++; }
 
        const char* err;
        const char* what()
        {
            return err;
        }
    };
}
 
namespace Lexer
{
    enum Token_value 
    {
        NAME,       NUMBER,     END,       
        PLUS='+',   MINUS='-',  MUL='*',    DIV='/',
        PRINT=';',  ASSIGN='=', LP='(',     RP=')'      
    };
 
    std::string string_value;
    double number_value;
    Token_value curr_token = PRINT;
 
    Token_value get_token();
}
 
namespace Parser
{
    double expr(bool);
    double term(bool);
    double prim(bool);
}
 
namespace Driver
{
    std::map<std::string, double> table;
    std::istream* input;
    void skip();
 
    using Parser::expr;
}
 
#endif

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
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
//calc.cpp
#include "calc.h"
#include <cctype>
 
Lexer::Token_value Lexer::get_token()
{
    char ch;
    
    do
    {
        if (!Driver::input->get(ch))
        {
            return curr_token = END;
        }
    } while (ch != '\n' && isspace(ch));
 
    switch (ch)
    {
        case 0:
            return curr_token = END;
        case ';':
        case '\n':
            return curr_token = PRINT;
        case '+':
        case '-':
        case '/':
        case '*':
        case '=':
        case '(':
        case ')':
            return curr_token = Token_value(ch);
        case '0':   case '1':   case '2':   case '3':   case '4':
        case '5':   case '6':   case '7':   case '8':   case '9':
        case '.':
            Driver::input->putback(ch);
            *Driver::input >> number_value;
            return curr_token = NUMBER;
        default:
            if (isalpha(ch))
            {
                string_value = ch;
                while (Driver::input->get(ch) && isalnum(ch))
                {
                    string_value.push_back(ch);
                }
                return curr_token = NAME;
            }
            throw Error::Syntax_error("bad token");
    }
}
 
double Parser::expr(bool get)
{
    double left = term(get);
 
    for ( ; ; )
    {
        switch (Lexer::curr_token)
        {
            case Lexer::PLUS:
                left += term(true);
                break;
            case Lexer::MINUS:
                left -= term(true);
                break;
            default:
                return left;
        }
    }
}
 
double Parser::term(bool get)
{
    double left = prim(get);
    
    for ( ; ; )
    {
        switch (Lexer::curr_token)
        {
            case Lexer::MUL:
                left *= prim(true);
                break;
            case Lexer::DIV:
                if (double v = prim(true))
                {
                    left /= v;
                }
                throw Error::Zero_divide();
            default:
                break;
        }
    }
}
 
double Parser::prim(bool get)
{
    if (get)
    {
        Lexer::get_token();
    }
 
    switch (Lexer::curr_token)
    {
        case Lexer::NUMBER:
            {
                double v = Lexer::number_value;
                Lexer::get_token();
                return v;
            }
        case Lexer::NAME:
            {
                double& v = Driver::table[Lexer::string_value];
                if (Lexer::get_token() == Lexer::ASSIGN)
                {
                    v = expr(true);
                }
                return v;
            }
        case Lexer::MINUS:
            return prim(true);
        case Lexer::LP:
            {
                double e = expr(true);
                if (Lexer::curr_token != Lexer::ASSIGN)
                {
                    throw Error::Syntax_error("')' expected");
                }
                Lexer::get_token();
                return e;
            }
        default:
            throw Error::Syntax_error("pimary expected");
    }
}
 
 
void Driver::skip()
{
    char ch;
 
    while (*input)
    {
        input->get(ch);
 
        switch (ch)
        {
            case '\n':
            case ';':
                break;
        }
    }
}

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
 // main.cpp
#include "calc.h"
#include <iostream>
#include <sstream>
 
int main(int argc, const char* argv[])
{
    switch (argc)
    {
        case 1:
            Driver::input = &std::cin;
            break;
        case 2:
            Driver::input = new std::stringstream(argv[1]);
            break;
        default:
            std::cerr << "to many arguments" << std::endl;
            return 1;
    }
 
    while (*Driver::input)
    {
        try
        {
            Lexer::get_token();
            if (Lexer::curr_token == Lexer::END)
            {
                break;
            }
            if (Lexer::curr_token == Lexer::PRINT)
            {
                continue;
            }
            std::cout << Driver::expr(false) << std::endl;
        }
        catch (Error::Zero_divide)
        {
            std::cerr << "divide by zero" << std::endl;
            Driver::skip();
        }
        catch (Error::Syntax_error e)
        {
            std::cerr << "error: " << e.what() << std::endl;
            Driver::skip();
        }
    }
 
    if (Driver::input != &std::cin)
    {
        delete Driver::input;
    }
 
    return Driver::no_of_errors;
}

@diagon · 06.05.2013, 23:18

У вас один и тот же хедер включается в 2 cpp'шника. Линкер не знает, что делать в такой ситуации.
Вы можете помочь ему, если объявите ваши функции как inline либо static.

Ну либо переписать все в ООП-шном стиле через классы.

@xtorne21st · 07.05.2013, 00:11 **[ТС]**

Пробую я различные комбинации, но что-то всё равно не получается... Интересно, а как тогда пишутся программы на С? Всё время static использовать?

Croessmah · 07.05.2013, 00:14

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

Теоритически вроде всё делаю правильно...

Теоретически в хедерах только объявляют. Поиск соответствующих объявлениям определений задача линкера.

Добавлено через 1 минуту

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

Всё время static использовать?

можно для объявления переменных указать extern, прототипы функций по умолчанию extern

@xtorne21st · 07.05.2013, 19:20 **[ТС]**

Croessmah, спасибо что напомнили. Также вспомнил, что если использован модификатор const, то extern нужно задавать явно.

Добавлено через 6 минут
Хотя вот только что проверил на gcc и без extern всё ок компилируется.

Добавлено через 36 секунд
Ps. Врут в учебниках

Добавлено через 38 минут
Хотя, это скорей всего зависит от компилятора

Новые блоги и статьи Все статьи Все блоги /
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	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? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .

@diagon Higher 1953 / 1219 / 120 Регистрация: 02.05.2010 Сообщений: 2,925 Записей в блоге: 2
	06.05.2013, 23:18
	У вас один и тот же хедер включается в 2 cpp'шника. Линкер не знает, что делать в такой ситуации. Вы можете помочь ему, если объявите ваши функции как inline либо static. Ну либо переписать все в ООП-шном стиле через классы. 1

@xtorne21st интересующийся 311 / 282 / 93 Регистрация: 25.09.2010 Сообщений: 1,056
	07.05.2013, 00:11 [ТС]
	Пробую я различные комбинации, но что-то всё равно не получается... Интересно, а как тогда пишутся программы на С? Всё время static использовать? 0

@xtorne21st интересующийся 311 / 282 / 93 Регистрация: 25.09.2010 Сообщений: 1,056
	07.05.2013, 19:20 [ТС]
	Croessmah, спасибо что напомнили. Также вспомнил, что если использован модификатор const, то extern нужно задавать явно. Добавлено через 6 минут Хотя вот только что проверил на gcc и без extern всё ок компилируется. Добавлено через 36 секунд Ps. Врут в учебниках Добавлено через 38 минут Хотя, это скорей всего зависит от компилятора 0