Сложить два массива char'ов и не только

[alexzk]

Бг...никада б не подумал, еще в универе в 1998 (т.е. доС99) учили нас именно использовать юнион для трюков.
Хотя странности замечал, почти всегда нада его делать (packed), иначе не летает при передаче данных м-у разными битнесс или сохранении в файл.

[p51x]

Вы со структурами их не спутали?

[bugaboo]

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

одинаково, потому что (примерно)

strcat = strlen + strcpy

да, и в 1 случае мы получаем 2 strlen + 2 strcpy,
а во 2: 3 strlen + 2 strcpy

разве не быстрее?

[Croessmah]

И здесь выложу, пусть валяется

//clang 3.8.0
 
#include <iostream>
#include <memory>
#include <cstring>
 
#include <list>
#include <string>
 
namespace details__
{
template<typename C>
std::enable_if_t<std::is_class<C>::value, std::size_t>
get_length(const C &c)
{
    return c.size();
}
 
 
std::size_t get_length(const char *str)
{
    return strlen(str);
}
 
 
 
template<typename C>
std::enable_if_t<
    std::is_class<C>::value,
    std::pair<typename C::const_iterator, typename C::const_iterator>
>
get_beginend(const C &c, std::size_t)
{
    return {std::begin(c), std::end(c)};
}
 
 
std::pair<const char *, const char *>
get_beginend(const char *str, std::size_t size)
{
    return {str, str + size};
}
 
 
 
template<typename Iterator>
char *
copy_to_buffer(char *buffer, const std::pair<Iterator, Iterator> &beginend)
{
    return std::copy(beginend.first, beginend.second, buffer);
}
    
    
}//details__
 
 
 
 
template<typename ... Args>
std::unique_ptr<char[]> concat(Args &&...args)
{
    using fake_array_t = int[];
    using length_array_t = std::size_t[];
    std::size_t all_size = 0;
    std::size_t cs = 0;
    length_array_t lengths_of_strings {
            (cs = details__::get_length(args), all_size += cs, cs)...
    };
 
    std::unique_ptr<char[]> result(new char[all_size + 1]);
 
    char *current = result.get();
    std::size_t length_index = 0;
 
    fake_array_t {
        (current = details__::copy_to_buffer(current, details__::get_beginend(args, lengths_of_strings[length_index])), ++length_index, 0)...
    };
 
    *current = 0;
 
    return result;
}
 
 
 
int main()
{
    char arr[] = "BLA";
    const char *ps = "->";
    std::string s = "-";
    std::list<char> lst {'e', 'n', 'd'};
    std::cout << concat("bla", s, arr, s, "bla", " ", ps, " ", lst).get() << std::endl;
}

http://rextester.com/OSO16460

[bugaboo]

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

Вот так ровно на 1 байт копирования быстрее
Код:

auto la = strlen(a);
char* c = new char[la + strlen(b) + 1];
memcpy(c, a);
strcpy(c + la, b);

Вы, наверно, имели в виду

	auto la = strlen(a);
	char* c = new char[la + strlen(b) + 1];
	memcpy(c, a, la);
	strcpy(c + la, b);

?

[alexzk]

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

Вы, наверно, имели в виду

	auto la = strlen(a);
	char* c = new char[la + strlen(b) + 1];
	memcpy(c, a, la);
	strcpy(c + la, b);

?

ну да...

[bugaboo]

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

ну да...

а что насчет скорости?
http://www.programmersforum.ru/showp...&postcount=173

[bugaboo]

А вот так без warningов, воще отлично

#include <cstring>
#include <iostream>
using namespace std;
int main()
{
	const char* a = "blabla";						// статический
	char* b = new char[10];							// динамический
	memcpy(b, "BLABLABLA", 10);
	auto la = strlen(a);
	auto lb = strlen(b);
	char* c = new char[la + lb + 1];					// динамический
	memcpy(c, a, la);
	memcpy(c + la, b, lb + 1);
	cout << c << endl;
	delete[] c;
	delete[] b;
	return 0;
}

[alexzk]

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

а что насчет скорости?
http://www.programmersforum.ru/showp...&postcount=173

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

[waleri]

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

чтоб его оптимизатор не выбросил

А еще кэширование надо отключить...