По какой причине происходит дубляж реализации?

Question

[Nightmare A]

webderiverxx все файлы .h, видимо реализация написана на .h файлах, есть лок в виде проверки препроцессора, почему неколько реализаций возникает, не совсем понятно.

koshmar@ACTPA:/media/koshmar/2DEDC4C311647F4A/РабочееПространство/cpp.машинный$ gcc --version
gcc (GCC) 11.1.0
Copyright (C) 2021 Free Software Foundation, Inc.
Это свободно распространяемое программное обеспечение. Условия копирования
приведены в исходных текстах.

  Без гарантии каких-либо качеств, включая 
коммерческую ценность и применимость для каких-либо целей.

Answer 1

[jcmvbkbc]

webderiverxx все файлы .h, видимо реализация написана на .h файлах

В .h файлах можно написать и так, что вот этого всего не будет. Если ты нам покажешь этот webdriverxx.h мы наверно сможем подсказать, что там нужно изменить для этого.

есть лок в виде проверки препроцессора, почему неколько реализаций возникает, не совсем понятно

Ты если такое говоришь, то, видимо, не вполне понимаешь, для чего используется защита макросом препроцессора.

Comments

[Nightmare A]

#ifndef WEBDRIVERXX_WEBDRIVER_H
#define WEBDRIVERXX_WEBDRIVER_H

#include "client.h"
#include "session.h"

namespace webdriverxx {

// The main class for interactions with a server. Automatically connects to a server,
// creates and deletes a session and gives access to session's API.
class WebDriver // copyable
	: public Client
	, public Session {
public:
	explicit WebDriver(
		const Capabilities& desired = Capabilities(),
		const Capabilities& required = Capabilities(),
		const std::string& url = kDefaultWebDriverUrl
		)
		: Client(url)
		, Session(CreateSession(desired, required))
	{}
};

inline
WebDriver Start(
	const Capabilities& desired,
	const Capabilities& required = Capabilities(),
	const std::string& url = kDefaultWebDriverUrl
	)
{
	return WebDriver(desired, required, url);
}

inline
WebDriver Start(const Capabilities& desired, const std::string& url)
{
	return Start(desired, Capabilities(), url);
}

} // namespace webdriverxx

#endif

[jcmvbkbc]

Nightmare A, да, одного этого заголовка будет недостаточно. Нужно определение функции которая дублируется -- webdriverxx::split.

[Nightmare A]

jcmvbkbc,

std::vector<std::string> split(const std::string str, char delimiter = ' ') {
    std::vector<std::string> result;
    result.reserve(str.size());
    int start = 0, end = 0;
    for (const auto& c: str) {
	if (c != delimiter) {
	    ++end;
	    continue;
	}
	result.push_back(str.substr(start, end - start));
	start = ++end;
    }
    if (start <= str.size()) {
	result.push_back(str.substr(start, end - start));
    }
    return result;
}

Функция как функция.

[jcmvbkbc]

Nightmare A, прекрасно. Остаётся либо перенести её определение в .cpp, а в .h оставить только объявление, либо добавить inline к определению в .h

Функция как функция.

Ты реально не видишь проблемы?

[Nightmare A]

jcmvbkbc, По идее не должно оно проходить дважды из за директив препроцессора.

[jcmvbkbc]

По идее не должно оно проходить дважды из за директив препроцессора.

Nightmare A, я тебе рекомендую разобраться, для чего используется защита макросом препроцессора -- для защиты от повторного включения файла в одну и ту же единицу трансляции. Потому что иначе была бы ошибка компиляции. От включения в разные единицы трансляции этот метод не помогает, а если ты посмотришь внимательно на сообщение об ошибке, то увидишь, что это не ошибка компиляции, а ошибка линковки, а конфликтующие символы определены в разных объектных файлах.

[Nightmare A]

jcmvbkbc, Дa, но hello world выводится только один раз за всю компиляцию ;d

[Nightmare A]

jcmvbkbc, Я не помню, я что то делал не изменяя код и оно работало .

[jcmvbkbc]

Nightmare A, а ты перенеси эту прагму в тот исходник где ты нашёл split и перекомпилируй.

[Nightmare A]

jcmvbkbc, В общем дополнительные inline директивы решили эту проблему .