drucken ein Vektor in c++

Ich bin neu in C++ und ich habe eine Funktion, die liest Elemente in einen Vektor und eine Funktion, druckt die Elemente. Das problem ist, dass die print-Funktion nicht gut funktioniert.

 #include <iostream>
    using namespace std;

1)

int read()
{
    int elements,i;
    cout<<"Give the number of elements:";
    cin>>elements;
    int v[elements];
    for(i=1;i<=elements;i++)
    {
        cout<<"v["<<i<<"]=";
        cin>>v[i];
    }
    return elements;
}

2)

void print_vector(int elements)
{
    int i;
    int v[elements];
    for(i=1;i<=elements;i++)
    {
        cout<<v[i];
    }
}

int main()
{
    int n=read();
    print_vector(n);
    return 0;
}

Example:
Give the number of elements:5
v[1]=1
v[2]=2
v[3]=3
v[4]=4
v[5]=5
1348

Druckt er 1348 anstelle der 12345. Was ist falsch an der Funktion und wie kann ich es ändern, zu drucken, die passenden Elemente?

  • Ihr code wird mit einem compiler-Erweiterung. Verwenden std::vector statt, die variable-Länge-array.
  • Bitte Lesen Sie whathaveyoutried.com und befolgen Sie die Ratschläge.
InformationsquelleAutor Cucerzan Rares | 2013-03-15
  1. 3
    int v[elements];

    Diesem erstellen Sie eine neue Integer-array.

    Ist es relativ schwer zu passieren, um arrays, wie Sie müssen vorsichtig sein, wie Sie mit der Zuweisung von Speicher, und Sie hark zurück zu den alten Tagen der C.

    Wie Sie lernen C++ können Sie eine class zum verwalten von Speicher-Probleme. std::vector ist syntaktisch ähnlich zu einem array, aber sieht nach Zuweisung und ändern der Größe für Sie, und erinnert sich, was Größe ist es auch.

    Übergeben werden können Objekte an Funktionen per Referenz &, und innerhalb der Funktion wird es sein, die Lage zu ändern das übergebene Objekt aus und haben eine nachhaltige Wirkung, nachdem die Funktion beendet wurde.

    void read(vector<int>& elements) {
    //in here you can resize the vector
}

void print(const vector<int>& elements) {
    //passing in elements by const reference means you can't modify it
}

    Also insgesamt:

    int main() {
    vector<int> elements;
    read(elements);
    print(elements);
}

    Alternativ, wenn Sie möchten, Objekt-orientiert, sollten Sie auf der Suche nach Möglichkeiten der Gruppierung von Daten und Funktionen zusammen, so dass Sie vermeiden, durch um Daten insgesamt. In diesem Fall können Sie die Vektor-und die read-und print-Funktionen. Vielleicht nennen es UserIntegers:

    class UserIntegers {
public:
    void process() {
        read();
        print();
    }
private:
    vector<int> numbers;
};

    Verwendung:

    int main() {
    UserIntegers userIntegers;
    userIntegers.process();
}
    • Und wie soll ich das design der print-Funktion, so dass es nicht erstellen Sie ein neues array mit Ganzzahlen?
    InformationsquelleAutor Peter Wood
  2. 2

    Haben Sie Fehler in der drucken-Funktion

    for(i=1;i<=elements;i++)
{                   //^^^^^
    cout<<v[i];
    ++i;  //You would like to skip some elements?
}

    Erhöhen Sie den index habe ich zweimal und array-index beginnt mit 0, nicht 1; es endet am Anzahl (Elemente-1) nicht die Elemente, mittlerweile, Sie sind mit array, nicht Vektor.

    InformationsquelleAutor taocp
  3. 2

    In read() Lesen Sie die Elemente in einem lokalen array v. Wenn die Funktion endet, wird das array Weg, als gut. Das gleiche passiert in print_vector().

    Ein weiterer Fehler ist die Indizierung der Arrays. Sie Zugriff auf die Elemente von 1 zu elements inclusive, die ignoriert das erste element 0 an und greift schließlich ein element außerhalb der array-Grenzen (elements).

    Müssen Sie reservieren ein array, besser ein std::vector außerhalb der beiden Funktionen und pass, die in read() für Füll-und in print_vector() für den Druck.

    #include <iostream>
#include <vector>

void read(std::vector<int> &v)
{
    int elements,i, n;
    std::cout<<"Give the number of elements:";
    std::cin>>elements;
    for(i = 0; i < elements; i++)
    {
        std::cout<<"v["<<i<<"]=";
        std::cin>> n;
        v.push_back(n);
    }
}

void print_vector(const std::vector<int> &v)
{
    for(auto i = v.begin(); i != v.end(); ++i)
    {
        std::cout << *i;
    }
}

int main()
{
    std::vector<int> v;
    read(v);
    print_vector(v);
    return 0;
}
    • Ich denke, Sie sind fehlt den template-argument für std::vector
    • Vielen Dank, ich verpasst, dass. Behoben.
    InformationsquelleAutor Olaf Dietsche
Schreibe einen Kommentar