【初心者歓迎】C/C++室 Ver.72【環境依存OK】
■ このスレッドは過去ログ倉庫に格納されています
0001デフォルトの名無しさん
2010/03/05(金) 16:51:13ただしその場合、質問者は必ず環境を書きましょう。
※sage禁止です(と代々スレに書いてありますが自己判断で)。
【前スレ】
【初心者歓迎】C/C++室 Ver.71【環境依存OK】
http://pc12.2ch.net/test/read.cgi/tech/1264774545/
【アップローダー】(質問が長い時はココ使うと便利)
http://kansai2channeler.hp.infoseek.co.jp/cgi-bin/joyful/joyful.htm
http://codepad.org/ (コンパイルもできるし出力結果も得られるのでお勧め)
◆ソースのインデントについて
半角空白やTABでのインデントはスレに貼ると無くなります。
そのため、アップローダーに上げるのも手ですが直接貼る場合は、
全角空白か に置換すると見栄えだけはよくなります。
0705デフォルトの名無しさん
2010/05/04(火) 13:08:010706デフォルトの名無しさん
2010/05/04(火) 13:12:400707デフォルトの名無しさん
2010/05/04(火) 13:44:120708デフォルトの名無しさん
2010/05/04(火) 15:12:22出力先の配列サイズをチェックしない関数で危険だから
safe関数使えって
0709デフォルトの名無しさん
2010/05/04(火) 15:17:15引数、ローカル変数、メンバ変数くらいしか確認する必要ないし
普通初めて使う変数の型くらい確認するだろ
>>693>>694
ATL使えでFA
スマポなしでプログラムしてるのが悪い
>>698
初期化リストに突っ込むときは基本的に変数を作る時だから
自分で型分かってるだろ
0710デフォルトの名無しさん
2010/05/04(火) 15:26:42>普通初めて使う変数の型くらい確認するだろ
だから、初めて使う時以外、その変数は使わないのかってww
全部型覚えとくの?それとも使うたびに毎回確認するの?p って付いてるだけで一手間減るなら別にいいじゃない
0711デフォルトの名無しさん
2010/05/04(火) 15:30:45型も分からない変数を扱おうとする人の気が知れない
0712デフォルトの名無しさん
2010/05/04(火) 15:34:13C++でポインタ使うのはライブラリの中だけだし
0713デフォルトの名無しさん
2010/05/04(火) 15:35:42関数の定義を見なくても引数のどれがポインタなのか分かるように関数名を修飾してるのか?
そもそもポインタかどうか以外にも確認すべき事はあるんじゃないか?
ポインタかどうかだけ分かっても仕方が無いんじゃないか?
ポインタ止めて参照にした時変数名変えるのか?
ってこと
0714デフォルトの名無しさん
2010/05/04(火) 15:36:47そう考えたらIDEって便利だよな。 どれだけ適当でも行ける
0715デフォルトの名無しさん
2010/05/04(火) 15:38:150716デフォルトの名無しさん
2010/05/04(火) 15:40:10少なくともポインタ止めて参照に変えるようなケースなら
そのタイミングでむしろ変数名変えたっていいんじゃね?
とりあえずおちつけ
0717デフォルトの名無しさん
2010/05/04(火) 15:58:28・どうせ変数の型は使うときに確認する
・名前の変え忘れが多い(ポインタじゃないのにpつける、ポインタじゃなくなったのにpのまま)
っていう実務上の教訓から、うちの会社はpつけんな、っていうコーディング規約になってるよ。
特にインテリセンスに慣れた輩は、変数名の記述ミス、スペルミスがあっても平気で放置するしな。
0718デフォルトの名無しさん
2010/05/04(火) 16:02:27読むだけならヒントにはなるかもね〜程度に考えた方がいい
そして読むだけなら周辺をチラ見すりゃポインタかどうかの判断ぐらいすぐつく
0719デフォルトの名無しさん
2010/05/04(火) 16:05:46Hoge hoges[num];
Hoge *hoge = new Hoge();
Hoge *hoges = new Hoge[num];
shared_ptr<Hoge> hoge;
vector<shared_ptr<Hoge>> hoges;
p つけると配列なのかそうでないのかとか
変数名の命名に困る
0720デフォルトの名無しさん
2010/05/04(火) 16:09:54名前付ける時点で型がわからないかもしれないじゃん
そういう時はpの人はどうしてるの?
0721デフォルトの名無しさん
2010/05/04(火) 16:34:29C++で今時生ポは無いだろう。
0722デフォルトの名無しさん
2010/05/04(火) 16:36:270723デフォルトの名無しさん
2010/05/04(火) 16:40:140724デフォルトの名無しさん
2010/05/04(火) 17:00:23今のC++でも普通に生ポは使うだろう。
俺はshared_ptr愛用するけども。
0725デフォルトの名無しさん
2010/05/04(火) 17:03:130726デフォルトの名無しさん
2010/05/04(火) 17:03:260727デフォルトの名無しさん
2010/05/04(火) 17:04:180728デフォルトの名無しさん
2010/05/04(火) 17:06:550729デフォルトの名無しさん
2010/05/04(火) 21:49:29スマポなんて,絶対つかっちゃだめ
0730デフォルトの名無しさん
2010/05/04(火) 21:54:390731デフォルトの名無しさん
2010/05/04(火) 22:05:24しないなら状況に最適化されたスマポを自作
カプセル化された環境で管理しきれるなら生ポ
0732デフォルトの名無しさん
2010/05/04(火) 22:07:01デストラクタで廃棄
0733急にごめんなさい。。。
2010/05/04(火) 22:10:50構造体を使い、メンバー変数に氏名、所属コース、出身高校を持つものを作成し
キーボードからこれらの変数を入力し、出力するプログラムを作成しなさい。
という実習課題がでたのですが自分が作成したプログラムでは
セグメンテーション違反です
と表示されるばかりで。。。
教えてください。どこがおかしいのでしょうか?
#include<stdio.h>
struct profile{
char *Name;
char *Course;
char *Koukou;
};
int main(void){
struct profile seito1 ;
printf("氏名を入力してください。\n");
scanf("%s",&seito1.Name);
printf("所属コースを入力してください。\n");
scanf("%s",&seito1.Course);
printf("出身高校を入力してください。\n");
scanf("%s",&seito1.Koukou);
printf("氏名:%s\n",seito1.Name);
printf("所属コース:%s\n",seito1.Course);
printf("出身高校:%s\n",seito1.Koukou);
return 0;
}
0734デフォルトの名無しさん
2010/05/04(火) 22:17:130735デフォルトの名無しさん
2010/05/04(火) 22:17:540736急にごめんなさい。。。
2010/05/04(火) 22:19:56セグメンテーション違反です。と表示されます。。。
どこを変えればいいんでしょうか?
0737デフォルトの名無しさん
2010/05/04(火) 22:20:21char Name[100]; //保存場所が無い。
scanf("%s",seito1.Name) //配列はポインタとして使える
0738デフォルトの名無しさん
2010/05/04(火) 22:22:58入力された文字列のデータはどこに格納してるつもりだい
0739急にごめんなさい。。。
2010/05/04(火) 22:26:27できました!!!ありがとうございます。
0740デフォルトの名無しさん
2010/05/04(火) 22:27:45リアルゆとりにそんなこというなよ
>>733は低脳日本人が生み出した超すごい世代なんだから
>>733 ゆとりだから質問するスレが分らないのはしょうがないけど、普通は
http://pc12.2ch.net/test/read.cgi/tech/1269438098/l50
で宿題関連は質問する。次回からそこで質問汁
0741デフォルトの名無しさん
2010/05/05(水) 01:01:51下記のようにすると、ビルドは成功するのですが
実行時に"The variable 't' is being used without being defined"
というエラーメッセージが出ます。
関数test(t)で*tに文字列を入れるにはどうしたらいいでしょうか。
教えてください。
int main(int argc, char *argv[])
{
char *t;
test(t);
cout << t << endl;
return EXIT_SUCCESS;
}
void test(char *t)
{
t = new char[100];
int i = 0;
cout << "step1" << endl;
t[i++] = 'a';
t[i++] = '\0';
}
0742741
2010/05/05(水) 01:09:57で質問しました。
0743デフォルトの名無しさん
2010/05/05(水) 01:18:04void test(char *t)
これはポインタを値渡ししてるからポインタを引数で返すことはできない
ポインタを返すにはポインタのポインタ
void test(char **t)
にしなければならない
後は自分で治して
0744デフォルトの名無しさん
2010/05/05(水) 01:31:13int main(int argc, char *argv[])
{
char *t;
test(t);
cout << t << endl;
return EXIT_SUCCESS;
}
void test(char *&t)
{
t = new char[100];
int i = 0;
cout << "step1" << endl;
t[i++] = 'a';
t[i++] = '\0';
}
0745デフォルトの名無しさん
2010/05/05(水) 01:32:30つ void test(char* &t)
0746デフォルトの名無しさん
2010/05/05(水) 01:34:56ポンタポンタ版もよろ
0747デフォルトの名無しさん
2010/05/05(水) 01:56:40int main(int argc, char *argv[])
{
char *t;
test(&t);
cout << t << endl;
return EXIT_SUCCESS;
}
void test(char **t)
{
*t = new char[100];
int i = 0;
cout << "step1" << endl;
(*t)[i++] = 'a';
(*t)[i++] = '\0';
}
0748741
2010/05/05(水) 02:04:30アドレス版(*&t)とポインタポインタ版(**t)共に
ビルドと実行がうまくいきました。
本当にありがとうございました。
いと難しいこと限りなし
0749デフォルトの名無しさん
2010/05/05(水) 03:42:08実装が↓みたいな感じで
かなり定型的で面倒なのですが,
こういうものなのでしょうか.
というより,もっと簡便な方法があったりするでしょうか?
char dst[256];
memset( dst, 0, sizeof( dst ) );
strncpy( dst, src, sizeof( dst ) / sizeof( char ) - 1 );
0750デフォルトの名無しさん
2010/05/05(水) 04:00:42char dst[256];
int len = strlen(src);
if (len >= sizeof(dst)) {
len = sizeof(dst) - 1;
}
memcpy(dst, src, len);
dst[len] = '\0';
か、
こうする
sprintf(dst, "%.*s", sizeof(dst)-1, src));
0751デフォルトの名無しさん
2010/05/05(水) 04:02:20マクロ使うぐらいじゃないかなぁ。 あとそのコードだとmemsetはいらないと思うけど。
Better CとしてのC++に移行するとちょっと楽になるよ。
0752デフォルトの名無しさん
2010/05/05(水) 04:11:46あとmemsetはdstの最後に終端文字を付けてるんですね.
0753デフォルトの名無しさん
2010/05/05(水) 04:18:390754デフォルトの名無しさん
2010/05/05(水) 07:05:54「定型的」ってことは、文字列を入れる配列は「とりあえず0クリア」
みたいに思ってるのかね。
文字列の後ろはゴミを入れたままでいいよ。
0755デフォルトの名無しさん
2010/05/05(水) 07:53:56他のメンバがあったりしてコンストラクタから例外が飛ぶと漏れることがあるから気をつけろよ。
0756デフォルトの名無しさん
2010/05/05(水) 08:55:01{
//初期化メンバ関数
this->init(hWnd, tyui, width, height);
}
C++の勉強をしていたのですが、とあるソースのコンストラクタの中に以上のような記述がありました
クラスの中で、コンストラクタとは別に初期化用関数を作るメリットなどがありましたら教えて頂きたいです
0757デフォルトの名無しさん
2010/05/05(水) 09:12:25コンストラクタ以外でも init() を使える
0758デフォルトの名無しさん
2010/05/05(水) 09:13:54引数違いのコンストラクタが複数ある場合には必須だとか
デフォルトコンストラクタを定義したいとか
色々あるだろうけど、this->厨は爆発すればいいと思う。
0759デフォルトの名無しさん
2010/05/05(水) 09:27:12参考になりました。ありがとうございます。
ちなみに「this->」は何が問題なのでしょうか・・・?
0760デフォルトの名無しさん
2010/05/05(水) 09:34:170761デフォルトの名無しさん
2010/05/05(水) 10:49:16とりあえず0クリアがコーディング規約になってる場合もあるからなあ
でも char dst[256] = ""; だけで0クリアできるんだけどね
配列の初期化では、初期化を行っている場合、値の指定されていない要素は0で初期化される
(明示的に初期化していない場合は実際に何も初期化されないので、とりあえず何かで初期化する必要はある)
0762デフォルトの名無しさん
2010/05/05(水) 10:52:16純粋にキモい
0763デフォルトの名無しさん
2010/05/05(水) 12:42:39でも例えばその文字配列を丸ごとどこかファイルに永続化するような場合には、
予めそこにゴミが残ってる事もわかった上で使わないといけない
気にするべきポイントなんてそんなもん
0764デフォルトの名無しさん
2010/05/05(水) 12:53:34今やってるのは
・keyの長さ(4 byte)
・valueの長さ(4 byte)
・key (x byte)
・value (y byte)
を1つのデータと見て
データの長さを読む→keyとvalueを読む→次のデータの長さを読む→・・・
といった感じで先頭から繰り返して欲しいkeyに対応するvalueをメモリに読み込みます
ですがこれだと死ぬほど遅いので効率をよくしたいです
なにか典型的な解決方法とかってありますか?
0765デフォルトの名無しさん
2010/05/05(水) 12:58:42ってか.
0766デフォルトの名無しさん
2010/05/05(水) 12:59:12まとめてメモリに読み込んでから処理する。
0767デフォルトの名無しさん
2010/05/05(水) 12:59:520768デフォルトの名無しさん
2010/05/05(水) 13:02:30メモリに読んでから構造体(POD型)で取り出しちゃう
0769デフォルトの名無しさん
2010/05/05(水) 13:03:12memset( this, 0, sizeof( this ) );
とすると良くないという話を前に聞いたんですが,
どこが良くないか
ご意見頂けませんでしょうか.
0770デフォルトの名無しさん
2010/05/05(水) 13:10:12sizeof( this )これは置いといて。
コンストラクタで初期化されたメンバを0で塗りつぶしちゃいかんでしょ。
0771デフォルトの名無しさん
2010/05/05(水) 13:11:540772デフォルトの名無しさん
2010/05/05(水) 13:17:110773デフォルトの名無しさん
2010/05/05(水) 13:46:13このような回答もあるのですが...?
0774デフォルトの名無しさん
2010/05/05(水) 13:58:33外部から叩いてもらうような処理って、外部からだとインスタンスを区別する事が出来ないから
やっぱ無理なんだろうか
0775デフォルトの名無しさん
2010/05/05(水) 14:01:06がしかし、
> double 型やポインタ型は、これらを構成する全ビットが 0 になったとしても、
> オブジェクトの値が 0 になるかどうかは分からない
てか何でクラスのメンバ全部をmemsetで初期化しようとするのか理解できん
0776デフォルトの名無しさん
2010/05/05(水) 14:04:210777デフォルトの名無しさん
2010/05/05(水) 14:12:32下記コードだと、ISO なんちゃら警告が出てコンパイル不能。
class Foo;
typedef int (Foo::*FUNC)(); //typedef int (*FUNC)();
void test(FUNC f) {
printf("%d", f());
}
class Foo {
private: int x;
public:
Foo(int a): x(a) {}
int getValue() { return x; }
};
int main() {
Foo f(123);
test(f.getValue); //test(&(f.getValue));
return 0;
}
0778デフォルトの名無しさん
2010/05/05(水) 14:12:45メンバ関数ポインタとオブジェクトのアドレスを渡してあげれば出来るよ
でもまあ普通は関数オブジェクトにして渡すけど
0779デフォルトの名無しさん
2010/05/05(水) 14:14:03メンバ関数へのポインタを得る記述は &Foo::getValue だ。インスタンスからは取れない。
0780デフォルトの名無しさん
2010/05/05(水) 14:16:26静的メンバとしてそれで取り出せるのは知ってたけど、やはりインスタンスからは無理か。
外から見たらそれがどのインスタンスのメンバなのかを特定出来ない(this不明)からか
>>778
Invokeさせるって話ですね。それはたまに書いてます
0781デフォルトの名無しさん
2010/05/05(水) 14:17:330782デフォルトの名無しさん
2010/05/05(水) 14:51:49#include<stdio.h>
int main(void)
{
printf("hello world!");
return 0;
}
というプログラムを実行すると
実行画面が表示されてすぐに消えるのですが
キーを押すまで表示したまま、というのはどうすればよいのでしょうか
0783デフォルトの名無しさん
2010/05/05(水) 14:58:450784デフォルトの名無しさん
2010/05/05(水) 14:59:29VisualC++ 2005なら
「デバッグ→デバッグなしで開始」(Ctrl + F5)でいけるよ
「デバッグ開始」(F5)だとすぐに消える
なぜかは分かりません。当方は仕様として認識してます。
0785デフォルトの名無しさん
2010/05/05(水) 15:01:21そういうのは個別に初期化すればいいでしょ
0786デフォルトの名無しさん
2010/05/05(水) 15:01:22終了次第後始末する → 閉じる って事じゃね
デバッグ無しだと起動しっぱなしになるってだけ。違うかな
0787デフォルトの名無しさん
2010/05/05(水) 15:05:34exeをダブルクリックしてないか?
コマンドプロンプトから呼び出せばすぐに消えないよ
myapp.exeというプログラムを作ったとしたら
コンパイルしたディレクトリでmyappと入力してエンターだ
0788デフォルトの名無しさん
2010/05/06(木) 12:15:00デバッグなしで実行のときは、余計なお世話なことにpauseコマンド(相当)を噛ましているだけ。
デバッグ開始の場合は、止めたければ勝手に止めればいいので余計なお世話もしない。
0789デフォルトの名無しさん
2010/05/06(木) 17:14:31ありがとうございます
先に言っておくべきだったと思うけどbcpadを使ってます
0790デフォルトの名無しさん
2010/05/06(木) 21:44:01int work[10000000];
int work2[10000000];
int main()
{
for(i=0;i<N_MAX;i++){
work[i] = i;
work2[i] = i;
}
t1 = my_clock();
for(i=0;i<N_MAX;i++){
a[i] = work[i];
}
t2 = my_clock();
t3 = my_clock();
for(i=0;i<N_MAX;i++){
a[i] = work2[i];
}
t4 = my_clock();
printf("%f %f\n",t2-t1,t4-t3);
}
質問です。以上のプログラムを実行したところ、aの配列にwork2の配列の要素を代入するほうが処理時間が早くなっています。
なぜ同じような操作を行っているのに処理速度が変わってくるのでしょうか?
0791デフォルトの名無しさん
2010/05/06(木) 21:59:23原因はここに書いてない部分にある。
0792デフォルトの名無しさん
2010/05/06(木) 22:04:40派生元と派生先両方のが実行されるんだよね?
0793デフォルトの名無しさん
2010/05/06(木) 22:10:24図書いてみて
言いたい事を整理し直してきなさい
0794デフォルトの名無しさん
2010/05/06(木) 22:23:23最初のループでa[i]も初期化してみたら結果が変わるかもね。
0795デフォルトの名無しさん
2010/05/06(木) 22:32:37最初のループは初期化を行っていて、
二回目のループはコピーになってるから?
0796790
2010/05/06(木) 22:41:35>>790
my_clock()関数の記述は省いたのですが、そちらに原因があるということでしょうか?
それとも環境的なものということでしょうか?
>>794
a[i]を初期化すると、早くはなったんですが、まだ差があるようです。
この差はなぜ生まれるのでしょうか?たぶん誤差の範囲では無い気がします。
0797デフォルトの名無しさん
2010/05/06(木) 22:49:09ぜんぜんわからない。
0798790
2010/05/06(木) 22:53:23double my_clock()
{
struct timeval tv;
gettimeofday(&tv, NULL);
return tv.tv_sec + (double)tv.tv_usec*1e-6;
}
0799デフォルトの名無しさん
2010/05/06(木) 23:05:57あまりにも勉強不足。
猫でもわかるプログラミング
http://www.kumei.ne.jp/c_lang/
ロベールの部屋
http://www7b.biglobe.ne.jp/~robe/
Codian
http://www.kab-studio.biz/Programing/Codian/
C++入門
http://www.asahi-net.or.jp/~yf8k-kbys/newcpp0.html
ATLAS Japan C++ Course
http://www.icepp.s.u-tokyo.ac.jp/~sakamoto/education/atlasj/cplusplus/index.html
上から順にここにあるぐらいの内容は読め。
0800デフォルトの名無しさん
2010/05/06(木) 23:18:25cygwin で gcc で再現した。
0.051000 0.043000
for(i=0;i<N_MAX;i++){
work[i] = i;
work2[i] = i;
a[i] = i;
}
にしてみたら、
0.043000 0.043000
になった。不思議だな…
0801790
2010/05/06(木) 23:27:41初期化すると処理時間一緒になってますね。
私が実行すると
0.037232 0.036249
と、差が出ています。環境はemacs+gccです。
0802デフォルトの名無しさん
2010/05/06(木) 23:30:47その実行ファイルを2回か3回連続して実行してみて
0803790
2010/05/06(木) 23:34:483回続けてやってみました。
0.037251 0.036251
0.037230 0.036211
0.037259 0.036257
0804デフォルトの名無しさん
2010/05/06(木) 23:41:37なんだろな。 俺も不勉強でわからん
■ このスレッドは過去ログ倉庫に格納されています