プログラマ専用SNS ミクプラ

あたっしゅさんが書きました: ↑
2週間前
実は、GDP の 4GB 版を持っているので

国内総生産…？
国内総生産 - Wikipedia

もしかして：GPD
gpdjapan.com

「GDP」ならこれかな…？メモリ4GBだし
12.1W型タッチパネルコンピュータ | ガウディ

前回出力したバイト数がsに格納されているので、
bufが指す位置をその分進めて、前回書き込んだ内容の次から書き込むようにしていますね。

どうしてごみが出るのかね？有効なデータの範囲内に終端のナル文字('\0')が無いため、続けて無効な(範囲外で何があるかわからない)領域を読み、ナル文字の前まで出力してしまうからです。 printfはもともとの仕様？ printfは少なくともC89にはあります。 The C89 Draft あとこのコードって *ae = *tempae; 普通に値を入れているっと思うだけどどうだろうかこれはポインタaeが指している場所にポインタtempaeが指しているものを代入するコードですね。 aeは関数名としても使われており紛らわしいですが、ここでは引数(char*型)として解釈され、この引数には...

ぷぷぷジューシーさんが書きました: ↑
1ヶ月前
情報がフェイクです

何のことでしょうか？

sprintf(rhost, "%s", temphost); の部分が配列ではないかったのかっと思うci 申し訳ありません。意味がよくわかりません。問題の部分を取り出し写したコード printf("%s\n", ae);は、文字列(ナル終端された文字の配列)へのポインタを要求する%sに対してナル終端されていない領域へのポインタを渡しているため、未定義動作になります。よくある挙動としては、tempaeの最初の要素である「a」に続いて、ゴミが出力されることがあります。 printf("%s\n", a);は、文字列へのポインタ(char*)を要求する%sに対してchar型のデータ(可変長引...

最初から引数にNULLを与え、それを書き換えなければ、NULLのままになるでしょう。例えば、第一引数を "http://example.com/hoge" とした場合、rpathへの代入は行われません。どうして geturlparserfunc の呼び出し元を開示してくれなくなったのでしょうか？指摘から逃げてもコードが勝手に改善することはないでしょう。さらに、この部分 rpath = malloc(bmfunc_rlen); sprintf(rpath, "%s", temppath); rhost = malloc(bmfunc_rlen); sprintf(rhost, "%s",...

ぷぷぷジューシーさんが書きました: ↑
1ヶ月前
コード:
  rhost = '\0', rpath = '\0';

ここでポインタに0を代入しています。
0をポインタに変換するとNULLになります。

大量に未定義動作をしていますね。また、定義があるlimits.hをincludeせずに MB_LEN_MAX を使っているという問題もあります。 #include <stdio.h> #include <uchar.h> #include <stdlib.h> #include <string.h> int main(void) { char32_t in[] = U"ttp://google.com"; // or "z\u00df\u6c34\U0001F34C" mbstate_t mbstate = {0}; char rpath[MB_LEN_MAX+1]; char* as = ...

c32rtomb | Programming Place Plus　Ｃ言語編　標準ライブラリのリファレンス注意あくまで文字の変換なので、末尾に終端文字(U'\0') は付加されない。文字列ではないもの(ナル終端されていない文字配列)を文字列を要求するstrcatに渡しているため、不定である未初期化の自動変数の値の参照が発生し、未定義動作になります。さらに、asも未初期化のままstrcatに渡されているため、未定義動作になります。 printf(" ここだよ %s\n", rpath); の前に as = calloc(MB_LEN_MAX+5, sizeof(*as)); if (...

#2とはこれです。 int main(void) { char hostname, pathname; geturlparserfunc("", &hostname, &pathname); printf("ホスト名；　%s\n", hostname); printf("パッチ名：　%s\n", pathname); } これはひどいですね。 printfの%sは文字列(終端にNUL文字がある文字の配列)を指すchar*型のデータを要求するのに対し、 hostnameとpathnameはchar型です。これは未定義動作になり、一般的な環境ではアクセス違反になる可能性が高いでしょう。さらに、...

まず、マルチバイト文字の変換エラーを起こすような入力ではないので、 geturlparserfunc関数内のエラーの出力は実行されません。さらに、#2の指摘内容が修正されていないので、 printfの処理内容がバッファを抜けて出力に反映されるより前に落ちてしまい、出力がされないのでしょう。 char型は可変長引数として渡す時はintに変換されるので、printfで出力するには%d(など)を用います。また、範囲外への書き込みを行う input[inputlen] = '\0'; も残っていますね。未定義動作です。さらに、足し算を行っている*rhostや*rpathは初期化されていないsta...

ホスト名とパス名をただ返すだけのプログラム返しませんね。まずは空文字列ではなくちゃんとURLを入力として与え、求める結果が得られるプログラムにすることをおすすめします。求める結果が得られずに困っているのであれば、そのような質問をすることをおすすめします。 2. 配列にした文字列をfor分でワイド文字に変換する for分ではなく、for文ですね。 2. '/"二個になったらrhostを+=で増えるようにする 3. '/"が三個以上になったらrpathを+=で増えるようにする C言語において、+=は文字列の結合ではなく数値(もしくはポインタ)の加算を行います。文字列に文字列を結合(追加...

int main(void) { char hostname, pathname; geturlparserfunc("", &hostname, &pathname); printf("ホスト名；　%s\n", hostname); printf("パッチ名：　%s\n", pathname); } これはひどいですね。 printfの%sは文字列(終端にNUL文字がある文字の配列)を指すchar*型のデータを要求するのに対し、 hostnameとpathnameはchar型です。これは未定義動作になり、一般的な環境ではアクセス違反になる可能性が高いでしょう。さらに、「ホスト名とパス名...

mbrtoc32 | Programming Place Plus　Ｃ言語編　標準ライブラリのリファレンス c32rtomb | Programming Place Plus　Ｃ言語編　標準ライブラリのリファレンスこれらは「文字」を変換する関数であるため、1文字しか変換しません。文字列全体を変換するには、最後まで1文字すつ変換するのがいいでしょう。また、ぷぷぷジューシーさんのコードには、他にも printf()に不適切なデータを渡している(%dはint型のデータを要求するが、渡しているmbfuncerrorはsize_t型) 配列の範囲外に書き込んでいる(outputにはsizeof(...

yk230 さんが書きました: ↑
1ヶ月前
コード:
    for(i = 0; i < 28123 + 1; i++){
        printf("%d\n", i);
        ans += i * abd[i];
    }

ここで配列の範囲外にアクセスしているので、未定義動作です。
したがって、任意の挙動が許されます。

Convert_Str_10関数が間違ってますね。
まず、配列iは１要素しか無いのに、2要素目以降に書き込んでいるため、範囲外へのアクセスとなり危険です。
十分な要素数を確保してください。
次に、n *= 64;を1回ずつ余計に実行しているため、値が64倍になっています。
該当部分のyのループを0ではなく1から始めるなどするといいでしょう。

スタートが「aaa(10進数だと0）」、終了が「baa(10進数だと262144)」以降だとすると、 Convert_10_64(start_num, &set64,strlen("aaa")); このようにして文字数を渡して、0から262143までは64進数変換後の文字数に応じて先頭に「a」と追加するということでしょうか？そうなると、無駄(というと語弊がありますが)な処理をしていると思うのですが、どのように対処するべきでしょうか？まず、毎回strlenを呼ぶのは無駄(運が良ければ最適化してくれるかもしれませんが、無駄な可能性)ですね。ループ前に長さを1回取得し、それを使うといいでしょ...

「a」でも「aa」でも「aaa」でも「aaaaaaaaaaaaaaaaa」でも10進数にすると「0」になり、
10進数には桁数の情報は含まれません。
よって、桁数は別に保存し、64進数に変換する関数に渡すといいでしょう。

また、Convert_10_64関数の中に

コード:

	*set64 = set;

という行がありますが、これはこの関数から抜けた時点で消えるローカル変数へのポインタを代入しています。
関数から抜けた後は、このポインタは無効な場所を指しているので、デリファレンスしてはいけません。

そのまんま埋め込むのが簡単でしょう。 #include <stdio.h> int main(void) { static const char* data[] = { "1234", "1243", "1324", "1342", "1432", "1423", "2134", "2143", "2314", "2341", "2431", "2413", "3214", "3241", "3124", "3142", "3412", "3421", "4231", "4213", "4321", "4312", "4132", "4123", NULL }; const char** p;...

ガウス＝ザイデル法 - Wikipedia
行列の操作を多く行うようなので、行列を表す構造体を用意し、
その構造体について行列の基本演算(足し算・引き算・掛け算)を行う関数をそれぞれ用意し、
それらの関数を用いてアルゴリズムを実装するといいでしょう。
ベクトルも、n行1列の行列として扱えます。

また、まずは作るプログラムの仕様をはっきりさせるのが重要です。
まずはどのような入力(読み込む値および書式)をどこから(標準入力・コマンドライン引数・ファイル・ネットワークなど)取り、
どのような出力(書き出す値および書式)をどこへ(標準出力・ファイル・ネットワークなど)出すのかを決めましょう。

コケさんが書きました: ↑
2ヶ月前
上記のようにnumを100以下に設定してもうまくいきません

まず「3435は開き直り数」という例が出ているのに、100以下というのは、探索範囲が明らかに全然足りなくてダメです。
さらに、探索する過程で、外側のループのカウンタである変数numの値を破壊してしまうのもダメです。

「すべて見つける」ということなので、何も考えないと無限に探索しないといけません。そこで、まず探索範囲を見積もります。おそらく、どこかで桁数を増やせば増やすほどもとの数の方が大きくなり、操作結果が追いつかなくなるでしょう。当然9をいっぱい使うほうが操作結果が大きくなるため、 9を並べた数とその操作結果を調べてみます。数操作結果 999999999 3486784401 9999999999 3874204890 99999999999 4261625379 9を10個並べた時、もとの数の方が操作結果より大きくなりました。よって、きちんと証明はしていませんが、10桁以下の数だけ調べれ...

ソースコードを提示する際は、BBCodeが有効な(無効にしない)状態で、 BBCodeのcodeタグの開始タグと終了タグの組(開始タグが先)で囲んでいただけると、見やすくてありがたいです。まず、このプログラムで使われている制御構文(プログラムに単純な「上から下」以外の動きをさせる文)は、 For - Next だけで、その他のifなどは使われていないようですね。フローチャート-繰返し型の記法が参考になるでしょう。例えば 05 For i=2001 to 2004 については、変数はi、初期値は2001、終了値は2004、増分は(明示されていないが)1、となりますね。このプログラム...

Kenta さんが書きました: ↑
2ヶ月前
簡易的な戦闘プログラムをc言語を用いて以下のように実装したいです。

実装すればいいでしょう。

なお、この掲示板は質問用です。
日記が書きたいのであれば、掲示板ではなく日記に投稿することをおすすめします。
(日記は、ユーザー登録してログインしていただけると使えます)

Kenta さんが書きました: ↑
2ヶ月前
よろしくお願いします。

何をお願いするのですか？

C言語はチューリング完全なので、理論上チューリングマシンでできることはなんでもできます。
(現実的にはメモリの容量などの制約を受けます)

まずは書きたい「条件を満たすまで変数を加算しつづけるプログラム(最大の試行回数は4まで)」
というのがどのようなものなのか、具体的に定義しましょう。
例えば
・「条件」とは具体的にどういうものか？どうやって決めるのか？どうやって判定するのか？
・「変数を加算」って、具体的に何に何を加算する？
・「最大の試行回数」ってあるけど、「試行」って具体的に何？
・やることは「変数を加算」するだけ？出力などはしないの？
など。

判定の後、同じ位置に重ねてa0.pngを描画しているので、
a0.pngの内容によりますが、これで隠してしまっている疑いがあります。
また、GOODの描画もMISSの描画も同じ座標に同じa2.pngを描画、というのも不自然ですね。

変数を静的配列の初期化に使おうとしている変数 sum, ave, max, minを宣言せずに使っている関数callを呼び出すときにa[]というのは間違っている (カッコが余計) *sumを計算する時にgを入れ忘れている *aveを計算する時に割る数が間違っている *maxや*minの計算に入力が反映されていない配列aの要素の値を使っている修正版： #include<stdio.h> void call(int n, int a[], int *sum,int *ave, int *max, int *min); int b,c,d,e,f,g,n; int a[]; int main...

1. GOOD / MISS を表示するための変数を用意する (ループ前に追加) enum judge_value { NONE, GOOD, MISS; }; const int JUDGE_DISPLAY_TIME = 30; // 判定を表示する時間 (フレーム数) judge_value judge = NONE; // 表示中の判定 int judge_time = 0; // 判定を表示する残り時間 (フレーム数) 2. ノーツを消す時に、判定を表示させるように状態を更新する //bar座標更新 for (int i = 0; i < LANE_SIZE; i++) { for (...

%[^\n]は入力前に空白文字を読み飛ばさないので、%dで読み込んだ後の改行で引っかかってしまいます。 %[^\n]の前に半角スペースを入れることで空白文字を読み飛ばすようにできますが、これだと行頭に半角スペースがある場合、それも読み飛ばされてしまいます。 %dおよび%[^\n]の後に%*cを入れることで、1文字(すなわち、改行文字)を読み飛ばすことができます。 #include <stdio.h> int main(void){ int n; char str[100]; scanf("%d%*c",&n); for(int i=0;i<n;i++){ scanf("%[^\n]%*c",...

C++では文字列リテラルはconst char*型に変換され、char*型の引数に(キャストなしで)使うことはできません。
文字列を書き換える処理は無いようなので、該当の引数をconst char*型にするといいでしょう。

ファイルを読み込む際は、読み込みが成功したかを確認するべきです。修正前 /* ファイルが終わりでない「かつ」配列を飛び出さないうちは，読み込みを続ける */ while ( ! feof(fp) && n < 10000) { fscanf(fp, "%lf %lf %lf %lf %lf\n", &(r[n]), &(z[n]),&(vr[n]),&(vz[n]),&(vphi[n])); n++; } 修正後 /* ファイルが終わりでない「かつ」配列を飛び出さない「かつ」読み込みに成功するうちは，読み込みを続ける */ while ( ! feof(fp) && n < 10000) {...

やっぱ、static int* a; を free(a); しちゃ、まずいだろ。 static int* a;はヌルポインタに初期化され、 freeにヌルポインタを渡した場合は何もしないと決まっているので、問題ないはずです。また、#6ではmallocに0を渡しているようですが、malloc(0)が何を返すかは処理系定義です。 #6の例では、malloc(0)はヌルポインタではないポインタを返すタイプの処理系であり、その返されたポインタをfreeしていないため、メモリリークの発生として検出されているようですね。以下、 N1570 より引用 6.7.9 Initialization If ...

呼び出された時に、要素数nの配列を確保して破棄しない（static付けた時のように）関数を作りたいんですけど、どうしてそのようなことがしたいのですか？ ( The XY Problem の疑いがあります) #include <stdlib.h> void main(){return;} void f (int n) { static int* a; a = (int*)malloc(sizeof(int)*n); free(a); return; } このプログラムは、この質問の回答としては間違っています。 free(a);にとり、せっかく確保した配列が破棄されてしまいます。 #inclu...

staticでなくてよければ、gcc拡張、もしくはC99の仕様で可変長配列(VLA : Variable-Length Array)が使えます。

コード:

void main(){return;}
void f (int n)
{
	int a[n];
	return;
}

【追記】
すいません、質問をよく読んでいませんでした。
「確保して解放しない」というのが条件なのですね。

ポインタに対する加算は、「1」を足すとそのポインタが指す型のサイズ分先へ進む、ということになります。
今回のreadRange.Beginには既にUINT64のサイズが掛けてあるので、
pDataをUINT64*にキャストしてから足すとUINT64のサイズを二重に掛けることになり、おかしくなります。
また、標準ではvoid*に加算を行うことはできません。
そこで、1バイトのデータ型へのポインタであるUINT8*にキャストをしていますね。

n>=2のとき、n項目を求めるにはn-1項目とn-2項目を使うので、n項目は必ずn-1項目よりも後に求まります。従って、前に出力した項の次の項が求まった時に出力するといいでしょう。また、いきなりreturnするのではなく、一旦結果をresultに格納して後で処理できるようにしました。 #include <stdio.h> int lastN = 0;//グローバル変数 int f(int n) //フィボナッチ数列を計算する関数 { int result; if(n == 0){//2項目を1と表示するため、return0としている result = 0; } else if(n == 1...

fgetsとsscanfを用いて、標準入力から2行の文字列を比較して一行目の文字が2行目に含まれているかどうかのプログラムを作成したいです。「sscanfを用いて」という条件があるのですね。うーん…返り値を作るのにでも使っておきましょうか。 #include <stdio.h> #include<string.h> int main(void){ char str[1000]; char abc[1000]; fgets(str, sizeof(str), stdin); char* lf = strchr(str, '\n'); // 改行文字を探す if (lf != NULL) *...

#2の指摘通り、sscanfの結果は使われていないので、sscanfの2行とbufの宣言は消してよさそうですね。また、char xyzではなくchar* xyzとするべきですね。 fgetsは改行文字が入力された場合その改行文字もバッファに保存するので、それを削除してからstrstrに渡さないとうまく検索できないという可能性があります。 #include <stdio.h> #include<string.h> int main(void){ char str[1000]; char abc[1000]; fgets(str, sizeof(str), stdin); char* lf =...

ワドさんが書きました: ↑
4ヶ月前
C言語初心者です。

あたっしゅさんが書きました: ↑
4ヶ月前
C++ 17 で確認。

うーん…まあ、C言語初心者だからといってC言語を使っているとは限らない、
むしろ初心者だからC言語を使うのを避けるという可能性もあるからなあ…
ただ、C++ならmalloc/freeではなくnew/deleteやstd::vectorを使うべきではないかなあ…？

nums[0].valueに入れている0についても最大値や最小値を求める対象としているため、全て正の数を入力した時にはmin = 0となってしまいます。また、全て負の数を入力したときにはmax = 0となってしまいます。これを回避するには、例えばnums[0].valueにはINT_MIN (limits.hで定義) を入れておき、 &nums[0]ではなくnums[0].biggerから最大値や最小値の探索を始める、という方法が考えられます。ただし、この場合引数を与えずに実行した時にNULLをデリファレンスしてエラーになる可能性があるので、このケースへは別途対処するべきでしょう。 ...

Meta3 さんが書きました: ↑
4ヶ月前
読みやすくして

掲示板の不親切設計によって[/i]が無いのにも関わらず[i]が斜体を表すタグとして解釈され、消えています。
その消えたままのコードを使用しているため、不正なコードになっています。
codeタグを使わずに貼り付けられたコードは、表示された本文から直接コピペするのではなく、
引用ボタンを押して引用投稿用のフォームからコピペするといいでしょう。

ソースコードを提示する際は、BBCodeが有効な(無効にしない)状態で、 BBCodeのcodeタグの開始タグと終了タグの組(開始タグが先)で囲んでいただけると、見やすくてありがたいです。 []で囲まれていないcodeや/codeは、codeタグではありません。参考書に書いてあるコードを書いていたのですが、配列に不定定数を入れる部分でエラーが出ており、これの直し方がわかりません。これは可変長配列(Variable-Length Array)といい、C99では必ず認められ、C11ではオプションの仕様です。したがって、対応するコンパイラ(gccなど)をC99モード(gccなら-std=c...

書式%nを使うとそこまでに読み込んだバイト数を取得できるので、これを用いて読み込む位置をずらしていくといいでしょう。 #include <stdio.h> int main(void){ char str[1000]; int n; int m; int i; char* p; fgets(str, sizeof(str), stdin); sscanf(str, "%d", &n); fgets(str, sizeof(str), stdin); p = str; for (i = 0; i < n; i++) { int len; if (sscanf(p, "%d%n", &m, &l...

ソースコードを提示する際は、BBCodeが有効な(無効にしない)状態で、 BBCodeのcodeタグの開始タグと終了タグの組(開始タグが先)で囲んでいただけると、見やすくてありがたいです。提示されたコードでは、aおよびbが宣言なしで使用されているため、コンパイルエラーになりますね。以下のような手順で処理を行うといいでしょう。ただし、擬似コード中の配列の添字は0から始まり、ループの範囲は両端を含むものとします。 xとyを読み込む y行x列の配列を確保する for jを0からx-1までループ配列の0行j列目の値を(x-j)にする next for iを1から(y-1)までループ for...

どうしてもlong型を使わないといけないのですか？
int32_t型に置き換えるなどではいけないのですか？

エンター(改行文字)が入力されるまで次に進まないようにすればいいでしょう。 #include <stdio.h> #include <stdlib.h> /* exit()用 */ void wait_enter(void) { int c; while ((c = getchar()) != '\n') { if (c == EOF) { puts("bye!"); exit(1); } } } void print(); char c; int main() { print(); printf("END\n"); return 0; } void print() { printf("1行目...

「32行目」で記述エラーを発見しました。というのは再現できませんでしたが、 gccでコンパイルしたところ、以下のコンパイルエラーが出ました。 (gccの-Werrorオプションを用いて警告もエラー扱いにしました) ロジックはチェックしていません。 2行目：あってはいけない位置に全角スペースがある (消すべき) 42行目：ifが無いのにelseがある (39～41行目を{}で囲むのを忘れた？) 45行目・74行目：ifが無いのにelseがある (代わりに} else {とするべき？) 74行目：ifが無いのにelseがある (代わりに} else {とするべき？) 82行目：代入できないも...

matcha さんが書きました: ↑
4ヶ月前
投稿した質問を編集する方法もしりたいです

投稿した記事の右上にある鉛筆っぽいマークのボタンを押すことで編集できます。
ただし、投稿してから5分間しか編集できません。

標準のrand関数は、乱数の質が低い可能性があります。ＤＸライブラリには、 GetRand という乱数を取得する関数があります。 int image = LoadGraph(".\\Media\\a.jpg"); // ループ while (ProcessMessage() == 0 && CheckHitKey(KEY_INPUT_ESCAPE) == 0) { // 画面を初期化する ClearDrawScreen(); for (int i = 0; i < 10; i++) { int n = GetRand(640 - 1); int m = GetRand(480 - 1); D...

足りないところを補ってみましたが、ちゃんと動きそうに思えます。 #include <iostream> #include <vector> #include <string> #include <algorithm> using std::vector; using std::string; using std::sort; int main(void) { vector<string> g_vLink1; g_vLink1.push_back("cat"); g_vLink1.push_back("dog"); g_vLink1.push_back("rabbit"); g_vLink1.p...

aが正の実数、qが0以上の実数で、xが実数のとき、 -2/(a*sqrt(x*x)) -2*q/(a*sqrt((x-1)*(x-1))) -pow(x-q/a, 2.0) は全て0以下 (ただしxが0, 1の時は分母が0になるので無効) ですね。 aが正、qが0のとき、 -2/(a*sqrt(x*x)) はx≠0のとき負、 -2*q/(a*sqrt((x-1)*(x-1))) はx≠1のとき0、 -pow(x-q/a, 2.0) はx≠0のとき負、x=0のとき0 なので、0, 1以外の任意の実数xについてfuncは負になります。 q=1.0/25, a=q+1のとき、q/a≒0.038であり...

ソースコードを提示する際は、BBCodeが有効な(無効にしない)状態で、 BBCodeのcodeタグの開始タグと終了タグの組(開始タグが先)で囲んでいただけると、見やすくてありがたいです。 ([]で囲まれていないcodeや/codeは、BBCodeのcodeタグではありません) 式を Wolfram Alphaにかけてみましたが、0になるのは複素解しか出てきませんでした。 (funcが0になる実数xは無いと推測できます) また、 const double q = 1/25; は、C言語では整数同士の除算は切り捨てられるので const double q = 0; と同じになります。これ...

まずは作るものの仕様をはっきりさせましょう。まずは入力について。入力ファイルの「.csv」は RFC 4180 - Common Format and MIME Type for Comma-Separated Values (CSV) Files に準拠したCSVファイルですか？それとも、そのサブセットですか？もしくは、独自の形式ですか？いずれにしろ、まずはどのような入力を扱うかを決めましょう。入力されるデータそのものについても仕様を決めないといけないでしょう。ヘッダ行やその他無視するべき行はありますか？データの長さはどの程度までですか？書式は？無視するべき行ではないはずなのに、...

コード:

    char d;
    scanf("%s",&d);

これはダメですね。
dには1文字分の領域しか無く、それは終端のナル文字で使うため、0文字までの文字列しか入力できません。
1文字以上の文字列を入力してしまうと、範囲外への書き込みを起こし、周辺のデータの破壊に繋がります。
今回の場合、dはこのあと使われていないので、
dを経由せずに直接bが指す領域に対して文字列を入力した方がいいでしょう。

0と、「下位から連続したjビットと、他に1ビット」だけが1の数のみを出力するようになっているのに対し、
110は下位0ビットと他に2ビットが1なので、出力されないですね。
どのような出力が欲しいのかわかりませんが、素直にこれではいけないのでしょうか？

コード:

for(i=0;i<(1<<h);i++){
    aa(i,h);
}

また、コードはインデントをして構造をわかりやすくした方がいいと思います。

あなたが「関数を作って欲しい」ということはわかりました。質問はありますか？課題の丸投げは禁止です。フォーラムルールを読み、従ってください。ちなみに、ココナラで「プログラム作成」で検索したらこんなのがありました。簡単なC/C++コード／実行ファイルつくりますお安く早く作ります。仕様相談、レビューもできます。 | プログラミング | ココナラ C/C++のプログラム開発を承ります業務や学校の課題でプログラム作成が必要な方どうぞ | プログラミング | ココナラ C言語やJavaのプログラミングをお手伝いします C言語やJavaでプログラミングが必要な方ぜひご相談下さい！ | プロ...

フォーラムルールより質問は具体的にする詳しくは「回答者が困る質問例」をご覧下さい「うまくいきません」という質問は大抵回答に困ります。自分は今何がしたくてどう取り組んで(作ったプログラムはどれでどのようなエラーやトラブルで困っていて自分は何が解らないのか、知りたいのか今のＣの知識はどの程度なのかこの５点をしっかりと明記して下さい。環境に依存する場合やライブラリを使っているときは使っているＯＳ名・コンパイラ名・ライブラリ名も明記しましょう。コンパイルエラーの質問時は必ず最低限のエラーメッセージも書きましょう。ただし、短いビット列の生成に再帰は必要ないでしょう。コンピ...

こんなんで、どうでせう。ダメですね。 if (a > MAX) { a = 0; } という処理だと、a = MAX の時はそのまま次のループに行き、R[a++]が範囲外へのアクセスになってしまいます。 if (a >= MAX) { a = 0; } とするべきでしょう。もしくは、R[a++]をR[a]として、このif文の代わりに a = (a + 1) % MAX; とするか。また、 SetDrawScreen(DX_SCREEN_BACK); をしていないのに ScreenFlip(); をしているのも、おかしいと思います。

出た数字をatari (-1とか？) に変える仕様だと仮定すると、以下のようにできます。 for (int r = 0; r < 5; r++) { // どの行を調べるか int bingo = 1; // その行が全部atariか？ for (int c = 0; c < 5; c++) { if (b[r][c] != atari) bingo = 0; // その行にatariじゃないマスがあるか？ } if (bingo) { // 全部atariだった！ (atariじゃないマスが無かった！) printf("おめでとう！%d行目がビンゴです！", r); } } for (int...

ブロックの描写位置がループ変数に依存しなくなってしまいましたね。
なぜそのように変更したのでしょうか？

この後、生成した乱数とビンゴカードの数がアタリかどうかを判定し、『アタリ』『ハズレ』と表示するソースを書きたいのですが、このときif文を繰り返しで行うのと、switch文で書くのとどちらが良いでしょうか？ ifを繰り返すのがいいでしょう。すなわち、ビンゴカード全体について判定を行い、どれかが一致すればアタリとするのが一般的でしょう。 for (n = 0; n <= 100; n++) { x = rand() % (n + 1); printf("抽選　%2d回目：%3d", n, x); int atari = 0; /* 当たったか */ /* カード全体に対し判定を行う *? ...

エラーの原因は、初期値 PositionY = 0 ではループ条件 PositionY >= 700 が真にならないことですね。
条件は PositionY <= 700 の間違いではないでしょうか？

また、この条件を直しても、画像が2×701枚全部一気に描画され、モーションにはならないでしょう。
モーションにするには、1回描画するごとにウェイトと画面クリアを入れるといいでしょう。
(本当は描画位置を状態として管理し、1フレームごとにメインループに戻すべきですが、
この段階ではメインループが無いようなので…)

fg->g[i][j]<=γ

という条件がおかしいですね。
緑を2回比較してしまっています。
正しくは

fg->b[i][j]<=γ

でしょう。

ループ条件のwidthとheightが逆ですね。
他の関数にも同じ間違いが見られます。

maxとminは個別の入力値ではなく、最初に入力した値に初期化するといいでしょう。 #include <stdio.h> int main(void) { int data, max = 0, min = 0, is_first = 1; for(; scanf("%d", &data)==1;) { if (is_first || data > max) max = data; if (is_first || data < min) min = data; is_first = 0; } if (is_first) { puts("no input"); } else { printf("%...

出力指定子%fに対してint型のデータを渡しているのが間違っていますね。

beki関数から返した結果をmain関数側で全く使わず、投げ捨てているのが間違っています。また、beki関数の引数numおよびansの値を使わずにローカル変数のように使っていること、およびせっかくdoubleで計算した結果をintで返しているのは不自然に思えます。 #include <stdio.h> /* 計算結果をdoubleにし、引数numとansを削除する */ double beki ( double input, int jou ); int main(void) { double real; double result; int power; int i; printf( "...

とりあえず、元のプログラムをできるだけ残して、修正。コンパイルエラーになりました。 prog.cc: In constructor 'samp::samp()': prog.cc:12:31: error: invalid conversion from 'char' to 'char*' [-fpermissive] 12 | samp() { cout << "a"; s = '\0'; } | ^~~~ | | | char どのようにすれば、このプログラムはきちんと動くのでしょうか。 samp::sはポインタなので、文字'\0'ではなくポインタを代入するようにする sampのコピ...

同じ条件のwhile文を3回も使うのでまとめたいのですが、以下のように、ループを使ってまとめることができそうですね。 //***入力された文字列を文字列、1文字ずつ、10進数、16進数それぞれに変換して表示***// #include <stdio.h> #define MAX 10 //格納上限 int main(void) { const char* headers[] = {"文字", "10進数", "10進数"}; const char* formats[] = {"%c", "%d", "%x"}; char moji[MAX] ; //文字入力 printf("文字列を入力し...

判断、というのはif文を使えば良いのでしょうか？効率は落ちますが、if文を使ってもいいです。 int main(void) { int b[5][5]; int a[25] = {0}; int i, j, m; for (i = 0; i < 5; i++) { for (j = 0; j < 5; j++) { for (m = 0; m < 25; m++) { /* i,j,mの値からして、a[m]をb[i][j]に格納するべきか？ (仮で全て偽) */ if (0) { /* もし格納するべきなら、格納する */ b[i][j] = a[m]; } } } } return 0;...

「aの要素」を代入したい場合は int main(void) { int b[5][5]; int a[25] = {0}; int i, j; /* #2 の (1) */ for (i = 0; i < 25; i++) { /* iに基づき、a[i]をbのどこに格納するかを求める (仮で(0, 0)) */ int y = 0; int x = 0; /* そこに格納する */ b[y][x] = a[i]; } /* #2 の (2) */ for (i = 0; i < 5; i++) { for (j = 0; j < 5; j++) { /* (i, j)に基づき、b[i][j]...

5×5の多次元配列bの各要素に、要素数25の1次元配列aを代入するにはどうしたらよいですか？「aの要素」ではなく「a」を代入するんですよね？ #include <string.h> int main(void) { int b[5][5][25]; /* 5x5の多次元配列 (各要素はint[25]) */ int a[25] = {0}; /* 要素数25の1次元配列 */ int i, j; /* bの各要素にaを代入(コピー) */ for (i = 0; i < 5; i++) { for (j = 0; j < 5; j++) { memcpy(b[i][j], a, sizeo...

IntやDoubleは独自に定義した型の可能性があり、誤りだとは限りません。
明示的に初期化されていないグローバル変数は「ゼロ」に初期化されるので、=がないからといって誤りとは限りません。

C言語のソースコードを教えてしまうとtxkrさんが作成する余地が無くなってしまいそうなので、とりあえずPythonで書いてみました。 import sys def add(a, b): return a + b def mul(a, b): return a * b def add_num(n): return lambda s: s.append(n) def calc(f): def do_calc(s): s[-2] = f(s[-2], s[-1]) s.pop() return do_calc q = sys.stdin.readline().rstrip() converted_...

画像はユーザー登録してログインしていただければ、添付ファイルの所から貼れるはずですよ。
新しいスレッドを立てても画像が貼れるかは変わらず、続きの話なのにスレッドが分割される方が有害でしょう。

: てすと; tesuto-20201019.png (4.14 KiB) 閲覧数: 1971 回

JJJuto さんが書きました: ↑
5ヶ月前
ただ、ソース後、dayの方が数字で表示されます。できれば、SUN,MON･･･といったように文字で表示したいのですが、どうすれば良いでしょうか･･･？

ソート前の表示と同様に、値で分岐して対応する文字を表示すれば良いと思います。
同じ長い処理を複数回書くのはあまり良くないので、関数化するのも良いでしょう。

未定義のsizeが使われているというメッセージですね。
sizeのかわりにkosuを使うべきであうように思えます。

まず、コードタグのスペルが違います。 JJJutoさんはco rd と書いていますが、正しくはco de です。次に、必要なヘッダのインクルードや、構造体、定数の定義が無いのが間違っています。再現可能な短いサンプルコードを提示していただくと、答えやすくなります。最後に、配列の要素数の取得方法が違います。配列の要素数は配列全体のサイズを配列の1要素のサイズで割ることで得られますが、 JJJutoさんのコード int i, j, tmp, changeday, a = sizeof(week[7].day) / sizeof(week[0].day); int n,m,temp, cha...

ソースコードを提示する際は、BBCodeが有効な(無効にしない)状態で、
BBCodeのcodeタグの開始タグと終了タグの組(開始タグが先)で囲んでいただけると、
見やすくてありがたいです。

Hyoka さんが書きました: ↑
6ヶ月前
どのようにすれば、エラーがでずにこのよう形で実行できるのでしょうか？

ThreadFunc関数だけを見た時にgameの初期値がわからないため、
初期値によってはplayerが初期化されずに使用されることがある、というメッセージですね。
ループの前にplayerを適当な値で初期化するようにするといいでしょう。

scanf関数に渡す書式指定文字列中の空白は、
「次の空白文字でない文字の直前まで読み飛ばす」という意味になります。
そのため、空白文字でない文字が入力されるまで入力を要求し続けることになります。
この空白を削除することで、動作を改善できるでしょう。

あたっしゅさんが書きました: ↑
6ヶ月前
＞ 1 行の長さに上限を勝手に設けてはいけなく，メモリを動的に確保してはいけないとする.

この条件では、作成不可能じゃ...コンピュータ通訳で、意味が逆になったんでしょうか ?

先頭のK文字と最後のK文字だけを保持しておけばいいので、作成不可能ではないと思います。
再帰だと末尾再帰最適化がされる保証が無く、スタックオーバーフローが怖いので、
仕方がないからsetjmp/longjmpを使うのが吉…ですかねえ…？

課題の丸投げは禁止です。フォーラムルールを読み、従ってください。質問は具体的にする詳しくは「回答者が困る質問例」をご覧下さい「うまくいきません」という質問は大抵回答に困ります。自分は今何がしたくてどう取り組んで(作ったプログラムはどれでどのようなエラーやトラブルで困っていて自分は何が解らないのか、知りたいのか今のＣの知識はどの程度なのかこの５点をしっかりと明記して下さい。環境に依存する場合やライブラリを使っているときは使っているＯＳ名・コンパイラ名・ライブラリ名も明記しましょう。コンパイルエラーの質問時は必ず最低限のエラーメッセージも書きましょう。宿題の質問は O...

-842150451は2の補数として16進数にすると0xCDCDCDCDとなります。これはVisual C++のデバッグモードでは「newなどで確保されたが、まだ初期化されていない領域」を表します。実際、コードを見ると、Data::outputではnewで確保した配列のデータを明示的に初期化せずに出力していますね。この配列の要素であるクラスMonsterのメンバはint型であり、コンストラクタやメンバの宣言に初期化はありません。今回のnewでは、初期化用のデータが指定されていないので、確保された領域はdefault-initializeされます。 (N3337 5.3.4 Newの15...

C++のenumの値は明示的に指定しないと0から始まるので、 mode = 0のときwhile文の条件の mode < MODE_FINISH || mode >= MODE_MAX すなわち mode < 0 || mode >= 5 は偽になり、入力を受け付けずにそのままswitch文へ行き、 case MODE_FINISH:を実行して終了することになります。 modeを0ではなく-1やMODE_MAXなどwhile文の条件を真にする値で初期化すると改善するでしょう。もしくは、while文のかわりにdo-while文を使うようにしてもいいでしょう。ちなみに、このコードではData::...

OpenGLは詳しくないですが、ざっと見た感じだとこれから使うはずのテクスチャをいきなりglDeleteTexturesで消してしまっているテクスチャを参照せずに固定の色を返すフラグメントシェーダを使用しているというのがまずそうだと思います。【参考】 glDeleteTextures は、紐付けを解除するらしいです。 If a texture that is currently bound is deleted, the binding reverts to 0 (the default texture). 一方、 glDeleteShader は、紐付けを解除しないらしいです。 If...

この問題の場合は、2次元配列の方がいいだろ。この問題で入力される座標の範囲はx, yともに-10**6～10**6なので、この範囲のタイルを全て2次元配列で用意すると、 1個のタイルを1バイトで表現すれば約4TBになってしまいます。 1個のタイルを2ビットで表現しても、約1TBです。これは現在一般的なコンピュータの主記憶で扱うには大きすぎるでしょう。また、この巨大な配列の要素を埋めるのにも、現在一般的なコンピュータでは長時間かかることになるでしょう。従って、わざわざ指定の場所に到達するまでの全タイルの情報を持つのではなく、塗った範囲の四隅の座標だけを管理する方がいいでしょう。 (...

四隅の座標はxとyに分け、タイル1個分を1とした値で複数のint型の変数で保持するといいでしょう。

指定の位置が次に色を塗る領域に入っているかは、次に色を塗る領域を求めた後、
「次に色を塗る領域の東の端のx座標 ≦ 指定の位置のx座標」かつ
「指定の位置のx座標 ≦ 次に色を塗る領域の西の端のx座標」かつ
「次に色を塗る領域の南の端のy座標 ≦ 指定の位置のy座標」かつ
「指定の位置のy座標 ≦ 次に色を塗る領域の北の端のy座標」
かで判定するといいでしょう。 (条件を全て満たす ⇔ 入っている)

過去問｜パソコン甲子園2019
に載っている予選の問題ですか？
そうだとしたら、色が塗られている領域は常に長方形になるので四隅の座標を保持するだけで管理でき、
次に色を塗る領域もそこから簡単にわかるので、
指定の位置が次に色を塗る領域に入っているかを判定し、入っていればその色を出力するようにすればよさそうですね。

自作関数とのことなので、通信のログを取るなどしてデバッグをするのがいいと思います。

エラーの原因は、プログラムの実行をmain関数から始めることになっているのに、そのmain関数が無いことですね。あと、ファイルの先頭に GameMsg002()など、まだ定義されていない関数のプロトタイプ宣言があるようですが、コンパイラから見たら、｢は？あるって書いてあったのに GameMsg002関数ねぇじゃん❗｣ってなるので、コンパイラから見たら、宣言がある時点でその関数は「ある」ということになります。すなわち、関数を呼び出すコードがあるとき、引数や戻り値をどう扱うかがわかります。コンパイルの段階ではそれで処理に十分であり、関数の定義は必要ありません。 (このため、関数の宣...

弾の移動処理においてEbulletCounter[j] = 0;を実行しているが、
ここでのjは敵ではなく敵の弾の位置なので、関係ない敵のデータを書き換えてしまうことに繋がり、不適切です。
EbulletCounter[j] = 0;は弾を出す処理の方で行うべきでしょう。

くろのまさんが書きました: ↑
7ヶ月前
これはどうですか！直しました！

ブロックと最初と最後でインデントの深さが違う以外は、問題なさそうだと思います。

くろのまさんが書きました: ↑
7ヶ月前
敵の弾はこれであっていますか？自分が動かしたいようには動いています。

ここには敵本体の処理しか書かれていないようなので、「敵の弾」としては論外ですね。
書かれている処理については、
敵を出現させる時のループの上限がENEMYではなくマジックナンバーの3になっているのが気になりますが、
今のところ出現させる敵は3体のみで、かつENEMYは3以上なので、異常動作の原因にはならないでしょう。

敵が画面上に現れてから60フレームごとに1発弾を撃つようにしたいです。弾を出したらカウンターを0にすればいけるかなと思ったのですが、全然できませんね… カウンターを敵ごとに用意するといいかもしれません。コードを下のように変更してみたら、とりあえず出てきた敵は倒せるようになったのですが、これであっているでしょうか？間違っています。外側のループの継続条件が i < SHOT+j となっており、 jは内側のループを抜けるときにはSHOTになっているので、iは SHOT+SHOT-1 までループしてしまいます。一方、bulletFlagなど弾に関する配列の要素数はSHOTしか無いため、範囲外...

配列上で同じ位置にある敵と弾しか見ていないというのがよく分かりません。詳しく教えていただけないでしょうか？複数あるもの同士の通常の当たり判定では、二重のループを用い、存在しているものの組み合わせ全てについて判定を行います。 mixcpp-20965-1-20200821.png 一方、くろのまさんのコードでは、ループを一重しか用いておらず、限られた組み合わせしか判定をしていません。 mixcpp-20965-2-20200821.png このため、判定しない組み合わせでは判定が行われず、「当たらない」ということになります。それと、今まで更新処理はシーン変更のみと思い込んでいたのですが、...

・敵を出現させる処理において、フラグを確認していないため、常に最初の敵を上書きしてしまう
・敵や敵の弾を動かす処理において、余計なbreak;があるため、最初の有効な敵しか処理されない
・自機の弾と敵の当たり判定において、敵のフラグを確認していない上、
　配列上で同じ位置にある敵と弾しか見ていないため、不適切な動作になる

といった問題がありそうです。
また、Game_Update関数があるのに、Game_Draw関数で大量の更新処理をしているのも、よくないと思います。

詳しくみてはいませんが、とりあえずざっと見たところ、以下の怪しい点がありました。 ●13行目に不正な文字列 */ があり、コンパイルエラーになる ●learn_w関数の for (p = 1; p < PatternNo; p++) { w[i][j] = pattern[p][i] * pattern[p][j]; } という部分は、代入は前の値を上書きするため実質 w[i][j] = pattern[PatternNo - 1][i] * pattern[PatternNo - 1][j]; と同じ意味であり、ループの意味がない (和を求めるなどの計算をしたかったのではないか？) ●inp...

オフトピック

あたっしゅさんが書きました: ↑
7ヶ月前
インデントがなくなってしまったソースコードを[ code ][ /code ]で囲っても、しょうがないだろう。

最初の投稿の引用画面でもインデントが無いため、
「なくなってしまった」ではなくて最初からインデントが無い可能性も考えられます。
また、今回は対象が無いようですが、codeタグには[i]などがBBCodeのタグとして処理されるのを
防ぐ効果もあるため、しょうがないとは限らないでしょう。

これはfscanfが問題なのではなく、その次のbubbleSort関数の処理にかなり長時間(実験はしていないが、1時間くらい～？)かかるだけだと思います。 topコマンドなどでCPU使用率をチェックし、CPU使用率が下がるのを待ってから入力をしてみましょう。なお、普通のバブルソートはO(n^2)であり、10000件であれば数秒～数十秒程度でできるはずですが、ここで使われているbubbleSort関数はO(n^3)なので、 nが大きくなっていくに従いバブルソートよりさらにどんどん処理時間が伸びます。また、このコードには他にも・binary_search関数の引数imaxに与える値が...

似た質問がありました。
同じ人のマルチポストだとは断定はできませんが…

C言語プログラミングy = sin(x)のグラフを 0≦x≦2π の範囲について π/10... - Yahoo!知恵袋

作成したいのであれば、作成すればいいでしょう。
希望だけで質問が書かれていないようですが、何をお願いするのですか？

		4月 2021
日	月	火	水	木	金	土
				1	2	3
4	5	6	7	8	9	10
11	12	13	14	15	16	17
18	19	20	21	22	23	24
25	26	27	28	29	30

検索結果 6297 件