メモリと構造体を用いたコード

Question

メモリ領域のことを勉強し始めて、メンバがchar[50],int priceの構造体Expenseを表示させたいのですが、最初の方に以下のようなコードを書いて、エラーを解決しようと模索しているうちに、頭がパンクしてしまい、以下のコードの意味もよくわからなくなってしまいました。
誰か初学者にもわかるようにコードの書き方を教えていただけませんか？

コード
#include<stdio.h>
#include<string.h>
#include<stdlib.h>

typedef struct{
    char item[50];
    int price;
} Expense;
int main() {
    // メモリの動的確保
    int n;
    Expense *a = (a*)malloc(n, sizeof(Expense));

    Expense product;
    printf("名前 :"); scanf("%s", product.item);
    printf("値段 :"); scanf("%d", &product.price);

    // メモリの解放
    free(a);
}

コンパイル結果
7-2.c: In function ‘main’:
7-2.c:12:21: error: expected expression before ‘)’ token
   12 |     Expense *a = (a*)malloc(n, sizeof(Expense));
      |                     ^

Comments

[1T2R3M4]

先生とか講師に相談してみては。
独学ならばメンターサービス等をご検討されては。

[dodox86]

いくつもの過去の質問が回答をいただいているにも関わらず開いたままですが、ひとつひとつ理解して進めましょう。誰も回答してくれなくなりますよ。むしろ、なぜ今回のような質問が今になってされるのか謎です。

[hoshi-takanori]

n が初期化されてません。
malloc の引数は一つです。(calloc なら二つですが。)
せつかく malloc したやつ使ってないんですが、何をしたいのでしょうか？

[退会済みユーザー]

まあ処理の内容や関数の機能を調べたり理解したりせず、他人にデバッグして貰ってるだけだから、一生進歩しないわな。大人しく入門書からやり直しなさい。teratailは初心者相手に1から10まで丁寧に指導するサイトじゃないです。
[1-1. teratailはサポートセンターではなく、思考するエンジニアの為のQ&Aコミュニティです]
https://teratail.com/help/question-tips#questionTips1-1
> teratailを「サポートセンター」だとは思わないでください。
> 作業依頼のような投稿をして、課題や仕事を無償でやってもらえる場ではありません。
> あなたがそのようなつもりで質問しているのだと他のユーザーから誤解されないように、丁寧な質問を心がけましょう。

もしサイトのヘルプが読めないのであれば、まず国語の勉強からすることをお勧めします。

Answer 1

エラーを解決しようと模索しているうちに、頭がパンクしてしまい …（中略）…

誰か初学者にもわかるようにコードの書き方を教えていただけませんか？

Study and understand before coding.

When considering using the function malloc, you should have checked the specification of the function (before using it).
Therefore, it is usually unthinkable to give the function malloc two arguments.

Similarly, if you know the cast, you can never write (a*).

Comments

[fana]

> コードの書き方

という観点での話として．

初学者だろうがなかろうが，「mallocってどう使うんだっけ？」ってなったら調べる．
malloc( どうのこうの ); とかコードを書く前に確認する．

どこかでmallocの手前に (XXX*) みたいなのがくっついてるのを見かけたことがあるから似たような謎文字列をくっつけとけばよいかな，とかいう話ではない．

コードを書くにはわからんことは調べなきゃならない．
そういうことをやらない姿勢がもうダメ．

[fana]

他の回答で

> ロジックを書く

と述べられているけども，それ以前の問題と言える．

Answer 2

プログラミングってのは「こう書けばいい」っていうものではありません。ましてや「手当たり次第にやるもの」でもありません。ロジックを書くのです。

まずメッセージを読んでください。エラ―メッセージは暴言でも罵倒でもなく、ましてやランダムに出るわけのわからないものでもなく、「コンパイラ等からのメッセージ」です。
今の状態は「相手の話を聞かずに自分で勝手な解釈をして逆ギレしている人」です。そんな人はコミュニケーションなんて取れませんよね。まずはメッセージぐらい読みましょう。

7-2.c: In function ‘main’:
7-2.c:12:21: error: expected expression before ‘)’ token
   12 |     Expense *a = (a*)malloc(n, sizeof(Expense));
      |     

[訳]
7-2.cのmain関数内にて:
7-2.cの12行目でエラーがあるよ。')'の前のやつ、なんか間違ってない?
Expense *a = (a*)malloc(n, sizeof(Expense));　の部分だけど。

では、7-2.cの12行目を見てください。Expense a = (a)malloc(n, sizeof(Expense));の部分です。

このExpense *a = (a*)malloc(n, sizeof(Expense));をそのまま読むと、

Expense型のサイズ(?)のn個分、動的確保してa* という型としてキャスト。そしてそれをExpense *aに入れる。

的なものです。ただし、このmalloc関数の引数部もおかしいです。ここを見ると、引数はsize_t sizeだけです。ということは、関数の使い方も間違っている。

仮に「Expense型をn個分確保する」とかだとしても、キャストするときに(a*)としている。
これは「aという型のポインタとして見なす」です。(キャストの場合)

となると、Expenseとも合わないし、そもそもこのaって本当に型としてあるのでしょうか?

つまり、基礎をないがしろにしていることが原因。まずは基礎からやりましょう!!!!

Answer 3

Expense a = (a)malloc(n, sizeof(Expense));

Expense a = (Expense)malloc(n, sizeof(Expense));

ですね

Comments

[angel_p_57]

malloc(n, sizeof(Expense)) にも突っ込みましょうよ。
※それ以前になんのために malloc するのが不明なので「なんか知らんけど少なくとも使い方から間違ってる」としかコメントはできないんですが

Answer 4

Expense a = (a)malloc(n, sizeof(Expense));

(a *) ここは、型を書くところです。
従って、
Expense *a = (Expense *)malloc(n * sizeof(Expense));

malloc()関数は、ポインタchar *で確保した領域を返すので、多くの場合、そのままでは使えません。
目的とする領域へのポインタ(今回は、(Expense *))に変換(キャスト)する必要があります。

---

malloc()について、修正。
キャストの件は、解決済みのようなので、このまま。

Comments

[退会済みユーザー]

C++ではなくC言語ではキャスト不要ですし、そもそもmallocの引数がおかしいです。

[pepperleaf]

既に解決済みなので、放置してましたが、
malloc()のみ修正