C言語なら俺に聞け 151

2019/04/02(火) 11:23:37.62

!extend:checked:vvvvv:1000:512
C言語の話題のみ取り扱います C++の話題はC++スレへ
質問には最低限の情報(ソース/コンパイラ/OS)を付ける
数行で収まらないソースは以下を適当に使ってURLを晒す
https://paiza.io/
https://ideone.com/
http://codepad.org/

C11
http://www.open-std.org/jtc1/sc22/WG14/www/docs/n1570.pdf

C99
http://www.open-std.org/jtc1/sc22/WG14/www/docs/n1256.pdf
http://kikakurui.com/x3/X3010-2003-01.html

C FAQ 日本語訳
http://www.kouno.jp/home/c_faq/

JPCERT C コーディングスタンダード
https://www.jpcert.or.jp/sc-rules/

VIPQ2_EXTDAT: checked:vvvvv:1000:512:----: EXT was configured

2019/05/12(日) 15:34:15.26

質問です
テキストファイルから文字を1文字ずつ読み込んでchar型の2次元配列matrixに入れていきたい。
この時改行は配列に入れたくないときどうすれば良いですか？

うまくいかないコード
for (i=0;i<10;i++) {
for (j=0;j<10;j++) {
if (fgetc(fp) == ‘\n’) {
continue;
}
matrix[i][j] = fgetc(fp);
}
}

テキストファイルは行数と列数があらかじめ分かってるとして
例えば
AAAAAAAAAA
BBBBBBBBBB
…
のようなファイルです

2019/05/12(日) 15:51:11.62

fgetc二回呼びたいの？

2019/05/12(日) 16:10:46.32

>>498
continueだと内ループ先頭に戻るから違うんじゃないの

2019/05/12(日) 16:24:47.11

>>498
.for ... {
. for ... {
. char c = fgetc(fp);
. if (c != '\r' && c != '\n') {
. matrix[i][j] = c;
. }
. }
.}

2019/05/12(日) 17:12:05.74

>>499
呼ばなくていいです
>>501
これでもうまくいかないです

**経団連** ◆T6xkBnTXz7B0 (ﾜｯﾁｮｲ 1701-PJeF) · 2019/05/12(日) 17:22:08.08

for (i=0;i<10;i++) {
for (j=0;j<10;j++) {
matrix[i][j] = fgetc(fp);
}
fgetc(fp); //読み捨て
}

2019/05/12(日) 17:29:14.22

変な例出すなよ…

2019/05/12(日) 17:29:29.19

サイズが固定なら fread でよくない?
そうすべきというほどのものでもないだろうが。

2019/05/12(日) 17:35:10.27

>>498
「うまくいかない」症状を見れば、もう少し見当がつくんだけど…。
格納先の配列を char matrix[LINES][CHRS]; とか定義して、
printf("%s\n", matrix[n]); で表示してるの違うかな。

各行の文字列の末尾にあるべき '\0' が格納されてない、みたいな。

2019/05/12(日) 17:36:33.22

2重ループやめても良いか？
　while (1) {
　　int c = fgetc(fp);
　　if (c == EOF) {
　　　break;
　　}
　　if (c == '\n') {
　　　continue;
　　}
　　matrix[i][j] = c;
　　j++;
　　if (j == 10) {
　　　i++; j = 0;
　　}
　}

2019/05/12(日) 17:38:28.80

501の方法だと'\n'読んだ時に、スキップしているけど
ループ変数の方は ++1 してるからズレてくる

2019/05/12(日) 18:35:08.99

>>503
>>507
どちらも行けました
507のは理解できるんですが
503の読み捨てというのはどう言うことなのでしょうか

2019/05/12(日) 18:43:30.54

>>509
10個読んで1個捨てる

2019/05/13(月) 20:38:42.19

バッファにつめるところを簡単にしてみた。
https://ideone.com/fJ91BI

2019/05/14(火) 06:41:01.67

OSSで構造体を公開しようとおもってる

そのサイズをバージョンアップしても固定にしたいんだけどいいアイデアないかな？
struct foo {
int a;
double b; // ここをバージョンアップで追加
}

char[100] buffer;

上記を予め含めておいて、追加サイズ分だけ減らすってのがよくある手段だと思うけど、intや他の型が固定サイズの保証はないから、処理系をかんがえると使えない
パディングでサイズも変わっちゃうしね

C++ならテンプレートでいけるけど、OSSだから利用者が多いC言語で書いておく必要がある
構造体を公開しないで、ハンドラとセッタで扱えばいけるだろうけど、今回は構造体でいきたい

2019/05/14(火) 08:01:00.18

「sizeof(int)が環境依存」を前提にするなら
バージョンアップ以前のint a;だけでもう既にサイズ固定じゃないじゃん
int32_tですら必ずある保証はないよ

「汎用な記述で全ての環境に対応したい」ってのが、土台無理な注文なのよ
低レベルなことやるなら、サポートする環境を列挙限定しなければならない

素直に「よくある手段」でやりなさいってことだ

2019/05/14(火) 08:07:10.74

>>512
処理系によるパディングとかの問題を別にしたら、bufferをunionにしといたらわざわざ減らさなくても良いのでは？

2019/05/14(火) 08:25:38.55

>>513
たしかにint aですでに違うね
ただ今回考えてるのは、コンパイルせずに共有ライブラリを差し替えるって状況を想定してる
そのときにセグメンテーションフォールトがでないようにしたい
だから、すべての処理系で同じサイズにしたいってわけじゃなくて、同じ処理系の中ではバージョンにかかわらずサイズを保ちたいってことなんだよね
説明不足ですまん

2019/05/14(火) 08:36:30.40

あらゆる環境だったらシリアライズしないとだし。
同じ処理系だったら同じ構造体でいいだろうし。

2019/05/14(火) 13:41:09.36

>>514 のはこんな感じですかの

union foo {
　struct {
　　int a;
　　double b; // ここをバージョンアップで追加
　};
　char dummyBuffer[100];
};

2019/05/14(火) 13:59:44.65

構造体の大きさが char[100] に収まってる範囲での話
>>512 のように末尾に用意したギャップを引き算していくにしても、引ききれなくなった段階で破綻するよね

※ 突破したことをチェックするには sizoef(union foo) と sizeof(char[100]) とを比較すんのかなぁ…

**経団連** ◆T6xkBnTXz7B0 (ﾜｯﾁｮｲ 1701-PJeF) · 2019/05/14(火) 14:09:24.00

C99 <stdint.h>のuint*_t型を使った方がいいと思う。

2019/05/14(火) 14:39:04.31

普通符号付きでしょ

2019/05/14(火) 14:51:20.14

符号プログラミングか

2019/05/14(火) 15:42:40.54

今はstatic_assertとかいうものがあるらしいぞ
これでコンパイル時にチェックすればよい

2019/05/14(火) 21:12:50.85

>>512

不格好だけどこれでいけるよ
全部プリプロセッサで処理できて、構造体サイズは100バイト固定になる。
予想を超えてサイズを突き破った場合は別構造体作るしかないね。

#include <stddef.h>

struct inner_foo {
int a;
char buf[];
};

#define MAX_STRUCT_SIZE 100
#define BUF_SIZE (MAX_STRUCT_SIZE - offsetof(struct inner_foo, buf))

struct foo {
int a;
double b;
char buf[BUF_SIZE];
};

2019/05/15(水) 07:47:15.99

>>523 ちょっと間違ってるみたい。
struct foo には struct inner_foo に double b; が追加されてるせいで
sizeof(struct foo) == MAX_STRUCT_SIZE にならない。
sizeof(struct foo) == MAX_STRUCT_SIZE + sizeof(double) になっちゃう。

inner_foo と foo とで char buf[] より前のメンバを一致させる必要がある。

2019/05/15(水) 08:35:12.84

>>515
なんでサイズが違うとセグメンテーションフォールトになるのかよくわからんが…
Win32APIみたいに頭にサイズ入れときゃいいんじゃね？

2019/05/15(水) 08:56:22.39

データファイルの話なら分かるが、intのサイズが違う環境の共有ライブラリってそもそも使えるのか？

2019/05/15(水) 09:12:43.64

コンパイルは自分でしろって話だと思うよ。
そうすると単にメモリ配置を意識したプログラムだな。
当然、バイトオーダーも気にしろよっていう話に

2019/05/15(水) 09:32:21.44

やっぱ int32_t みたいな環境に関係なく同じビット数になるやつにしといた方が良いんじゃないかな。

2019/05/15(水) 09:45:02.94

515の
＞ただ今回考えてるのは、コンパイルせずに共有ライブラリを差し替えるって状況を想定してる
＞そのときにセグメンテーションフォールトがでないようにしたい

構造体のパラメータを追加し共有ライブラリを差し替えつつ
呼び出し側のコンパイルは行わないって無理ないか？

旧ソースで追加されたパラメータ部分は不定値であって
その不定値をうけて共有ライブラリ内部が正しく動作できるとは思えないが…

2019/05/15(水) 11:40:27.07

構造体には、サイズの変わらない、ポインタだけを置いておいて、

実際の領域は、ライブラリ内で確保して使って、解放すれば？

2019/05/15(水) 12:19:01.09

さすがに実体を共有するという目的がないとこういう発想にはならんでしょ。
共有メモリとか、ファイルとか、通信とか。

2019/05/15(水) 12:31:29.02

>>529
呼び出す側がバージョン情報をもっておいて、処理側で振り分けてアクセスしないようにすれば問題ないね

#define version 2

void func (const struct* foo) {
func_impl(foo, version);
}

void func_impl(const struct* foo, int version) {
if (version > 1) {
foo.b;
}
}

2019/05/15(水) 12:43:05.74

>>532
そのバージョン情報の数字を埋め込むのに一番適度なのは実構造体のサイズって話に(>>525)

>>531 のような背景なら
ペイロードサイズは触りたくない　ってのは納得できるな
バージョン違いの吸収法は別途必要だけど

2019/05/15(水) 12:53:24.07

構造体の中身を公開しないでポインタだけ使わせて読み書きする関数を別途用意するって方法もあると思う。オブジェクト指向の言語でのプロパティのようなものね。
内容のファイルへの読み書きやネットワークでのやり取りはそれようにデータをpack、unpackする処理を作ってそこにバージョン番号も埋め込む。

2019/05/15(水) 12:59:51.65

exeがnew(malloc)するけど中をいじるのは全てdllの関数ってのはままある
dllだけ差し替えで済ましたい、サイズが変わらなければexeはビルドし直さなくていいのに…
ってことだろう

>512の
>intや他の型が固定サイズの保証はないから、処理系をかんがえると使えない
>パディングでサイズも変わっちゃうしね
は全く関係ない話だったということだ

素直に「よくある手段」でやればいい

>514,517のunionもいいと思うけど
foo.aでアクセスできないよね？foo.body.aとかしないと

2019/05/15(水) 13:12:20.84

>>517 の foo.a でいきなりアクセスできる union が記述できるのは C11 からみたい
無名構造体/無名共用体

2019/05/15(水) 14:41:00.06

>>535
intが4バイトなんて保証はないから、char[] bufの大きさを決められないって話でしょ？
intメンバを追加したとして、bufのサイズはどれだけ減らせばいいのかわからないのが問題
4の環境が多いだろけど、2の環境だと2を減らさなきゃいけないわけで

dll差し替えで再ビルドしない場合、構造体サイズが変わると特に配列渡しの場合はセグメンテーションフォールトになる可能性が高くて危険
セーブデータみたいに構造体をそのままファイルに落とすとしても、サイズはずっと不変なのが扱いやすいね

2019/05/15(水) 14:56:07.34

数字付かないintのでかい方に合わせるなら、それこそunion使って明示的にオフセット決めるとか。
本当にそんなことしたいのか？という話は置いといて。
もちろんint32_tとかint_least32_t使う方がいいし、本当に位置合わせするならバイト位置指定して読み書きするし、そもそもサイズ固定せずにシリアライズ用のライブラリ使うのが正しいよ。

2019/05/15(水) 19:35:47.81

>>537
> intが4バイトなんて保証はないから、char[] bufの大きさを決められないって話でしょ？
そんな話はとっくに終わっとる
>>515をちゃんと読めよ

2019/05/15(水) 20:46:54.27

>>536
ほう知らんかった
なら>517でほぼデメリットないね

>>537
「その環境」でのintのサイズはsizeof(int)でおｋ

2019/05/15(水) 21:55:29.06

win32apiはバージョンによって構造体の変数が追加になるから構造体のサイズを構造体の先頭にセットする

2019/05/16(木) 08:13:26.18

windows.hのsmallで一時間潰した

2019/05/16(木) 19:38:59.34

文字配列列sを逆順に文字配列tに格納して出力したいのですが、
以下のコードでは実行してもt=[]と出力されてしまいます
#include <stdio.h>
#define MAXLINE 1000
main()

{
char s[MAXLINE],t[MAXLINE];
int c,i,I;

for(i=0; i<MAXLINE-1 && (c=getchar())!=EOF && c!='¥n'; ++i)
{
s[i]=c;
}
if(c=='¥n'){
s[i]=c;
++i;
}
s[i]='\0';

I=i;
for(i=i; 0<=i; i=i-1)
{
t[I-i]=s[i];
}

printf("t=[%s]",t);
}

どこがおかしいのでしょうか…

2019/05/16(木) 19:48:10.73

後のループの一発目は

s[i]='\0';
I=i;
/* ここのforを削って */
t[I-i]=s[i];

こういうことになるから

t[0] = s[i] ってことになって　t[0] = '\0'; をやってることになる
上記のは
　　1　2　3　4　\0 →　\0　4　3　2　1　のコード
実装すべきなのは
　　1　2　3　4　\0 →　4　3　2　1　\0　となるコード

2019/05/16(木) 20:27:22.18

惜しいねえ。綺麗に逆に入ってはいるんだよな。

2019/05/16(木) 20:38:11.53

おっしゃる通りでした…今度は上手くいきました
ありがとうございます

2019/05/16(木) 20:38:46.73

おっしゃる通りでした…今度は上手くいきました
ありがとうございます

2019/05/16(木) 21:09:34.65

自己解決できたようでなにより
整数の配列を逆に整列させるなら正解だったんだけどねー
「文字列」操作の落とし穴ですな

2019/05/18(土) 22:03:40.11

今C言語教室でSQLとGUIやってる初心者なんだがCでDB弄る仕事ってあるのか
JavaとかC#に取られそうだけど

2019/05/18(土) 22:06:44.17

sqliteは使うよ

◆QZaw55cn4c (ﾜｯﾁｮｲ ea47-8NB0) · 2019/05/18(土) 22:07:47.63

>>549
ODBC 経由ならできますが…昔、ハンドルのキャストをとちったミスを、その意味不明の動きから突き止めるのは大変でした
ODBC を直書きする情報はネットにほとんどありませんね

2019/05/18(土) 22:19:48.67

SQL使うのか慣れてないからあんまやりたくないんだよなあ
頑張るか

2019/05/18(土) 22:37:47.44

>>549
C++ならまだしもC言語でGUI？
どこの教室だよ…

2019/05/18(土) 22:53:12.41

むかーし、Pro*C って使った事がある
Cのソース内にSQLを埋め込み
プリコンパイルする感じ
今は流行らないかもしれない

2019/05/18(土) 22:58:45.45

SQL使いこなしてる人ってすごいけど、正直ああはなりたくないという感じだよな。

2019/05/18(土) 23:01:53.11

賢いファイルシステムとして利用すると超便利だぞ

2019/05/18(土) 23:02:38.45

同じ事をCのロジックで組むとなると、アタマおかしくなるかも

2019/05/18(土) 23:08:58.89

普通にアプリケーションの規模が大きくなるとsql使いたくなるだろ

そういう場合はsqlite使う

2019/05/18(土) 23:10:30.33

５ちゃんねる専用ブラウザもsqlite使っているそうですね？
プログラムコードはCではないですが

2019/05/19(日) 10:01:29.20

IoTのデバイス開発やってるけど、C言語でDBのクライアントライブラリ使ってるよ

2019/05/19(日) 16:30:57.06

リーナスさんはC++嫌いつーてるしな
トレードマークのfuck口調で

2019/05/19(日) 17:49:51.40

この頃は間に何かライブラリ挟んでDB変わってもSQL文がほぼ同じままで行けるようなのが流行りなのでは？

2019/05/19(日) 19:08:51.80

大学の情報科の人たちってどのレベルでC言語使えるんですかね？
Cに限ったことではないですが大学4年でどの程度の練度が身につくのか

2019/05/19(日) 19:17:39.10

人によるだろ
そもそも好きなやつは大学入る前からやってるからな

2019/05/19(日) 19:22:28.74

>>563
別に使えなくてもいいよ

2019/05/19(日) 21:05:03.54

>>565
おまえは情報科に来るなってだけ
やる気ねえやつが畑違いな学部に入ること自体が害悪だ

2019/05/19(日) 21:14:52.17

>>566
お前が入ったの専門学校だろ？

2019/05/19(日) 21:19:32.43

>>567
化学科だけど

2019/05/19(日) 21:24:37.32

学校だとアルゴリズムやシステム設計手法は学べるだろうが
練度が上がるのはやはり仕事を始めてからだと思う

2019/05/20(月) 12:44:26.31

コンピュータサイカヨウの基本は一通りやるんじゃない？
情報処理試験の基本レベルくらいのもの
初心者は実務にはそのままは使えないけど、中級以上のエンジニアならこのあたりの知識があると差がついてくるね

2019/05/20(月) 14:12:04.87

業務のほとんどは手作業の自動化だから差は出ない。そしてみんなリストラされる。

2019/05/20(月) 18:18:34.53

仕事で出会った、いろいろな人を思い出すに、ガチに使える人材って凄いからなあ。
学校や仕事を通じて学ぶ、みたいな次元じゃないんだよねえ。

2019/05/20(月) 19:49:19.18

おまえが使ってるんだったらおまえが偉いよ

2019/05/21(火) 00:42:18.33

setbuf関数っていつなんの為に使うんや？
バッファリングのためにあるのはわかるけどそもそもバッファリングがなんの為にあるのかわからん

2019/05/21(火) 00:44:38.83

ちまちま出力するより溜めてからまとめて出力した方が高速

2019/05/21(火) 01:29:10.75

1万文字くらいのテキストファイルを標準出力でやってみたけど変わった気しなかったぞ
もしかして数百万文字レベルの巨大ファイルだと劇的に変わるんか？

2019/05/21(火) 01:34:19.46

10kbって小さくね

2019/05/21(火) 01:46:59.05

というか普通はバッファリングする関数ばっかりだが、自前でバッファ用意したいときもあるって話。
基本変わらんよたぶん。

2019/05/21(火) 01:47:43.41

確かに少ないな
と思ったんで2,700kbでやってみたらむしろ4秒長くなったぞ

2019/05/21(火) 02:21:13.21

>>574
FILE * 経由で出力する printf() 等は内部でバッファリングしていてバッファがいっぱいになったらまとめて出力する(出力先が端末の場合は行単位で出力する)。
そのバッファを自分で指定した領域を使わせるか、またはなしにできるのが setbuf()。更に自由に大きさやモードも指定したいなら setvbuf() を使う。

バッファリングに関して考慮しなくても良い入出力をする場合は read(), write() で行う。この場合は全くバッファリングされないので少ないバイト数の入出力が多いと遅くなる。
それと printf() のようなフォーマットを考慮した出力はないので必要なら自作する。

2019/05/21(火) 02:26:07.71

>>576
open()してwrite()で1バイトづつ書くのとまとめて書くのとではバイト数によっては大幅に違いが出ると思う。環境依存ではあるが。

2019/05/21(火) 03:47:59.45

>>581
大きなファイルの入出力を高速化したい時はsetbuf(fp,buf)
バッファリング無しにしてバッファをフラッシュするタイミングを順番に逐一実行させたい時はsetbuf(fp,NULL)
みたいな感じでええんかな？

2019/05/21(火) 03:58:34.93

>>582
setvbuf()で自分でバッファの大きさ決めてしまった方が良いかも。

2019/05/21(火) 04:24:32.31

既出かもしれませんがconstがjavaのfinalと違うと知りました。どう違うのでしょうか？c初心者です

2019/05/21(火) 05:39:22.97

>>576
OSによってはHDDへ遅延書き込みするから、結局バッファしてるのと大して変わらなかったりする。

2019/05/21(火) 06:44:08.98

>>584
20年前くらいの知識で、しかも完全に理解してるわけではないので、間違っていたらゴメンだけど。

変数について言えば、Java の final 変数は、値が未定義かどうか、という情報を内部的に持っていて。
もし、変数の値が未定義の場合は、1回だけ代入できたはず。例えば

final int i;
System.out.println("test");
i=4;
System.out.println(i);
i=5;
System.out.println(i);

の場合、1回目の代入（i=4）は許されるけど、2回目の代入（i=5）は怒られる。

それに対して、（C89の頃のだけど）Cのconst変数は・・・

const int i;
printf("test\n");
i=0;
printf("%d\n",i);

1回目の代入（i=0）であっても怒られる。

みたいな感じの違いはあったような。最近のCやJavaは知らないんだけどね。
あまり自信がないので、詳しい方のレスを待ちましょう。

2019/05/21(火) 06:49:56.40

>>585
そっちもあるけどAPI呼出し回数の削減もあるから

2019/05/21(火) 07:08:29.24

>>585
今どきのディスクならほぼ遅延書き込みしてるといってもいいね
バッファには二段階あって、
①プロセスが持ってるバッファ
②OSが持ってるバッファ

flush関係のライブラリ関数やシステムコールは①を②に吐き出す
②を吐き出すにはLinuxだとfsyncもしくはfdatasync

2019/05/21(火) 09:19:53.31

>>584
Cはポインタの指す先が const という指定ができるがJavaは配列の要素まで final にする指定ができなかったと思う。
つまり配列やクラスそのもののインスタンスへの参照は final にてきる。しかし参照される中身までは final にできず書き換え可能(クラスの場合はpublicのクラス変数とか、外部から書き換えられるようになっていれば書き換えられる)。

ま、ちょっと試してみな。

2019/05/21(火) 10:05:33.15

LTOのオプション
を使ってコンパイルすれば
インライン関数をヘッ
ダーに書かなくても
勝手にインライン化できるものはインラ
イン化してくれる
という理解で
あってますか？

2019/05/21(火) 10:21:18.62

>>590
それコンパイラ依存なのでは？

2019/05/21(火) 12:55:43.68

>>589
これな
Javaを最初に設計した奴はバカなんじゃないかと final の仕様を見たとき思ったな

2019/05/21(火) 15:23:18.72

Linux では、HDD へ書き込むのは、デフォルトで、5秒に1回

USB メモリで、書き込み回数を減らしたい場合は、
15秒に1回とかに設定する人もいる

2019/05/21(火) 18:23:31.31

>>593
書き出し周期はファイルシステムに依存するよ
例えばext3かext4かで挙動は違う

2019/05/21(火) 18:37:21.97

C言語のfprintfを用いるときに、前行の改行を消す方法をご存知ないですか？
エスケープシーケンスでバックスペースを試したのですか上手くいかなくて困っています。

状況としてはfor文を用いて改行を含んだ文字列を繰り返し出力させているんですが、最後の行だけ改行を無くし、続けて別の文を出力したいのです。
繰り返しの回数を変えて最後に改行なしの処理をいれればいいと思うのですが、簡単な方法があればと思い質問させていただきました。
よろしくお願いします。

2019/05/21(火) 18:40:11.67

ストリームの撒き戻しは難しいんじゃないか？　表示装置なのかファイルなのかでも変わるだろうし…

2019/05/21(火) 18:54:48.20

ファイルポインタは巻き戻せる
標準出力ならエスケープシーケンスで右上に行けばいい

2019/05/21(火) 19:19:14.41

こんな感じでいいのか?

#include <stdio.h>
int main(void) {
FILE* fp;
fp = fopen("Test.txt","w");
fprintf(fp,"hoge\n");
fseek(fp,-1,SEEK_CUR); /* 改行2byteなら-2 */
fprintf(fp,"fuga\n");
fclose(fp);
return 0;
}