JSpace-SPOJ błędna odpowiedź w c++

Question

Link do zadania->https://pl.spoj.com/problems/JSPACE/

Jak w temacie, próbowałem dodawać endl lub "\n" po wyświetleniu kodu, ale spoj nadal nie zalicza, proszę o pomoc. Przeglądnąłem także podobne wątki, ale nie znalazłem odpowiedzi jeszcze raz proszę o pomoc.

Kod

#include <iostream>
#include <string>

using namespace std;

int main()
{
    string s;

    getline(cin,s);

    for(int i = 0; i < s.length(); i++)
        if((int)s[i]==32)
        {
            while((int)s[i]==32)
                s.erase(i,1);
            s[i] = (char)(toupper(s[i]));
        }

    s[0] = (char)toupper(s[0]);

    for(auto i: s)
        cout << i;
    cout << "\n";

    return 0;
}

 

Answer 1

Wczytujesz tylko pojedynczy wiersz, a musisz wczytać ich więcej. W treści zadania jest przykład z dwoma wierszami.

Pomyśl o takiej konstrukcji

    string input;
    string output;

    while( getline(cin,input) && input != "" )
    {
        output += textReduce(input) + "\n";
    }

    cout << output;

gdzie w funkcji textReduce znajduje się właściwy algorytm do skracania wiadomości. 

string textReduce( const string& text )
{
    string result;
    for(....)
    {
        // twój algorytm do skracania wiadomości.
    }
    return result;
}

 

Comments

Dzięki za pomoc, ale mam takie pytanie czy const string i & jest konieczny w definicji argumentu? skoro konkatenacja i tak odbywa się w while'u, dzięki za pomoc.

---

czy const string i & jest konieczny

Nie jest, możesz użyć textReduce( string text ), ale w takim przypadku "przesyłasz" stringa prze wartość, co wiąże się z niepotrzebnym narzutem czasowym i pamięciowym związanym z tworzeniem nowego obiektu typu string ( tutaj text ) w momencie wywołania funkcji. Jeżeli przesyłasz argument przez referencję (jak w przykładzie powyżej) żaden nowy obiekt nie jest tworzony.

---

Rzeczywiście, dzięki za pomoc i odpowiedź.

Answer 2

Chyba nie przetestowałeś programu dla przykładu pod zadaniem.

Program ma wczytywać dowolny tekst, który może się składać z wielu linii.

Ponadto wydaje mi się, że nie powinieneś dotykać pierwszego znaku (jeżeli nie jest spacją).

Comments

Sprawdzałem dla przykładu, ale nie wydaje mi się by było tam wyszczególnione że program wczytuje kolejne linie, ale dzięki za pomoc.