【初心者歓迎】C/C++室 Ver.102【環境依存OK】

**デフォルトの名無しさん** · 2017/11/04(土) 16:33:35.07

エスケープシーケンスやWin32APIなどの環境依存なものもOK
そのような質問は必ず環境を書きましょう
半角空白やタブでのインデントはスレに貼ると無くなります

コードを貼れる所
http://codepad.org/
https://ideone.com/

前スレ
【初心者歓迎】C/C++室 Ver.101【環境依存OK】
https://mevius.5ch.net/test/read.cgi/tech/1500329247/

**デフォルトの名無しさん** · 2017/11/08(水) 06:36:55.45

これのがまだマシ

**デフォルトの名無しさん** · 2017/11/08(水) 06:40:10.32

普通に>>23でいい

**デフォルトの名無しさん** · 2017/11/08(水) 09:37:48.99

gotoとかｗｗｗｗ
って思った人いるんだろうか

**デフォルトの名無しさん** · 2017/11/08(水) 10:35:10.40

switch である必要ないな

**デフォルトの名無しさん** · 2017/11/08(水) 11:33:23.57

>>34
ああ、素人はgoto嫌うよな

>>35
もちろん必要は無い
テーブルよりは色々な面でマシってだけ
普通に if ( '0' <= c && c <= '9' ) で良い
一目で意味がわかるし高速

**デフォルトの名無しさん** · 2017/11/08(水) 12:27:45.50

&&って条件分岐が入るけど、本当に高速？

**デフォルトの名無しさん** · 2017/11/08(水) 12:44:08.80

そこはctypeで判定だね

**デフォルトの名無しさん** · 2017/11/08(水) 13:01:31.63

>>37
とおもってアセンブリ吐かせてみたら、すごかった。
48引いた上で、符号なしで10未満か判定してるので、
余分な条件分岐してなかった。

**はちみつ餃子** ◆8X2XSCHEME · 2017/11/08(水) 13:02:27.97

>>37
&& では分岐しなかった。
'0' <= c && c <= '9' を clang で -O3 付きでコンパイルしてみたらこうなった。

addl $-48, %eax
cmpl $10, %eax
setb %al

賢いな！
直後に分岐するなら結果をレジスタに書き込む必要はないから実質的に二命令だ。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/11/08(水) 13:37:52.71

俺の知識も時代遅れになったな。シンプル伊豆ベスト。

**はちみつ餃子** ◆8X2XSCHEME · 2017/11/08(水) 18:14:28.55

もっと >>40 を高速化できないかと思って考えてみたが、
'0' を引くんじゃなくて 0x30 と xor を取ってから 10 より小さいか比較するっていうのでもいける。

ただ、最近はビット演算が算術命令より速いとも限らないみたいなんだよね～

**デフォルトの名無しさん** · 2017/11/08(水) 18:22:33.84

>>42
まるで昔は速かったかのようだ

**はちみつ餃子** ◆8X2XSCHEME · 2017/11/08(水) 18:37:11.31

>>42-43
Z80 ですら足し算と XOR は 1 クロックだったわ。スマソ

**デフォルトの名無しさん** · 2017/11/08(水) 18:43:03.66

ん？
8086はビット演算の方が速いぞ

**デフォルトの名無しさん** · 2017/11/08(水) 18:57:26.39

>>21
> 仕様として「両方で停止は問題ない」よな？
相談者でないとわからん
途中に'\0'があるケースについては書いてないから

> そもそも相談者の仕様がおかしい
それはお前が決める話じゃない

**デフォルトの名無しさん** · 2017/11/08(水) 18:59:47.82

>>22
> ショートサーキット前提の記述をよしとするかどうかはわからん
if(p != 0 && *p != '\0')
みたいなのは普通に書かない？

**デフォルトの名無しさん** · 2017/11/08(水) 19:03:40.31

>>47
俺は書くけど　それを"普通"とまで言い切ってしまって良いという自信はない

**デフォルトの名無しさん** · 2017/11/08(水) 19:14:43.13

>>46
わからないならだまってれば
正確な要求仕様がわからない事くらい誰でもわかる

両方でとまる関数は片方で止まる関数の機能を含むので
問題になる場面は普通は考えられない
質問に書いてない、文字列の途中にNULL文字とか
いうアホな事を心配しない限り

**デフォルトの名無しさん** · 2017/11/08(水) 19:18:16.64

全角数字の方がまだ可能性がある

**デフォルトの名無しさん** · 2017/11/08(水) 19:42:17.13

>>49
こういう思い込みの激しい奴が要らんバグを作り込むんだよな...

**デフォルトの名無しさん** · 2017/11/08(水) 19:51:21.87

isdigit関連で性能測ってみた。
環境: Core i7-2600 / gcc 6.4 (-O2)
rund() & 0xff の値で条件分岐して、真ならさらに rand() を呼ぶ、を10億回ループ。
条件式はASCII前提。

数字: 条件式 < int配列ルックアップ < isdigit
英字: 条件式 <= int配列ルックアップ < isalpha
数字英字: int配列ルックアップ < 条件式 < isalnum

ロケールの影響か、isalphaなどはisdigitより遅い。
char配列ルックアップはint配列より遅い。
奇妙なことに、配列ルックアップでも、数字だけより、英字も入ってくると遅くなる。なぜだ。

52 · 2017/11/08(水) 19:54:52.03

あ、ごめん、rand() 呼ぶ分、数字 < 英字 < 英数字の順で遅くなるの、当然だった。

**デフォルトの名無しさん** · 2017/11/08(水) 20:30:53.41

最速なら前スレの>>995や>>997

>>997は演算の依存性が高いので
>>995の方が少し速いかな？
まあコンパイラや環境依存でしょう

x64前提なら、
64bit一気に持ってきて偶数奇数桁に分けて、
LEAで5倍してシフトして足すとか
AVXで乗算してから水平演算とか
色々と考えられるけど
まあ一応このスレのC言語の範囲で

**デフォルトの名無しさん** · 2017/11/08(水) 20:49:19.11

このスレの結論はこれか。
10000*a[0]+10000*a[1]+1000*a[3]+100*a[4]+10*a[6]+a[7]-5333328;

これより速くする方法あるのか

**デフォルトの名無しさん** · 2017/11/08(水) 21:08:26.39

何時の間に判定せずにすむようになったんだ

**デフォルトの名無しさん** · 2017/11/08(水) 21:11:25.51

前スレの>>997が質問者だから
フォーマットは固定と思われる

**デフォルトの名無しさん** · 2017/11/08(水) 21:12:49.95

さすがに '0' * 111111 をコンパイル時に計算しないコンパイラは無いと思うので
このままで良い気はする
当然こちらの方が意味が分かりやすい

**デフォルトの名無しさん** · 2017/11/08(水) 21:14:04.15

( ) を色々とかえて時間を測ってみたけどほとんど同じ
やっぱりLEAを多用してる
VS2017の64bit Releaseビルドの場合

**デフォルトの名無しさん** · 2017/11/08(水) 21:39:10.43

VPMOVZXBD xmm0, a
VPMADFWD xmm0, xmm0, [掛け算テーブル]
VPHADDD xmm0, xmm0, xmm0
VPHADDD xmm0, xmm0, xmm0

**デフォルトの名無しさん** · 2017/11/08(水) 21:40:02.30

AVXが使えるならこれ

**デフォルトの名無しさん** · 2017/11/08(水) 21:42:47.06

微妙に間違った
メモリが連続してれば複数同時に出来る
連続してなくても複数同時にやった方が速い

**デフォルトの名無しさん** · 2017/11/08(水) 21:43:54.20

>>29でメモリアクセスに言及されてる話の流れの中で
10億回ループがいちいちキャッシュクリアされてるのかが気になる

**デフォルトの名無しさん** · 2017/11/08(水) 21:49:19.34

普通はキャッシュに入ったままだが、
処理次第で可能性が無いことはない
他のキャッシュを汚染する可能性もある
HTTなどでは特に

他に良い方法が有るのにわざわざ面倒なテーブルにする意味がわからない

**デフォルトの名無しさん** · 2017/11/10(金) 07:46:10.46

std::vector<std::string>からchar*const[]を作成するいい方法ってないですか。
newするしかないですかね。

**デフォルトの名無しさん** · 2017/11/10(金) 10:26:44.91

>>65
vector<char*> に各要素 s の &s[0] を入れて済むなら new は要らなさそう。

**デフォルトの名無しさん** · 2017/11/10(金) 20:15:55.61

>>66
まあどうみても内部でnewしてるんだろうけど

**デフォルトの名無しさん** · 2017/11/10(金) 21:01:13.10

クラスの概念がわかりません

**デフォルトの名無しさん** · 2017/11/10(金) 21:19:13.94

構造体みたいなもの

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/11/10(金) 21:39:23.58

>>68
クラスは構造体に関数を追加したようなもの。
クラス内部の関数はメソッドと呼ばれる。
メソッドからは、クラスのデータとメソッドを参照できる。
データとメソッドのそれぞれの項目をクラスのメンバーと呼ぶ。
クラスの実体はインスタンスと呼ばれる。メソッドからは、thisポインターにより、クラスのインスタンスを参照できる。
メンバーは、public:、protected:、private:によってアクセスを制御できる。アクセスできないメンバーを参照するとコンパイルエラーになる。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/11/10(金) 22:11:59.59

class A
{
int a;
};
と書くとA::aは、privateになる。構造体と同じアクセスにするには
class A
{
public: int a;
};
と書かないといけない。このようにクラスのデフォルトのアクセスはprivateになり、構造体はpublicになるという違いもある。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/11/10(金) 22:14:05.73

クラスはコンストラクタと呼ばれる特殊なメソッドがあり、インスタンスの構築に使われる。
また、インスタンスが破棄されるときに、デストラクタという特殊なメソッドが呼ばれる。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/11/10(金) 22:17:22.95

コンストラクタの例。
class A
{
public:
int m_a;
A(int a) {m_a = a + 1; }
};

int main() {
A a(2);
printf("%d\n", a.m_a);
}

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/11/10(金) 22:19:15.61

別のコンストラクタの例。
class A {
public: int m_a;
A(int a, int b) : m_a(a + b) { }
};
int main() {
A a(2, 3);
printf("%d\n", a.m_a);
}

**はちみつ餃子** ◆8X2XSCHEME · 2017/11/10(金) 23:02:09.42

そんなの説明してやらなくてもネット上に無数の解説があるだろ。
ちゃんとした説明を読んで理解できないなら掲示板で短い説明読んだってわかりゃしねーよ。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/11/10(金) 23:14:40.61

>>75
情報が多過ぎて迷子になってるんじゃないか。マニュアル読め、規格票読め、で終わればいいのだが。

**デフォルトの名無しさん** · 2017/11/10(金) 23:21:03.62

そう思うならなぜ現在地を聞く前に世界地図を渡すのか

**デフォルトの名無しさん** · 2017/11/10(金) 23:32:23.63

>クラスは構造体に関数を追加したようなもの

こういうアホってどこにでもいるんですね

◆QZaw55cn4c · 2017/11/10(金) 23:40:07.53

>>78
その理解で8割がたOKだと思うのだが、何が問題なの？

**はちみつ餃子** ◆8X2XSCHEME · 2017/11/10(金) 23:46:33.90

>>76
チュートリアルだってたくさんあるじゃん。

**デフォルトの名無しさん** · 2017/11/11(土) 00:24:04.64

>>79
クラスがわからんやつにこんな説明しても通じないだろ

**デフォルトの名無しさん** · 2017/11/11(土) 00:49:17.73

山田さんってどんな人？と聞かれて
原子の集まりだよ。って答えるようなもの
形式的にウソじゃあないんだが
そこじゃないだろっていう

**デフォルトの名無しさん** · 2017/11/11(土) 02:47:19.31

純粋にC++のクラスそれ自体の概念というなら
C言語(ベース)でのオブジェクト指向プログラミングをサポートするために言語機能で用意された部品
オブジェクト指向プログラミングにおけるオブジェクト、オブジェクトの状態、
メッセージの送受信、メッセージを受けたときの振る舞いといった概念を具体化した仕組みの一例

**デフォルトの名無しさん** · 2017/11/12(日) 13:22:39.97

記憶クラスについての質問の可能性

**デフォルトの名無しさん** · 2017/11/12(日) 23:51:21.74

実のところはOOPを知らないからクラスをどう使えばいいのかわからない／
他人の真似でなんとなく使ってるけどなぜクラスを使うのかがわからないといったあたりの話だろ

**デフォルトの名無しさん** · 2017/11/22(水) 02:18:46.71

下手にクラスを説明するやつが多いせいで、コーディング経験なくても理解できる概念的なものだと思われちゃう
クラスは実用あってこそ

**デフォルトの名無しさん** · 2017/11/22(水) 07:58:58.54

アニマルクラスがわんにゃー鳴くのがオブジェクト指向です

**デフォルトの名無しさん** · 2017/11/22(水) 17:16:56.90

メンバ関数を呼び出す際の
「オブジェクトにメッセージを送る」って言い回しも
C++だと分かりにくいね。

この表現がしっくりくるプログラミング言語というか
環境もあるのだろうけど。

**デフォルトの名無しさん** · 2017/11/22(水) 17:52:35.50

>>88
そういうのをベタで実現しているOO言語はないかな…

**デフォルトの名無しさん** · 2017/11/22(水) 18:04:22.80

smalltalkは？

**デフォルトの名無しさん** · 2017/11/22(水) 20:55:50.24

>>90
あれも結局、関数（メソッド）の動的なコールをそう呼ばせてるだけ
同じSmalltalkでもSmalltalk-72までさかのぼるとメッセージ送信と呼べなくもない仕組みにはなっているけど
それでもオブジェクトに（メッセージを）「送る」というよりは（トークン列を）「流し込む」って感じかと

まあ、ともあれC++の「オブジェクト指向」を説明するときには「メッセージを送る」とかはすっかり忘れていいよ
そもそもC++の設計者の提唱するオブジェクト指向は「抽象データ型（端的にはユーザー定義の型）」を
Simulaという言語から拝借した「クラス」という言語機能で実現することが主軸のアイデアなので
http://www.stroustrup.com/whatis.pdf

**はちみつ餃子** ◆8X2XSCHEME · 2017/11/22(水) 21:08:20.60

言語の意味論とそれを実現するメカニズムは別物だから、
仕組みがどうなっているかから考察するのは無粋だと思うけどなぁ。

まあ C++ は低水準レイヤでの仕組みを意識せざるを得ない、
意味論とメカニズムが不可分なデザインではあるから概念としてのメッセージ指向よりも
メカニズムに意識が引きずられてしまうというのは確かにあるかもしれない。

**デフォルトの名無しさん** · 2017/11/22(水) 21:21:24.87

メソッド呼び出しってメッセージパッシング感すげー出てない？

**デフォルトの名無しさん** · 2017/11/22(水) 21:28:03.86

>それでもオブジェクトに（メッセージを）「送る」というよりは（トークン列を）「流し込む」って感じかと

違いがわからないw

**デフォルトの名無しさん** · 2017/11/22(水) 22:33:35.60

>>93
メッセージパッシング言うなら、何かが静的に決定してる時点で興醒めだよ
http://wiki.c2.com/?AlanKayOnMessaging

**デフォルトの名無しさん** · 2017/11/22(水) 23:02:11.68

>>95
C++のポリモーフィックなメソッドは動的ディスパッチだろう

**デフォルトの名無しさん** · 2017/11/22(水) 23:09:00.03

>>91
>抽象データ型
これがすべてなのに、どうしてメッセージ駆動の話が今でももちきりなのだろうか？

**デフォルトの名無しさん** · 2017/11/23(木) 08:26:45.55

>>96
C++に「動的なものは何もない」という主張ではなく
「静的に決まる何かが一つでもある」時点でそれをメッセージングに例えるのは興醒めという意味

少なくともケイは彼のOOPにおけるメッセージングを「遅延結合の徹底」の目的で提案している
http://d.hatena.ne.jp/katzchang/touch/20080807/p2
「私が考えるOOPはメッセージング、状態処理のローカルでの保有・保護・隠蔽、
そして全ての物に対する強力な遅延束縛、これだけだ。
これはSmalltalkとLISPだけが実現できている。他のシステムでも可能かもしれないが、私は知らない。」

つまるところ、これはすなわち、SmalltalkやLispみたいな変態言語でもなければ、
（あるいはアクター機構のように、並列化を意識して本当にメッセージを送るのでもなければ）
無理して「メッセージパッシング」なんてたとえる必要は無いんやで、という話でしかない

**デフォルトの名無しさん** · 2017/11/23(木) 08:27:38.45

>>97
まったくもってその通り

**デフォルトの名無しさん** · 2017/11/23(木) 14:15:04.16

Ｃのキャストについて教えてください

int a,b;
long c;

の条件下で、a+b 、a*100+b*100（a*100、b*100のどちらもlong型になる）
がそれぞれlong型の解になるような場合のキャストとしては

1 c=(long)a+b;
2 c=(long)a+(long)b;

3 c=(long)a*100+b*100
4 c=(long)a*100+(long)b*100

とそれぞれ、どの記載が正しいのでしょうか？

**デフォルトの名無しさん** · 2017/11/23(木) 14:19:02.83

>a*100、b*100のどちらもlong型になる

intでは表せなくてlongが必要
って意味なら3以外

**デフォルトの名無しさん** · 2017/11/23(木) 14:20:46.83

ありがとうございます。
１と３は似ていますが１はa,bそれぞれキャストされるが
３ではb*100はキャストされずint型で計算されてしまう、ということなのですね

**デフォルトの名無しさん** · 2017/11/24(金) 16:57:14.87

よろしかったら教えてください

　　if(a==0){ }

と記述するのにfor文では
　　
　　for(i=0;1<100;i++){}

と書き、for(i==0;i<100;i++){}

と書かないのはなぜでしょうか？（==の使い方）

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/11/24(金) 17:05:28.65

>>103
代入と比較は違う。

**デフォルトの名無しさん** · 2017/11/24(金) 17:07:52.77

スーパー初心者か?
forでも書いてもいい。だが意味がないか低い。
「=」は代入だが「==」は代入でない。

**デフォルトの名無しさん** · 2017/11/24(金) 17:08:50.74

片山先生、ありがとうございました！
そういうことなのですね。理解できました、いつもありがとうございます！

**はちみつ餃子** ◆8X2XSCHEME · 2017/11/24(金) 17:38:47.92

わかってる人にとってはクソみたいな質問だけど、
数学だと代入 (定義) するっていうのと等しいっていうのは区別がはっきりしないので、
そういうメンタルモデルで考えてると混乱しちゃうってのはわからんでもない。

とはいうものの、よっぽど出来の悪い入門書でもそこらへんわからんようなのは無いと思うんだが、
どういう学び方をしてるんだかちょっと疑問。

**デフォルトの名無しさん** · 2017/11/24(金) 17:51:57.71

なんかすみません
ネットで優しそうなサイトで進めてます

**デフォルトの名無しさん** · 2017/11/28(火) 12:50:21.53

数学のイコールには比較（の結果等しい）の意味しかないだろう
Pascalみたいに代入が:=なら良かったのに

**デフォルトの名無しさん** · 2017/11/28(火) 12:57:44.05

プログラムを記述する際、等しいかを比較するより、代入のほうが頻度が高いから
Cでは多く入力するであろう代入側を 1文字にしてそうでない側を 2文字にした
って聞いたことがある

**デフォルトの名無しさん** · 2017/11/28(火) 13:22:50.66

Fortranだって代入は=だから、Cで採用されたのも仕方ない

とはいえ、文字数ごときにこだわらずに、:=を採用してくれていたらタラレバ

**デフォルトの名無しさん** · 2017/11/28(火) 13:44:24.58

力添えを頂きたい。
SunAwt系のソフト内で表示されているテキストを取得したいんだが、尻尾も掴めず困り果て、どうすれば良いか全く分からない状態。
何かヒントを頂けないだろうか。

**はちみつ餃子** ◆8X2XSCHEME · 2017/11/28(火) 13:50:31.60

>>112
別のプロセスから取得するって意味？

**デフォルトの名無しさん** · 2017/11/28(火) 18:08:18.05

>>113
YES。
別プロセスからテキストを取りたい。
他の例えばIEやハンドルが取れる様なソフト内の文字ならgettextやcom操作で取ったりできるがSun系のソフトは同じ様にはいかず、メモリを読むのかどうすればいいのか、知識が足りなくて検索も上手に出来ず...頭打ち状態。
何か足掛かりになる物を教えて頂きたい。

**デフォルトの名無しさん** · 2017/11/29(水) 00:32:03.31

>>114
試したことないけどこれは？
http://www.oracle.com/technetwork/articles/javase/index-jsp-136191.html

**デフォルトの名無しさん** · 2017/11/29(水) 13:08:28.50

#include<stio.h>
int main(){
int i,num[5];

for(i=0;i<5;i++){
printf("***\n");
scanf("%*c%d"&num[i]);
}
}

実行し、num[]に、10、20,30,40,50と入力すると
num[0]の値は0、num[1]は20、その後50まで代入されてます
num[0]に10が代入されないのは何故なのでしょうか？

**デフォルトの名無しさん** · 2017/11/29(水) 13:26:13.79

>>116
"%*c"のせいじゃね

**デフォルトの名無しさん** · 2017/11/29(水) 13:36:20.41

>>117
そうでした。ありがとうございました。

**デフォルトの名無しさん** · 2017/11/30(木) 14:49:53.66

テフ？

**デフォルトの名無しさん** · 2017/12/01(金) 03:20:07.32

boost::qiでマルチバイト文字を扱うにはどうすればいいんですかね

**デフォルトの名無しさん** · 2017/12/01(金) 12:48:46.21

５人の生徒に３教科のテストをし、その点数を入力する
入力後、
　　　　　　　　　　テストＡ　　テストＢ　　テストＣ
　０点～１９点　　　　０人　　　　０人　　　　０人
２０点～３９点　　　　０人　　　　０人　　　　０人
４０点～５９点　　　　２人　　　　０人　　　　１人
・・・
８０点～１００点　　　１人　　　　３人　　　　２人

と集計する方法に悩んでいます。
どういう考え方が考えられますか？

**デフォルトの名無しさん** · 2017/12/01(金) 13:06:08.20

>>120
char_に替わるパーサー書けばいいんじゃないかね
マルチバイトの最初の文字を認識する自作パーサーをmb1、マルチバイト文字のパーサーをmbcharとして
mbchar = (mb1 >> char_) || char_;
とか？

**デフォルトの名無しさん** · 2017/12/01(金) 13:14:35.24

>>121
宿題は自分で考えるのが原則。どこまで考えた？

**デフォルトの名無しさん** · 2017/12/01(金) 13:21:20.74

>>123
ありがとうございます。
入力された点数を３次元配列に格納するところまでです。
今は、繰り返し分を使い、配列に格納された数字が０でなかったら
その数字をカウントして人数を集計しようとしてるのですが、できていません

**デフォルトの名無しさん** · 2017/12/01(金) 13:56:35.70

>>124
なぜ3次元？生徒と教科がインデックスだから2次元では？
あと、表示さえすればいいだけなら、
効率度外視である教科のある得点範囲の人数をカウントするくらいできるでしょ。
効率考えるなら表示用に0で初期化した2次元配列を別に用意する。

**デフォルトの名無しさん** · 2017/12/01(金) 14:01:47.55

>>125
ありがとうございます。2次元です。
ten[5][3]で得点を格納したあとが進まないのです
得点範囲の人数のカウントについてもう少し考えてみます

**デフォルトの名無しさん** · 2017/12/02(土) 00:04:56.97

中古本でプログラミング言語c++の3版が4000円で4版が6000円なんだが
やっぱり4版を買ったほうがいいのかな

**デフォルトの名無しさん** · 2017/12/02(土) 00:18:15.85

あんまり良い本じゃないから買わなくていい

**デフォルトの名無しさん** · 2017/12/02(土) 00:54:45.18

>>128
そうなのか・・・聞いておいてよかった
ありがとう

**デフォルトの名無しさん** · 2017/12/02(土) 09:08:29.93

>>125
昨日はありがとうございました
90行程度と長いプログラムになりましたが
人数をカウントし表示させることができました

カウントは、教科毎にfor文で呼び出し
20以下、20～39、・・・80～99、100とif文で条件指定し
各範囲で人数をカウントし、最後に表示させました

また分からないことがありましたらご指導お願いいたします。

**デフォルトの名無しさん** · 2017/12/02(土) 14:58:42.41

ファイルを開いた回数を記録するというプログラムを作っています
int main(){
int count;
FILE *fp;

if((fp=fopen("log.txt","r"))==NULL){
count=0;
}else{
fscanf(fp,"log.txt",&count);
fclose(fp);
}

if((fp=fopen("log.txt","w"))==NULL){
printf("Error\n");
return -1;
}

count++;
printf("%d回目のプログラムを実行です\n",count);
fprintf(fp,"%d\n",count);
fclose(fp);
}
これを実行するたびに
2752513回目の・・・
3444737回目の・・・
2953217回目の・・・
となってしまい、ファイルを開いた数が記録されません

最初にlog.txtがない状態で実行したときの、一番最初だけ、countに１が代入され
log.txtにも1が記載されるのですが、2回目以降のcountには7桁の乱数みたいのが代入され記載されてしまいます
どうして不具合が出るのか分からなくなっております。よろしくご教授くださいませ

**デフォルトの名無しさん** · 2017/12/02(土) 15:12:26.31

fscanfで書式の指定ミス