How to create a char array from string constant?

Question

I try to create a char* values[] for my unittests that will be written but initialized from constants. The naive way throws a warning that ISO C++ forbids it:

char* single[1];
single[0] = "foobar";

I tried the following that obviously does not work:

std::string executable = "foobar";
std::array<char, executable.size()> data; // Error: size not known at compile time
std::copy(executable.begin(), executable.end(); data.data());

char* single[1];
single[0] = data.data();

There must be a way like:

std::array<char> data = { "foobar" };
char* single[1];
single[0] = data.data();

In C++ string literals are constant character arrays. So you have to write for example const char* single[1]; single[0] = "foobar"; — Vlad from Moscow
– Vlad from Moscow, Commented Mar 16, 2017 at 10:00
@VladfromMoscow But I need char** so I can freely manipulate the data. The constness is in the way. — aggsol
– aggsol, Commented Mar 16, 2017 at 10:01

cpplearner · Accepted Answer · 2017-03-16 10:25:02Z

4

I suggest you to add to_array to your toolbox. Then you can simply write

auto data = to_array("foobar");
char* single[1];
single[0] = data.data();

answered Mar 16, 2017 at 10:25

cpplearner

19k3 gold badges60 silver badges91 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

aggsol Over a year ago

This is a great find, too bad it's not standard yet.

Community · Accepted Answer · 2017-05-23 11:54:48Z

3

std::string foo = "foobar";
std::vector<char> data(foo.data(), foo.data()+foo.size()+1u);
char* single[1];
single[0] = data.data();

How to convert a std::string to const char* or char*?

You can use some kind of transformation function if this pattern reoccurs:

template<class CharT, std::size_t N>
std::vector<CharT> literal_vector(CharT const (&a)[N])
{
    return std::vector<CharT>(&a[0], (&a[0]) + N);
}

and then

std::vector<char> lv = literal_vector("Hello");
single[0] = lv.data();

edited May 23, 2017 at 11:54

CommunityBot

11 silver badge

answered Mar 16, 2017 at 10:15

Pixelchemist

25.1k8 gold badges51 silver badges72 bronze badges

Comments

metalfox · Accepted Answer · 2017-03-17 16:30:57Z

3

Even though your question has not been tagged as c++1z, it is worth mentioning that the constness restriction of the data function of std::string has been lifted in the current working draft of the Standard (N4618), which is about to be published as C++17. In [basic.string], one can find both:

const charT* data() const noexcept;
charT* data() noexcept;

Your compiler may already have support for it, possibly needing the compiler flag -std=c++1z or similar. If so, you should be able to write:

std::string executable = "foobar";
char *single[1];
single[0] = executable.data();

edited Mar 17, 2017 at 16:30

answered Mar 17, 2017 at 16:14

metalfox

6,5362 gold badges25 silver badges49 bronze badges

1 Comment

aggsol Over a year ago

This is the minimal approach I want and hopefully will use soon.

Jorge Omar Medra · Accepted Answer · 2017-03-17 01:23:31Z

1

Here you have tow ways, both of them are using memcpy.

Using dynamic memory.
Using static memory.

 

    using namespace std;

    #define MAX_BUFFER_SIZE 250
    int main()
    {
        const char* myData = "This is My Data";

        int sizeOfmydata = std::strlen(myData);;

        //1.- Dinamic Memory
        char* data1 = new char[sizeOfmydata];
        std::memcpy(data1, myData, sizeOfmydata);

        //2.- Static Memory
        char data2[MAX_BUFFER_SIZE];
        memset(data2, '\0', MAX_BUFFER_SIZE);
        std::memcpy(data2, myData, sizeOfmydata);


        delete[] data1;
        return 0;
    }

answered Mar 17, 2017 at 1:23

Jorge Omar Medra

9881 gold badge9 silver badges20 bronze badges

Comments

OrMiz · Accepted Answer · 2017-03-16 10:02:56Z

-1

You can use std::vector<std::string> to store your data, and use c_str() of std::string to parse your string to char*

answered Mar 16, 2017 at 10:02

OrMiz

2751 gold badge2 silver badges12 bronze badges

5 Comments

aggsol Over a year ago

c_str() returns const char* not char *

OrMiz Over a year ago

You can change it back like @pergy did

Pixelchemist Over a year ago

Which is probably unsafe and UB.

aggsol Over a year ago

@Pixelchemist What does UB mean?

Pixelchemist Over a year ago

@aggsol: Undefined behaviour which essentially means: The c++ standard doesn't provide a guarantee about it. Using const_cast on the return value of c_str and manipulating the data through the resulting pointer might work... today... if you jump twice ... and if the sun is brighter than yesterday... on your compiler ...

Community · Accepted Answer · 2017-05-23 11:54:48Z

-1

This is a bit long but it seems to do what you need. This is file foo.cpp:

#include <iostream>
#include <vector>
#include <string>
#include <algorithm>
#include <cstdio>

int main(int argc, char* argv[])
{
    std::vector<std::string> strings = {"foobar", "barbaz"};

    char* values[strings.size()];

    for (int i = 0; i < strings.size(); ++i) {
        values[i] = new char[strings[i].size() + 1];
        std::copy(strings[i].cbegin(), strings[i].cend(), values[i]);
        values[strings[i].size()] = '\0';
    }

    for (int i = 0; i < strings.size(); ++i) {
        printf("%s\n", values[i]);
    }

    return 0;
}

Then:

$ make foo
g++     foo.cpp   -o foo
$ ./foo
foobar
barbaz

You should of course delete each of the elements of the array of char*, and maybe put the loop for copying from std::string to char* in a function and so on.

PS: auch, this solution is identical to this answer that someone already linked.... The linked other answer has more detail. Does that mean your question is a duplicate? I let experienced users decide.

edited May 23, 2017 at 11:54

CommunityBot

11 silver badge

answered Mar 16, 2017 at 10:00

user7473772

3 Comments

aggsol Over a year ago

No I want char* val[] or char** val because I need to manipulate the array and it's values.

user7473772 Over a year ago

@aggsol sorry for misunderstanding

user7473772 Over a year ago

@aggsol Now I think I did better :) at least it is easy to write all literals and then you just copy them to memory you create with new which is I am afraid unavoidable if you don't want constness

Collectives™ on Stack Overflow

How to create a char array from string constant?

6 Answers 6

1 Comment

Comments

1 Comment

Comments

5 Comments

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

1 Comment

Comments

1 Comment

Comments

5 Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related