Указатели - надо начальную букву в слове сделать большой используя указатели

[FerrariF70]

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

#include<stdio.h>

void uppercase(char *s);

int main()
{
	int i;
	char *words[] = { "Kuku","abc","good" };
	int size = sizeof(words) / sizeof(int);
	
	for (i = 0; i < size; i++) {
		uppercase (words+i);
	}
	for (i = 0; i < size; i++) {
		printf("%s ", *words + i);
	}
}

void uppercase(char **s)
{
	for (; *s != '\0'; s++) {
		if (**s >= 'a' && **s <= 'z') {
			**s -= 32;
			break;
		}
		break;

	}
}

_____
Код программы нужно выделять (форматировать) тегами [CODE] (читать FAQ)
Модератор

[Cuprum5]

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

char *words[] = { "Kuku","abc","good" };

- что-то здесь не то, в этой строке, не могу понять что, но определенно что-то здесь не то. Звездочку указателя там не надо ставить, по-моему.
Может быть Вы хотели написать вот так:

string words[] = { "Kuku","abc","good" };

Что-то мути много в программе. Берете просто первую строку в массиве words, потом эту строку рассматриваете как массив и берете первый элемент. Смотрите большая там буква или маленькая, кажется есть функция IsUpper() или нет не помню уже, но в любом случае можно по ASCII-кодам посмотреть. Если маленькая, то прибавляете 32. Все.

[FerrariF70]

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

- что-то здесь не то, в этой строке, не могу понять что, но определенно что-то здесь не то. Звездочку указателя там не надо ставить, по-моему.
Может быть Вы хотели написать вот так:

string words[] = { "Kuku","abc","good" };

Что-то мути много в программе. Берете просто первую строку в массиве words, потом эту строку рассматриваете как массив и берете первый элемент. Смотрите большая там буква или маленькая, кажется есть функция IsUpper() или нет не помню уже, но в любом случае можно по ASCII-кодам посмотреть. Если маленькая, то прибавляете 32. Все.

string *words[] = { "Kuku","abc","good" };

это массив указателей без звёздочки будет ошибка. насчёт isUpper() не проверял спасибо за совет. я проверял с toUpper() но это не то я так понял.

[Cuprum5]

isupper() и islower() - с маленькой буквы пишется, я щас посмотрел. А звездочки там не надо, это я Вам 100-пудово говорю. Да, звездочку точно не надо, щас посмотрел в C++Builder 6.
А, вычесть нужно 32, а не прибавить.

[pproger]

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

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

#include<stdio.h>

void uppercase(char *s);

int main()
{
	int i;
	char *words[] = { "Kuku","abc","good" };
	int size = sizeof(words) / sizeof(int);
	
	for (i = 0; i < size; i++) {
		uppercase (words+i);
	}
	for (i = 0; i < size; i++) {
		printf("%s ", *words + i);
	}
}

void uppercase(char **s)
{
	for (; *s != '\0'; s++) {
		if (**s >= 'a' && **s <= 'z') {
			**s -= 32;
			break;
		}
		break;

	}
}

_____
Код программы нужно выделять (форматировать) тегами [CODE] (читать FAQ)
Модератор

ты объявил массив указателей на строки, которые лежат в read only области памяти, их нельзя модифицировать.

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

void uppercase(char *s)
{
	*s = toupper(*s);
}

int main()
{
	int i;
	char words[][32] = { "Kuku", "abc", "good" };
	int size = sizeof(words) / sizeof(words[0]);

	for (i = 0; i < size; i++) {
		uppercase(words[i]);
	}

	for (i = 0; i < size; i++) {
		printf("%s ", words[i]);
	}
}

[FerrariF70]

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

isupper() и islower() - с маленькой буквы пишется, я щас посмотрел. А звездочки там не надо, это я Вам 100-пудово говорю. Да, звездочку точно не надо, щас посмотрел в C++Builder 6.
А, вычесть нужно 32, а не прибавить.

islower() и isupper() это всего лишь проверка на символы нижнего и верхнего регистра.

[FerrariF70]

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

ты объявил массив указателей на строки, которые лежат в read only области памяти, их нельзя модифицировать.

спасибо большое не учёл а вернее не знал что есть такая память только для чтения
это похоже больше как на const. и что получается когда я объявляю массив указателей

 *words[]={"..." , "..." , "..."}

их получается нельзя изменить? потому что они присутствуют как в виде массивов у которых есть свои адреса что по сути является массиву нельзя изменить адрес.

#include<stdio.h>

int main()
{
char *words[] = { "abc", "good" ,"Kuku" };
/*это одно и то же что и это?*/

char arr1[] = "abc";
char arr2[] = "good";
char arr3[] = "Kuku";
char *arr4[] = { arr1, arr2, arr3}; 

*words == *arr4 /* Одно и тоже? */
}

спасибо за решение задачи! Я думал так сделать но хотел всё таки с указателями дойти до этого решения.

[Cuprum5]

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

islower() и isupper() это всего лишь проверка на символы нижнего и верхнего регистра.

- ну да, разве тебе не это нужно? Проверяешь символ - маленький/большой, если маленький, то отнимаешь 32. Понял?

[Alex11223]

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

и что получается когда я объявляю массив указателей

Нет, когда вы используете строковые литералы ("..."). Массив не причем.

[FerrariF70]

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

- ну да, разве тебе не это нужно? Проверяешь символ - маленький/большой, если маленький, то отнимаешь 32. Понял?

не совсем это. просто надо было поменять лишь начальную букву можно и таким способом было. но меньше кода красивее смотрится .