Функция C toLowerCase не работает - Нарушение прав доступа

Question

У меня есть простая функция, которая принимает массив символов в качестве аргумента и преобразует все символы в нижний регистр. Тем не менее, я получаю странную ошибку нарушения прав доступа. Вот код:

void toLower(char *rec) 
{ 
int i=0; 
while (rec[i]!='\0') 
{ 
    if (rec[i]>='A' && rec[i]<='Z') 
       rec[i]='a'+rec[i]-'A'; //this is where I get an error - assigning the 
             //the value to rec[i] is the problem 
    i++; 
} 
} 

Не могли бы вы рассказать мне, в чем моя ошибка? Благодаря

Answer 1

В комментарии вы говорите, что вы передаете в символьной строки в функции, например:

palindrome("In girum imus nocte et consumimur igni") 

где palindrome передает свой аргумент toLower. Это не будет работать, потому что строковые литералы доступны только для чтения, и вы пытаетесь его модифицировать. Вместо этого, вы можете использовать:

char str[] = "In girum imus nocte et consumimur igni"; 
palindrome(str); 

Или вы можете palindrome скопировать свой аргумент в массив и вызвать toLower по этому вопросу.

Answer 2

Ты слишком много работаешь :)

#include <algorithm> //For std::transform 
#include <cctype> //For std::tolower 
#include <cstring> //For std::strlen 

void toLower(char *rec) 
{ 
    std::transform(rec, rec + std::strlen(rec), rec, std::tolower); 
} 

Answer 3

Похоже, вы не прошли в допустимом буфере. Но не пишите это сами; используйте что-то вроде strlwr.

Answer 4

Вы передаете (даже косвенно) строковый литерал? Если это так, то он может быть загружен в неприступную память; вам нужно будет внести изменения в копию.

То, что ваш прототип функции принимает char *, а не const char *, предполагает, что вы, вероятно, этого не сделали, но я подумал, что я его выброшу.