Q&A
配列の変数宣言部分も示してください。
char*で宣言したarrayにstrtolを使って、longで宣言したarrayiに代入しようと
したのですが、_p!=nullptrとエラーが出てうまくいきません。
C++
1struct Arrays{ 2 char* array[ex][ey][3]; 3 long arrayi[ex][ey][4]; 4} 5~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 6 7 Arrays arrays; 8 fp = fopen(file, "rb"); 9 fread(header, 1, 54, fp); 10 for (int i = 0; i < ex; i++) { 11 for (int j = 0; j < ey; j++) { 12 for (int k = 0; k < 3; k++) { 13 fread(&arrays.array[i][j][k], sizeof(int), 1, fp); 14 } 15 } 16 } 17 for (int i = 0; i < ex; i++) { 18 for (int j = 0; j < ey; j++) { 19 for (int k = 0; k < 3; k++) { 20 arrays.arrayi[i][j][k] = strtol(arrays.array[i][j][k], (char**)NULL, 10); 21 } 22 } 23 }
どなたかご教示お願いします…。
array[][][] の型は?
arrays.array[i][j][k] には longに変換可能な文字列がちゃんと入っていますか? 確認しましたか?
もしかして、char array[ex][ey][3]; ということ? そしたら、当然、エラー。
char* array[ex][ey][3]で宣言しているのですがエラーになります…
文字列は入っていると思うのですが、freadを使い慣れておらずもしかしたら間違っているかもしれません
> 文字列は入っていると思うのですが
printfでもすればわかるでしょ。なんで確認せんのですか?
>printfでもすればわかるでしょ
coutで出してみたのですが、コード -1073741819 で終了してしまいます。
「文字列は入っていると思うのですが」がマチガイだった、と。
なら strtol がどーのこーの以前の問題です。変換不能な文字列を渡してるんだから。
もう少し文字列が入ってるかどうか確認してみます。
バイナリモードのファイルから char* に sizeof(int) で文字列を読み込んでいるそうなのでめちゃくちゃでしょう。
array を char にしてもう1次元追加する必要があるのでは。
>バイナリモードのファイルから char* に sizeof(int) で文字列を読み込んでいるそうなのでめちゃくちゃでしょう。
>array を char にしてもう1次元追加する必要があるのでは。
やってみたのですが、読み込んだデータがめちゃくちゃになって文字化けしていました。
「やってみた」ではこちらには全く修正内容が分かりませんが。
それと、いったいどのようなファイルを読んで arrayi に入れようとしているのでしょうか。ex,ey は幾つになっているのでしょう。
回答6件
0
ベストアンサー
BMP ファイルを対象にしたプログラムなのでしたら、そのようにご質問に書かれていればもっと簡単にお話が進んだと思うのですが…。
BMP のフォーマットの中でも一部に限定するとしてヘッダ解析を飛ばしてあるとしまして、まず strtol は不要ですから arrayi は無くすものと考えます。
array が 3 次元なのは 幅・高さ・RGB(各1バイト)でしょう。
なら最小単位は R・G・B の各バイトですので array の型は (char*では無く) char でなければならないです。(なぜポインタにしているのか謎です。)
それを 3 重ループで読むのであれば、 fread では 1 バイトずつ読むことになるはずです。
読み込むサイズは (sizeof(int) では無く) sizeof(char) もしくはすっきり 1 と書くことになります。
…としましたが。
どうせ連続して配列の大きさ分を全部読むのでしたら、ちまちま 1 バイトずつ読まなくても header のように丸ごと前部読んでしまえば簡単ではないでしょうか。
投稿2021/11/01 20:21
編集2021/11/01 20:34総合スコア13209
0
BMPファイルのデータはバイナリであり文字列ではありませんから、
strtol を使うことは全くの見当違いです。
投稿2021/11/01 13:05
総合スコア8224
0
freadは基本的にはバイナリーデータを読むために使います。
文字列データを読みたいのなら、fgetsとか、fscanfとかを、入力ファイルの形式に合わせて使います。
投稿2021/10/31 12:39
総合スコア85901
bmpファイルなのでバイナリデータを読み込めばRGB値（0～255）が出てくると思ったのですが、違うのでしょうか？文字列として読み込んだほうがいいですか？
ああ、バイナリデータなのであれば、strtol するのがおかしいです。
strtolしているので、てっきり文字列データだと思いました。
バイナリデータのbmpをfreadで読み込んだのですが、int arrayだと値が4302920みたいにおかしな数字になってしまいます。なのでchar型で宣言してそこにfreadでarrays.arrayに格納しました。てっきり文字列の扱いになっていると思ってstrtolを使ってしまいました。
BMPファイルを読むのであれば、ファイルフォーマットをちゃんと理解してないといけませんが、
> int arrayだと値が4302920みたいにおかしな数字になってしまいます。
と言っている時点で、理解の入り口にすら立ってない気がしますので、かなりスキルアップしないと駄目でしょうね。
ヘッダもきちんと飛ばして読み込んでいるんですが、unsigned char arrayにしてfread(arrays.array[i][j][k], sizeof(char), fp)にしているのに文字化けしてしまうんです。これでも間違っているんでしょうか…？
読んだ後それをどう解釈するかですね。
…どういう意味でしょうか？すみません、ヒントをください
> ファイルフォーマットをちゃんと理解してないといけません
と書いたとおりです。理解すればどうしたらいいかわかるはずです。
...formatもわからんデータを闇雲に読んでるのか。まるでデタラメやな。
0
C++
1struct Arrays{ 2 char* array[ex][ey][3]; 3 long arrayi[ex][ey][4]; 4} 5... 6 fread(&arrays.array[i][j][k], sizeof(int), 1, fp);
arrays.array[i][j][k] には fread で読んだ文字列を格納できるだけの十分な領域が確保されていますか?
投稿2021/10/31 12:28
総合スコア16612
0
arrayi[i][j][k] = strtol(array[i][j][k], (char**)NULL, 10);
strtol()の宣言を見ると、long strtol(const char *s, char **endptr, int base);
第一引数が、 char *で、戻り値がlong これに対し、~~同じ arrayi[i][j][k] を与えてる時点で不適切。 ~~
よく見ると、array と arrayi で違った。すみません。
改めてみると、 array[][][] は char *で定義されているけど、実体(実際の文字列を入れる領域)が無いみたいです。それを確保しないとダメでしょう。(書かれてないところにある?)
例えば、char array[ex][ey][3][50];とか。ただ、単純にそうしても ex,ey のサイズ次第でエラーになるかも。
投稿2021/10/31 12:22
編集2021/11/01 11:33総合スコア6385
すみません、知識不足で何が不適切なのかが分かりません。
引数は array / 結果は arrayi で受けてるので、そこは問題なさげ
変数が違う事を見逃してました。他にもありそうですが、少しだけ、見直し。(多分、足りない)
0
どー見ても使い方がおかしいです。
関数の仕様を調べてみては
投稿2021/10/31 12:13
総合スコア88042
調べてみましたがどこが間違っているのか見当がつかず…すみません…
C言語 strtol、でぐぐれば、関数の仕様から使い方にサンプルコードまででてきますけど、どのあたりがわからないんでしょうか
それ通りにやっているつもりなのですが、全くエラーの箇所の検討がつかないのです。
その関数は、文字列を整数に変換するものです
関数の引数には、文字列のアドレスを与える必要があります
コードを見る限り、arrays.arrayには何も代入されてませんけど、ホンマに文字列のアドレスが入ってるんでしょうか
なんかどんどん質問文が変更されてますねｗ
きちんと全文出るまで待つことにしますね
あなたの回答
tips
プレビュー
