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

フォーラムルールを読み、従ってください。

niwaka1117 さんが書きました: ↑
11時間前
C言語でスリンショットゲームを作りたいのです、、、

作ればいいでしょう。
少なくとも、作ろうとするべきでしょう。

niwaka1117 さんが書きました: ↑
11時間前
学校の課題なのですが、説明が下手ですみません、、

課題の丸投げ (問題文だけ書くこと) は禁止です。

niwaka1117 さんが書きました: ↑
11時間前
有識者の皆様、よろしくお願いいたします。

何をお願いするのですか？
具体的にここのやり方がわからないなどの質問はありますか？

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

ググれば例がいっぱい出てきますね。
C言語カレンダー - Google 検索

・共通の出力を行う部分は1個にまとめる・それぞれの値が中央値になっているかを確認し、中央値になっているものを出力する #include <stdio.h> int main(void) { int a, b, c; /* a, b, c の入力 */ printf("(%d, %d, %d)の中央値は ", a, b, c); if ((b <= a && a <= c) || (c <= a && a <= b)) printf("%d", a); else if ((a <= b && b <= c) || (c <= b && b <= a)) printf("%d", b); els...

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

コードしか書かれていないようですが、何か質問はありますか？

もう一点質問になってしまうのですが、→の移動判定が高速になってしまい、キーを押し続けると高速で移動することで選択が困難な状態になってしまいます。原因としては、毎フレームキーの入力判定を取っているため高速で移動し、選択が困難になっていると思うのですが、適切に処理をするにはどうすればいいでしょうか。例えば、CheckHitKey関数のかわりにグローバル変数の扱いに気を使ってみよう (龍神録プログラミングの館) で紹介されているCheckStateKey関数を用い、戻り値が1 (キーが押され始めた直後のフレームである) のときだけカーソルを移動するようにするといいと思います。 #2 ☆海美...

関数 ChooseTime において、例えば最初 Choose == 0 で右キーを押した時 1. Choose == 0 なので、 Choose += 1; により Choose = 1 となる 2. Choose == 1 なので、 Choose == 1; により Choose = 0 となるとなり、結局Chooseの値が変わらないことがあります。以下のように、1回のChooseTimeの実行につき1組のキー判定のみ実行するようにすると改善するでしょう。 (この例ではswitchを使いましたが、else ifなどでもいいでしょう) int ChooseTime() { switch ...

うーるさんが書きました: ↑
1ヶ月前
コード:
       moji1[n]=moji2[n]

そもそもここでセミコロンが抜けているため、
提示するコードをC言語としてコンパイルしようとするとコンパイルエラーになるはずです。
何をどうやって実行しているのですか？

プログラミングではイコールの左側に右側を入れるのですね C言語では=演算子を用いることで右辺の値を左辺の変数に代入できますが、一般に「プログラミングではイコールの左側に右側を入れる」とは限りません。例えば、BASIC言語では=を代入ではなく比較に用いることがあります。なぜなんの文字列も帰ってこないのか scanf("%d",n); という行において、本来渡すべきnへのポインタ &n ではなくnの値そのものを渡してしまっているため、 nは未初期化なので読み込んだ値をデタラメな場所に書き込むようscanf関数に指示してしまい、その結果アクセス違反となり、文字列を出力する前に強制終了してし...

変数dを配列ではなく、普通のint型にするといいでしょう。

int *w = 5; ポインター変数wに、即値を直接代入することは未定義のはずです。 (0を除く)整数からポインタへの変換は「処理系定義」であり、「未定義」ではないと思います。 ( N1570 6.3 Conversions → 6.3.2.3 Pointers) ただし、 int *w = 5; printf("%d\n",w); のように、書式 %d に int* 型の値を渡すのは、書式と渡されるデータの型がマッチしないので未定義動作になります。 (N1570 7.21.6.1 The fprintf function の 9) もし、アドレス値を出力したいのであれば、 #includ...

動画の再生状態を得る（GetMovieStateToGraph）のは停止中か再生かであって、終了とは関係なさそうで…… なぜそう考えるのですか？ GetMovieStateToGraph 関数の説明を見ると、もし０が返ってきたら停止中、１なら再生中となります。　因みに動画の再生が終了するか PauseMovieGraph 関数で再生にポーズを掛けると停止中となります。と書かれており、停止中かどうか(かつポーズをしていないか)で動画の再生が終了したかを判定できることがわかります。さらに、 PlayMovieToGraph 関数の説明に載っているサンプルコードを見ると、 // ルー...

box さんが書きました: ↑
1ヶ月前
参考までに、自分だったらこう書きます。

これはバブルソートではなく、選択ソートですね。
バブルソート縛りではなく適当なソートアルゴリズムでいいなら、
自分ならわざわざ自分でソートを実装せずにqsort関数に任せます。

box さんが書きました: ↑
1ヶ月前
あと、元のコードで問題なのは、
外側のiに関するループは何をしてるんですか？ってことです。
比較にも何にも使ってないので、そのループ、なくていいんじゃないですか？

外側のループは、処理を繰り返すことでソートを進める役割があります。

now_on_key が char 型なので、60fpsの時約2秒を超えて長押しすると誤動作する可能性がありますが、
他は間違いを見つけられませんでした。
(見落としがあったらごめんなさい)

MonitoringKey() を1フレームに複数回呼び出すなど、提示されていないコードに間違いがある疑いがあります。

OnTimer()内で呼んでいるのですが、それで実行するとウィンドウが一瞬で閉じてしまいますこのような場合、バグにより強制終了になっていることを疑うべきですね。今回の場合、mPLength がグローバル変数のデフォルト値である0のままUpdate関数を実行しているため、 //★座標を後ろのパーツから更新(ひとつ前のパーツの座標にする) for ( auto i = mPLength-1; i; --i ) { の部分で i が -1 からループを開始するために範囲外へのアクセスが発生し、 i の値が十分小さくなるとアクセス先が有効なメモリ領域を外れてしまい、強制終了になることがあるようです。

cupa さんが書きました: ↑
2ヶ月前
結果：mX:260 mY:20（もちろん、ランダム取得しているので毎回結果は異なります）
値は正当です。（座標）

この値は正当ではありません。
mField の各次元の要素数は FIELD_SIZE + 2 すなわち 22 しかありません。
mY:20 は大丈夫ですが、 mX:260 は範囲外になってしまいます。
範囲外にならないよう、配列の要素数を大きくするか、
座標の値が範囲内 (0以上 FIELD_SIZE + 2 未満) になるようにするべきです。

コンパイルコマンドの main.cpp と *.cpp で main.cpp が二重にコンパイル (リンク) されてしまっているようですね。
main.cpp を消して *.cpp だけにするといいかもしれません。

put()の呼び出し以降に描画処理が全く無いようですね。
gFieldを参照した描画処理をしているforループの前にput()を呼び出すべきではないでしょうか？

定義した GLOBAL を function.h 内で使うからだと思います。
C言語やC++では、定義された識別子を使いたいときは、今見ている場所より前で定義された識別子しか使えません。

龍神録プログラミングの館 - 宣言の仕方を工夫しよう

cupa さんが書きました: ↑
3ヶ月前
この場合抜けるのはFOR文ですよね？While文は抜けないですよね？

はい。

cupa さんが書きました: ↑
3ヶ月前
０番目の弾のフラグが０だった場合、他の弾はみられず後は飛ばされるのではないのでしょうか？

飛ばされます。

cupa さんが書きました: ↑
3ヶ月前
ここで、breakしてると思うのですが、何故でしょうか？

最初に見つかった空き枠1個だけを利用し、弾を1個だけ出すためですね。

cupa さんが書きました: ↑
3ヶ月前
コード:
DrawFormatString(0,0,GetColor(255,255,255),"%d",image);
このようにしてみたのですが変な値が出てダメでした。

imageは配列のはずなので、それぞれの要素を出力しないとダメですね。
例えば↓

コード:

for (int color = GetColor(255, 255, 255), i = 0; i < 16; i++) {
	DrawFormatString(20 + 300 * (i / 8), 50 + 30 * (i % 8), color, "image[%d] = %d", i, image[i]);
}

C言語の館（９章）では、||になっていたのですがどういうことでしょうか？ if(!(x<10 || x>FIELD_MAX_X-10 || y<5 || y>FIELD_MAX_Y-5)){//計算結果移動可能範囲内なら ch.x=x , ch.y=y;//実際に移動させる } ( https://dixq.net/rp/9.html より) || を使って「移動先が範囲外か」を判定し、! で論理を反転することで「範囲内か」を判定しているようですね。また、C言語の館は初期の位置がちゃんと下の方の真ん中で、どこで初期のx,yを定義しているのでしょうか？ first_ini 関数のようですね。...

後もう一つ、frame_cntを毎フレーム０で初期化しているので、宣言だけにしたのですが、そこは直ったのですがwarningが出ます。修正したほうがいいでしょうか？初期化されていない自動変数の値を使うと未定義動作になるので、やってはいけません。 (C++のクラスの中には何も書かなくても初期化されるものもありますが、intなどのプリミティブは初期化されません) また、この関数内でこの関数で使う変数を宣言・（初期化）しちゃっているのはよくないでしょうか？ C言語の館が毎フレーム呼ばれる関数内で宣言・（初期化）していたので・・・ものによります。この関数の中でしか使わず、関数の実行が1回終わっ...

起こる現象としては、起動して何もキー入力をしないと普通に前向き時のアニメーションをするのですが、一回でも方向キーを押すと、ニュートラル状態でも正常に動きません。方向キーを押している間は表示されるのですが、離すと消えてしまいます。今のプログラムには、・左右キーを押した後離した時に、ch.imgをニュートラル用の値に戻す処理が無いので、ch.imgがどんどん増えていく・左右キーが押されている場合、毎フレームch.imgを特定の値にセットするので、アニメーションにならないという問題がありますね。 //左右キ－が押されていなかったら for (int i = 0; i < 2; i++)...

まず、前のトピックで指摘されている通り、トピックのタイトルの付け方を直すべきでしょう。その際、タイトルは「質問」なんかではなく、投稿の内容を端的に示す具体的なものにしましょう。次に、簡単なところから直していきましょう。実行結果は「検索する文字を入力」なのに、プログラムでは「検索する文字列を入力」となっており、余計な「列」が入っています。また、実行結果は「文字列apple_banana_orange中に」なのに、プログラムでは「文字列%sの中に」となっており、余計な「の」が入っています。インデントも整えるといいでしょう。最後に、アルゴリズムを直すべきでしょう。適当な方法で文...

・宣言と定義でstr_copyの型が違う・初期化していない配列str2の要素の値を使おうとしている・constが付いたポインタの指すものを書き換えようとしている・pstr1の指す位置を動かそうとしていない・必ず2要素以上コピーするため、コピー元に空文字列を渡すと誤動作するという問題がありますね。 ↓ 修正版 #include<stdio.h> #define SIZE 100 unsigned str_copy(char *pstr1,const char *pstr2); int main(void){ char str1[SIZE]; char str2[SIZE]; print...

Yuhiwa さんが書きました: ↑
3ヶ月前
ありがとうございます。でもまだ０になります。

合計が0になるような入力を与えると、0になりますね。
https://wandbox.org/permlink/D9vuxLtdErngiuIM

オフトピック

訳：検証して正しい合計が出ることを確認しているので、「0になる」とだけ言われても原因はわかりかねます。

0にならないようにするには、psum ではなく、psum「が指している変数」を書き換えるようにするといいと思います。 void array_sumof(int *vc,int no, int *psum){ int i; for(i=0;i<no;i++){ // psumを書き換える // psum+=*vc; // psumが指している変数を書き換える *psum+=*vc; } } また、配列の先頭要素 *vc を no 回足すのではなく、配列の各要素 vc[i-no] を足すようにしたほうがよさそうですね。「今日中」に間に合わないようにするために待ったのではなく、単に質問を見たタイミ...

cupa さんが書きました: ↑
3ヶ月前
GLOBAL＝externなのではないのでしょうか？別物なのでしょうか？

はい、別物ですね。
extern はC言語の予約語であり、ユーザが定義しなくても使えます。
GLOBAL はただの識別子であり、ユーザが定義(または宣言)しないと使えません。

cupa さんが書きました: ↑
3ヶ月前
また、調べたらところ、staticやexternが付与されていない場合はデフォルトでexternがついたのと同じ、
ということらしいのですが・・・

cupa さんが書きました: ↑
3ヶ月前
後半の（また、～）部分については関数の場合です

GLOBAL は関数でないもの(変数)について使われていますよね？

「C4530」でググった最初の2個
コンパイラの警告 (レベル 1) C4530 | Microsoft Docs
アンワインドセマンティクスって何？ | の回想録

メッセージの通り、オプション /EHsc を追加すればいいのではないのでしょうか？

cupa さんが書きました: ↑
3ヶ月前
window.h
コード:
#ifdef WINDOW_H
#define WINDOW_H

int test();    //  つまりはこういうことです

#endif WINDOW_H

また ifndef のかわりに ifdef を使っている、ということですか？

ヘッダファイルに宣言を書いてうまくいきました。 main.cpp #include <iostream> #include "test.h" int main(void) { std::cout << "main 1\n"; test(); std::cout << "main 2\n"; return 0; } test.cpp #include <iostream> void test(void) { std::cout << "test\n"; } test.h #ifndef TEST_H #define TEST_H void test(void); #endif header_te...

cupa さんが書きました: ↑
3ヶ月前
＃をつけてもう一度やってみたのですが、

ifdef を ifndef にする修正もしましたか？

cupa さんが書きました: ↑
3ヶ月前
まだ、testが定義されていません、と言われます・・・

これはコンパイル時ですか？リンク時ですか？

box さんが書きました: ↑
3ヶ月前
window.cpp
の2行目に
#include "window.h"
と書いたらどうなりますか？

結果は変わらないと予想できます。

条件式が悪いです。 menu[i].y >= mouseY && menu[i].y + 50 <= mouseY は、書き換えると menu[i].y + 50 <= mouseY && mouseY <= menu[i].y となりますが、これを満たす整数 menu[i].y および mouseY は存在しません。 xについて、同様に textWidth が正のとき、条件を満たす整数 menu[i].x および mouseX は存在しません。 (オーバーフローは考慮しないこととします) if (menu[i].x <= mouseX && mouseX <= menu[i].x + t...

コード:

if (upCount == 1 // 1フレーム目
    || ( // のいずれかの時
        upCount >= LONG_THRESHOLD // aフレーム目以上
        && // で、
        (
            upCount // フレーム数から
            - LONG_THRESHOLD // aを引いた数
        ) % MOVE_INTERVAL // をbで割った余り
        == 0 // が0
    )
)

ボタンが押されているフレーム数をカウントし、・1フレーム目・aフレーム目以上で、フレーム数からaを引いた数をbで割った余りが0 のいずれかの時に移動するようにするといいでしょう。 aは長押しと判定されるまでのフレーム数、bは長押し時に移動する間隔を表します。こんな感じかな。 (動作未確認です。typoなどあるかもしれません。) const int LONG_THRESHOLD = 30, MOVE_INTERVAL = 5; int upCount = 0, downCount = 0; while(ProcessMessage() == 0 && CheckHitKey(KEY_INP...

行列を読み込む所で scanf に渡している書式に余計な空白が入っているため、
数値と空白を読み込んだ後、また空白以外の文字を読み込むまで次の処理に行かなくなっています。
余計な空白を削除するといいでしょう。

どうしても「角度」を求めたいのであれば、三角関数が目的の値を返す角度を二分探索する方法もあるかもしれません。

dic さんが書きました: ↑
3ヶ月前
atan2 は中身がわからないので、使えないのです。

atanは使えますか？

「角度」を求めようとせず、正規化した「方向ベクトル」を求め、それを用いて移動するのはどうですか？ void Player1Input() { int PosX, PosY; PosX = PosY = 0; GetTouchInput(0, &PosX, &PosY, NULL, NULL); if ((PosX > 0) && (PosY > 0)) { double X, Y, speed; X = Y = 0; speed = 1.0; // 各辺の長さを求める double a, b, c; a = b = c = 0.0; a = (PosX - s_iPlayerPosX); b ...

ソースコードを提示する際は、BBCodeが有効な(無効にしない)状態で、 BBCodeのcodeタグの開始タグと終了タグの組(開始タグが先)で囲んでいただけると、見やすくてありがたいです。 'n' was not declared in this scopeというエラーが変数nを定義せずに使ってしまっているので、定義を追加するといいでしょう。ついでに、変数iも定義せずに使っていますね。 // 変数nの宣言をする // | 唐突に出てきた謎の変数iの代わりに、宣言した変数nを使う // | | // v v for (int n = 0; n < NUM_LEDS; n++ ){ dig...

例えば、「残り時間」を直接持たず、「開始時刻」と「現在時刻」から計算している場合は、
「開始時刻」を前にずらすことで「その分余計に時間が経過した」すなわち「残り時間が減った」
状態を作ることができます。

提示されているコードは残念ながらあまり関係無さそうなので一旦消すか脇に置いて、
まずは処理対象の「配列1」と「配列2」を用意しましょう。

配列を用意したら、以下のような手順で計算をするといいでしょう。
(各配列の要素に、配列内での重複が無い場合)

コード:

1. 共通点の数を0に初期化する
2. 配列1のそれぞれの要素について
   2-1. その要素が配列2にあるか検索する
        (線形探索、二分探索、ハッシュテーブルなど適当に)
   2-2. その要素が配列2にあれば、共通点の数を1増やす

配列内に重複した要素が存在しうる場合、まずは重複した要素をどう数えるかを定義するべきですね。

usao さんが書きました: ↑
4ヶ月前
そいつらをリンクする記述（-L オプションかな）も書く必要がある．

-L オプションはライブラリがあるディレクトリを指定するオプション、
実際にライブラリをリンクする指示は -l オプションですね。

できるはずです。
読み込もうとしているファイル名が正しいか、exeファイルを適切なワーキングディレクトリで実行しているかを確認してください。

for (i = 0; i < Ysize; i++) { for (j = 0; j < Xsize; j++) { if (iとjが表す位置が正方形の内部である) { iとjが表す位置のピクセル値を255にする; } else { iとjが表す位置のピクセル値を0にする; } } } という感じのプログラムにするといいと思います。この中でわからない部分はありますか？ char OutFile[200]; strcpy(OutFile, argv[1]); この部分は、argv[1]を直接ファイル名として使えばいいのに、無駄にデータのコピーを行い、無駄にバッファオーバーランを起こすリスク...

あいうえおさんが書きました: ↑
4ヶ月前
for文の初期値を2にしてはいけない理由を知りたいです。

学校の先生の教えに反し、試験の時にそうすると不正解とされるから…とかでしょうか？

コードを見ると、画像ファイル中にヘッダやパディングなどは無く1ピクセルを1バイトで表現するようなので、出力サイズを表す変数を追加し、それに従って出力データを確保、設定、書き出しするといいでしょう。また、提示されたコードはC言語ではなくC++ですね。 C言語にするには、newのかわりにmalloc、delete[]のかわりにfreeを使うといいでしょう。 Man page of MALLOC (freeの説明も載っています) #define _CRT_SECURE_NO_WARNINGS #include <stdio.h> #include <stdlib.h> #include <str...

「文字に関する関数」を使ってはいけないというのに注意したいですね。例えば、・書式指定文字列を扱うprintf系やscanf系の関数・文字の入出力を行うgetchar関数、putchar関数、getc関数、fgetc関数、putc関数、fputc関数・文字列の入出力を行うgets関数、puts関数、fgets関数、fputs関数などは使えなそうですね。となると、通常のパソコン環境であれば、入出力はfread関数やfwrite関数を使用するのがいいでしょう。システムコールや、マイコン環境におけるメモリマップドI/Oを使うという選択肢もあるでしょう。さらに、「数学関数や文...

g_pPlayer が NULL かをチェックせずに g_pPlayer->getPos を使ったことで落ちているようなので、
g_pPlayer->getPos など g_pPlayer をデリファレンスする前に必ず g_pPlayer が NULLでないかをチェックし、
NULL だった場合はデリファレンスしないようにするといいかもしれません。

複雑ですぐにはわからなそうなので、とりあえず作った入力データを置いておきます。

コード:

void atari_hantei_jiki_to_tekitama(void) { if (shouhai != 0) { return; } if ((tekitama_x < jiki_x + 20) && (tekitama_x > jiki_x - 20) && (tekitama_y < jiki_y + 20) && ( tekitama_y > jiki_y - 20)) { tekitama_sonzai = 0; HP = HP + 1; } if (HP == 3) { shouhai = -1; } } 当たった時に tekitama_sonzai を0にしているもの...

・printf関数を使うため、 #include <cstdio> を追加します・ POINT構造体の要素はLONGなので、printf関数の仕様に合わせるためにintにキャストします・座標を取得したいだけなのにSetCursorPosをするのは意味不明なので、削除します #include <windows.h> #include <iostream> #include <cstdio> int main() { while(1) { POINT pt; //マウスの現在の座標を取得する GetCursorPos(&pt); printf("X座標%dY座標%d", (int)pt.x,...

呼び出そうとしている関数 S_calc が定義されていないためにエラーになっていると考えられます。
提示されたコードの後ろに

コード:

}

function S_calc(a,b) { return 0; }
function T_calc(a,b,c) { return 0; }
function solve_linear_equations(a) { return 0; }

を追加すると、エラーは出なくなりました。

メニューは出たのですがキーを何も押せずにプログラムが終わってしまいます。 //終了 if(m == 'Q' || 'q'){ break; } この条件式は、「m == 'Q'」と「'q'」のうち1個以上が真である(0でない)とき、真になります。 'q' の値は(少なくともShift_JISおよびUTF-8では)113であり、0ではないので、これは真になります。よって、mの値にかかわらず、条件は真となり、プログラムが終了します。 if(m == 'Q' || m == 'q'){ とするとよいでしょう。また、名前、住所、電話番号をそれぞれ &name[j]、&jusho[j]、&denw...

別のソースファイルからでも使用できる配列を指すポインタを指定すれば、
別のソースファイルからでも使用できるようになると思います。

具体的なうまくいっていないコードなどはありますか？

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

課題は「大文字のアルファベットを入力」のはずなのに、数値の読み書きしかしていない、というのもおかしいですね。

tmsk さんが書きました: ↑
6ヶ月前
コード:
ValueError: could not convert string to float: ' 0.5185022.336320e-001'

これは小数点が2個含まれる不正な文字列が入力に混ざっているために起こったエラーのようですね。
不正な文字列が含まない入力を使うことで、改善できると推測できます。

tmsk さんが書きました: ↑
6ヶ月前
また、入力されたデータの数値を一緒にcsvファイルに書き込むプログラムの書き方がわかりません。

まず具体的にどのような形式で出力するのかを決め、それに合わせたコードを書くのがいいでしょう。

質問なのですが、std::listに抽象クラスを用いることは可能でしょうか。使えるようですね。 #include <iostream> #include <list> #include <memory> struct AbstractClass { virtual ~AbstractClass(){} virtual void printName() const = 0; }; struct ConcleteClass1 : public AbstractClass { void printName() const override { std::cout << "ConcleteClas...

C言語では、配列の添字は0から始まり、宣言時には最大の添字ではなく要素数を指定します。今回の場合、配列 b は float[2] 型の要素を1個しか持たない配列として宣言されているので、 b[1] は確保されておらず、範囲外となります。したがって、 scanf("%f",&b[1][0]); は、値の読み込みに成功した場合、範囲外への書き込みが発生し、周りのデータが破壊される原因となります。質問文で言及されている b[0][1] へのアクセスがコード内に見当たらないので、この行を scanf("%f",&b[0][1]); とするべきであると予測できます。他の修正方法としては、・範囲...

オフトピック

box さんが書きました: ↑
7ヶ月前
ポインター値を%sで出力するって、これは何がしたいんですか？

なお、%sは文字列を出力するものであり、「ポインター値を%sで出力する」ことは普通ではありません。
処理系によってはNULLを%sに渡すと(null)と出力されることがありますが、そうなるとは限りません。
ポインタをprintfで出力するには、void*にキャストして%pを用います。
(CTFでは関数のポインタが格納された場所のポインタを%sに渡して
ポインタをバイナリで出力させるテクニックが使われることがありますが、これはまた別の話…)

for (j = 0; j < 61; j++) { space[i][j] = ' '; } このループを抜けた直後、jの値は61になります。その後 space[i][j] = 0; が実行されますが、space[i]は61要素しか無いので、これは範囲外へのアクセスとなり、エラーの原因になります。 C言語では配列の添字は0から始まること、および配列の宣言時には最大の添字ではなく要素数を指定することに注意し、十分な数の要素を確保しなければいけません。ポインター値を%sで出力するって、これは何がしたいんですか？ %sは文字列(ナル文字で終端された文字の配列)の先頭へのポインタをとり、...

void CheckHit::CollisionAll()関数にあるfor(auto enemy : _list)の部分でエラーがでてしまい、デバッグが出来ない状態です。該当部分で使われている変数？ _list の宣言が提示されたコードの中にありません。どのようなエラーが出ますか？他クラスのstd::listにアクセスする方法があるのでしたら教えていただきたいです。「他クラス」でstd::listにアクセスする方法を作って(アクセサの関数を作るとか、publicにするとか)、それを利用し、「他クラス」の名前(staticの場合)またはインスタンス(staticでない場合)を指定して...

「assertに引っかからなければOK」なんですよね？ /** * @desc BookListの中で，出版社の本を数える * @param (bookList) BookList * @param (publisher) 出版社 * @return 指定した出版社の本の冊数 */ int countBookByPublisher(BookList* bookList, const char* publisher) { /* [宿題] ここを実装する */ void exit(int status); /* 標準ライブラリ関数exitの宣言 */ exit(0); /* assertが呼ばれる...

さらに、pop 関数内において、値を移動するだけで「余計な要素を削除する」処理が抜けているようですね。

あたっしゅさんが書きました: ↑
7ヶ月前
コード:
        cout << i << ":" << p << endl;

コンパイルエラーになりました。
p の代わりに、 *p とするべきではないでしょうか？

Heap_Max においても、 Heap_Min においても、pop 関数内において

コード:

            if (heap[child1] <= x)
                break;

という処理があります。
この2個のクラスでは比較が反対になるはずなのに、これは不自然ですね。

あたっしゅさんが書きました: ↑
7ヶ月前
STL 使うのなら、ありもん使えばいいみみ。

同意します。
std::priority_queue が使えるでしょう。

ＤＸライブラリは静的リンクなので、(他に依存しているランタイムが入っているなど対応した環境ならば)
ＤＸライブラリをダウンロードしていないPCでも実行できるはずです。
「ＤＸライブラリをダウンロードしていないPCでは実行してはいけない」的なライセンスの条件も無いはずです。

実際に「実行しても大丈夫」かどうかは、プログラムの内容などによるでしょう。
(例えば、Cドライブの内容を全部消そうとするプログラムを実行するのは危険でしょう)

質問はありますか？
フォーラムルールを読み、それに沿った投稿をお願いします。

とりあえず、変数 wheel_cnt が、宣言されているだけで全く使用されていない。 MouseInput::GetStateMouse がせっかく int を返すのに、GetMousePoint のエラー処理がされていない。 leftPrevState = leftState; および rightPrevState = rightState; が、if文の両方の節に書かれている。 prevCurX や prevCurY の処理と同様、前に出した方が良さそう。 leftState や rightState のインクリメント時のオーバーフローチェックが無い。多分20億フレームも連続して使うや...

どのように解決したかを書いてください。

フォーラムルールより

トピックを立て、解決した場合は「解決しました」とだけ書かず、どうやって解決したか他の人に分かるように書いて　からトピックを終了して下さい。

具体的にどこで詰まったかがわからないので、とりあえずPythonで書いてみました。あとはこれを移植すればいいでしょう。 # coding: utf-8 import sys def solve(suuzi): nen = int(suuzi[0:4]) tuki = int(suuzi[4:6]) if nen < 2019 or (nen == 2019 and tuki < 5): gengou = "平成" wareki_nen = nen - 2019 + 31 else: gengou = "令和" wareki_nen = nen - 2019 + 1 if wareki_nen...

・それぞれの配列を出力しきるまで別の配列を出力しないようにする・初期化していない部分の値を使わないようにするなどに注意して、出力がイメージ通りになるように直せばいいと思います。 #include <iostream> #include <iomanip> #include <string> using std::cin; using std::cout; using std::setw; using std::string; // 表示を開始するべき桁 (0でない最初の桁、または(全部0の場合)最後の桁) の位置を返す int getDisplayStartPoint(const int ...

課題の丸投げは禁止です。フォーラムルールを読んで従ってください。 a. 質問は具体的にする詳しくは「回答者が困る質問例」をご覧下さい「うまくいきません」という質問は大抵回答に困ります。自分は今何がしたくてどう取り組んで(作ったプログラムはどれでどのようなエラーやトラブルで困っていて自分は何が解らないのか、知りたいのか今のＣの知識はどの程度なのかこの５点をしっかりと明記して下さい。 c. 宿題の質問は OK! でも丸投げは NG! 宿題の文章だけ書いて「誰か答え教えて下さい」は× 課題の丸投げ(問題文だけ書く事)は禁止です。ただし上のように記載してもらえればこれは当てはまり...

CheckSoundMem関数はエラーの時は -1 を返します。
-1 は、条件式として使った場合は真とみなされます。
削除後のサウンドハンドルを渡すと、エラーになると予想できます。

コード:

		if (CheckSoundMem(_images[i]) == 1) {

として、「再生中」の時のみハンドルを返すようにするとどうでしょうか？

その戻り値をStopSoundMem(ImageMusic::getIns() -> getMusic) みたいな感じで引数に用いればうまくいくと思ったのですがダメでした。「みたいな感じ」って具体的にどのような感じなのでしょうか？ getMusic関数を呼ばず、関数のポインタをStopSoundMemに渡す感じ、ということですか？「ダメでした」「うまく機能してくれませんでした」とは、具体的にどううまく行かなかったのですか？ (デタラメな値が帰る、帰る値はそれっぽいけど再生の停止ができない、など) また、このgetMusicの実装だと、_images の要素に再生中の音声が無い場合に未初...

ジェネリック型についてはよくわからないですが、とりあえずコンパイルが通るようにしてみました。・ std::function を使うため、ヘッダ functional を include する・HogeTest のコンストラクタ内の文の最後にセミコロンを補う・HogeTest のコンストラクタとデストラクタをpublicにする・HogeTest にメンバ hoge が無いため、追加する・main関数の返り値の型をvoidではなくintにする・Filter 関数に渡す第一引数をstd::vectorへのポインタではなくstd::vectorにする・ForEach が定義されていないの...

GameScene::update() では if (Keyboard::getIns()->getPressingCount(KEY_INPUT_ESCAPE) == 1) { というように1と比較しているのに対し、MenuScene::update()では if (Keyboard::getIns()->getPressingCount(KEY_INPUT_ESCAPE)) { と、0以外の任意の値の時に処理を行うようにしていますね。 Keyboard::getIns()->getPressingCount の仕様がわからないので確実ではないですが、 MenuSceneでもGameScen...

関数

コード:

bool PlayerOptionCollection::update()

内の行

コード:

void move();

はmove関数を宣言しているだけであり、move関数の呼び出しは行われません。
move関数を呼び出したければ、

コード:

move();

とするべきです。

ちなみにつっこまれていました配列の要素数が25だったりDefine::PBULLET_NUMだったりするのは、const static intで宣言すると配列の定義の際、要素数に用いることができなかったためです。 const static int で宣言しても要素数に使えそうですが、どのようなエラーになりましたか？ namespace Define { const static int a = 4; const static int b = 100; }; void x(){ int c[Define::a][Define::b / 4]; } PlayerOptionクラス(ソースは省きま...

star_main 関数において、 if (count % 60 == 0) //60カウントに一回 star_enter(); //発射としていますが、 count を更新している部分が無いので、 count は明示的に初期化されていないグローバル変数の初期値の 0 のまま変化せず、 0 を 60 で割った余りは 0 なので、 star_enter 関数が毎回呼び出されます。さらに、start_enter 関数において、 srand((unsigned)time(NULL)); を毎回呼び出しており、 time 関数の戻り値は秒単位の環境が多いため、短い間隔で呼び出すと rand 関数...

まず訂正をします。 100個のデータから生成されるデータは、1001個ではなく991個でした。 (最初のデータはそのまま使い、残り99個のデータ1個につき10個のデータが生成されると考えられるので) すると、1000個のデータを得るには残りの9個をどうやって生成するのかを考える必要があります。そして、991個のデータを生成する部分を一般化してみました。後は、1000個ではなく991個でよいのであれば、入出力と間の値を計算する部分の仕様を決め、実装すれば、プログラムができるでしょう。 #include <stdio.h> #include <stdlib.h> #include <stri...

つまり，100個のデータだとしたら，それぞれの間で等分割して1000個のデータに変換したいです．これは無理そうですね。それぞれの区間を10分割すると、100個のデータからは1001個のデータが生成されます。したがって、「等分割」「100個のデータを1000個のデータに変換」という条件を同時に満たすのは無理と考えられます。ただし、データの最後が例示されていないため、最後のデータを省くことで1000個にする、という選択肢はあります。 1001個でよければ、データの分割は以下のようにできます。 #include <stdio.h> int main(void) { double dataA...

c言語で構造体に1から順にナンバーをつけて、それをソートしてからまた、1からつけなおすにはどうすれば良いんでしょうか？希望の処理を行うコードを書き、実行すればいいです。初期化のための関数を作り、代入してからfor構文を使う感じでしょうか？自分の書きやすい形、そしてできれば後日の修正や改良がしやすい(保守性が高い)形で書けばいいでしょう。自分がしたいのは例えば下のような用意してある構造体をあいうえお順に並び替えて②①③にしてから、それを新しく上から①②③に書き換える関数を作りたいのですがどうすればいいのでしょうか？ 1. あいうえお順に並び替える 2. 番号を書き換えるという処理を行...

hanoha1631 さんが書きました: ↑
9ヶ月前
これ以上間違いを見つけることができません。
指摘してくれると助かります...

とりあえずすぐにわかるのは

・i=0 で範囲外の s1[i-1] や、j=0で範囲外の s2[j-1] にアクセスしている
・strcmp で比較するデータに終端のナル文字を入れていない
・変数 i を用いたループの内側にまた同じ変数 i を用いたループがあるので、
　内側のループによって変数 i の値が破壊されて外側のループがうまく回らない

あたりですね。

boxさんのサンプルをもとに、aiueoとiueoaのようなローテートした文字列だけを一致とするようにしてみました。 1. 長さが違っていたら、不一致とする 2. 一方の文字列を2個くっつける 3. もう一方の文字列が、2個くっつけた文字列の中に入っているかを判定する #include <stdio.h> #include <string.h> #define N (128) void getData(char s[2][N]) { int i; for (i = 0; i < 2; i++) { printf("%dつめ＞", i + 1); fgets(s[i], N, stdin); s...

This question already has answers here: c - Crash or "segmentation fault" when data is copied/scanned/read to an uninitialized pointer - Stack Overflow ポインタ str を初期化せずにデリファレンスしてしまっているのが誤りです。有効なバッファのアドレスに初期化してから入力の読み込みや処理をしないといけません。また、バッファオーバーランを防ぐため、%s で文字列を読み込む時は読み込む最大の長さを指定するべきです。この最大の長さは、最後にナ...

みけCAT さんが書きました: ↑
9ヶ月前
まずはコンパイラ様に教えてもらいましょう。

ごめんなさい、リンク先間違いです。
(編集の制限時間ギリギリでリンクが抜けていることに気がついたのででよく確認せずにリンクを貼り、
そのまま時間切れになってしまいました)
正しくは https://wandbox.org/permlink/rcRUewltC5f1jSwk

ソースコードを提示する際は、BBCodeが有効な(無効にしない)状態で、 BBCodeのcodeタグの開始タグと終了タグの組(開始タグが先)で囲んでいただけると、見やすくてありがたいです。他の回答者さんへ：掲示板の配慮を欠いた設計により、[i]が終了タグが無いにも関わらず斜体のタグと解釈され、隠されているようです。コードは引用画面から読むのが良いです。どこが間違っているのか教えて欲しいです。まずはコンパイラ様に教えてもらいましょう。 prog.c: In function 'main': prog.c:39:5: error: array size missing in ...

コード:

int maxof(int n,a[n]) {

ここで第2引数の型名が抜けています。

コード:

int maxof(int n,int a[]) {

としましょう。
ついでに、

コード:

int minof(int n,int b[n]) {

も宣言に合わせて

コード:

int minof(int n,int b[]) {

としましょう。

h1j1k1 さんが書きました: ↑
9ヶ月前
以下のinclude文を悪用(?)した形は許されますか?

コーディングルールなどによるでしょう。

h1j1k1 さんが書きました: ↑
9ヶ月前
コード:
char credit_list[][] = {
#include "credits.csv"
};

このコードは、配列の要素の型が不完全なのでコンパイルエラーになるでしょう。

コード:

char credit_list[][5] = {
#include "credits.csv"
};

ならコンパイルは通るでしょう。

make_unique さんが書きました: ↑
9ヶ月前
私がやりたいのは円柱の丸い面の上で球のコリジョンを持ったキャラクターが戦う
というものなんですけどその判定がいるのですけどどうしましょう？

どう実装するかを考えて実装すればいいと思います。

・円柱は何個あるのか？
・円柱は動くのか？
・「戦う」とは具体的にどのような動作か？
・キャラクターの可能な動きは？ (たとえばジャンプはできる？)

などによって、具体的な適切な実装は違ってきそうだと思います。

usaoさんの言うように、円と長方形の当たり判定に帰着させる方法が良さそうですね。まず、点と直線と平面と交点と距離に載っている方法で、球の中心点から円柱の中心軸に垂線を下ろした時の交点を求めます。すると、この交点を原点、円柱の中心軸をx軸、交点と球の中心点を通る直線をy軸 (軸の向きは適当に) とした平面を考えることができ、その平面における球の中心点、および円柱の2個の底面と中心軸の交点の座標 (≒距離) がわかるはずです。あとは、これらの座標と、球や円柱の半径を用いて、当たり判定をする円と長方形の座標が求まります。 mixcpp-20210809.png 円と長方形の当たり判定は...

変換せずにDrawFormatString関数で直接描画するのではダメですか？

逆に五体倒しても画面外に行ってもEND画面に行きません。どうしたらよいでしょうか。デバッグをすればいいと思います。敵が弾に当たった時に加え、敵が画面外に出たときにも、「該当の敵を消して Tekikazu に1を足す」という処理をするようにするといいかもしれません。 if (Teki1Y[j] > WORLD_HEIGHT && Teki1Y[j] < -65 && Teki1X[j] > WORLD_WIDTH && Teki1X[j] < 0) { Tekikazu += 1; } この条件式だと、「WORLD_HEIGHT (正の数) より大きく、かつ-65未満」な数や、「WOR...

どうして敵を描画するごとにTekikazuを１プラスするのか、説明できますか？

敵を倒しきったかの判定なら、敵を倒した、
すなわち弾と敵が接触して敵が消えた時にプラスした方がいいような気がします。

あたっしゅさんが書きました: ↑
9ヶ月前
　FILE* が引数なら、
コード:
FILE* fp = fopen( "filename", "r" );
tap_manage_open( fp );
とか、するのかみみ ?

tap_manage_open の引数は FILE ** なので、「FILE* が引数なら」という前提は成り立ちません。
したがって、この予想は正しいが無意味であると考えられます。

この時のmanage get , open, close に入る引数がわかりません。どなたか教えてくださいませんか。よろしくお願いします - プログラマ専用SNS ミクプラの続きですね。規約には明示されていないことですが、同じ内容のトピックを複数立てるのは、情報の分散に繋がるので個人的にはよくないと思います。また、このトピックで指摘させていただいた、「manage get,manage open, manage close」の仕様や目的についても書かれていないようですね。 2枚目の512,513,514の宣言は気にしないでください。どうしてですか？回答に繋がる重要ポイントかもし...

フォーラムルールを読んでください。
コードの貼り方も書いてあります。

1. 自機の弾や敵と同様に、「爆発アニメーション」の状態を監理する配列群を用意する　(存在するか、座標、アニメーションの位置を監理する) 2. 爆発した時、自機の弾や敵と同様に「爆発アニメーション」を出す 3. 存在する「爆発アニメーション」について、状態の更新と描画を行うというようにするといいと思います。 #define WORLD_WIDTH 640 #define WORLD_HEIGHT 480 #define BOMB_MAX 200 #include "DxLib.h" int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPre...

ソースコードを提示する際は、BBCodeが有効な(無効にしない)状態で、 BBCodeのcodeタグの開始タグと終了タグの組(開始タグが先)で囲んでいただけると、見やすくてありがたいです。余計な空白が入った [ code ] や [ /code ] は、BBCodeのcodeタグではありません。この時のmanage get , open, close に入る引数がわかりません。どなたか教えてくださいませんか。よろしくお願いしますこれらの関数の仕様も目的もわからないので断定はできませんが、 p_open → 適当なFILE*型変数へのポインタ p_close → p_openにポインタ...

・pの値を0で固定しており、最初の要素以外の弾についての判定をしていない・敵と弾の両方が「存在する」とき判定をするべきなのに、「存在しない」とき判定をするようになっている・敵と弾の存在(しない)判定の通過後、接触していないと判定した時、不適切にbreak;で判定処理を打ち切っている・敵と弾の存在(しない)判定の通過後、接触していると判定したとき、iの値を不適切に設定している ↓直したコード for (i = 0; i < Tama; i++) { // 自機の弾iの移動ルーチン( 存在状態を保持している変数の内容が１(存在する)の場合のみ行う ) if (JikiTamaFlag[i] ...

前言われた配列を走査する、というのがまだ理解できていなくて…どういう意味なのでしょうか配列の要素を前から1個ずつ処理していく、という意味です。配列走査 - Google 検索みけCATさんに言われたとおりに「配列を走破し・・・」ていうのを前と同じやり方でやってみたのですがループ後に条件を満たす要素が見つかったかをチェックし、見つかっていたらそこに対して処理を行う、という処理が無いため、同じやり方ではないですね。弾を撃つとフリーズしてしまいます for (i = 0; i < Tama; i++) のループの内側で、変数 i の値を不適切に書き換えているため、特殊な場合以外は無...

		5月 2022
日	月	火	水	木	金	土
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	31

検索結果 6504 件