【初心者歓迎】C/C++� ..
303:デフォルトの名無しさん
08/02/23 15:06:12
>>300
論理的に記述しようぜ。
「while を終了する」 と 「指定位置に移動する」 では
前者の方がより論理的だ。
論理的なコードは、一般に変更に強く、バグが出にくい。
まあ、break すらみだりに使うべきじゃないという人もいるけどね。
304:デフォルトの名無しさん
08/02/23 15:07:50
関数の引数で、char * 型とchar []型を判別する方法ありますか?
305:デフォルトの名無しさん
08/02/23 15:09:39
>>304
不可能
306:デフォルトの名無しさん
08/02/23 15:10:05
void foo(char* const& hoge);
template <size_t N> void foo(char (&hoge)[N]);
と区別できなくはない。
307:デフォルトの名無しさん
08/02/23 15:12:02
char[N]とchar[]は別物だと思うよ。
308:デフォルトの名無しさん
08/02/23 15:12:33
void foo(char* hoge, size_t size);
template <size_t N> inline void foo(char (&hoge)[N]) { foo(hoge, N); }
もちろん、こう実装するんだぜ。
309:デフォルトの名無しさん
08/02/23 15:13:12
そう言う意味での char [] 型なんてそもそも存在しないが。
310:デフォルトの名無しさん
08/02/23 15:13:14
>>285
＞cuで２が、gguで３が、ijcoで4が返ってきます
その書き方はおかしい
strch_idxには引数が2つあるので、cuとかgguだけでは結果は決まらないはずだ
311:デフォルトの名無しさん
08/02/23 15:13:24
>>300
逆。
gotoの方がどこにでも飛ばせるせいで行き先が不明確になる。
自分の書いたコードを他人に読ませることを想像してごらん。
breakならどこに飛ぶのか一目瞭然だけど、gotoだといちいちラベルを検索しないといけない。
しかもbreakなら「ループの終了」という意図が一目瞭然だけど、
gotoだとどういう意図で飛ばしたのかを考えないといけない。
そういう理由でgotoは避けられるため、安易にgotoが入っているとさらに、
「gotoを使わなければいけないどんな理由があったのか？」と考えさせることになる。
312:デフォルトの名無しさん
08/02/23 15:14:13
int⇔floatの変換だけでも、毎回static_cast使う？
あと、コンテナの最後の要素以外を処理する時、
std::vector<T> container;
for (int n = 0; n < static_cast<int>(container.size()) - 1; ++n) container[n] = ...;
ってやる？
313:デフォルトの名無しさん
08/02/23 15:23:01
>>312
俺はstatic_castを使う
コンテナの方は
for (size_t n = 0; ～
って書けばいいのでは
本当は std::vector<T>::size_type の方が適切なのかもしれんが
314:デフォルトの名無しさん
08/02/23 15:23:54
>>312
＞int⇔floatの変換
うん。
＞最後の要素以外を処理
unsigned使う。
315:デフォルトの名無しさん
08/02/23 15:31:10
＞int⇔floatの変換だけでも、毎回static_cast使う？
使う。
＞コンテナの最後の・・・
for (std::vector<T>::size_type n = 0, size = container.size(); n + 1 < size; ++n) container[n] = ...;
ってやると思う。
符号無しの値を符号付きにキャストするのは基本的に危険だと思う。
316:デフォルトの名無しさん
08/02/23 15:32:52
後者の場合、
if( !c.empty() )
for( size_t n=0; n<c.size()-1; ++n)
で何か問題あるのか？
317:デフォルトの名無しさん
08/02/23 15:34:18
>>316
n + 1 < c.size() で判定すれば empty チェックは要らない。
318:デフォルトの名無しさん
08/02/23 15:36:12
typename 忘れてた。
まあ T をそういう意味で使ってるとは限らないが。
319:デフォルトの名無しさん
08/02/23 15:39:01
>>302 >>303
>>311での説明とほぼ同じという理解でよろしいでしょうか
>>311
gotoよりbreakを使うべきというのは、よく理解でき、納得できました
しかし、怖いというのは一体
>>310
すみませんが、もうすこし噛み砕いて説明願えないでしょうか
const char* sには、任意の文字列が入って、char cで、検索する文字を固定という事ではないんですか？
そのconst char* sにcuですとか、gguですとかが入っているので、これで十分だと考えているのですが
320:デフォルトの名無しさん
08/02/23 15:43:08
>>319
goto はまず特定の状況でしか使われない。
それ以外で使われると混乱するし、
何のために使われているかすぐに分からないので不気味で怖い。
321:デフォルトの名無しさん
08/02/23 15:44:11
>>319
goto は意図が不明確。
goto のあるコードを変更することになった場合、
その意図が 100% 分からなければ、
手を加えて変なことになりかねないので怖い。
322:デフォルトの名無しさん
08/02/23 15:44:56
>>319
一番怖いのは、バグの温床になること。
コードの分かりにくさってのはバグに直結する。
323:デフォルトの名無しさん
08/02/23 15:47:14
>>315
イテレータ使うならこうだな。
前進イテレータでさえあれば、これでいけるはず。
if(! container.empty()) {
Iter it_next = container.begin();
++it_next;
for(Iter it = container.begin(), end = container.end(); it_next != end; ++it, ++it_next) *it = ...;
}
324:デフォルトの名無しさん
08/02/23 15:52:18
>>319
const char* s が cu でも c の方は
strch_idx("cu", 'c') とか
strch_idx("cu", 'u') とか
strch_idx("cu", 'x') とか色々指定できるわけで
325:デフォルトの名無しさん
08/02/23 15:53:30
>>320-322
なるほど、よく理解できました
ご親切にありがとうございます
１、意図が不明で不気味で怖い
２、バグを招きやすいから怖い
この二点ですね
そして、本筋のほうもできたら説明していただきたいのですが
326:デフォルトの名無しさん
08/02/23 15:53:54
STLつかうとコンパイルに時間かかりますけど、クラスも多少鈍くなるんですか?
327:デフォルトの名無しさん
08/02/23 15:58:00
そもそも while じゃなくて for 使おうぜ。
328:デフォルトの名無しさん
08/02/23 15:58:55
>>326
一般に、コンパイルに時間がかかる方が、
コンパイル時にある程度処理を行ってくれているため、速くなる。
ただ、コードがあまりにも肥大してくると、
キャッシュの効きが悪くなって遅くなる事もある。
329:デフォルトの名無しさん
08/02/23 16:02:03
>>325
いくつか修正したみたいだけど今はどうなってるの？
330:デフォルトの名無しさん
08/02/23 16:05:44
>>324 >>329
最初から全文現すべきでしたね、申し訳ありません
#include <iostream>
using namespace std;
int strch_idx(const char* s, char c)
{
int i = 0;
do {
if(s[i] == c) goto end;
else if(s[i] == 0)
{
i = -1;
break;
}
i++;
} while(s[i]);
return i;
}
int main()
{
cout << "文字列を入力："; char p[] = ""; cin >> p;
char c = c; cout << strch_idx(p, c);
return 0;
}
>>327
forだと、終了させるための条件が思いつかなかったので、whileにしましたが、やはりforのほうが良い理由があるのでしょうか
331:デフォルトの名無しさん
08/02/23 16:07:30
>>330
＞char c = c;
これはなによ
332:デフォルトの名無しさん
08/02/23 16:08:43
>>330
continue した時でもインデックスが増えてくれるのと、
インデックスをなめていく操作を行っているという意図が伝わりやすいのと。
終了条件は while の時と同じでいいじゃん？
333:デフォルトの名無しさん
08/02/23 16:16:31
>>330
goto end ってどこに飛んでるの？
334:デフォルトの名無しさん
08/02/23 16:17:17
>>330
goto endが一個消し忘れ。
×char c = c;
○char c = 'c';
-1を返したいのは文がヌル文字だけだった場合？それとも文字が見つからなかった場合？
前者ならループのたび判定するのは無駄だから最初に一回だけ判定させたらいい。
後者ならヌルが見つかった時点でループが終わってるから中のlese-if節は判定してくれない。
最初に文字列の長さを測って最後までループが回った場合に-1を返すようにしないと。
335:デフォルトの名無しさん
08/02/23 16:26:11
f(x)のあとでstrlen(ch)が1になることがあります もとのソースを簡略にしました
構造体を参照渡ししてもだめでした 下は期待通りの動作をします 何がいけないのかわかりません
#include <string.h>
#include <stdio.h>
typedef struct STRDATA{ char *start; char *end;}strdata;
f(strdata x){
delete x.start;
x.start = new char [20];
strcpy(x.start,"++++++++++++++"); }
main(){
char *ch=new char [10]; strcpy(ch,"uuuu");
strdata x;
x.start=ch; x.end=ch+strlen(ch);
f(x);
printf("%s",ch);}
336:デフォルトの名無しさん
08/02/23 16:26:35
なんで do-while になってるんだろう。
もっと単純に考えようぜ。
まず、文字を１つずつ取得していくループを書く。
for(i = 0; /* 文字列が終了するまで */; i++) {
/* s[i] で文字を先頭から順番に走査していける */
}
んで次に、c が見つかったらインデックスを返すようにする。
for(i = 0; /* 文字列が終了するまで */; i++) {
/* s[i] が c なら i を返す */
}
そして、検索しても c が見つからなかった場合は -1 を返す。
for(i = 0; /* 文字列が終了するまで */; i++) {
/* s[i] が c なら i を返す */
}
/* -1 を返す */
これでおk。
337:デフォルトの名無しさん
08/02/23 16:29:05
>>304
Cならシーキビだろ
C++か？
template<typename T>
struct IsArray {
enum { value = 0 };
};
template<typename T, size_t N>
struct IsArray<T[N]> {
enum { value = 1 };
};
template<typename T>
bool test(const T&) {
return IsArray<T>::value;
}
template<typename T>
void f(const T& t) {
if(test(t)) cout << "配列だ" << endl;
else cout << "配列じゃない" << endl;
}
338:デフォルトの名無しさん
08/02/23 16:29:46
てか、C++ならブースト使えよか
339:デフォルトの名無しさん
08/02/23 16:30:19
>>335
f の呼び出し後、ch は既に delete された後のアドレスを保持することになるから
絶対にアクセスしちゃダメ。
そもそも生のポインタで確保されたメモリを扱ってるからそういう事が起きる。
コンテナ使え。
340:デフォルトの名無しさん
08/02/23 16:30:21
これだとエラーで止まります どうすればいいですか
f(strdata *x){
delete x->start;
x->start = new char [20];
strcpy(x->start,"++++++++++++++"); }
main(){
char *ch=NULL;
strdata x;
x.start=ch; x.end=ch+strlen(ch);
f(&x);
printf("%s",ch);}
341:デフォルトの名無しさん
08/02/23 16:31:18
>>335
chに割り当てたnew char[10]はfの中でdeleteされてるから、
printf("%s",ch) は違法。
それからnew[]した配列はdeleteではなくdelete[]しないといけない。
342:デフォルトの名無しさん
08/02/23 16:34:08
>>340
char *ch=NULL;
x.end=ch+strlen(ch);
ヌルポインタに整数を足しても有効な値にはならない。
343:デフォルトの名無しさん
08/02/23 16:35:22
バイナリ文字列の始めと終わりを構造体で渡して、内容、サイズを書き換えるには
>>335をどう変更すればできますか?
344:デフォルトの名無しさん
08/02/23 16:37:35
普通に先頭のポインタとstrlen使って適当に操作したらいいんちゃう？
345:デフォルトの名無しさん
08/02/23 16:38:25
バイナリ・・・文字列？？
それはともかく、std::string 使え。
#include <string>
void f(string& str) {
str = "++++++++++++++";
}
int main() {
std::string str("++++");
printf("%s\n", str.c_str());
f(str);
printf("%s\n", str.c_str());
}
346:デフォルトの名無しさん
08/02/23 16:38:32
このふたつは動きますが内容が変化しません なぜですか
f(strdata x){
delete x.start; x.start = new char [20];
strcpy(x.start,"++++"); }
main(){
char *ch=NULL;
strdata x;
x.start=ch;
f(x);
printf("%s",ch);}
f(strdata *x){
delete x->start; x->start = new char [20];
strcpy(x->start,"++++"); }
main(){
char *ch=NULL;
strdata x;
x.start=ch;
f(&x);
printf("%s",ch);}
347:デフォルトの名無しさん
08/02/23 16:38:49
１つ目、std:: 忘れた。
348:デフォルトの名無しさん
08/02/23 16:41:16
Ｃ言語だけで、0を含む文字列を変化させたいのですが、できないですか?
349:デフォルトの名無しさん
08/02/23 16:42:49
>>346
ポインタが参照渡しでどうのこうの言ってた奴だよな？
頼むからポインタとは何かを一から勉強し直してくれ。
350:デフォルトの名無しさん
08/02/23 16:44:23
>>346
関数に渡された数値は数値がコピーされただけの別物。
だから関数内でx.start = new char [20];とかやっても呼び出し元のxには変化は無い。
351:デフォルトの名無しさん
08/02/23 16:44:32
f(x)で、(バイナリ)文字列を書き換えられるやり方教えてください それみて勉強します
352:デフォルトの名無しさん
08/02/23 16:45:30
>>351
まともな本読め。
そこにいくらでもサンプルは書いてあるし、
詳細な解説も載ってる。
353:デフォルトの名無しさん
08/02/23 16:46:24
>>350
後者はアドレス渡しですけど・・・
354:デフォルトの名無しさん
08/02/23 16:47:42
>>353
後者では x は書き換えられるが ch は書き換えられない。
355:デフォルトの名無しさん
08/02/23 16:47:47
>>353
アドレスの指す先の内容を書き換えると呼び出し元へ反映されるというだけのこと。
356:デフォルトの名無しさん
08/02/23 16:50:34
x->start = ch
ってやっても、ch と x->start がリンクされるわけじゃない
x->start の内容が変化しても、chには関係ない
357:デフォルトの名無しさん
08/02/23 16:52:12
これで正解でしょうか?
#include <string.h>
#include <stdio.h>
typedef struct STRDATA{ char **start; char **end;}strdata;
f(strdata x){
delete *(x.start); *(x.start) = new char [20];
strcpy(*(x.start),"++++++++++++++"); }
main(){
char *ch=NULL;
strdata x;
x.start=&ch;
f(x);
printf("%s",ch);}
358:デフォルトの名無しさん
08/02/23 16:52:50
図を描こうぜ。図を。
ch, x, そして動的に確保されたメモリが
実際にメモリ上でどう置かれていてどう参照していて
何を実行するとどう変化するか。
359:デフォルトの名無しさん
08/02/23 16:54:53
>>357
それでとりあえずまともに動くね。
推奨されるコードかと言うとまたそれは別だが。
new 使ってるから C++ なんでしょ？ コンテナ使えば楽だぜ。
360:デフォルトの名無しさん
08/02/23 16:58:05
横から質問で申し訳ないんだけど、
・>357のfに渡す前に、xはいつnewされてるの？
・関数の一行目がdeleteって、ものすごく気持ち悪いんだけど、よくつかう手法なの？
361:デフォルトの名無しさん
08/02/23 17:00:09
>>360
deleteにNULLを渡しても何も起こらないことになっているから、
最初にnewされていないというのは大丈夫。エラーにはならない。
最初にdeleteというのは必要に応じて使えばいいと思うけど、
俺も書いた覚えない（clear()とかいかにもそれだけを行う関数というのでもない限り）。
362:デフォルトの名無しさん
08/02/23 17:03:44
ｻﾝｸｽ たびたびどうもありがとうございます
363:デフォルトの名無しさん
08/02/23 17:04:30
俺も横レスだけど、配列なのに delete [] にしてないのは問題ないのか？
364:デフォルトの名無しさん
08/02/23 17:04:41
>>363
大問題。
365:デフォルトの名無しさん
08/02/23 17:05:56
だからコンテナを使おうぜと言ってるのに。
366:デフォルトの名無しさん
08/02/23 17:06:16
いろいろな意味で気持ち悪い。というか何をしたいのかよくわからない。
367:デフォルトの名無しさん
08/02/23 17:07:46
>361
>deleteにNULLを渡しても何も起こらないことになっているから、
>最初にnewされていないというのは大丈夫。エラーにはならない。
なるほど、どうもです。
>最初にdeleteというのは必要に応じて使えばいいと思うけど、
>俺も書いた覚えない（clear()とかいかにもそれだけを行う関数というのでもない限り）。
なるほど。
よく考えたらC++になってからnewなんてほとんど使ったこと無かった気がします。
動的な配列が必要になったらだいたいvectorにつっこんでた。
368:デフォルトの名無しさん
08/02/23 17:10:42
clear はメモリが解放される訳じゃないんだよな。
std::vector<int>().swap(v); みたいにしないとメモリは解放できない。
解放した方がいいかどうかは状況次第だが。
369:デフォルトの名無しさん
08/02/23 17:32:26
>>364
大問題ってことで軽くググってみたら、
・配列に対して、delete [] としなかった場合、１個目の要素のみデストラクタが走り、残りの要素は走らない
・確保された領域（配列）は一応開放される（デストラクタで開放されるべき領域は除く）が、
要素数保持のための隠れた確保領域は開放されずに残る
ってな、感じかな
370:デフォルトの名無しさん
08/02/23 17:37:02
勘違いしてないか?タスクマネージャで確認してみ
#include <stdio.h>
main(){
char *ch=new char[200*1024*1024];
getchar();
printf("delete 実行\n");
delete ch;
getchar();}
371:デフォルトの名無しさん
08/02/23 17:40:15
これでも解放するし
#include <stdio.h>
main(){
int n;
char **ch=new char*[200];
for(n=0;n<200;n++)ch[n]=new char [1024*1024];
getchar();
printf("delete 実行\n");
for(n=0;n<200;n++)delete ch[n];
getchar();}
372:デフォルトの名無しさん
08/02/23 17:42:39
>>331
strch_idx関数に渡すためのcを、main関数のほうで作っておかないとだめかなと考えまして
>>332
あまり理解できませんが、なるべくforを使うようにします
>>333
消し忘れです
ご指摘ありがとうございます
>>334
理屈を丁寧に解説していただきありがとうございました
>>336を参考にやってみて、できました
>>336
ご丁寧にありがとうございました
その指針でできました
最後にご丁寧に、教えるのもわずらわしいような初歩的な愚問に答えていただき、ありがとうございました
373:デフォルトの名無しさん
08/02/23 17:44:35
これが初心者歓迎スレの良心。
374:デフォルトの名無しさん
08/02/23 17:47:40
>>370
>>371
特に勘違いしてるところはないと思うが・・・
要素数を保持する領域はないってことを言いたいのか？
組み込み型にはないけど、クラスにはあるみたいなことが書いてあったんだが
375:デフォルトの名無しさん
08/02/23 17:49:37
これだと解放しないけど・・・ *xと定義されているなら、deleteを使うのでは? []は**xを解放する場合でしょ
#include <stdio.h>
main(){
char **ch=new char*[200];
for(int n=0;n<200;n++)ch[n]=new char [1024*1024];
getchar(); printf("delete 実行\n");
delete[] ch; // delete ch;
getchar();}
376:デフォルトの名無しさん
08/02/23 17:50:41
>>369
そもそもnew[]したものをdeleteするとどうなるかは未定義。
そういう挙動になったという話は、たまたまその実装ではそうだったということでしかない。
377:231
08/02/23 17:51:43
今更ですが>>231のようなことをした時に
MyArrayの方はデフォルトコンストラクタか引数を省略できるコンストラクタを
呼んでいるようなのですが、これのタイミングが分かりません。
コンストラクタを通過するそぶりもないし、一応初期化はされてるっぽいし・・・
特にデストラクタのタイミングも分からないのが心配です。
いつ開放されるんでしょうか?
378:デフォルトの名無しさん
08/02/23 17:54:33
>>377
どこかでnew[]やdelete[]しているだろ？そのときだ。
379:デフォルトの名無しさん
08/02/23 18:04:57
>>375
newで割り当てたものはdelete、new[]で割り当てたものはdelete[]で解放します。参照の深さは関係ありません。
380:デフォルトの名無しさん
08/02/23 18:12:04
終端をセットしたいのですがどうやったらいいですか? コンパイルが通りません
main(){
char *ch=new char [50];
char **start, **end;
start=&ch; //これは成功します
end=&&ch[20];
// end=&(ch+20);
}
381:デフォルトの名無しさん
08/02/23 18:24:57
ch と &ch[0]は同じアドレスを表しますよね 20個目のアドレスは、&ch[20]ですよね
それを参照渡ししようとしたら&&ch[20]のはず・・・
382:デフォルトの名無しさん
08/02/23 18:26:38
char * chend = &ch[20];
end = &chend;
383:デフォルトの名無しさん
08/02/23 18:33:33
>>382
コンパイルはできましたが、startと動作が違うようです
main(){
char *ch=new char [50]; strcpy(ch,"abcdef");
char **start, **end;
start=&ch;
char * chend = &ch[20];
end = &chend;
printf("%c\n", ((*start+1)[2]) );
printf("%c\n", ((*end-5)[2]) );
}
384:デフォルトの名無しさん
08/02/23 18:34:48
間違えました 20個を定義していませんでした
これだとうまくいきました ｻﾝｸｽ
char * chend = &ch[6];
end = &chend;
printf("%c\n", ((*start+1)[2]) );
printf("%c\n", ((*end-5)[2]) );
385:デフォルトの名無しさん
08/02/23 18:38:15
なぜ「end = &&ch[20];」というか「end = &(&ch[20]);」ができないかは理解しておいてね。
386:デフォルトの名無しさん
08/02/23 18:41:41
再使用できるように動的確保したら複雑になってきました・・・
main(){
char *ch=new char [50]; strcpy(ch,"abcdef");
char **start, **end;
start=&ch;
char **chend =new char *;
*chend=&ch[6];
end = &(*chend);
printf("%c\n", ((*end-5)[2]) );
}
387:デフォルトの名無しさん
08/02/23 18:43:04
先ずお前はnewを使うのをやめろ。話はそれからだ。
388:デフォルトの名無しさん
08/02/23 19:53:27
これはひどい
389:デフォルトの名無しさん
08/02/23 20:04:00
何がしたのか全然わからない
390:デフォルトの名無しさん
08/02/23 20:04:21
何度基礎をやり直せと言ったことか
391:デフォルトの名無しさん
08/02/23 20:26:55
なぜ、再使用しようとすると動的確保することになるのかもわからない。
文章で理由を説明してほしい。
392:デフォルトの名無しさん
08/02/23 20:36:55
なるほど、stringが出来るわけだ
393:デフォルトの名無しさん
08/02/23 20:53:56
おまえらほんと我慢強いよな。感心するよ
俺は>>386と同レベルのコードを保守する羽目になって殺意を覚えた。
394:デフォルトの名無しさん
08/02/23 21:00:35
我慢強いというか、読んでない
395:デフォルトの名無しさん
08/02/23 21:02:02
C++の授業で先生が、
int main() {
int i;
cin>>i;
double a[i];
...
というコードはC++では出来ない(やりたいならnewでやるべき)と言われたのですが、
g++とiccではできました。これってだめだけど、gccやiccの拡張機能によって
できているのでしょうか？
396:デフォルトの名無しさん
08/02/23 21:04:05
>>395
C++ が変化し続けているだけの話
397:デフォルトの名無しさん
08/02/23 21:09:33
>>395
現状のC++標準規格じゃ無理。C99は可能。
C99サポートしてるコンパイラなら期待してもいい。
398:デフォルトの名無しさん
08/02/23 21:11:02
newも使うべきではないよな。いやnew[]ではなくてstd::vector。
399:デフォルトの名無しさん
08/02/23 21:13:17
C99は正直あんまやらないでほしい
400:デフォルトの名無しさん
08/02/23 21:18:40
可変引数マクロはマジﾎｼｽ
401:デフォルトの名無しさん
08/02/23 21:24:18
0xでおｋ
402:デフォルトの名無しさん
08/02/23 21:35:03
おしえてください 下から2行目を動かすとデータが壊れるのですが原因がわかりません
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
typedef struct STRDATA{ char **st; char **end; }strdata;
f(strdata x){ printf("%s",*(x.st)); }
strconv(strdata *q, char **p){
char **chend =(char **)malloc(sizeof(char **));
q->st=p;
*chend=&(*p)[strlen(*p)];
q->end = &(*chend);}
strconstconv(strdata *q, char *p){
int n=strlen(p);
char *ch=(char *) malloc(n+1);
strcpy(ch,p);
q->st=&ch;
char **chen =(char **)malloc(sizeof(char **));
*chen=&(ch[n]); q->end = &(*chen);}
main(){
strdata str;
#define STR "abcdefgh"
strconstconv(&str, STR); //ここをコメントアウトして一つ下を動かしても平気です
//char *ch=new char [50]; strcpy(ch,STR); strconv(&str,&ch);
//char *x=new char [1]; ここを動かすとおかしくなります
f(str); }
403:デフォルトの名無しさん
08/02/23 21:37:15
もうお前いい加減諦めたら。
ローカル変数のアドレスを関数外に持ち出すな。
404:デフォルトの名無しさん
08/02/23 21:39:05
char *x=new char [1]; が、なぜstrdata strを書き換えられるんでしょうか?
405:デフォルトの名無しさん
08/02/23 21:39:21
なんかまぁ・・・・・・いろいろとおつかれさん
406:402
08/02/23 21:45:20
おなじやつですが短くしました 下から2行目を動かすと壊れるのはなぜでしょうか
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
typedef struct STRDATA{ char **st; char **end; }strdata;
f(strdata zzz){ printf("%s",*(zzz.st)); }
strconstconv(strdata *q, char *p){
int n=strlen(p);
char *ch=(char *) malloc(n+1);
char **chen =(char **)malloc(sizeof(char **));
strcpy(ch,p);
q->st=&ch;
*chen=&(ch[n]); q->end = &(*chen);}
main(){
strdata str;
strconstconv(&str, "abcdefgh");
// char *test=new char [1];
f(str); }
407:デフォルトの名無しさん
08/02/23 21:46:36
まず変数のスコープを勉強しよう
408:402
08/02/23 21:48:09
なぜコメントアウトを外すとデータがこわれますか?
409:デフォルトの名無しさん
08/02/23 21:53:31
newやmallocの確保は、解放しない限り残るんですよね そのアドレスは参照で受け取っているので問題ないと思うのですが
参照渡しにしてもだめです
f(strdata *zzz){ printf("%s",*(zzz->st)); }
main(){
strdata str;
strconstconv(&str, "abcdefgh");
f(&str); }
410:デフォルトの名無しさん
08/02/23 21:54:33
基礎からやりなおせよ
411:デフォルトの名無しさん
08/02/23 21:56:58
ヒントください
412:デフォルトの名無しさん
08/02/23 21:57:01
いいから基礎からやりなおせって。スタックとヒープの概念すら分かってないだろ。
413:デフォルトの名無しさん
08/02/23 21:57:11
>>409
トリップをつけてくれるとあぼーんしやすいのですがいかがでしょうか
414:デフォルトの名無しさん
08/02/23 22:00:25
スタックとかヒープとか、この際関係ないレベルじゃん
415:デフォルトの名無しさん
08/02/23 22:02:39
より短くしました これでも外すと壊れます なぜですか?
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
typedef struct { char **st; char **end; }strdata;
f(strdata *q){
char *ch=(char *) malloc(10);
strcpy(ch,"abcdef");
q->st=&ch; }
g(strdata *zzz){ printf("%s",*(zzz->st)); }
main(){
strdata str; f(&str);
//char *test=new char [1];
g(&str); }
416:デフォルトの名無しさん
08/02/23 22:03:36
>>409
多少手を入れて(int mainとするとか)こっちで動かしてみましたが、ちゃんとabcdefghと表示されました。
でも、newだけc++の機能をつかってるけど、あとは全部(結構年季が入った感じの)cだし、
c++のコンパイラなら、関数の戻り値を指定しないのはダメだと思うし、
mallocとnew は併用したらダメだってどこかで聞いたけどな。
いまいちやろうとしていることの意図がつかめません。
(mallocとnewを併用して、どういう状況でまずいのかしらべようとしているのか)
もしc++を勉強しようとしているなら、なにか適当な本とかで勉強するのを勧めます。
Cを上記くらいご存知なら、すぐにC++も使えるようになりますよ。
417:デフォルトの名無しさん
08/02/23 22:03:53
ヒントもなにも>>403がずばり回答してるんだが。
これでもわからないなら、なんでもいいからポインタの無い言語に行ってくれ。
そして帰ってくんな。
418:デフォルトの名無しさん
08/02/23 22:08:06
>>415
なんでお前は答えを貰っても全くレスポンスを返さずに、
そんなヘドロみたいなコードを貼り続けるんだ。
頭沸いてるのか
419:デフォルトの名無しさん
08/02/23 22:08:35
>>415
これなら動く。
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
typedef struct {
char *st;
char *end;
} strdata;
void f(strdata *q) {
char *ch=(char *)malloc(10);
strcpy(ch,"abcdef");
q->st = ch;
}
void g(strdata *zzz) {
printf("%s", zzz->st));
}
int main() {
strdata str;
f(&str);
g(&str);
return 0;
}
420:デフォルトの名無しさん
08/02/23 22:11:12
>>419
おまいやさしいな
421:デフォルトの名無しさん
08/02/23 22:14:15
>>419
それだと、ポインタは値渡しのため、strに渡した文字列が書き換えられなくなります
削ったのですがこれでも原因が不明です コンパイラはBCC5.5です
#include <stdio.h>
#include <stdlib.h>
typedef struct { char **st; }strdata;
f(strdata *q){
char *ch=(char *) malloc(10);
ch[0]='X';ch[1]='Y';ch[2]='Z';ch[3]=0;
q->st=&ch; }
main(){
strdata str; f(&str);
//char *test=(char *) malloc(1);
printf("%s",*(str.st));}
422:デフォルトの名無しさん
08/02/23 22:17:54
>それだと、ポインタは値渡しのため、strに渡した文字列が書き換えられなくなります
( ﾟдﾟ) ・・・
(つд⊂)ｺﾞｼｺﾞｼ
(；ﾟдﾟ) ・・・
(つд⊂)ｺﾞｼｺﾞｼｺﾞｼ
_, ._
（；ﾟ Дﾟ） …！？
423:デフォルトの名無しさん
08/02/23 22:17:55
419でどこの文字列がどう書き換えられないっていうんだ言ってみろ
424:デフォルトの名無しさん
08/02/23 22:20:55
VCCやGCCやDMCでも実行中にエラーになります
#include <stdio.h>
#include <stdlib.h>
typedef struct { char **st; }strdata;
void f(strdata *q){
char *ch=(char *) malloc(10);
ch[0]='X';ch[1]='Y';ch[2]='Z';ch[3]=0;
q->st=&ch; }
int main(){
strdata str;
char *test;
f(&str);
test=(char *)malloc(1);
printf("%s",*(str.st));
getchar(); return 0;}
425:デフォルトの名無しさん
08/02/23 22:21:34
>>421
>それだと、ポインタは値渡しのため、strに渡した文字列が書き換えられなくなります
ｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗ
>>419よ、これが現実だｗｗｗｗｗアホは相手にするなｗｗｗｗｗｗ
426:デフォルトの名無しさん
08/02/23 22:22:15
ぶっちゃけポインタ全く理解してないだろ。
427:421
08/02/23 22:26:27
誤解していましたすみません
428:デフォルトの名無しさん
08/02/23 22:28:05
さすがに・・・
初心者を抜け出した程度の俺でも酷いと思うｗ
429:デフォルトの名無しさん
08/02/23 22:28:08
>>424
変数testの用途が不明だったので、削除させてもらった。
#include <stdio.h>
#include <stdlib.h>
typedef struct {
char *st;
}strdata;
void f(strdata *q) {
char *ch = (char *)malloc(10);
ch[0] = 'X';
ch[1] = 'Y';
ch[2] = 'Z';
ch[3] = '\0'; //まあ0のままでもいいんだけど
q->st = ch;
}
int main() {
strdata str;
f(&str);
printf("%s", str.st);
getchar();
free(str.st);
return 0;
}
430:421
08/02/23 22:28:56
すみません 誤解していませんでした 書き換えられません
#include <string.h>
#include <stdio.h>
typedef struct { char *st; }strdata;
f(strdata *x){
delete x->st;
x->st = new char [9];
strcpy(x->st,"++++++"); }
main(){
char *ch=NULL;
strdata x;
x.st=ch;
f(&x);
printf("%s",ch);}
431:デフォルトの名無しさん
08/02/23 22:34:49
>>430
お前が421や424で書いたコードは
＞printf("%s",*(str.st));
だったのになぜ
＞printf("%s",ch);
になってるんだ？納得のいく説明を聞こうか？
432:デフォルトの名無しさん
08/02/23 22:34:55
>>429
int main() {
strdata str;
f(&str); ←ここで初期化された文字列の内容、長さを変更したいんです あと変数testほ確保しても落ちないようにしたいんです
printf("%s", str.st);
みなさんは、>>424はまともに動きますか?
433:デフォルトの名無しさん
08/02/23 22:37:26
>>432
動かすまでもなく、まともに動かないのは分かる
434:430
08/02/23 22:37:28
>>335>>343>>348>>351がもともとの質問でこれを実現したいんです
435:デフォルトの名無しさん
08/02/23 22:37:36
>>430
printf("%s",ch);をprintf("%s",x.st);とすればいい。
嫌か？
じゃあこれでどうだ。
#include <string.h>
#include <stdio.h>
typedef struct {
char *st;
} strdata;
void f(strdata *x) {
delete x->st;
x->st = new char[9];
strcpy(x->st,"++++++");
}
int main() {
char *ch = NULL;
strdata x;
x.st = ch;
f(&x);
ch = x.st;
printf("%s",ch);
delete[] ch;
return 0;
}
436:デフォルトの名無しさん
08/02/23 22:39:36
>>433
test=(char *)malloc(1);をはずすと上手くいきます なぜですか
437:デフォルトの名無しさん
08/02/23 22:40:58
>>436
それは上手く動いてるんじゃなくて、たまたま動いてるだけ
438:デフォルトの名無しさん
08/02/23 22:41:46
>>335
をチラ見しかしてないけどそれが動くことがとても不思議。
とりあえず期待する動作はなんなのかを日本語で書いてください。
439:デフォルトの名無しさん
08/02/23 22:42:06
>>435
それだと始めにchが巨大に確保されていたらメモリーリークになると思うのですが
440:デフォルトの名無しさん
08/02/23 22:43:20
>>436
偶然。
あのコードでは、すでになくなったローカル変数を読み取ろうとしている。
運良くメモリ上に残っていれば、一見正しく動いているように見える。
そのtestの行は、残っていた変数の値を破壊する決定打になったのだろう。
441:デフォルトの名無しさん
08/02/23 22:43:21
>>439
思うだけだ
442:デフォルトの名無しさん
08/02/23 22:44:24
見る気もしない。
高速スクロールで華麗にスルー。
違うお題ないの？
443:デフォルトの名無しさん
08/02/23 22:46:11
Perlで言うところの
use LWP::Simple;
sub foo{ return get('URLﾘﾝｸ(hoge.com)'); }
これをVC++で極力シンプルにやるにはどうすればいいでしょうか？
環境は以下のとおりです。
VisualC++.net2003 Standard
managedC++不使用 MFC不使用 ATL/WTL使用 STL/Boost使用
444:デフォルトの名無しさん
08/02/23 22:46:57
>>438
バイナリ文字列 (\0を含む) の初めと終わりを構造体で関数に渡して、バイナリ文字列の内容、長さを書き換えたいんです
445:444
08/02/23 22:48:45
純粋なC言語だけでそれを実現したいんです
446:デフォルトの名無しさん
08/02/23 22:49:59
>>445
…
一般的に知られている C言語の定義とは違うものを扱ってらっしゃるのですね
447:デフォルトの名無しさん
08/02/23 22:50:53
最終的には、C言語のみにしますけど今は実験段階なので・・・
448:デフォルトの名無しさん
08/02/23 22:52:02
>>444
なんでchとかtestとか別の変数が出てくるの？
構造体のまま扱い続ければいいでしょ。
449:443
08/02/23 22:54:26
>>443
単純にC++でネット上のものをダウンロードさせるには
どうすればいいですか？って聞くべきだった orz
450:デフォルトの名無しさん
08/02/23 22:55:14
chは現物の文字列です この初めと終わりのアドレスを構造体で渡して変化させるようにしたいんです
451:デフォルトの名無しさん
08/02/23 22:55:32
本でも読んで出直してこいよ
本格的に他人に頼りたいなら金でも払え
452:デフォルトの名無しさん
08/02/23 22:59:15
現物の文字列ってなんだよ
ただのポインタだろ
str.st と等価だ
453:デフォルトの名無しさん
08/02/23 23:01:55
たとえば、
char *ch=new char [100]; strcpy(ch, "---------");
という文字列に対して、その先頭と後方のアドレスを関数に渡して、関数側で書き換えられるようにしたいんです
454:デフォルトの名無しさん
08/02/23 23:04:50
何にそんなもんをつかうんかわからんが
とりあえずそんなことを考える前に
入門書なりきちんと読め。
たのむから
455:デフォルトの名無しさん
08/02/23 23:09:26
初めの文字列が200M程度確保されていたとして作業領域も200M使うとします
このとき初めの文字列を解放するか、上書きすれば最高でも400Mしかメモリを使いません
しかし、それができずに返却すると最大 200M + 200M + 200M必要になります
456:デフォルトの名無しさん
08/02/23 23:11:03
>>449
ソケット使ってカリカリやるしか思いつかん
457:デフォルトの名無しさん
08/02/23 23:11:12
>>443
手軽さならWinInet
真剣にやるならWinsock
458:デフォルトの名無しさん
08/02/23 23:13:38
>>450
435みたいに、構造体で渡した後、元の変数に代入し直せばいい。
459:デフォルトの名無しさん
08/02/23 23:15:30
もはやポインタいらんがなｗ
460:デフォルトの名無しさん
08/02/23 23:19:19
文字列ではなく、1byte変数の配列を可変長で扱いたいと。
だったら、
typedef struct {
char *baka;
size_t length;
} aho;
とでもして、構造体をやり取りすればいいだろ。
C++だったら、vector<char>、deque<char>でも使えや。
461:443
08/02/23 23:20:13
WinInetででググったらいっぱい出てきたんで調べてみる。
ｻﾝｸｽ
462:デフォルトの名無しさん
08/02/23 23:20:26
>>443
URLDownloadToFile
463:デフォルトの名無しさん
08/02/23 23:26:44
>>458
なぜ直接書き換えられないのかわかりません
464:デフォルトの名無しさん
08/02/23 23:29:10
>>463
こっちからしてみたら、なんで構造体を使いたくないのかわからない。
465:デフォルトの名無しさん
08/02/23 23:29:25
>>463
もっと勉強してから出直せ
466:デフォルトの名無しさん
08/02/23 23:40:17
もともと構造体は使っていますよ あとC言語のみで動かせるようにしたいんです
開始のアドレスと長さを渡しても上と同じだと思います
467:デフォルトの名無しさん
08/02/23 23:41:34
あぼーんしたいからトリップつけてくれ
468:デフォルトの名無しさん
08/02/23 23:44:18
たぶんトリのつけ方を知らない。
「名前欄にレス番つけること」すら今日知ったっぽい
469:デフォルトの名無しさん
08/02/23 23:44:39
釣りだろう
470:デフォルトの名無しさん
08/02/23 23:46:32
文字列を入力して、
表示するとき右から３文字ごとにコンマをつける方法ってどうすれば？
（例：入力>>344fru38fh4tgiur
出力>>3,44f,ru3,8fh,4tg,iur）
この続きお願いします・・
#include <iostream.h>
main()
{
char a[50] ;
cout <<"aに文字を入力してください>>" ;
cin.getline(a, 50) ;
int a_count = 0 ;
while( a[a_count] != '\0'){
a_count++ ;
}
cout <<"aを表示します→" ;
for(int i=0 ; i<a_count ; i++){
cout << a[i] ;
}
}
471:デフォルトの名無しさん
08/02/23 23:48:55
これがたまたまうまく動いたりしますが、実際は間違っているから困っているんです
#include <stdio.h>
#include <string.h>
typedef struct { char *start ; char *end; } bin;
void f(bin *x) {
delete x->start;
x->start = new char [3];
strcpy(x->start,"aa"); }
main() {
char *ch = new char [2];
strcpy(ch,"a");
bin x;
x.start=ch; x.end=ch+strlen(ch);
f(&x);
printf("%s",ch);
}
472:デフォルトの名無しさん
08/02/23 23:50:11
>>470
URLﾘﾝｸ(www.google.co.jp)
473:デフォルトの名無しさん
08/02/23 23:53:13
x.start=ch; とした時点で、数値がコピーされるだけになり、文字列を書き換えられなくなります
だから x.start=&ch;と参照渡しをするために
構造体を { char **start ; char **end; }に変更します
そうすると上記のような現象が出ます
474:デフォルトの名無しさん
08/02/23 23:53:48
>>471
f(&x);
ch=x.start;
printf("%s",ch);
でいいだろ
って・・・釣られちまったよ～～～～ん
475:デフォルトの名無しさん
08/02/23 23:56:53
>>473
fを呼んだあとのchが指す先はfの中でdeleteされていることに気づいているか？
476:475
08/02/23 23:57:42
×>>473
○>>471
477:デフォルトの名無しさん
08/02/24 00:03:07
>>475
>>471ではdeleteやnewが効かない場合があるんです これはちゃんと表示されますか?
#include <stdio.h>
#include <string.h>
typedef struct { char *start ; char *end; } bin;
void f(bin *x) {
delete x->start;
x->start = new char [3];
strcpy(x->start,"aa"); }
main() {
char *ch = NULL;
bin x;
x.start=ch;
f(&x);
printf("%s",ch);}
478:デフォルトの名無しさん
08/02/24 00:06:58
Cで動くようにしたいとかホザいておきながら、new や deleteを使うバカ
しかも、
baka = new aho[shine];
としたら
delete []baka;
とすることしら知らないバカ。
マニュアルでも読め
479:デフォルトの名無しさん
08/02/24 00:07:28
>>477
じゃあもういいから、こうしとけ
#include <stdio.h>
#include <string.h>
typedef struct { char **start ; char **end; } bin;
void f(bin *x) {
delete[] *x->start;
*x->start = new char [3];
strcpy(*x->start,"aa"); }
void g(bin *x) {
delete[] *x->start;
*x->start = new char [3];
strcpy(*x->start,"bb"); }
main() {
char *ch = NULL;
bin x;
x.start=&ch;
f(&x);
printf("%s",ch);
g(&x);
printf("%s",ch);
}
これで終了、もう来るな
480:デフォルトの名無しさん
08/02/24 00:08:23
>>477
deleteもnewも効いているはずだよ？
・chが指す先をdeleteしようが、x->start=new・・・をしようが、chの中身は変わらない。
・deleteしても絶対に書き込めなくなるとは限らない。
これらは理解している？
481:not 477
08/02/24 00:10:08
>>478
あ・・・そういえばdelete []baka;としないといけなかったんだ・・・・
C厨ですまん。
482:デフォルトの名無しさん
08/02/24 00:15:24
構造体はCでないというつもりか？
483:デフォルトの名無しさん
08/02/24 00:17:32
>>477
main関数内、chだけに注目すれば、
chはNULLで初期化された後、一切変更を受けないだろ。
printfにNULLを渡しているので、未定義。
484:デフォルトの名無しさん
08/02/24 00:18:48
>>479
それだと、 bin x;を初期化する関数と、適当な配列を入れるとバグるんです なんでうごかないんですか
#include <stdio.h>
#include <string.h>
typedef struct { char **start ; char **end; } bin;
void f(bin *x) {
delete[] *x->start;
*x->start = new char [3];
strcpy(*x->start,"aa"); }
void h(bin *x){
char *ch=new char [3];
ch[0]='X';ch[1]=0;
x->start=&ch; }
main() {
bin x;
h(&x);
f(&x);
char *test=new char [1];
printf("%s",*(x.start));
}
485:デフォルトの名無しさん
08/02/24 00:23:27
バグるじゃ分らんだろ。
486:デフォルトの名無しさん
08/02/24 00:24:02
>>484
h内のローカル変数chのアドレスをx->startに格納しているが、
chはhを抜けると無くなるので、x->startは存在しない場所を指している。
487:デフォルトの名無しさん
08/02/24 00:24:26
>>484
じゃあこれでいいだろう
#include <stdio.h>
#include <string.h>
typedef struct { char **start ; char **end; } bin;
void f(bin *x) {
delete[] *x->start;
*x->start = new char [3];
strcpy(*x->start,"aa"); }
char *ch;
void h(bin *x){
ch=new char [3];
ch[0]='X';ch[1]=0;
x->start=&ch; }
main() {
bin x;
h(&x);
f(&x);
char *test=new char [1];
printf("%s",*(x.start));
}
488:デフォルトの名無しさん
08/02/24 00:25:03
char *test=new char [1];をのぞくと、bcc5.5では動作します
ほかのコンパイラでは、入れなくても実行時に落ちます 原因は何ですか?
489:デフォルトの名無しさん
08/02/24 00:25:46
いいように使われてるぞ、デバッグ隊ｗ
次ページ最新レス表示スレッドの検索類似スレ一覧話題のニュースおまかせリスト▼オプションを表示暇つぶし2ch
4987日前に更新/243 KB
担当:undef