スレを勃てるまでもないC/C++の質問はここで 17at TECH
スレを勃てるまでもないC/C++の質問はここで 17 - 暇つぶし2ch733:デフォルトの名無しさん
10/11/11 14:30:22
数千行のプログラムである処理を行うとき、
関数の引数としてポインタを利用する場合とグローバル変数を
用いる場合ではどのような違いがありますでしょうか?

734:デフォルトの名無しさん
10/11/11 14:30:22
ひ、ひじかた…

735:デフォルトの名無しさん
10/11/11 14:34:27
グローバル変数は極力使わないようにするべき

736:デフォルトの名無しさん
10/11/11 15:14:44
グローバル変数を使うと悪意を持った同僚がバグを仕込む確率が高くなります

737:デフォルトの名無しさん
10/11/11 15:28:41
関数コールの時にスタックに積まない分、グローバルの方が早いって事?

738:デフォルトの名無しさん
10/11/11 15:40:55
>>733
仕様をいちいち客に問い合わせるか、
仕様書を見るかの違い。

739:z
10/11/11 16:54:20
C言語で分からない問題があります。
{if-else文} 身長と体重のデータを読み込み、下の定義から体型を判定するプログラムを作成せよ。

定 義:身長h(cm),体重w(Kg)とすると

w < (h-95)*0.82 やせすぎ(SLIM)

w > (h-95)*1.10 ふとりすぎ(FAT)

その中間 正常(NORMAL)



740:デフォルトの名無しさん
10/11/11 16:56:19
マルチしね

741:デフォルトの名無しさん
10/11/11 17:41:13
>>737
誤差らしい

742:デフォルトの名無しさん
10/11/12 11:29:00
>>737
最近のコンパイラはstaticな関数はどんどんインライン展開してしまうのでスタックに積むコストはなくなる場合が多くなります。
また、グローバル変数はCPUキャッシュの外にある場合があるので、アクセスコストが馬鹿にならない場合があります。

そんなことよりも、それが問題になるようなレベルの高速化をする前にやることは腐るほどあるので、保守性を上げる為に
どうしても必要がある場合以外はグローバル変数は禁止の方向で。まして、externで外部変数を参照するなんて以ての外。

743:デフォルトの名無しさん
10/11/12 12:15:27
>>733
マルチスレッド化に際し、大きな障害になる。

744:デフォルトの名無しさん
10/11/12 16:14:21
応用的なクラスを効率とか安全性考えて試行錯誤するんだけどstlとかboost使ってとりあえずでやっつけたほうが良いものが出来上がって悲しい

745:デフォルトの名無しさん
10/11/13 08:38:18
>>744
俺たちの代わりに試行錯誤してくれたんだと考えれば悲しくない。

746:デフォルトの名無しさん
10/11/13 19:14:16
DirectXでゲームを作ってます

外部ファイルで持たせてあるデータのロードに失敗した時に、
エラーメッセージを表示して強制終了したいのですが、
MessageBox()関数には引数として、hwndが必要です。

クラス設計を進めていくと、わざわざ全てのクラスのコンストラクタに
hwndを渡すのが煩わしいのですが、他の方法はないでしょうか?

hwndをグローバル変数で持たせる、という方法も思いついたのですが、
エラーメッセージのために、普通、そこまでするものでしょうか?

ちなみに、_ASSERT_EXPR()は、リリース時には動かないので、除外してください

747:デフォルトの名無しさん
10/11/13 19:50:19
例外を投げてUIを扱えるところまで戻ったところでcatchする

748:デフォルトの名無しさん
10/11/13 19:57:00
>>746
言わんとするところは分かるが根本的な設計が間違ってる

ウィンドウハンドルみたいなハード依存部分を
グローバルスコープで参照したり各クラスオブジェクトに渡すのは良くない
ハードに依存する部分としない部分はキッチリ分けないと再利用性が無くなる

あと、その言い方だとたぶん
各オブジェクトごとにファイルを読んでインスタンスを生成するような設計っぽいけど
それだと同じファイルを使うオブジェクトを複数生成するときに
オブジェクトの数だけ読み込みを行わなくならなくならないか?

やるべきことはデータロード部分と各クラスオブジェクトを分離すること
まずデータロードクラスなりを1つ作って
データ読み込みは全てコイツが行う
ウィンドウハンドルもこコイツだけが内部で知ってれば良くて
メッセージ表示等も全てコイツがやる

各クラスオブジェクトはこのデータロードクラスで
読み込んだデータを参照してインスタンスを生成し
それ自身はファイルの読み込みはしないようにする

749:デフォルトの名無しさん
10/11/13 19:59:00
なるほど!

750:デフォルトの名無しさん
10/11/13 20:11:08
>>748
便乗して質問してしまうが「データの参照」って具体的にどうやって書けばいいの?


751:デフォルトの名無しさん
10/11/13 20:17:36
>>746
コンストラクタとか関数の引数全てにhwnd渡すぐらいなら、
グローバル変数の方がいい

どうしても気なるなら、staticな変数にhwndを覚えるようなクラスを作って、
メッセージボックス表示をそのクラスで行なうとか

752:デフォルトの名無しさん
10/11/13 20:22:51
強制終了ならFatalExceptionとか作ってスローしてmainでキャッチすればいいじゃん

753:デフォルトの名無しさん
10/11/13 21:41:33
>>746
けっこうみんな悩んでるからここで聞いて解決する問題じゃないと思う


最新レス表示
レスジャンプ
類似スレ一覧
スレッドの検索
話題のニュース
おまかせリスト
オプション
しおりを挟む
スレッドに書込
スレッドの一覧
暇つぶし2ch