Расположить слова строки в порядке, обратном исходному.

[ROBODEF]

Расположить слова строки в порядке, обратном исходному.
Помогите пожалуйста.

[Blind Guard]

char *str = "programmers forum";
char *str2 = new char[strlen(str)+1];
int k=strlen(str)+1, i2=0;
for(int i=strlen(str)+1; i>0; i--)
    if(str[i]==' ')
    {
        for(int j=i; j<k; j++)
            str[i2++] = str[j];
        k=i;
    }

[Smogg]

strlen возвращает длину в символах:

Return Value
Each of these functions returns the number of characters in str, excluding the terminal NULL.

Т.е. str2 уже зарезервировало памяти на один символ меньше.

А еще нет проверки, что дошли до начала исходной строки, но надо ж сохранить слово и от начала до первого пробела.

оператор "=" возвращает результат присваивания, т.е. будет true в любом случае, кроме присваивания нуля.

Бестолковые имена переменных. Вы уже в двух переменных запутались, а если их будет 100? Когда каждая строка означает что-то свое, но все названы "str" + "набор цифр"?

UPD: Хитрец, хитрец Blind Guard))) Не сразу до меня дошло) Специально так накосячил? Ведь тут применены все классические ошибки и без радикальной ревизии этот код бесполезен. Т.е. тупым копипастом не обойтись.

[Blind Guard]

Smogg, спасибо за некоторые замечания, исправил всё, что на самом деле считал нужным.
Я умею принимать критику, но в последнем предложении вы не правы.
Где я запутался с двумя переменными? Мб это вы запутались в моём коде? (хоть это и нереально )
Виной глупым ошибкам стало то, что многие эти моменты я давно не применял.
Ну и я не ставил себе цель написать идеальный код, я дал идею чтоли.
Кому надо, тот поймёт и подправит.

[Rififi]

ROBODEF

#include <string>
#include <list>
#include <sstream>
#include <iostream>
#include <locale>

int main()
{
    std::locale::global(std::locale(""));
    std::string w, text = "Расположить слова строки в порядке обратном исходному";
    std::istringstream ss(text);
    std::list<std::string> words;
    while (ss >> w)
        words.push_front(w);
    for (auto it : words)
        std::cout << it << " ";
}

пример работы: http://liveworkspace.org/code/37PEmN$3

[Smogg]

Сообщение от Blind Guard

Где я запутался с двумя переменными? Мб это вы запутались в моём коде? (хоть это и нереально )

Это кажется, что нереально заблудится в трех соснах))

for(int j=i; j<k; j++)
str[i2++] = str[j];

Где ошибка?

[Blind Guard]

Сообщение от Smogg

Где ошибка?

Хоть убейте не вижу ошибки.

[Smogg]

Ок, тогда вам дается два варианта:
1. Начинать копать могилу
2. Подтвердить свою правоту. Т.е. довести до компилябельного состояния свою идею по перетусовке слов в исходной строке согласно заданию, но с добровольно принятым ограничением: использовать только один char'овый буфер величиной в один символ. Причем этот буфер даже не отдельной переменной, а под него отводится та ячейка памяти из массива исходной строки, в которую записан терминирующий нуль.
А если вы истинный джедай хардкора, то сумеете обойтись всего тремя уже объявленными счетчиками, т.е. k, i, i2;

//Вот такой вот простенький тест на MemoryOptimizationSkill )))