C++ Drucken einer Zeichenfolge in umgekehrter mithilfe einer for-Schleife

Ich habe ein Programm, das druckt die Zeichen einer Zeichenfolge mit einer for-Schleife. Es muss auch drucken die gleichen Zeichen in umgekehrter, das ist, wo ich Probleme. Kann jemand mir helfen herauszufinden, warum die die zweite for-Schleife nicht ausgeführt wird?

int main()
{
    string myAnimal;

    cout << "Please enter the name of your favorite animal.\n";
    cin >> myAnimal;

    //This loop works fine
    int i;
    for(i = 0; i < myAnimal.length(); i++){
        cout << myAnimal.at(i) << endl;
    }

    //This one isn't executing
    for(i = myAnimal.length(); i > -1; i--){
        cout << myAnimal.at(i) << endl;
    }
    return 0;
}
Sie sollten nicht mit int zu speichern, die Länge der Zeichenfolge die Länge der Zeichenfolge überschreitet den integer-Bereich.

InformationsquelleAutor Bobby | 2014-01-26

  1. 5

    Weisen Sie ich zunächst auf die Länge minus eins, oder der Letzte index-Wert in das array.

    for(i = myAnimal.length()-1; i >= 0; i--){
    cout << myAnimal.at(i) << endl;
}
    Nachtrag: dies ist, was die runtime error Meldung "terminate aufgerufen, nach dem werfen einer Instanz von 'std::out_of_range'", oder ähnlich, das solltest du bekommen haben, wurde versucht, Ihnen zu sagen.
    Das war es, vielen Dank!
    wenn Sie fand diese Antwort hilfreich, sollten Sie akzeptieren der Antwort.

    InformationsquelleAutor Lokno

  2. 1

    Da die Zeichen-Positionen starten bei 0, das Letzte Zeichen der myAnimal an position (myAnimal.length()-1) nicht myAnimal.length() so dass Sie wollen, starten Sie die zweite Schleife gibt.

    InformationsquelleAutor moonshadow

  3. 0

    @Lokno Ihnen schon die richtige Antwort. Lassen Sie mich allerdings auch Fehler Ihren code ein bisschen mehr, Ihnen zu zeigen, einige andere alternativen und zu beheben einige kleinere Fehler.

    Erste, du hast nicht wirklich ein post kompilieren Beispiel, weil Sie vergessen haben, zeigen die enthaltenen Header <iostream> und <string> und auch nicht zeigen, die using namespace std; das war implizit in Ihrem code.

    Zweite, für die regelmäßige for Schleife, halten lieber die loop-variable innerhalb der Schleife, es sei denn, Sie wirklich brauchen, verwenden Sie es als einen Wert zurück. Auch lieber pre-Inkrement ++i über post-Inkrement i++. Darüber hinaus, weil Sie gemacht haben, sicher, dass die richtigen loop-Indizes, es gibt keinen Grund, die Grenzen-geprüft-element zugreifen at() über die ungeprüften [] version.

    In C++11 haben Sie die range-for Schleife, die ermöglicht, auch die kürzeren und mehr narrensicher code, wo ich auch auto wo Sie benutzt haben könnte char. Leider gibt es keine reverse range-for Schleife. Die richtige index-basierte reverse for-Schleife ist wahrscheinlich einfacher zu Lesen, wenn Sie i >= 0 eher als i > -1.

    Dann gibt es einen Algorithmus, basierend Schleife mit std::copy wo Sie die Verwendung der iterator-Schnittstelle std::string (insbesondere die reverse-Iteratoren rbegin() und rend()) zum kopieren von einzelnen Zeichen durch ein ostream_iterator gebunden standard-Ausgabe.

    BTW, ich benutzte den separator "|" anstatt die neue zu sehen, Sachen, die einfacher sind, anpassen, um Ihrem Geschmack. In jedem Fall, mit std::endl können Auswirkungen auf die Leistung, weil es leert den Ausgabe-Puffer zu jeder Zeit.

    #include <algorithm>
#include <iterator>
#include <iostream> //you forgot this
#include <string>   //you forgot this

int main()
{
    using namespace std;    //you forgot this

    //let's pretend this is the string
    string myAnimal = "Please enter the name of your favorite animal.";

    //keep the loop variable local, prefer pre-increment
    for (int i = 0; i < myAnimal.length(); ++i)
        cout << myAnimal[i] << "|";    //prefer [] over at()
    std::cout << "\n";

    //C++11 range-for
    for (auto c : myAnimal)
        std::cout << c << "|";
    std::cout << "\n";

    //index-based reverse loop
    for (int i = myAnimal.length() - 1; i >= 0; --i)
        cout << myAnimal[i] << "|";
    std::cout << "\n";

    //algorithm-based reverse loop
    std::copy(myAnimal.rbegin(), myAnimal.rend(), ostream_iterator<char>(cout, "|"));
    std::cout << "\n";

    //main implicitly return 0
}

    Live Beispiel. PS: main() implizit zurück 0 auf Erfolg.

    InformationsquelleAutor TemplateRex

  4. 0

    Könnten Sie reverse-Iteratoren

    #include <iostream>
#include <string>

int main() {
  std::string myAnimal;

  std::cout << "Please enter the name of your favorite animal.\n";
  std::cin >> myAnimal;

  //Iterate in reverse order
  for(auto c = myAnimal.rbegin(); c != myAnimal.rend(); ++c) {
    std::cout << *c << std::endl;
  }
}

    Beachten Sie, dass Sie haben, erhöhen Sie die variable " c " (und nicht abnehmen), da dies ist ein reverse-iterator.

    InformationsquelleAutor Gautham

  5. -1
    for(myAnimal.length(); i > -1; i--){
    ^

    dies nicht tun. Holen Sie sich den Wert und werfen Sie es dann Weg.

    Meinst du i = myAnimal.length() - 1 ?

    Sorry, habe eine schnelle Bearbeiten. Sollte ich = myAnimal.Länge().

    InformationsquelleAutor Karoly Horvath

  6. -3

    Statt

    int i;
for(i = 0; i < myAnimal.length(); i++){
    cout << myAnimal.at(i) << endl;
}

//This one isn't executing
for(i = myAnimal.length(); i > -1; i--){
    cout << myAnimal.at(i) << endl;
}

    schreiben

    for ( string::size_type i = 0; i < myAnimal.length(); i++ ){
    cout << myAnimal.at(i) << endl;
}

//This one isn't executing
for ( string::size_type i = myAnimal.length(); i != 0; ){
    cout << myAnimal.at( --i) << endl;
}

    In Ihrem code, den Sie versuchen, Zugriff auf ein element des Strings ist außerhalb des zulässigen Bereichs, ist gleich [0, length() - 1]

    Auch anstelle des Typs int ist es besser, verwenden Sie die type, std::string bietet für den Rückgabetyp der Funktion Mitglied der Länge, die std::string::size_type.

    Sollten Sie zumindest Versuch zu erklären, warum die änderung hilft. Auch, bewegen Sie den loop-Inkrement in die cout Betrieb ist clever, wird aber allgemein als schlechter Stil.
    Ich denke dieser code ist mehr verwirrend für Anfänger.

    InformationsquelleAutor Vlad from Moscow

Schreibe einen Kommentar