-

[drozdvadym]

No description provided.

[drozdvadym]

Яку функцію ти не назвав би оптимальною?

[Youw]

загалом. ти робиш копію рядка, а потім його розвертаєш. Лишній прохід по цілому рядку.

[Youw]

думаю це варто випраивити, як і в інших файлах також

[drozdvadym]

Звідки ви знаєте що використовується в strdup? в студентській реалізації так, повний обхід, в нормальній тільки strlen повністю обходить, а далі або memcpy або розгортання циклу і так далі, тобто оптимізації.

На рахунок swap згоден.

Якщо ж писати в C++ стилі взагалі краще використати std::swap :).

На рахунок макросів - почитайте буст і сирци різних контейнерів, здивуєтеся що там є багато макросів. Або ж Qt почитайте, там теж багато.
Макроси це не зовсім сішний стиль - це інструмент препроцесора, який і надалі присутній в C++.

Звичайно в більшості випадків, в який використовують макроси студенти в С, в С++ краще їх замінювати const, inline, template (в моєму випадку теж краще було б inline використати)

Якщо ж писати повністю в C++ стилі, то я зробив би так:

#include
#include // std::reverse
#include

static const char* str = "The string!";

int main () {
std::string result(str);

std::reverse(result.begin(), result.end());

char* reverse_string = &result[0];

std::cout << "Original: " << str << std::endl;
std::cout << "Reversed: " << reverse_string << std::endl;

return 0;
}

Дякую за критику. Завдяки ній і учимося :)

[drozdvadym]

Напевне зараз і закомічу такий варіант :)