Piramidka - problem z wyświetlaniem znaków w konsoli

adampp93 2018-02-10 10:45:33 UTC #1

Witam,

Problem związany jest z prawdopodobnie jednym z łatwiejszych zadań na Spoju akim jest piramidka.

Treść zadania:

Wariant 1
Dla programu napisanego po najmniejszej linii oporu wszystko działa bez zarzutu.

Kod

Widok programu

adampp93 2018-02-10 10:46:05 UTC #2

Wariant 2
Przy próbie ulepszenia programu i zamknięciu w funkcji pętli wyświetlającej stringa w pierwszej linii wyjścia pojawia się następujący problem

Kod

Widok programu

Chciałbym nadmienić, że w drugim wariancie sędzia również przyjmuje program, zatem problem dotyczy wyłącznie wyświetlania znaków w konsoli. Zaznaczam również, że funkcja display() zupełnie poprawnie wyświetla stringa s1. Czy ktoś zatem potrafiłby mi wyjaśnić co może być przyczyną tych krzaczków?

PS Przepraszam za rozbicie, ale nie mogę wstawić więcej niż 1 obrazka w poście.

tarpauwatratar 2018-02-10 10:57:54 UTC #3

Wklejając kod na Ideone wybieraj dobry kompilator.
Robiąc zadania na SPOJu nie rób walidacji wejścia - jedyne co może z tego wyniknąć to WA bądź TLE, a na pewno jest to błąd w sztuce
Piszesz poza zaalokowaną przez stringa pamięcią. Widać to już po losowości wyświetlanych znaków. Po i przed wykonaniem pętli rozmiar stringa wynosi zero.

adampp93 2018-02-10 11:30:13 UTC #4

Nie zauważyłem tam tej opcji, na przyszłość będę wybierał właściwy kompilator.
Dzięki za uwagę, nie wiedziałem, że to błąd, ale będę tego unikał.
Zgodnie z Twoją uwagą, aby nadać stringowi s2 rozmiar zapisałem w kodzie linię “s2 = s1;” zaraz po wprowadzeniu s1 z klawiatury i pomogło

Dzięki za wszystkie uwagi

tarpauwatratar 2018-02-10 11:35:02 UTC #5

W sztuce - błąd. Ale oczywiście formalnie jest to po prostu Twój wybór. To trochę jakbyś kazał komuś podać jakąś liczbę od 1 do 10, a potem zapytał go, czy ta liczba jest mniejsza od zero lub większa od 10 Autor podaje zakresy i po prostu się ich trzymamy. I tyle
Poczytaj o klasie string. Są inne (lepsze?) rozwiązania tego problemu, np. zamienienie = na += albo metoda resize, … Twoja metoda oczywiście też przejdzie, ale poza światem SPOJa wymaga: więcej pamięci, więcej czasu na obliczenia i nie jest zbyt profesjonalna (tak zwany zelentyzm)