C++, strupr и указатели

[[DarkNet]Alpha]

Внезапно оказалось, что в стандартной (не от микрософта) string.h нет функции strupr и подобных. Решил я написать расширение.
Вот порождение моего больного сознания:

Код: [Выделить]

void strupr(char *s){
    char *res=s;
    while(*res!='\0'){
        *s=toupper(*res);
        s++;
    }
}

Код: [Выделить]

#include <iostream>
#include <string.h>
#include "h/strex.h"
using namespace std;

int main()
{
    char *pstr1="ololowcase";
    strupr(pstr1);
    cout << *pstr1 << endl;
    return 0;
}На выходе получаю ошибку сегментирования. ЧЯДНТ?

[Kwah]

Забываешь cдвигать res?

[[DarkNet]Alpha]

Всё равно ошибка сегментирования.
Пользователь решил продолжить мысль 06 Октября 2010, 00:43:41:На статической строке сработало. Правда, я не понял, как...

[VestniK]

Вот такой код выглядит куда логичней:

Код: [Выделить]

void strupr(char *s){
    char *ptr = s;
    while(*ptr != '\0'){
        *prt = toupper(*ptr);
        ptr++;
    }
}

З.Ы. сам не тестил
Пользователь решил продолжить мысль [time]Wed Oct  6 00:53:17 2010[/time]:А почему бы не std::string использовать, там через итераторы сложней вылезти за пределы строки и получить сегфолт

Код: [Выделить]

std::string s;
for (std::string::iterator it = s.begin(); it != s.end(); it++) {
    *it = toupper(*it);
}


[[DarkNet]Alpha]

О, спасибо, добрый человек.
Можете кратко рассказать или дать хорошую ссылку про указатели? Я испорчен паскалем и питоном, и такое насилие над строками не могу понять...

[Kwah]

Вот так работает

Код: [Выделить]

#include <stdio.h>
#include <string.h>
#include <ctype.h>

void strupr(char*);

int main()
{
    char pstr1[] = "ololowcase";
    strupr(pstr1);
    printf("%s\n", pstr1 );
    return 0;
}

void strupr(char *s){
    char *res=s;
    while(*s!='\0'){
        *res=toupper(*s);
        s++;
        res++;
    }
}


[VestniK]

Ну а что именно интересует? Указатель это просто адрес области памяти где лежат твои данные. В C/C++ есть арифметические операции над указателями, но с ними лучше быть аккуртней. Через них можно получить указатель на область памяти которая твоему приложению не принадлежит и при попытки что-нибудь с этой памятью сделать ты олучишь сегфолт.

А так всё просто например обрати внимание на "случайное" совпадение в выводе следующей программы:

Код: [Выделить]

#include <stdio.h>

int main(int argc, char **argv)
{
    int  a[5];
    long long b[5];

    int  *ptr_i  = a;
    long long *ptr_ll = b;

    size_t dist_i  = (size_t)(ptr_i+1) - (size_t)ptr_i;
    size_t dist_ll = (size_t)(ptr_ll+1) - (size_t)ptr_ll;

    printf("dist_i  = '%d'; sizeof(int)       = '%d'\n", dist_i , sizeof(int));
    printf("dist_ll = '%d'; sizeof(long long) = '%d'\n", dist_ll, sizeof(long long));

    return 0;
}

@Kwah а зачем обоих сдвигать? Это же пустая трата времени на ненужную дополнительную арифметическую операцию в теле цикла.

[[DarkNet]Alpha]

Такс... strlwr и strupr осилил, до кучи написал strrev. Попытался встроить это в string.h, но этот файл поверг меня в ужас. Стоит ли искать в нём начала и концы, или пусть себе будет strex?

[Toxic]

[DarkNet]Alpha, ошибку получаешь, потому что константную строку пытаешься модифицировать. Создай буфер, пихни в него строку и передавай указатель на буфер. Посмотри как сделан pstr1 у Kwah. И не забываем про arguments sanity check - внутри функции обязательно проверку аргумента на NULL поставь.

[Kwah]

@Kwah а зачем обоих сдвигать? Это же пустая трата времени на ненужную дополнительную арифметическую операцию в теле цикла.

Ну, оригинал написан с использованием двух указателей. Из-за того, что res++ отсутсвует получается бесконечный цикл, затирающий сначала оригинальную строку, а потом всё подряд после неё. Ну... И инерционность мышления

Интересно, а оптимизатор эту лишнюю арифметическую операцию не распознает?

[VestniK]

ну можно попробовать скомпилировать с -O0 -O1 -O2 -O3 и посмотреть дизасемблером на эту функцию, вот только я не знаю как дизасемблером пользоваться (в особенности, как там найти конкретную функцию)