【初心者歓迎】C/C++� ..
661:デフォルトの名無しさん
08/02/25 12:37:27
定数文字列は値渡しにして、動的な文字列は参照渡しにしたいのですが、これだとメモリリークしますよね
どうやれば直りますか?
#include <iostream>
using namespace std;
class cl{
char *str;
public:
cl(char* const& x){ str=x; }
template <size_t n> cl(char (&x)[n]) {
str=new char[n]; strcpy(str,x);}
};
main(){
cl str="rrrr";
str="oooo";
getchar();
}
662:デフォルトの名無しさん
08/02/25 12:42:21
値渡しされているかフラグを作って、されていれば解放する様にすればいいのですが、
初期化前にコンストラクタで代入されてしまうとフラグが確認できません
663:デフォルトの名無しさん
08/02/25 12:47:11
>>661は間違えました 定数式が期待する方へ行っていませんでした それもどうやれば直りますか
664:デフォルトの名無しさん
08/02/25 12:56:55
>>661
やりたいことがよくわからん。せめて、コンパイルできるソース(の断片)を貼ってくれ。
665:デフォルトの名無しさん
08/02/25 13:00:43
まとめるとテンプレート関数で、
定数文字列" "と
固定文字列char [n]と
可変文字列char *を
区別して、上の二つならメモリを確保して値渡しにしたいんです
値を書き換えたいため
666:デフォルトの名無しさん
08/02/25 13:01:40
最近ずっとこいつ出没してるな
667:デフォルトの名無しさん
08/02/25 13:02:01
それと、新たにメモリを確保するときに、以前確保したメモリを解放してメモリリークしないようにしたいです
668:デフォルトの名無しさん
08/02/25 13:06:38
>>667
要は、const char *でコンストラクトするときはメモリを確保して、char *でコンストラクトするときは確保しなければ委員でね?
それと、書き換えのときはconst char *でもchar *でも以前の状態に応じて解放しないといけないんでね?
669:デフォルトの名無しさん
08/02/25 13:07:20
簡略化すると関数の引数で、" "型と、char[]型と、char*型を区別したいって事です
670:デフォルトの名無しさん
08/02/25 13:17:48
>>668
constつけるだけでは無理です
#include <iostream>
using namespace std;
class cl{
char *str;
public:
cl(char const *x) { cout<<"文字列は定数です\n"; }
cl(char *x){ cout<<"文字列は可変長です\n"; }
};
main(){
cl str="xxx"; //定数で初期化
char ch[]="yyy"; str=ch; //固定配列の代入
str="zzz"; //定数の代入
getchar();
}
671:デフォルトの名無しさん
08/02/25 13:19:34
すまん、そもそも char ch[] = "yyy"は固定長でも書き換え可能な配列なんだが
それを定数として扱ってはいけない理由を教えてくれ。
672:デフォルトの名無しさん
08/02/25 13:22:36
あ、聞き方が変だ。
char ch[] = "yyy"は書き換え可能なんだから可変長と同一視すればいいし
const char ch[] = "zzz"は書き換え不可能なんだから定数と同一視すればいいのでは?
673:デフォルトの名無しさん
08/02/25 13:22:41
可変長文字列と {定数文字列、固定長文字列} （ ← 一緒でいいです）
を分類する方法教えてください
あと、すでにメモリが確保してあればそれを解放する方法教えてください
674:デフォルトの名無しさん
08/02/25 13:23:36
長さも書き換えられないといけません
675:デフォルトの名無しさん
08/02/25 13:24:19
こいつ日本人なのかなぁ
676:デフォルトの名無しさん
08/02/25 13:26:00
＞あと、すでにメモリが確保してあればそれを解放する方法教えてください
これは簡単。コンストラクト時にメモリを確保したらフラグをセットしておいて、代入オペレータでそれをチェックすればいい。
勿論、代入オペレータも2種類用意して必要ならフラグをセットするのは当然として。
そのフラグはいずれにしてもデストラクト時に必要にナル死ね。
677:デフォルトの名無しさん
08/02/25 13:28:05
あー、ローカルな配列だとコンストラクト時にフラグが立ったら問題か。
ここは一つ、char配列は渡さないと言う運用でw
678:デフォルトの名無しさん
08/02/25 13:28:42
で、この馬鹿はなんで目的を説明できないんだろうなぁ。>675ってことなんだろうか。
679:デフォルトの名無しさん
08/02/25 13:43:23
目的は、参照渡しのできるstringを作りたいんです
現存するやつは値渡しです
680:デフォルトの名無しさん
08/02/25 13:45:33
いやだから、それを(態々)作る目的を聞いているのだが。
681:デフォルトの名無しさん
08/02/25 13:46:27
チャットじゃないんだから要旨をまとめてから書き込め。
682:デフォルトの名無しさん
08/02/25 13:46:53
>>676
クラスの確保と同時に代入されてしまうと、フラグが設定されていないので動作不明になりませんか?
683:デフォルトの名無しさん
08/02/25 13:50:39
>>681
可変長文字列は参照渡しにして、それ以外は値渡しにして、新たな文字列が渡されたら以前のを解放したいんです
684:デフォルトの名無しさん
08/02/25 13:52:58
>>683
>680
685:デフォルトの名無しさん
08/02/25 13:53:25
伸びてると思ったら、また例の日本語が不自由なヤツが現れたのか
686:デフォルトの名無しさん
08/02/25 13:56:58
何をしたいかと言うと
(バイナリ) 文字列 str , p , qに対して
replace(str, p, q) strの全文でpをqに置換する
という関数を作りたいんです
687:デフォルトの名無しさん
08/02/25 14:03:08
>>665
それぞれの型の応じて関数を振り分ければいいのか？
template<size_t N>
void g(const char (&a)[N]) {
cout << typeid(a).name() << endl;
}
template<size_t N>
void g(char (&a)[N]) {
cout << typeid(a).name() << endl;
}
void g(char *&p)
{
cout << typeid(p).name() << endl;
}
template<typename T>
void f(T& t)
{
g(t);
}
688:デフォルトの名無しさん
08/02/25 14:03:31
//テスト
int main()
{
f("abc");
char a[5];
f(a);
char *p;
f(p);
}
689:デフォルトの名無しさん
08/02/25 14:08:07
ぜんぶcです・・・・
#include <iostream>
#include<typeinfo>
using namespace std;
template<size_t N> void g(const char (&a)[N]) { cout << "a" << endl;}
template<size_t N> void g(char (&a)[N]) { cout << "b" << endl; }
void g(char *&p){ cout << "c" << endl; }
template<typename T> void f(T& t) { g(t); }
int main() {
f("abc");
char a[5]; f(a);
char *p; f(p);
}
690:デフォルトの名無しさん
08/02/25 14:12:45
>>689
そりゃコンパイラがだめなんだろうな。
g++ 4.2.3だとa b cになる。
691:デフォルトの名無しさん
08/02/25 14:12:57
>>686
これでいいのか？
#include <string>
#include <iostream>
using namespace std;
void replace(string& str, string p, string q)
{
int i = str.find(p);
while (i >= 0) {
str.replace(i, p.length(), q);
i = str.find(p, i + q.length());
}
}
int main()
{
string str = "abcdabcd";
replace(str, "bc", "BC");
cout << str << endl;
}
692:デフォルトの名無しさん
08/02/25 14:17:34
>>686
俺だったら、次のどっちかにする。だから、673のようなことをしたくなる理由が理解できない。
1. strは全く書き換えず、置換後の文字列は新しく確保したメモリに書き込んで戻り値に返す。
2. strは書き換えできるという前提でstrを書き換える。（書き換えできない
文字列をstrに渡したければ、書き換え可能な場所へコピーを作ってそれを渡す）
693:デフォルトの名無しさん
08/02/25 14:25:43
STR str="hogehogehogehogehoge"
replace(str, "geho", "(^_^)"); や、
int N=300*1024*1024;
char *ch=new char [N];
for(int n=0;n<N;n++)ch[n]=n; //巨大なバイナリ配列
STR str(ch,N);
replace(str, "geho", "(^_^)");
がしたいんです
694:デフォルトの名無しさん
08/02/25 14:29:10
巨大な配列を扱おうとすると、元のデータをコピーしたり、返却時に新規確保すると、
もとのデータと併せて領域が2倍必要になりますしコピー時間がかかります
>>691 バリナリデータも扱いたいです
695:デフォルトの名無しさん
08/02/25 14:32:38
>>693
できるぞ
>>694
バイナリデータも問題ない
#include <string>
#include <iostream>
using namespace std;
void replace(string& str, string p, string q)
{
int i = str.find(p);
while (i >= 0) {
str.replace(i, p.length(), q);
i = str.find(p, i + q.length());
}
}
int main()
{
string str="hogehogehogehogehoge";
replace(str, "geho", "(^_^)");
cout << str << endl;
int N=300*1024*1024;
char *ch=new char [N];
for(int n=0;n<N;n++)ch[n]=n; //巨大なバイナリ配列
string str2(ch,N);
replace(str2, "geho", "(^_^)");
cout << str2 << endl;
}
696:デフォルトの名無しさん
08/02/25 14:32:39
C++
開発環境 CentOS
コンパイラ g++
関数書き出しプログラムが機能せず。
どうしてでしょうか。
#include <iostream>
#include <fstream>
#include <cstring>
#include <string>
#define buffer_size 1000
using namespace std;
int main(int argc,char *argv[]){
string buffer;
int count=0;
char *p;
ifstream fin(argv[1]);
if(!fin)return 0;
while(fin>>buffer){
while(count!=1){
if((p=strchr(buffer.c_str(),'{'))!=NULL){
count++;
cout<<buffer;
}
}
while(count!=0){
if((p=strchr(buffer.c_str(),'{'))!=NULL)count++;
else if((p=strchr(buffer.c_str(),'}'))!=NULL)count--;
}
}
fin.close();
return 0;
}
697:デフォルトの名無しさん
08/02/25 14:35:18
>>694
置換前より後の文字列のほうが長い場合、
単純にその場で書き換えていくと、1ヶ所置換が行われる度に
置換部分以降をずらしたり、時にメモリの伸張をしたりで時間がかかる。
最近のOSはメモリの扱いも上手になっているし、
そもそもPCのメモリ搭載量も大きくなっているから、
新しいメモリに書き込んでいくも選択肢として捨てたものではない。
698:デフォルトの名無しさん
08/02/25 14:38:10
>>689
VC++ 2008でしか試してない。オーバーロード解決は規格に従って
いるんだが。コンパイラがテンプレート規則に対応していないのかね。
699:デフォルトの名無しさん
08/02/25 14:48:28
>>695 >>697
stringにコピーするのが一番なんですかね?
関数で作業領域を確保しようとするので 初めの3倍必要になります
コピー無しなら2倍ですみます 300メガとかだと動作に支障出ないですかね・・
>>698
うちのコンパイラだと全滅しました MinGWの最新版 、DMC、VC++6.0、bcc5.5.1
700:デフォルトの名無しさん
08/02/25 14:51:46
>>692
2番の方向で行こうとしているところだったんです
定数なら書き換え可能な領域を確保しようとしたら、その判別がコンパイラでできないのでここで質問していたんです
701:デフォルトの名無しさん
08/02/25 14:51:46
>>699
new char[]せずに最初からstringの中にデータを作ればコピーを減らせる
int N=300*1024*1024;
string str2;
str2.resize(N);
for(int n=0;n<N;n++)str2[n]=n; //巨大なバイナリ配列
replace(str2, "geho", "(^_^)");
702:デフォルトの名無しさん
08/02/25 14:57:26
わかりました
stringに直接挿入する方向で行こうと思います
巨大な入力があるのはファイルからなので、自作して直接stringに入れるようにします
703:デフォルトの名無しさん
08/02/25 15:00:35
>>699
＞コピー無しなら2倍ですみます 300メガとかだと動作に支障出ないですかね・・
stringの操作云々よりも、その情報量をどう扱うかと言うアルゴリズムの方が問題になりそうな希ガス。
少なくとも、GiBオーダ未満のメモリくらい今時普通に扱えるよ。
>>696
どう、機能していないのさ。
704:デフォルトの名無しさん
08/02/25 15:01:19
ちゃんと目的を説明すれば、妥当な回答が手早く得られると言う好例だな。
705:692
08/02/25 15:03:09
>>700
俺の場合、その判別は自分でやる。
必要なら自分でメモリ確保してmemcpyか何かでコピーしてから置換する関数へ渡すと言うこと。
メモリ確保その他をstringに任せているという点が違うけど、695案と同系統のアイデア。
706:デフォルトの名無しさん
08/02/25 15:04:31
>>696
関数突入を探すwhileの条件がcount != 1だから、関数突入後に中括弧を見つけるとcountが2になって脱出できなくなる罠。
707:デフォルトの名無しさん
08/02/25 15:10:50
>>696
>706に加えて、内側のwhileは両方ともifでいいだろよ。
つまり、最初のwhileは if (count == 0)で、次のwhileはif (count > 0)。
んで、fin >> bufferだと単語単位でしか入力できないから工夫が必要。
# 行単位入力すればいいのかな? 仕様がわからんからなんとも言えんが。
708:デフォルトの名無しさん
08/02/25 15:13:01
初めはC言語のみで、作成しようとして最後まで作ったのですが
ポインタのポインタとかやっているうちに動かない場合があることに気づき修復不可能になりました
そしてC++で参照や型判定して作ろうとしたらこれも上手くいきませんでした
簡明なプログラムが大事ですね
709:デフォルトの名無しさん
08/02/25 15:20:04
ここできいたのを参考に変換関数を作ってたのですが、そしたら全体がややこしくなり回復無理になりました
#define strconv(q, p) _memconv(&q, &p, strlen(p))
#define memconv(q, p, n) _memconv(&q, &p, n)
#define strconstconv(q, p) _strconstconv(&q, p)
_memconv(strdata *q, char **p, int n){
char **chend =(char **)malloc(sizeof(char **));
q->start=p; *chend=&(*p)[n]; q->end = &(*chend);}
710:デフォルトの名無しさん
08/02/25 15:52:07
>>708,709
無理して難しく書いても、あとあとメンテナンスやデバッグのことを考えると、シンプルに書いた方がいい場合が多い
メモリを何MBか余計に使ったり、実行時間が何分か伸びたところで、なんだというんだ
ややこしいコードは書くのもデバッグも何日もかかる
それだけの価値があればやってもいいが、たいていは割に合わない
711:デフォルトの名無しさん
08/02/25 17:21:33
やさしいC＋＋買って来た。
一日でマスターするのは無理なのかなぁ･･･
712:デフォルトの名無しさん
08/02/25 17:26:04
一週間くらいは覚悟しないといかんね
713:デフォルトの名無しさん
08/02/25 17:39:36
一年ぐらいは(ry
とか言いたいけど最低10年とかいう意見もあるし
やね某が何年か前に提唱した半年ぐらいで勘弁しておいてやろう
714:デフォルトの名無しさん
08/02/25 18:04:09
質問ですが、stringの+は、\0を特別視しませんよね?
"aa\0aa" + "bbbbb"はどのコンパイラでも、
"aa\0aabbbbb"ですよね?
715:デフォルトの名無しさん
08/02/25 18:38:41
いつからCは+演算子で文字連結できるようになったんだ
716:デフォルトの名無しさん
08/02/25 18:39:19
stringと書いてあるだろうがアホ
717:デフォルトの名無しさん
08/02/25 18:43:10
>>716
>>714はC++でもNGだろ。C文字列同士だぞ。
718:デフォルトの名無しさん
08/02/25 18:45:36
そこは中身がそうなっているstringインスタンスだと解釈してやろうぜ。
719:デフォルトの名無しさん
08/02/25 18:48:48
質問者はそういう中身のstringを造ろうとしてそこでつまりそうだが
720:デフォルトの名無しさん
08/02/25 19:21:16
>>718
アイアイ
721:デフォルトの名無しさん
08/02/25 19:30:23
std::auto_ptrって、ブロックを出ると自動的に開放されるポインタですよね？
これって、AUTO変数と比べてより便利な点ってあるのでしょうか？
722:デフォルトの名無しさん
08/02/25 19:36:53
×ブロックを出ると自動的に開放
○std::auto_ptrが破棄されたときインスタンスも自動的に開放
723:デフォルトの名無しさん
08/02/25 19:37:01
>>721
・スタックに置くには適さない巨大なデータも置ける
・最初はNULLにしておいて必要になってからnewできる
・ブロックを出るまで待たなくても好きなタイミングで削除できる
・releaseで手放せばブロックを出ても削除しないことも出来る
724:デフォルトの名無しさん
08/02/25 19:39:10
関数の戻り値にも使えるはず。
725:デフォルトの名無しさん
08/02/25 19:41:51
using namespace std;
string func1() {
string s = "Hello";
return s;
}
void func2(string s) {
cout << s << endl;
}
int main() {
func2(func1());
}
この渡し方って解放されなかったり破壊されたりしますか？
処理系によりけりですか？
726:デフォルトの名無しさん
08/02/25 19:43:39
>>725
しません。
どの処理系でも平気。
727:デフォルトの名無しさん
08/02/25 19:59:12
>>722-724
ありがとうです。
ところで、testは適当なクラスとして、
int main()
{
std::auto_ptr<test> tp(new test);
tp.release();
return 0;
}
とすると、メモリリークが検出されてしまうのですが、tp.release()をしないようにすると
大丈夫でした。これってrelease()の使い方が間違ってますか？
あと、最初にauto_ptr<test>だけ宣言しておいて、途中でnew testする方法もわかりません。
もうちょっとだけ教えてくださいな。
728:デフォルトの名無しさん
08/02/25 20:05:08
releaseは、以後、自分でメモリ管理するからauto_ptrはdeleteするなというもの。
729:デフォルトの名無しさん
08/02/25 20:14:08
>>728
最初は、
std::auto_ptr<test> tp(new test);
tp.release();
test* tp2 = tp.get();
delete tp2;
としていたのですが、以下のようにしなければならないということですね。
std::auto_ptr<test> tp(new test);
delete tp.release();
ありがとうございました。
730:デフォルトの名無しさん
08/02/25 20:27:13
>>729
tp.reset();
731:デフォルトの名無しさん
08/02/25 20:44:23
resetはVC6のauto_ptrには無いとか言い出すぜきっと。
732:デフォルトの名無しさん
08/02/25 20:52:38
resetもしないでデストラクタに任せればいい場面にまで
729のようなことをしていないか不安。
733:デフォルトの名無しさん
08/02/25 20:58:49
設計について色々調べていると（デザインパターン等々）これ無駄じゃね？みたいな部分に多々遭遇します
例えばコンストラクタとデストラクタで確保＆解放を行う為に全体をそれに合わせたり
それの為にメソッドやクラスを用意したり等々
確かに見通しもよくなりますし、わかりやすくなるとは思うのですが、
それらを犠牲にすれば短く出来たりオーバーヘッドを減らせたり出来るんじゃないか？と考えてしまいます
つまりコード全体の見易さ、転じてメンテナンスが容易になるなどのメリットと
速度を犠牲にするというデメリットがあると思います
とはいえ後者は今時気にするほどでもなく、詳しくないのですがここを気にするのは組み込み系と呼ばれる分野くらいなのかなとも思います
実際はやはり見易さ等を気にした手法（なんていうんだろう？）を用いて設計を行うべきなんでしょうか？
うまく表現できなく、長くなってしまってすみません
734:デフォルトの名無しさん
08/02/25 21:01:16
Visual Studioって 何処にヘッダファイル置けば
735:デフォルトの名無しさん
08/02/25 21:04:18
Visual Studioって 何処にヘッダファイル置けば
フルパス指定しなくて読み込めるんですか？
C:\Program Files\Microsoft Visual Studio 8\VC\include
でOK?
736:デフォルトの名無しさん
08/02/25 21:06:29
>>735
ソースファイルと同じ場所なら
#include"hoge.h"
でおｋ
737:デフォルトの名無しさん
08/02/25 21:09:06
>>733
大丈夫
君よりコンパイラのほうが賢いから
738:デフォルトの名無しさん
08/02/25 21:09:18
速度犠牲とか偉そうに言うやつほど、ダメなプログラマ
739:デフォルトの名無しさん
08/02/25 21:10:01
普通の初心者質問には答えないんだなぁ
740:デフォルトの名無しさん
08/02/25 21:11:48
本当に遅くなることばかりじゃないしな。
std::sort が qsort より概して速くなる、というみたいに、
インライン化されると問題ないことも多い。
741:デフォルトの名無しさん
08/02/25 21:14:36
>>738
色んな分野があるって前提で考えてるんだろう？
お前は組み込み系とビジネスソフト開発が同じだと思っているのか？
それにプログラミングしてればみんなプログラマって思考が気持ち悪い
お前見たいのは答える側に回るな
742:デフォルトの名無しさん
08/02/25 21:14:59
プログラマではないです、ただいろんな分野があってそれに合ったプログラミングがあることくらいはわかっています
でもやっぱり速度は気にしちゃいけないところですか、無駄だと思ってしまうところから直す必要がありそうですね
コンパイラの最適化というのも知らない分野ですし、当分は見通しの良いプログラムを目指してみようと思います
743:デフォルトの名無しさん
08/02/25 21:18:54
まず動くプログラムを書く。
そして遅ければ改善する。
この順だな。
744:696
08/02/25 21:19:34
696です。
先ほどのプログラムを訂正したら、
一行ほどしか出力されませんでした。
どこが間違っているのでしょうか。
URLﾘﾝｸ(www.borujoa.org)
745:デフォルトの名無しさん
08/02/25 21:26:58
これをちょっと弄って横長から縦長にしたいのですが、
いまいち組み方がわかりません。環境はLinuxでコンパイラはgccです。
#includeは省略
#define max 50
using namespace std;
class hist{
private:
string buffer;
int count;
int mat[max],i,j;
public:
int func(char *argv[]);};
int hist::func(char *argv[]){
ifstream fin(argv[1]);
if(!fin)return 0;
for(i=0;i<max;i++)mat[i]=0;
while(fin>>buffer){
count=strlen(buffer.c_str());
for(i=1;i<max;i++){
if(i==count)mat[i]=mat[i]+1; }
}
for(i=1;i<max;i++){
cout<<i;
for(j=0;j<mat[i];j++)cout<<"*";
cout<<'\n'; }
return 0;}
int main(int argc,char *argv[]){
hist hoge;
hoge.func(argv);
return 0;
}
746:696
08/02/25 21:28:43
すみません、あげるの忘れてました。
どなたか教えてください。
747:デフォルトの名無しさん
08/02/25 21:32:49
GNU global使えていう話じゃなくて？
748:744
08/02/25 21:37:58
>>747
そんなソフトがあったんですかｗ
ありがとうございます。
しかしそれではコーディングトレーニングにならないので、
ソースコードでよろしくお願いいたします。
どう考えてもイメージがわかないもので質問した次第です。
749:デフォルトの名無しさん
08/02/25 21:44:30
はじめまして
C++でプログラミングをしようと思っているのですが
開発環境等をどうしようか迷っています
将来的には
■ゲームを動かすプログラム(同人やエロゲのアプリ程度)
■補助ソフトやパッチ
■データを暗号化して格納するプログラム
等を作りたいと思っています
何かアドバイスがあればお願いします
750:デフォルトの名無しさん
08/02/25 21:45:47
VC++ でいいっしょ
タダだし
751:744
08/02/25 21:48:04
>>749
「C++の勉強したいんだがどの本買って勉強したらいいんですか？」
って聞いてるのと一緒だべ？
752:デフォルトの名無しさん
08/02/25 21:50:19
デザインパターン信者
私にもそんな時期がありました（マジで）
753:デフォルトの名無しさん
08/02/25 21:51:50
80:20の原則があるしな。経験つめばどこがホットスポットなのかが見当つくから、
プロファイルとってそこだけしぼって最適化すればいい。
最初から保守性犠牲にしてまでパフォーマンスチューニングが必要な分野って
限られてると思うよ。
754:デフォルトの名無しさん
08/02/25 21:55:02
>>749
今からC++勉強するよりもおまいは吉里吉里勉強したほうが幸せになれると思う
755:744
08/02/25 22:06:01
マイヘッドがクラッシュしてしまいそうなんですが、
エニバディー、ティーチ ミーしてくれませんか？
756:デフォルトの名無しさん
08/02/25 22:16:44
>>752
理解できずに挫折したか。
757:デフォルトの名無しさん
08/02/25 22:16:59
トオルさん帰れよ
758:デフォルトの名無しさん
08/02/25 22:18:58
>>744
関係はないが、とりあえずバッファを buffer_size しか確保してないのに
fgets で filesize 読もうとしてるのはヤバい。
759:デフォルトの名無しさん
08/02/25 22:28:27
>>744
else if(count > 0)
760:744
08/02/25 22:29:05
>>758
bufferはstring型でいこうとおもったんですが、
fin>>buffer;でやると、単語読み込みで一行読み込みにはならないし、
fgets(buffer.c_str(),filesize,fin);はコンパイラに怒られるし、
どうしようもないんで駆け込み寺としてここにきました。
たぶんこれも慣れた人なら簡単に直せるんだとは思いますが、
いままでC言語ばっかりやってたもんで、変な癖ついてるんです。
761:デフォルトの名無しさん
08/02/25 22:30:58
getline を知らんのか？
762:744
08/02/25 22:32:36
>>761
K＆Rにも載ってたんで知ってます。
763:デフォルトの名無しさん
08/02/25 22:32:53
>744
よくわからんが、
一行よむなら、std::.getline( fin, buffer );だが。むろん、finはifstream辺りの必要はある。
初めて"{"が存在する行をみつけたとき、上のifでcount++になって、
下のifも成立してでcount++で２になるんだが、そういうものなのかね？
764:デフォルトの名無しさん
08/02/25 22:33:42
どこのK&Rだ、どこのw
765:デフォルトの名無しさん
08/02/25 22:35:10
>>750
あと必要な物はやる気と根気ですかね?
>>751
"だべ"ってどういう意味ですか?
マルチリンガーじゃないので日本語以外わからないんです、本当にごめんなさい。
766:デフォルトの名無しさん
08/02/25 22:36:48
>>765
ｷﾓｽ
767:744
08/02/25 22:43:14
>>759
else ifですか。基本的なところ躓きました。
ありがとうございます。勉強になりました。
>>763
そんな関数があったんですか。
ちょっとgetlineの中をのぞいて使わせてもらいます。
>>764
Cアンサーブックのほうです。
バイブル本とか釣られてやってみたけど、
糞面白んないんで解くのやめました。
768:デフォルトの名無しさん
08/02/25 22:44:52
可変のchar *のバッファ用意したいとき
string buf (n, '\0' );
&buf0]; ← これはchar*のところへ入れられる
と使うと便利だ 自動で消滅するし
769:744
08/02/25 23:06:24
>>768
下のほうhtmlのタグみたいですね。
奥の手っぽいんでどうしようもないときに使ってみます。
ありがとうございました。
770:デフォルトの名無しさん
08/02/25 23:48:16
>>744
さっき、恐らくは想定したと思われる仕様を完璧に満たすソースを書いたんだが、
反応がなかったから会社に置いてきちゃった。次に会社に出るのは3日後なんで、
後で寝ちゃわなければ軽く再現してみるよ。
771:デフォルトの名無しさん
08/02/25 23:57:46
>>770
＞次に会社に出るのは3日後なんで
何でだ？
772:デフォルトの名無しさん
08/02/26 00:01:47
なんか、「それを記すにはあまりにも2chの投稿制限は厳しい」みたいに
続きそうな文章だなｗ
773:デフォルトの名無しさん
08/02/26 00:08:58
>>749
本体 > VC++
開発用ユーティリティ > TurboC++ Exploer (BCB)
でいいんでないの？
VC++はGUIアプリの構築はそれなりに手間かかるし、BCBは紙芝居の開発じゃ全然VCLの長所が生かせない。
774:デフォルトの名無しさん
08/02/26 00:16:21
>>771
単に、明日明後日は客先直行直帰だから。
>>744
手直ししながら、「クラスにする必要ないじゃん」とか
「変数は極力局在化しろよな」とか「マクロシンボルは大文字だろ」とか
「入力にfgets()を使うなら出力はfputs()じゃないか」なんて乗りで
修正してたらこうなった。
#include <cstdio> #include <cstring>
#define BUFFER_SIZE 1000
int func(char *argv[]) {
FILE * fin = fopen(argv[1],"r");
if (fin == NULL) return 1;
unsigned count = 0;
char buffer[BUFFER_SIZE];
while(fgets(buffer, sizeof(buffer), fin)!=NULL) {
if (count == 0) {
if (strchr(buffer,'{') != NULL) {
fputs(buffer, stdout);
count++;
}
} else if (count > 0) {
if (strchr(buffer,'{') != NULL) count++;
if (strchr(buffer,'}') != NULL) count--;
fputs(buffer, stdout);
}
}
fclose(fin);
return 0;
}
int main(int argc,char *argv[]) {
return func(argv);
}
775:744
08/02/26 00:36:44
>>774
まぁ人には癖があるからね。
ありがとう。
人と自分のソースを比較するのが、
一番勉強になるよ。
776:デフォルトの名無しさん
08/02/26 00:45:12
>>775
>774は常套句の宝庫だと思うよ。fgets(buffer, sizeof(buffer), fin)とか。
>744のクラスは、変数をばら撒くだけでクラスとしては何もしない無意味な設計だったし。
後はあれだな、関数に渡すのはconst char *にしてmain()で予めargcのチェックと
argv[1]の抽出位しておくのが常套手段か。
777:744
08/02/26 00:57:36
>>776
途中で配列にしたら楽だろうなぁとかは思ったが、
配列だと途中でサイズの変更が利かないから。
ポインタだと動的メモリの割り当てで適当なサイズに
割り当てできるし、と思っただけです。
別に機能さえあってればどのやりかたでもいいんだけど、
個人的には
stdoutの使い方がいまいちわかんなかったから、
そこが勉強になったのと、マクロの名前が大文字ね。
それ完全に忘れてたから勉強になった。
ありがとうってのは主にそこかなぁ・・・。
クラスは再利用に機能を発揮するから今の段階では必要としていないけど、
あとあと拡張していくにあたって必要になってくるから残しておきました。
確かにこの辺は言わんとわからんかったかもしれない。
突っ込まれたんで補足しておきました。
778:デフォルトの名無しさん
08/02/26 05:08:28
>726
ありがとう、ちょっと不安だったんだ。
名前も無いまま渡されるオブジェクトってどうなるんだろ、って。
779:デフォルトの名無しさん
08/02/26 05:56:26
ある関数に引数として構造体のポインタを渡し、そこにアドレスをセットして
返してもらうにはどう書けばよいのでしょう？
BOOL hoge( HOGE *ptr )
{
…略
ptr = address;
return TRUE;
}
としても上手く動かないようです。
780:デフォルトの名無しさん
08/02/26 06:00:17
>>779
整数を受け取るには整数のポインタを渡す必要がある。
構造体を受け取るには構造体のポインタを渡す必要がある。
構造体のポインタを受け取るには構造体のポインタのポインタを渡す必要がある。
781:デフォルトの名無しさん
08/02/26 06:03:19
あ！なるほど。こんな感じでしょうか？
BOOL hoge( HOGE **ptr )
ちなみにC++なのですが、参照渡しを使ってこれは良いのでしょうか？
BOOL hoge( HOGE *&ptr )
一応コンパイルは通るのですが…
782:デフォルトの名無しさん
08/02/26 07:25:27
OK
783:デフォルトの名無しさん
08/02/26 11:34:24
同じクラスなら、別のインスタンスのprivateなデータも参照出来ますよね。継承関係が
ある場合には、親オブジェクトから子オブジェクトを通して親オブジェクトのprivateデータを
参照できたのですが、これってこういうもの？
#include <iostream>
using namespace std;
class test2;
class test {
public:
void func(const test2 &a);
private:
int p;
};
class test2 : public test {
public:
void func2(const test2 &a);
};
void test::func(const test2 &a) {
cout << a.p << endl;
}
test2 T2;
int main()
{
test t;
t.func(T2);
return 0;
}
784:デフォルトの名無しさん
08/02/26 11:37:48
そういうもの。
785:デフォルトの名無しさん
08/02/26 11:40:09
>>783
というか出来ないとまずいだろ。
コピーコンストラクタとか代入演算子の定義が出来なくなってしまうよ。
786:デフォルトの名無しさん
08/02/26 12:06:04
protectedじゃなくてもアクセス出来ちゃうの？
787:デフォルトの名無しさん
08/02/26 12:22:23
protectedにすると、さらにtest2からも参照できるようになる
788:デフォルトの名無しさん
08/02/26 13:01:03
プログラムにZIPを解凍する機能を盛り込みたいのですが、そのような
機能のスタティックリンクできるライブラリをご存じないでしょうか？
環境はVisualC++.net2003です。
789:デフォルトの名無しさん
08/02/26 13:07:03
>>788
zlib
790:デフォルトの名無しさん
08/02/26 13:30:25
>>783じゃないが試してみたけど
同じクラスのインスタンスへの参照やポインタを持つとアクセス権限ブッチぎれちゃうのか
なんかカプセル化とかそういう面で考えると気持ち悪いというかなんというか
仕様上出来ないと仕方ないと言うのも分かるのだが・・・
791:デフォルトの名無しさん
08/02/26 13:33:53
789㌧
792:デフォルトの名無しさん
08/02/26 13:34:46
C++のprivateはクラスプライベートであってインスタンスプライベートの意味ではない
793:デフォルトの名無しさん
08/02/26 14:02:36
>>782
すみません、寝てました。返答有難うございます。
ちなみに「ポインタ変数を参照で渡す」と言う意味ならむしろ
BOOL hoge( HOGE &*ptr )
のような気がするのですが、なんで*&ptrなのでしょう？
それが仕様と言われればそこまでですが。
794:デフォルトの名無しさん
08/02/26 14:24:18
>>793
URLﾘﾝｸ(kmaebashi.com)
これの応用。
ptrのほうからreference to pointer to HOGEと読める。
795:デフォルトの名無しさん
08/02/26 14:26:03
>>793
typedef HOGE * HOGE_PTRしてみれば判る。
796:デフォルトの名無しさん
08/02/26 14:37:54
>>794>>795
なるほど。正直きちんと理解できたかは自信ないですが、ちゃんと法則が
あってそうなっているのは分かりました。
どうも有難うございましたｍ（＿＿）ｍ
797:デフォルトの名無しさん
08/02/26 14:46:05
vc2008のコマンドラインコンパイラだけ手に入りますか
vc6でコンパイルしたいです
sp1で鈍いパソコンです
798:デフォルトの名無しさん
08/02/26 14:48:16
EEを入れてみたか？
799:デフォルトの名無しさん
08/02/26 14:49:23
>>797
コンパイラだけ手に入れるのは無理だが
コマンドラインでの利用はできるよ
800:デフォルトの名無しさん
08/02/26 14:56:40
sp1にインストールできますか
801:デフォルトの名無しさん
08/02/26 15:02:40
>>792
kwsk
according to standard
802:デフォルトの名無しさん
08/02/26 15:49:24
すみません
DOSの実行ファイルで、標準出力へは書き出されないメッセージを取得する方法ありませんか?
hoge.exe > log.txt
として取得できないやつです 画面には出ます
標準出力、標準エラーの取り方はわかります
803:デフォルトの名無しさん
08/02/26 16:09:38
>>802
それのどこがC/C++に関係があるの?
804:デフォルトの名無しさん
08/02/26 16:18:38
>>802
つ printscreen
805:デフォルトの名無しさん
08/02/26 16:34:10
>>803
標準出力はこれで受け取れますが、exeが返す実行結果がとれません
#include <windows.h>
main(){
HANDLE hFile = CreateFile("log.txt",GENERIC_WRITE,FILE_SHARE_WRITE, NULL,CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL );
SetStdHandle(STD_OUTPUT_HANDLE , hFile);
STARTUPINFO si; PROCESS_INFORMATION pi;
ZeroMemory(&si, sizeof(si));ZeroMemory(&pi, sizeof(pi));
si.cb = sizeof(si);
si.hStdOutput =hFile ;
si.hStdError =hFile ;
si.wShowWindow = SW_HIDE;
CreateProcess(NULL, "HtoX32c.exe e:\\0.htm ", NULL, NULL, FALSE,0, NULL, NULL, &si, &pi);
WaitForSingleObject(pi.hProcess, INFINITE);
CloseHandle(pi.hThread);
CloseHandle(pi.hProcess);
CloseHandle(hFile);
}
806:デフォルトの名無しさん
08/02/26 16:51:04
HtoX32とやらが標準エラー出力に出している可能性は無い?
そもそも、まさかHtoX32はWindowを持っているアプリケーションなのか?
だとしたら、メッセージフックなどの手段が必要になるよ。
つーか、HtoX32とやらが何ものかにも依るけど同等のプログラムを自分で書いた方が手っ取り早かったりしてね。
807:デフォルトの名無しさん
08/02/26 16:59:39
HTMLをテキストに変換するプログラムです
ソースが公開されていたり定番のライブラリとかありませんか?
808:デフォルトの名無しさん
08/02/26 17:03:03
正規表現での消し方のサンプルみながら自分でやってみます
809:デフォルトの名無しさん
08/02/26 17:11:51
>>805
URLﾘﾝｸ(support.microsoft.com)
810:デフォルトの名無しさん
08/02/26 17:15:43
popenを使えたら楽だな
あればだけど
811:デフォルトの名無しさん
08/02/26 17:44:11
>>805
そのコードだと、自プロセスの標準出力が hFile になるけど、子プロセ
スの HtoX32c.exe の標準出力はコンソールのままでは？
812:デフォルトの名無しさん
08/02/26 17:44:15
1つの関数に複数の構造体を渡すことって可能でしょうか？
suruct typedef{
int x,y;
}SAMPLE;
SAMPLE sample;
suruct typedef{
int x,y;
}SAMPLE2;
SAMPLE2 sample2;
void AA(??? *s){
int a = s->x;
int a = s->y;
}
void main(){
AA(&sample);
AA(&sample2);
}
こんな処理をしたいのですが、???の所がわかりません。
良い方法を教えていただきたいです。
仮の構造体を１つ作って、その変数に代入→渡すってやり方だとできましたが
激しく面倒くさい。
813:デフォルトの名無しさん
08/02/26 17:52:41
関数オーバーロード、もしくはテンプレート関数でぐぐれ。
814:デフォルトの名無しさん
08/02/26 17:54:43
>>812
typedef SAMPLE SAMPLE2;
815:デフォルトの名無しさん
08/02/26 18:02:57
>>813
>>814
情報ありがとうございます。
テンプレート関数が一番やりたかった事を解決してくれそうでした。
ありがとうございました。
816:デフォルトの名無しさん
08/02/26 18:10:58
構造体のメンバを決め打ちするようなテンプレート関数は感心しないな。
テンプレートを使うということはC++で良いので、それならメンバ関数にしてしまった方が良い。
817:デフォルトの名無しさん
08/02/26 18:22:32
ifstream::readって実際に何バイト読めたか検出できないんですか？
freadみたいにふつうに戻り値が来るのかと思いきや違うみたいですし…。
818:デフォルトの名無しさん
08/02/26 18:28:13
つ gcount
819:デフォルトの名無しさん
08/02/26 18:29:13
>>817
自己レス。failで少なくとも期待通りの
バイト数読めたかどうかは取得できるのね。
820:デフォルトの名無しさん
08/02/26 18:30:30
>>818 と思ったらそんなメソッドがあったか。ｻﾝｸｽ
821:デフォルトの名無しさん
08/02/26 18:31:39
fstreamは重要なところでは使うべきではない
APIのほうがよい
822:デフォルトの名無しさん
08/02/26 18:55:55
んなこたーない
823:デフォルトの名無しさん
08/02/26 19:18:08
fstreamは一社の開発ではないがAPIはMS一社
どのコンパイラでも同じ動作になる
あと3000個とか開けない
824:デフォルトの名無しさん
08/02/26 19:18:50
if文って 文字列と文字列を比べることって出来ないですよね？
1 = あいうえお
2 = かきくけこ
3 = さしすせそ
4 = あいうえお
if (a = 1){
}
みたいなことをやりたいのですがどうしたら良いでしょうか？
825:デフォルトの名無しさん
08/02/26 19:19:44
stringはできる
826:デフォルトの名無しさん
08/02/26 19:20:33
> どのコンパイラでも同じ動作になる
コンパイラで差はないかもしれないが・・・
827:デフォルトの名無しさん
08/02/26 19:21:41
>>823
んなばかな
APIがMS一社で規定されてるなんて初めて知った
MSはついに世界を征服したのか
828:デフォルトの名無しさん
08/02/26 19:23:04
Windowsのファイル入出力APIのことを指す
829:デフォルトの名無しさん
08/02/26 19:55:08
>>824 strcmpしとけ
830:デフォルトの名無しさん
08/02/26 21:19:23
CreateProcessしたときに止まらなくなったら停止させたいんですけど
どうやったらいいですか?
831:デフォルトの名無しさん
08/02/26 21:22:25
ググって解決しました
832:デフォルトの名無しさん
08/02/26 21:58:13
マルチスレッドで動かしてる関数って_endthread()書かなくてもreturn;書いてたら
問題ない？
833:デフォルトの名無しさん
08/02/26 22:04:41
問題ない。
ところで_beginthreadは使うなよ。_beginthreadexにしろよ。
URLﾘﾝｸ(msdn2.microsoft.com)(VS.80).aspx
834:デフォルトの名無しさん
08/02/26 22:15:49
VC6.0です。
__declspec(dllimport) int __stdcall hoge( LPWORD, LPWORD, LPWORD );
という形の関数のアドレスが入る関数ポインタhageを作りたいのですが、
うまく宣言できません。
__declspec(dllimport) int __stdcall (*hage)(LPWORD, LPWORD, LPWORD );
などとやるとコンパイルエラーになってしまいます。
どうすれば宣言できるのでしょうか？
835:デフォルトの名無しさん
08/02/26 22:17:26
declspecは関係ないだろ
836:デフォルトの名無しさん
08/02/26 22:19:03
関係あるよ
837:デフォルトの名無しさん
08/02/26 22:19:32
関数ポインタで悩んだら、typedef
typedef __declspec(dllimport) int __stdcall hogetype( LPWORD, LPWORD, LPWORD );
hogetype hoge;
838:デフォルトの名無しさん
08/02/26 22:36:15
関数ポインタの便利さになれすぎてしまって
最近 if を見るたびにすぐ関数テーブル化してしまう。
間違い？
839:デフォルトの名無しさん
08/02/26 22:39:07
なるべくシンプルにかくのを推奨されますからね
840:デフォルトの名無しさん
08/02/26 22:43:09
関数テーブル間違いです
鈍いです
841:デフォルトの名無しさん
08/02/26 22:47:58
なんかCreateProcessすると不安定になる スレッドでsystemで実行してみる
CreateProcessを短期間に20回以上使うと動かなくなるエラーってありますか?
実行する側のexeのせいかもしれないですが
842:デフォルトの名無しさん
08/02/26 22:50:02
テーブルだめなのか・・
if の中にさらに if があるともうその地点でうわ・・って思ってしまう。
ちょっとソース見直してみます
843:デフォルトの名無しさん
08/02/26 22:50:41
>>833
ありがとう。今度からそうする
844:デフォルトの名無しさん
08/02/26 22:54:19
そこでBoost::functionとコールバックですよ。
845:デフォルトの名無しさん
08/02/26 22:59:28
>>842
無理して避けることもないよ。何事も程々が1番。
846:デフォルトの名無しさん
08/02/26 23:02:18
グローバル変数って絶対使わない方がいいんですかねえ
例えばゲームプログラムを作っていて、状態を管理するグローバル変数を使っているとする。
その変数には、ゲームの進行に合わせて、例えば
定数（TITLE=0, STAGE1, STAGE2, STAGE3, STAGE_CLEAR, GAME_OVER, ENDING)が入るとする。
処理は上の状態それぞれでモジュール化されているとすると、
どうしても状態管理変数はグローバル変数化してどの状態からでも
参照・設定できるようにせざるを得ないと思うのですが、何かいい手があるのでしょうか。
状態管理変数をファイル内static変数にしてセッターを使って書き換えるようにしたり、
状態管理変数とセッターをクラスにしてそのオブジェクトのスコープをグローバルにするもしくは状態管理変数とセッターを
staticにしてどのクラスからでも書き換えられるようにするくらいなら最初から状態管理変数をグローバルにしといた方が
シンプルだと思うのですがどうでしょうか。
847:デフォルトの名無しさん
08/02/26 23:03:57
数が多くなければOK、重要ならOK
848:デフォルトの名無しさん
08/02/26 23:05:38
>>846
静的なシングルトンもグローバル変数も似たようなもんだ
グローバル変数でいいんじゃね？
ただ、関数経由にしておけばデバッグするときに setter で引っ掛ければいいだけだから
ずいぶんと楽になる＊かもしれない＊
849:デフォルトの名無しさん
08/02/26 23:30:59
>>846
シングルタスクなら良いけど。
GUIでスレッド使って随時処理しながら入力待ちして、
入力はコールバックで処理、みたいのだと、
ゲッタやセッタ作った方が排他処理し易いかなぁ。
まぁケースバイケース？
850:デフォルトの名無しさん
08/02/26 23:32:07
ゲーム状態のインスタンスが1つだけなら、グローバル変数でいいと思う。
インスタンスが複数なら、
状態ごとの処理に、状態管理変数をクラス化したものを渡すとか、
もしくはstateパターンなんてのもいいかもしれない。
851:デフォルトの名無しさん
08/02/26 23:33:35
どうせグローバル変数的に使うとはいえ、
それゆえにどこから変更されてるかを追跡するためにも
>関数経由にしておけばデバッグするときに setter で引っ掛ければいいだけだから
というのは地味に大事
852:デフォルトの名無しさん
08/02/26 23:38:31
ゲーム状態も例えば、プレイヤキャラの他にAIキャラが裏で進行中なんてことになると、
グローバルだと破綻するよね。そんな場合はゲーム状態はキャラクタに依存する情報になるわけだけど。
853:デフォルトの名無しさん
08/02/26 23:46:14
シングルトンとグローバル変数は違う。
グローバル変数として使ってる間違ってる例が多いのが勘違いの原因かもしれない。
自分だったら>>846の場合、シーンマネージャクラスを作ってそいつに振る舞いを管理させる。
極力グローバル変数は使わない。
854:デフォルトの名無しさん
08/02/27 00:01:10
シングルトンは初期化のタイミングを選べるからグローバルとは大きく違う。
855:デフォルトの名無しさん
08/02/27 00:20:00
結局オブジェクト指向の導入でＣにおけるグローバル変数使用の問題は解決できてないんじゃねえの
Ｃを極めればオブジェクト指向言語を使わなくても良いコードが書けるんじゃねえの結局クラスとか
オブジェクトとかデザインパターンとか複雑で分かりにくくするなだけじゃね
856:デフォルトの名無しさん
08/02/27 00:21:36
system は空白が改行のように扱われてしまいます
パラメータは渡せませんか?
857:デフォルトの名無しさん
08/02/27 00:22:49
>>855
んなこたーない
858:デフォルトの名無しさん
08/02/27 00:26:23
ファイルの読み込みについての質問です。
----
2008/02/20，01：00：00
2008/02/21，02：00：00
2008/02/26，03：00：00
…
----
と書かれたファイルを、
tmp[0] = 2008;
tmp[1] = 2;
tmp[2] = 20;
tmp[3] = 01;
といったように取り込みたいです。
スラッシュとカンマとコロンを区切りにしてうまく取り込む方法を教えてください。
もし、スラッシュしかなければstrtokとatoiでできたのですが、3種類あるので困っています。
よろしくお願いします。
次ページ最新レス表示スレッドの検索類似スレ一覧話題のニュースおまかせリスト▼オプションを表示暇つぶし2ch
4987日前に更新/243 KB
担当:undef