質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.34%

  • C

    4007questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

  • C++

    3791questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

malloc関数、文字化け

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 788

strike1217

score 563

Windows で実験コードを作成していたら、表示がおかしくなりました。

#include<stdio.h>
#include<stdlib.h>
#include <string.h>
#define str "you will make me happy"

int main() {
    int* p = NULL;
    long i = 0;
    p = (int *)calloc(1, sizeof(int));  // sizeof(int)の大きさを1個作る.

    char* q = (char *)malloc(sizeof(char) * (strlen(str)));

    if (p && q) {
        *p = 100;
        for (i = 0; i < strlen(str); i++)
            q[i] = str[i];
        printf("Succeed!!%d : %s : %d", *p, q, strlen(str));
        free(p);
        free(q);
    }
    else
        printf("failed");
    return 0;
}


Succeed!!100 : you will make me happy86・モT・ : 22

という風に文字化けしてしまいます。
mallocのところをcalloc()でやるとうまくいきました。
このコードどこがおかしいのでしょうか??
ダメな部分の指摘をお願いします。

ちなみに、for文を以下のようにしたら、化けなくなりました。

for (i = 0; i <= strlen(str); i++)
            q[i] = str[i];


それから、文字列を短いとうまくいきます。
長くするダメになります。

#define str "HAPPY"

なぜですか??
配列イメージだと一番最初のコードでよさそうですが・・・・

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+2

strxxx系のファンクションは、長さを指定しない代わりに\0をストッパーとしています
ですので\0が見つかるまでを対象としています。
また、printfの%sも同様な動きをします。callocは\0で埋めてくれますが、mallocで、
同様とするには自身で\0埋める必要があります。定義された領域もたまたま後ろが\0で
有る場合が多く正常っぽく動く事があります。

#include<stdio.h>
#include<stdlib.h>
#include <string.h>
#define str "you will make me happy"

int main() {
    int* p = NULL;
    long i = 0;
    p = (int *)calloc(1, sizeof(int));  // sizeof(int)の大きさを1個作る.

    char* q = (char *)malloc(sizeof(char) * (sizeof(str))); //sizeofを使う23

    if (p && q) {
        *p = 100;
        for (i = 0; i <= strlen(str); i++) //最後に\0を入れる
            q[i] = str[i];
        printf("Succeed!!%d : %s : %d", *p, q, strlen(str));
        free(p);
        free(q);
    }
    else
        printf("failed");
    return 0;
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/19 20:31 編集

    今まで、配列のコピーとかは以下のようにやっていました。
    for (i = 0; i < strlen(str); i++)

    ただの配列の時は問題なくできるのですが、これではダメということですね!

    for (i = 0; i < strlen(str) + 1; i++)
    こうでもよいのでしょうか??

    キャンセル

  • 2017/01/19 20:55

    焦点は、q[i] = str[i];の部分たと思います。#defineは、プリコンパイラがstr文字列を "you will make me happy" に変えてCコンパイラに渡すと思います。ですのでstr[22]は、領域外となります。
    ですので「q」のエリアを1バイト長くmallocしてそこに\0を入れる様にしています。for文は22で抜けて
    来ますのでそれを使っています。

    キャンセル

  • 2017/01/19 21:02

    ふむふむ・・・

    #define str "you will make me happy"でも終端にヌル文字は付きますよね??

    キャンセル

  • 2017/01/19 21:09

    訂正します。
    すみませんstrは23バイトの領域を取り、NULLが付いています。ご指摘ありがとうございます。
    ですので for (i = 0; i < strlen(str) + 1; i++) for (i = 0; i <= strlen(str); i++) でもOKです。

    キャンセル

  • 2017/01/19 21:15

    ありがとうございます!

    キャンセル

  • 2017/01/19 22:28

    文字列のコピーはforでループを回さず、strncpy関数を使いましょう。

    キャンセル

  • 2017/01/20 17:25

    やはり標準関数を使用した方が良いんですね!
    ありがとうございます。

    キャンセル

+2

メモリの確保量が1バイト足りません。

char* q = (char *)malloc(sizeof(char) * (strlen(str)+1));

あと malloc したあとのメモリはゼロクリアされていないので自分で 0x00 を最後に入れる必要があります。マクロの場合は最後に 0x00 が入っているのでそれがコピーされます。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/19 19:54

    char* q = (char *)malloc(sizeof(char) * (strlen(str)+1));
    これだけでは、うまくいきませんでした。
    for (i = 0; i < strlen(str) + 1; i++)
    こうしないとダメみたいです。

    キャンセル

  • 2017/01/19 21:06

    #define str "you will make me happy"でも終端にヌル文字は付きますよね??

    キャンセル

  • 2017/01/20 14:35

    > こうしないと

    0x00 を最後に入れる必要があります。

    がその意味になります。

    キャンセル

  • 2017/01/20 14:36

    マクロで、と言った意味は strcpy を使えばそこまではコピーしてくれるという意味です。

    キャンセル

  • 2017/01/20 17:24

    ありがとうございます。

    キャンセル

+2

終端に 0 が必要だった気がします。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/20 17:25

    ありがとうございます。

    キャンセル

同じタグがついた質問を見る

  • C

    4007questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

  • C++

    3791questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。