【初心者歓迎】C/C++室 Ver.102【環境依存OK】
■ このスレッドは過去ログ倉庫に格納されています
エスケープシーケンスやWin32APIなどの環境依存なものもOK
そのような質問は必ず環境を書きましょう
半角空白やタブでのインデントはスレに貼ると無くなります
コードを貼れる所
http://codepad.org/
https://ideone.com/
前スレ
【初心者歓迎】C/C++室 Ver.101【環境依存OK】
https://mevius.5ch.net/test/read.cgi/tech/1500329247/ >>628
scanf("%d",&n);
int array[n]; //読み込んだ値を要素数にするってこういう事? 基本的な事なんだろうけど、vectorを範囲ベースのforループしたいのですができません。
auto v = vector<int>()だとfor (auto &i : v)でいけるのですが、
newした場合はどうすればいいのでしょうか?
autov = new vector<int>(); for (auto &i : v)だと
begin関数が見つかりませんといってコンパイルできません。
どうすればいいでしょうか? すみませんでした。普通に逆参照すればいいのか。
auto v = new vector<int>(); for (auto &i : *v) MFCのアプリで排他制御のルーチンを作っていて疑問に思ったことなのですが、
CやC++の仕組みのことなので、こちらで質問させていただきます。
複数のスレッドから呼ばれる関数の内部は、
CCriticalSectionを使って排他制御しています。
void Test()
{
static CCriticalSection cs;
cs.Lock();
:
cs.Unlock();
}
関数内のstatic変数は、その関数が初めて呼ばれたときに
実体が作成されると理解しているのですが、
1回目のTest()でCCriticalSectionのコンストラクタを実行している最中に
他のスレッドがTest()を呼んだ場合、このコンストラクタ自体は、
正しく排他制御されるのでしょうか?
コンストラクタが終わるまで、他のスレッドは待ってくれるのでしょうか?
それとも、CCriticalSectionの変数は、
関数の外に置かなくてはいけないものでしょうか?
MFCの内部でも、AFXPlaySystemSound()などで
同じようなことをやっているようなのですが。 >>633
C++11 でローカルなスタティック変数の初期化はスレッドセーフであると定められた。
(C++03 にはこの規定はなく、実際その頃はmsvcもgccもスレッドセーフではなかった)
msvc では 2015 から。
c言語バージョンの指定のほか、専用のオプションによっても有効化、無効化できる
以下を参照
https://docs.microsoft.com/ja-jp/cpp/build/reference/zc-threadsafeinit-thread-safe-local-static-initialization
gcc / clang では -fno-threadsafe-statics オプション 他の言語ではコンパイルエラーで悩むことはないのですが、コンパイルが通らなくてかれこれ2日。
エラーの行番号見ても、あってるよーな?よくわからない。
どうすりゃいいの。 >>636
コードすら貼らないオツムの足りなさじゃ諦めろ >>634-635
ありがとうございます。
VSは2015よりも前なので、このやり方はダメってことですね。
関数の外に置くようにします。
MFCの内部ではやっちゃってますが。 なんだよ。このクソ言語。やっとわかったわ。
ちょっとずつコンパイルしていかねぇとダメじゃん。このクソ言語。
エラーの内容はhファイルの方に"{"が期待されるところに"}"がありますなエラー。
ヘッダファイルと睨めっこしても、{}の数あってるし仕方なくcppファイルの方の実装の中身を全部コメントアウトしたら
コンパイル通った。で、cppファイルの方を数行ずつコメントアウト解除したらエラーわかったわ。
hファイルにエラーがあるとかミスリードしてんじゃねぇよ。 エラーも貼れないなら、助けるのはムリ。
テレパシーは秘密保持違反になるから、このスレッドでは君を助けられないね。
つーか、仕事中にパソコンで5チャンネル閲覧記録は、大きな企業なら会社にチェックされているだろうね。 そりゃ、ソースコード全部張れば分かるとは思うけど、
エラーの内容は
「hファイルの方に"{"が期待されるところに"}"がありますなエラー。」
みたいなエラーでて、この意味ぐらい自分で分かるし。
だから、このエラーの内容だけ張っても君たちが分かるとは思えなかったら張りませんでした。
お手数かけました片山さん。 May the source be with you. そうでしたか。まだ、C++やりだして1週間未満なものでして、そういうノウハウというか関連知識が
ありませんでした。
まだ、Visual Studioの「エラー一覧」ウィンドウしかまだ見てないもので。とりあえず、初心者の自分は
頻繁に数行書いたらコンパイルすることにします。 プリプロセッサによる不便さだから c++ の文法とは全く関係ないけどな >>642
今後のためにそのエラーになる最少構成でやってみ ヘッダファイルがこんなふうにconst用と非const用の関数があるとして
class A {
B b;
B& get() { return b; }
const B& get() const { return b; }
}
このBがくっそ長くて書くのが面倒な場合、autoで代用するのはまずい習慣ですか?
auto& get() { return b; }
auto& get() const { return b; } >>653
型の別名を定義することについての是非は置くとして、 typedef はオワコン。
using を使うのがモダンな C++ やで。 typedefやusingはなるべく使いたくないんですよね
やはりヘッダーの返り型くらいはちゃんと記述すべきですかね
イタレータとか長くなりがちだけど・・・ こうだな。動くかどうか知らんけど。
auto& get() { return b; }
const auto& get() const { return b; } 柴田望洋訳のC++の本とロベールのC++の本以外にまともな日本語のC++の本ってありますか? ロベール、独習(Herbert Schildt)、柴田望洋、林 晴比古
修験者になって、数年山籠もりしたいのなら(笑)、ドワンゴ江添の、
C++11/14 コア言語、江添 亮、2015 >>660-661
ありがとうございます。
Herbert Schildtの本は、ロベールに比べると細かい話がないように思います。
柴田さんの中級という本も細かい話がないように思います。 vector について質問です。
コピーコンストラクタ、operator=
について一応勉強したのですが、 vector などのクラスでは、
コピーコンストラクタや operator= はどうなっているのでしょうか? 実際にヘッダ読んで見るしかないと思うが
ディープコピーしてるよとしか Visual Studioなどでデバッグするとヘッダーが見れるよ。vectorはテンプレートのクラスだから、
読み進めるにはテンプレートの知識が必要かもね。 >>664
ありがとうございました。
ディープコピーでしたら、使う上では、何も考えずに関数の引数として渡したり、
関数から返したりしてもかまわないですね。
>>665
ありがとうございました。 >>666
それはディープコピーのコストがかさむのでおすすめしない >>666
>何も考えず引数で渡しても良いですね
何をもってして「良い」と評価するのかはわかりませんが、
そういうのは拙い知識で独断せずそこら中にある
経験豊富な先人の書いたコードを読んで真似するのがいいですよ
引数は大抵 const 参照で渡しているでしょう class TestA
{
TestA() {}
~TestA() {}
};
class TestB
{
TestB() {}
~TestB() {}
TestA getA() { return TestA(); }
};
int main() { TestB b; TestA a = b.getA(); }
TestBクラスの getA() メソッドがこのテストコードで正しく動作しているのですが、TestAの
インスタンスは一体どこで作られてるんでしょうか?
TestA getA() { TestA a; return a; }
としないでいきなり reurn TestA() で良い理由が良く分かりませんでした。 >>669
どこのメモリにどう作られているのかという質問だと思うけど、
C や C++ では式の値として構造体やオブジェクトが認められていて、
コンパイラがどこぞにメモリを用意して上手いこと作ってくれることになっている。
式や式の一部の値としてだけ現れて変数に格納されないオブジェクトはテンポラリーオブジェクトとか言うので、
詳細はテンポラリーオブジェクトでぐぐって。
>>669
のmain はテンポラリーオブジェクトを使って b を用意せず TestB().getA(); ともかける
ところで今回の質問の、関数の戻り値に関するコンパイラの動作は
「戻り値最適化」なる最適化によって色々複雑なことになっているので
これは自分で戻り値最適化でググって調べて欲しい。 >>669
質問が文法的なことについてだとすると、式中の項ととして
TestA()
std::string("foo")
などのように型名にコンストラタの引数を加えて関数呼び出しのように書くと
テンポラリーオブジェクトを生成してそれを値とする頃ということになる。
例
size_t l = std::string("abc").length();
std::string 型のテンポラリーオブジェクト(中身は"abc") が作られ、
それのメンバ関数 length が呼ばれ、l はその値(3)で初期化される。 >>670-671
詳細をありがとうございました。
大変良く解りました。より深くは自分でググってみます。 for (int i = 0; i < n + 1; ++i) {;}
とやると、 i が n + 1 未満かどうかの判定をする際、毎度
n + 1 を計算してそれを i と比較するということになるので
しょうか?
もし、そうなら、
m = n + 1
for (int i = 0; i < m; ++i) {;}
としたほうがよいのでしょうか? >>674
ただの整数の足し算程度なら最適化されることが期待できるのであまり気にしなくていい。 https://ideone.com/TVzVzF
上記のコードで、read関数を持ったIReadableクラス(基底クラス)があって、そのクラスから派生したCFileReaderクラス、
直接は派生していないけどread関数を持ったCMemoryReaderクラスがあって、その両方を受け取れるvector<IReadable*>を作りたいのですがコンパイルが通りません。
どうしたらいいでしょうか? どなたか教えてください。。。 >>674
いまだに最適化されないコンパイラもあることはある
8bitマイコンとかの話
あと、nがvolatileだと当然毎回計算する
nがSFRだったり複数スレッド共有変数だったりするなら気を付けよう
普通は気にしなくて大丈夫
コンパイラの最適化の基本の基本なので
このループの比較がパフォーマンスに大きく影響するなら
高速化テクニックは色々とある >>677
なんとなく互換性のある read を持つものをエンベロープして IReadable になるエンベロープクラスを用いるのはどう?
ifstream も IReadable じゃないからエンベロープする。
https://ideone.com/27yhhu >>674
> for (int i = 0; i <= n; ++i) {;} for ループといえばつい手癖で
for (int i =0; i < count-1; ++i ) {...}
と書いてて、
後日に count を int から size_t にしたとき
countが0のケースではまったことがあるなあ
for (int i =0; i + 1 < count; ++i ) {...}
と書けば良いだけなんだけど >>681
それcountが1でも全くループしないというヘンテコ仕様だけど
そんなもの手癖で書くの? >>683
隣り合った2項、例えばv[i] と v[i+1] を用いた処理をするとか
後続のものがある項だけ処理するとか良くある int **a を利用して2次元の動的配列を作ります。
a を関数に渡して、計算に利用します。
2次元配列 a の要素は変更しないので、
int func(const int **a){ ... }
としました。
main 内には以下のように書きました。
ret = func(a);
すると、 a を int** から const int** に変換できませんというエラーが出てしまいます。
これはなぜでしょうか? >>675
>>678
>>680
ありがとうございました。 >>683
そんなにヘンテコでもないと思う
for (size_t count = 0 ; count < 10 ; count++){
for (size_t i = 0 ; i < count - 1 ; i++){
....
}
}
こんなのはありがちかと >>687
レアケースっていうか
nが小数の場合だな ふむ、なくはないか。
やはり条件の方に+1とか-1とか書くんじゃなくて初期値変える方が素直だろうな。
最適化の下手なコンパイラ対策も含めて。 >>689
だからそう言うのをレアケースって言ってるんだが...
よく書くと言うなら多分住む世界が違う >>685
具体的なコードは以下です:
int func(const int **a) {
return 0;
}
int main() {
int n = 10;
int **a = new int*[n];
for (int i = 0; i < n; ++i) {
a[i] = new int[n];
}
func(a);
for (int i = 0; i < n; ++i) {
delete[] a[i];
}
delete[] a;
} >>692
さくらの続編を放送してる今日この頃、レイアースの第三章もワンチャン有るかもな! >>687
そんな事言ったら
countが0でループだってレアケースだぞ >>693
const付きに変換出来るのは
ポインタの先がconstに変わる場合
int finc(int * const * a) なら大丈夫 ありがとうございます。
const についてよくわからないのですが、
func の中で、
a[i][j] = 1;
みたいなことをできないようにしたいのですが、どうすればいいでしょうか? >>679
ありがとうございます。その方法で取りあえずの目的は果たせそうなのですが、ほかの方法はないでしょうか?
というのも、他のクラスでラッピングしたりキャストしたりせずにifstreamを食わせたいのです。
たとえば、STLの各種関数はポインタでもvector等のコンテナでも大体OKですよね?
あれと同様に関数ポインタでもクラスのインスタンスへのポインタでもread()を持つものなら何でも突っ込めるようにしてforでぶん回したいのです。
標準のコンテナや自作クラス等の違いを吸収できるような構造にしたいのです。条件が後出しになってしまって申し訳ありません。 キャスト
が一番コストが少ないと思う
関数の中で
delete [] a[i];
a[i] = new int[10];
は出来ても良いの? >>699
ありがとうございます。
ちょっと回答が理解できないため、質問を代えさせてください
以下のプログラムの func はNGなのに func2 はOKなのはなぜでしょうか? int func(const int **a) {
return 0;
}
int func2(const int *b) {
return 0;
}
int main() {
int n = 10;
int **a = new int*[n];
for (int i = 0; i < n; ++i) {
a[i] = new int[n];
}
func(a);
int *b = new int[n];
func2(b);
for (int i = 0; i < n; ++i) {
delete[] a[i];
}
delete[] a;
delete[] b;
} >>701
それは c / c++ の欠点の1つで、
T * は const T * に文句も言わず変換してくれるが
T ** は const T ** に変換してくれないという問題
T は const T と定数性以外 compatible だよね、という判断を 1 段階しかしてくれない。
キャストが必要
const_cast<const int**>(a) など >>702
皆が言っているのは質問の内容とはちょっと違うけど、
func の中で a[0] = 0; などを禁止するために内側のポインタ自体も const にして
const int * const * にした方がいいよということ いやごめん
func(int * const * )にすればコンパイル通るのか…
T * const * ==> const T * const * という変換は暗黙でokなのか
自分がバカでした >>703
欠点なの?
T ** を const T ** に勝手に変換されると困るんだが。 T * を const T * には勝手に変換するけどな >>706
ちょっと興味があるので困るコードを教えて >>705
慌ててたのかこのレス書き間違ってた
func(const int * const * )にすればコンパイル通る、
T ** ==> const T * const * という変換は暗黙でok、
と書きたかった(全然違うな) >>709
int** ipp;
const int ci=0;
const int ** cipp;
cipp = ipp;
*cipp= &ci;
**ipp = 3; おっと ipp の初期化忘れた
int** ipp;
const int ci=0;
const int ** cipp;
int *ip;
ipp = &ip;
cipp = ipp;
*cipp= &ci;
**ipp = 3; ロベールの本に、
ios::out | ios::trunc
が意味がないと書いてあります。
既存のファイルを破棄してから書き込むということだとすると意味があるように思います。
ios::in | ios::out | ios::trunc とした場合のみ意味があると書いてあります。
どういう意味なのでしょうか? truncate は、ファイルサイズを切り詰め・縮小する。
これは、読み書き両用時に使える機能
ファイルを読み込んで、ファイルサイズを縮小して書き込む
ファイルの書き込みでは、使えない。
書き込みでは、ファイルサイズ0で開くから、前のファイルが消える >>712
なるほどね。
T ** => cont T ** は許すとcont T を操作できてしまうから禁止されていて、
T * => const T *
T ** => const T * const *
...(以下略)
にはその問題が無いから許可されているわけか file.read((char*)buf, sizeof buf);
は buf のサイズ分だけ読み込めということだと思います。
ファイルの最後の部分を読み込むとき、サイズが WIDTH 未満の場合にはどうなるのでしょうか?
fstream file;
file.open("hello.txt", ios::in | ios::binary);
do {
unsigned char buf[WIDTH];
file.read((char*)buf, sizeof buf);
for (int i = 0, size = file.gcount(); i < size; ++i) {
printf("%02X ", buf[i]);
}
cout << endl;
} while (!file.eof());
file.close(); 👀
Rock54: Caution(BBR-MD5:1341adc37120578f18dba9451e6c8c3b) >>717
ありがとうございました。
あともう一つ質問させてください:
ロベールの本なのですが、
int n = 0x41424344;
file.write((const char*)&n, sizeof n);
というコードがあるページにあります。
その少し後ろのページには、以下のコードがあります。
char buf[BUF_SIZE];
dst.write(buf, src.gcount());
ここで、なぜ
dst.write((const char*)buf, src.gcount());
としていないのでしょうか? >>718
char *はconst char *に互換だから。constは書き換えないという意味で、constなしはconstありに対して互換性がある。 すみません。もう一つ質問です。
またロベールの本なんですが、以下のコードが書いてあります。
ファイルから入力中にエラーが起きたときの対処法です。
if(src.fail()) {
ではなく、
if(src.fail() && ! src.eof()) {
と書いてあります。その理由として、
「読み込みの場合はファイルの終端に到達した際も fail メンバ関数が
真となるため、 eof メンバ関数が真になる場合は除外しておきます。」
と書いてあります。
そこで質問です。
ファイルの終端の直前の部分を読み込む最後の読み込みの際に何等かのエラーが起こった場合、
src.fail() == true かつ src.eof() == true となります。
そのため、
src.fail() && ! src.eof() == false となってしまいます。
この場合、読み込みエラーが起こったにもかかわらず、その対処ができないことになりはしないでしょうか?
それともロベールさんのコードはOKなコードなんでしょうか? fstream src;
…
char buf[BUF_SIZE];
src.read(buf, sizeof buf);
if(src.fail() && ! src.eof()) {
error = true;
break;
} >>720
ありがとうございます。
つまりどちらもエラーにはならないということですね。
ですが、記述が統一していない理由というのは何か考えられるでしょうか?
int n = 0x41424344;
file.write((const char*)&n, sizeof n);
に const がついているのは、 n は int 型だから write に渡すときには絶対に
キャストしなければならない。(char *) でもいいが、どうせなら const もつけて
しまおうということですかね?
一方、
char buf[BUF_SIZE];
dst.write(buf, src.gcount());
の方は、キャストの必要がないからわざわざ (const char*) とキャストすることも
ないかなという感じですかね? ロベールさんの本は色々細かいことが書いてあって有用なのですが、なぜこういうコードなんだろう?という
疑問を持って読んでいくと次から次へと疑問が生まれてきて読み進むのが大変です。 Cスタイルのキャストは強力すぎて、効果が分かりにくいので、モダンでは、そのキャストは、reinterpret_cast<const char *>(&n)って書いた方がいい。 winsowsにはC#コンパイラが標準付属してますが、C++コンパイラは標準付属してないんですか? つまり、その本は古い。温故知新とは言うが、Cスタイルのキャストの使用はモダンではない。 ■ このスレッドは過去ログ倉庫に格納されています