VPF::ошибка начинающих - Форум программистов

sergioK1

Дата 14.5.2011, 11:55 (ссылка)

(голосов:1)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 417
Регистрация: 30.1.2011

Репутация: нет
Всего: нет

Код


#include <string.h>
#include <stdlib.h>

main(){
  const char* from = "12345678";
  char *to = (char*) malloc(6);
  strncpy(to, from+2, 5);
}

грамотно писать

Код


  char *to = (char*) malloc(6* sizeof(char));

предположение что char - 1 байт не всегда верно ,

Это сообщение отредактировал(а) sergioK1 - 14.5.2011, 13:32

borisbn

Дата 14.5.2011, 12:12 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 4875
Регистрация: 6.2.2010
Где: Ростов-на-Дону

Репутация: 21
Всего: 135

Цитата(sergioK1 @ 14.5.2011, 11:55

)

char *to = (char*) malloc(6* sizeof(char);

Цитата(Стандарт N1905=05-0165)

5.3.3 Sizeof
... sizeof(char), sizeof(signed char) and sizeof(unsigned char) are 1.

хотя, для самодокументированности кода можно и писать sizeof.
а ещё лучше

Код


char *to = (char*) malloc( 6 * sizeof(*to) );

чтобы в случае смены типа to меньше изменять в коде.

и ещё: у тебя закрывающей скобки в конце не хватает - malloc(6* sizeof(char); smile

Это сообщение отредактировал(а) borisbn - 14.5.2011, 12:13

--------------------

Женщины отличаются от программистов тем, что у них чары состоят из стрингов

Artemon

Дата 14.5.2011, 12:25 (ссылка)

(нет голосов)

Загрузка ...

а ты мне нравишься

Профиль
Группа: Завсегдатай
Сообщений: 1771
Регистрация: 24.2.2004
Где: Челябинск

Репутация: 1
Всего: 20

Грамотно - это вообще не использовать для хранения строк тип char*, вместо этого использовать std::string или аналоги.

--------------------

Контроль топлива на топливозаправщиках, мониторинг автотранспорта, расчет зарплаты водителей www.rscat.ru

boostcoder

Дата 14.5.2011, 13:00 (ссылка)

(голосов:1)

Загрузка ...

pattern`щик

Профиль
Группа: Завсегдатай
Сообщений: 5458
Регистрация: 1.4.2010

Репутация: 20
Всего: 110

Цитата(borisbn @ 14.5.2011, 12:12

)

и ещё: у тебя закрывающей скобки в конце не хватает

и еще не понятно, что за magic-number в аргументе malloc`а ? ;)

--------------------

компилятор онлайн с поддержкой C++11
сборки компиляторов MinGW

sergioK1

Дата 14.5.2011, 13:46 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 417
Регистрация: 30.1.2011

Репутация: нет
Всего: нет

Цитата(Artemon @ 14.5.2011, 11:25)

скобку подправил,

Код


  char *pChar =(char*) malloc(10*sizeof(char));  // еще лучше  не лениться саsting ставить

на С нет класса String.
зачем писать на С , когда есть С++ - это вовсем другая тема,
почему два разных языка идут вместе Я уже давно никак не могу понять ,

Это сообщение отредактировал(а) sergioK1 - 14.5.2011, 14:57

sergioK1

Дата 15.5.2011, 16:40 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 417
Регистрация: 30.1.2011

Репутация: нет
Всего: нет

Цитата(borisbn @ 14.5.2011, 11:12)

Цитата(sergioK1 @ 14.5.2011, 11:55

)

char *to = (char*) malloc(6* sizeof(char);

а ещё лучше

Код


char *to = (char*) malloc( 6 * sizeof(*to) );

чтобы в случае смены типа to меньше изменять в коде.

так воппще для ленивых , правда не для всех компиляторов,

Код



  char *to = (char*) malloc( 6 * sizeof( __typeof__( to)  ) );

borisbn

Дата 15.5.2011, 17:19 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 4875
Регистрация: 6.2.2010
Где: Ростов-на-Дону

Репутация: 21
Всего: 135

Цитата(sergioK1 @ 15.5.2011, 16:40

)

__typeof__( to)

Это ж будет указатель char* smile

а его sizeof = 4 (или 8)

--------------------

Женщины отличаются от программистов тем, что у них чары состоят из стрингов

gerov

Дата 15.5.2011, 23:13 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 1
Регистрация: 4.3.2011
Где: все равно в душе

Репутация: нет
Всего: нет

ааа. Спасибо. Очень познавательно!

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Для новичков \| Следующая тема »