Odwracanie tekstu

Question

Mógłby ktoś ocenić poniższy kod i napisać, czy i jak dałoby się to napisać tak, by funkcja była bardziej uniwersalna/efektywna/prostsza/lepsza pod jakimś innym względem?

void zadanie()
{
	const int n = 6;
	char tekst[n] = { 'a', 'b', 'c', 'd', 'e', 'f' };
	char rev[n];
	

	for (int i = 0; i < n; i++)		
	{								
		rev[i] = tekst[(n - 1) - i];
	}
	for(int i = 0; i < n; i++)
		cout << rev[i];
}

 

Comments

1. Po pierwsze lepiej kiedy tekst do odwrócenia będzie podawany jako argument dla funkcji ( możesz użyć tradycyjnych łańcuchów z C lub z std::string jeżeli piszesz to w c++ )

2. Co do samego tekstu, nie lepiej by był on w formie łańcucha? Tzn. dodajesz na koniec tekstu znak '\0' czyli nul, wtedy też nie będziesz musiał wypisywać tego tekstu w pętli for, lecz za pomocą np. printf()

3. Zamiast wyświetlać tekst w funkcji zwróć wskaźnik na niego ( jeżeli ma być to łańcuch jak w C to rzecz jasna musisz zrobić dynamicznie alokowaną tablice char'ów )

---

Zamiast wyświetlać tekst w funkcji zwróć wskaźnik na niego

O, a w jakim celu wskaźnik? :)

Dodałbym do listy po prostu skorzystanie z std::reverse()... :P

---

Dodałbym do listy po prostu skorzystanie z std::reverse()... :P

Pisanie czegoś samemu zamiast korzystać z gotowych metod chyba bardziej uczy

O, a w jakim celu wskaźnik? :)

chodzi mi o funkcję:

char* zadanie(const char* str); //zwróci odwróconą wersje 

oczywiście było by prościej użyć std::string

---

Tak właściwe to po co to pisać? Po co wyważać drzwi od nowa? 

 

PS sam nie ogarniał tych nowostek, nie jesteś sam

---

Pisanie czegoś samemu zamiast korzystać z gotowych metod chyba bardziej uczy ;)

Też, ale główna przyczyna jest taka, że po prostu  takie mamy zadanie domowe.

 

Answer 1

Nie no... można... 

#include <iostream>

void revStr(char * msg, std::size_t n) {
    char * end = msg + n - 2;
    while(msg < end) {
        char c = *end;
        *end-- = *msg;
        *msg++ = c;
    }
}

int main() {
    char msg[] = "Ala ma kota";
    revStr(msg, sizeof(msg));
    std::cout << msg << '\n';
}

Ale to takie kruche że ... lepiej nie... 

Answer 2

Podpinam się pod temat .

Dlaczego nie wczytuje mi pierwszego słowa ? 

#include <iostream>
#include <string>


int main() {
    
    const int ile = 2;
    
    std::string napis[ile];
    int dlugosc[ile];
    
    std::cout << "Podaj wyraz : ";
    std::cout << std::endl;
    
    for (int i = 0; i <= ile; i++)
    {
        std::cin >> napis[i];
        dlugosc[i] = napis[i].length();
    }
    
    
    for (int i = 0; i <= ile; i++)
    {
        std::cout << "Napis nr : " << i+1 << " to " << napis[i] << std::endl << dlugosc[i] << std::endl;
    }
    
    return 0;
}

 

Comments

Nie wiem czy na pewno problem jest w tym, ale deklarujesz 2 stringi a w pętli wczytujesz 3 razy - i=0, i=1, i=2. Dokładnie to samo jest w drugiej pętli

---

Dziękuję za nakierowanie.

String gdy zmieniłem na 3 to wczytywał napis 4 razy ( od 0 do 3 ) , więc wtedy usunąłem '=' z for i problem rozwiązało :)

---

@AizoOz, 

#include<iostream>
#include<string.h>
#include<stdio.h>
 
int main() {     
    const int MAX_CHAR_PROCESSING = 100;
    char sentence[MAX_CHAR_PROCESSING], * word;
    
    std::cout << "Podaj zdanie (max. dlugosc: " << MAX_CHAR_PROCESSING << " znakow):\n";
    std::cin.getline(sentence, MAX_CHAR_PROCESSING);
    
    int length_ = strlen(sentence);
    std::string line = "+-----------+-----------------+---------+\n";
    
    printf("%s", line.c_str());
    printf("| Wyraz nr. |  Wyraz (fraza)  | Dlugosc |\n");
    printf("%s", line.c_str());

    int count_word = 0;
     
    word = strtok(sentence, " ");
     
    while (NULL != word) {
        count_word++;
        printf("|%6d%5c|%12s%5c|%5d%4c|\n", count_word, 32, word, 32, (int)strlen(word), 32);
        word = strtok(NULL, " ");
    }
   
    printf("%s", line.c_str());
    std::cout << "  Ilosc zankow: " << length_ << " (wszyskie + spacje)\n";
    std::cout << "    Ilosc slow: " << count_word;

    return 0;
}