Fylla en strängvektor i c++

Jag blir förvirrad då du skriver char "strang[5][20];" en gång och char "strang[20][5];" en annan. Troligtvis menar du den sista, 5 strängar på 20 tecken var.

void fyllvektor(char **strang)
{
    strcpy(strang[0], "foobar");
}

Du bör använda någon strängklass (bra träning att skriva själv), annars blir stränghanteringen jobbig. Lätt att få minnesläckor, överskridningar osv.

Rättning: char ** kompilerar visserligen, men det bör vara "char strang[20][5]"

Det du försöker göra är inte möjligt med C-strängar (char-vektorer), i vart fall inte på det sätt du gör det. För det första är strang[] i sig en pekare så det är liksom ingen idé att att göra ett referensanrop. För det andra kan funktionen fyllvektor inte "veta" hur mycket minne som allokerats till strang[], eftersom C++ inte "lagrar" den informationen då pekaren används.

Använd hellre std::string, bättre och säkrare på alla sätt och vis.

Felet ligger i att din fyllvektor()-funktion aldrig kan "veta" vilka dimensioner den strang-vektor som deklareras i main() har. Det som skickas till funktionen är en minnesadress, varken mer eller mindre. Det innebär att om funktionsargumentet i fyllvektor() är char ** s, så saknar uttrycket s[1][2] mening i fyllvektor(), eftersom s inte har några dimensioner i den funktionen.

Följande kod fungerar dock:

#include <iostream>
#include <cstring>
#include <cstdlib>

using namespace std;

struct str {
	char	s1[20];
};

void fyllvektor(str *, int);

int main()
{
	struct str strang[5];
	fyllvektor(strang, 5);
	for (int i = 0; i < 5; i++)
		cout << strang[i].s1 << '\n';

	return 0;
}

void fyllvektor(str * s, int b)
{
	for (int i = 0; i < b; i++)
	{
		strcpy(s[i].s1, "foo bar");
		s[i].s1[19] = '\0';
	}
}

Använd std::string och std::vector.

#include <string>
#include <vector>

void fyll_vector( std::vector< std::string >& v )
{
    // lägger till 10 element i slutet av vectorn.
    for( int i = 0; i < 10; ++i )
        v.push_back( "hejsan" );
}

void fyll_vector2( std::vector< std::string >& v )
{
    // sätter varje element i vectorn till "hejsan"
    for( int i = 0; i < v.size(); ++i )
        v[i] = "hejsan";
}

int main()
{
    std::vector< std::string > vector_of_strings; // skapar en tom vector
    fyll_vector( vector_of_strings );


    std::vector< std::string > vec_of_str( 10 ); // skapar en vector med 10 element
    fyll_vector2( vec_of_str );
}

Blir mycket smidigare när du inte behöver hålla koll på storleken, kan lägga till element längst bak så sköter klassen om minneshantering själv.