【初心者歓迎】C/C++室 Ver.106【環境依存OK】

**デフォルトの名無しさん** · 2020/07/13(月) 13:51:48.09

エスケープシーケンスやWin32APIなどの環境依存なものもOK
そのような質問は必ず環境を書きましょう
半角空白やタブでのインデントはスレに貼ると無くなります

コードを貼れる所
http://codepad.org/
https://ideone.com/

前スレ
【初心者歓迎】C/C++室 Ver.105【環境依存OK】
https://mevius.5ch.net/test/read.cgi/tech/1556142878/

**デフォルトの名無しさん** · 2020/10/14(水) 20:28:11.47

>>237
+1

**蟻人間** ◆T6xkBnTXz7B0 · 2020/10/14(水) 20:29:17.83

>>238
#ifdef マクロ

**デフォルトの名無しさん** · 2020/10/22(木) 12:14:45.82

C++を使ってシューティングゲームを作ろうとしているものです。
https://bituse.info/game/shot/
こちらのサイトを参考にしてＤＸライブラリも使って作業をしているのですが
https://bituse.info/game/shot/5
ただいまこちらの章でエラーが出てしまい行き詰っています。

背景を線画する章なのですがサイトの指示通りに新しいクラスを作って実行すると
1>------ ビルド開始: プロジェクト: gamegame, 構成: Release Win32 ------
1>back.cpp
1>C:\Users\rikua\source\repos\gamegame\back.h(17,7): error C3861: 'LoadGraph': 識別子が見つかりませんでした
1>C:\Users\rikua\source\repos\gamegame\back.h(19,10): error C2065: 'MARGIN': 定義されていない識別子です。
1>C:\Users\rikua\source\repos\gamegame\back.h(23,22): error C2065: 'FALSE': 定義されていない識別子です。
1>C:\Users\rikua\source\repos\gamegame\back.h(23,2): error C3861: 'DrawGraph': 識別子が見つかりませんでした
1>control.cpp
1>main.cpp
1>プロジェクト "gamegame.vcxproj" のビルドが終了しました -- 失敗。
========== ビルド: 0 正常終了、1 失敗、0 更新不要、0 スキップ ==========

このようなエラーが出てしまいます。

こちらが現在作成しているファイルの全てです。
http://whitecats.dip.jp/up/download/1603336402/attach/1603336402.zip
パスワード1234
どなたか分かる方お願いします・・

**デフォルトの名無しさん** · 2020/10/22(木) 12:38:26.86

すまん
398MBとか見てそっ閉じしたわ・・・
各ヘッダが何をincludeしてるのか構造見てみて

**デフォルトの名無しさん** · 2020/10/22(木) 12:47:21.09

他人だけど1234で削除したら削除出来ちゃったωωω

**デフォルトの名無しさん** · 2020/10/22(木) 12:48:24.95

>>242
申し訳ありません！
ヘッダは今全部で５つありまして
back.h
control.h
define.h
pch.h
player.h
の5つです

back.h
includeなし

control.h
include "player.h"
include "back.h"

define.h
include <windows.h>

pch.h
include "DxLib.h"
include "define.h"

player.h
#include "pch.h"
#include "player.h"

それぞれは今このような形になっています

**デフォルトの名無しさん** · 2020/10/22(木) 12:49:30.64

最後の
#include "pch.h"
#include "player.h"
こちらの＃は貼るときに消し忘れなので気にしないでください

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/22(木) 12:50:25.91

>>241
質問のクロスポストは好ましくないので避けてね。
https://teratail.com/questions/299584

**デフォルトの名無しさん** · 2020/10/22(木) 12:55:08.56

>>246
申し訳ありません！以後気を付けます

**デフォルトの名無しさん** · 2020/10/22(木) 14:04:28.09

>>244
back.hにpch.hを追加で
player.hのplayer.hはいらない
でやってみて

**デフォルトの名無しさん** · 2020/10/22(木) 14:07:55.53

hoge.hファイルに#include "hoge.h"ってできたんか・・・

**デフォルトの名無しさん** · 2020/10/22(木) 14:25:21.78

>>248
帰宅したらすぐに試してみます！ありがとうございます！！

**デフォルトの名無しさん** · 2020/10/22(木) 16:15:13.07

>>249
コンパイルエラー選手権でそんなんあったな

**デフォルトの名無しさん** · 2020/10/22(木) 16:24:16.18

>>248
すみません！
player.hの内容は元々includeが一つもありませんでした！
なのでback.hにpch.hを追加して実行してみると
このようなエラーが出てきました

1>------ ビルド開始: プロジェクト: gamegame, 構成: Release Win32 ------
1>back.cpp
1>control.cpp
1>C:\Users\rikua\source\repos\gamegame\define.h(16,13): error C2011: 'SHOT': 'struct' 型の再定義
1>C:\Users\rikua\source\repos\gamegame\define.h(16): message : 'SHOT' の宣言を確認してください
1>main.cpp
1>C:\Users\rikua\source\repos\gamegame\define.h(16,13): error C2011: 'SHOT': 'struct' 型の再定義
1>C:\Users\rikua\source\repos\gamegame\define.h(16): message : 'SHOT' の宣言を確認してください
1>player.cpp
1>プロジェクト "gamegame.vcxproj" のビルドが終了しました -- 失敗。
========== ビルド: 0 正常終了、1 失敗、0 更新不要、0 スキップ ==========

**デフォルトの名無しさん** · 2020/10/22(木) 16:42:21.53

>>252
すみません先ほどのエラーは
違ったものでした

こちらが現在出ているエラーです

>------ ビルド開始: プロジェクト: gamegame, 構成: Release Win32 ------
1>back.cpp
1>control.cpp
1>main.cpp
1>pch.cpp
1>player.cpp
1>control.obj : error LNK2005: "public: void __thiscall BACK::All(void)" (?All@BACK@@QAEXXZ) は既に back.obj で定義されています。
1>control.obj : error LNK2005: "private: void __thiscall BACK::Draw(void)" (?Draw@BACK@@AAEXXZ) は既に back.obj で定義されています。
1>control.obj : error LNK2005: "public: __thiscall BACK::BACK(void)" (??0BACK@@QAE@XZ) は既に back.obj で定義されています。
1>main.obj : error LNK2005: "public: void __thiscall BACK::All(void)" (?All@BACK@@QAEXXZ) は既に back.obj で定義されています。
1>main.obj : error LNK2005: "private: void __thiscall BACK::Draw(void)" (?Draw@BACK@@AAEXXZ) は既に back.obj で定義されています。
1>main.obj : error LNK2005: "public: __thiscall BACK::BACK(void)" (??0BACK@@QAE@XZ) は既に back.obj で定義されています。
1>main.obj : error LNK2005: "public: void __thiscall PLAYER::All(void)" (?All@PLAYER@@QAEXXZ) は既に control.obj で定義されています。
1>player.obj : error LNK2005: "public: void __thiscall PLAYER::All(void)" (?All@PLAYER@@QAEXXZ) は既に control.obj で定義されています。
1>C:\Users\rikua\source\repos\gamegame\Release\gamegame.exe : fatal error LNK1169: 1 つ以上の複数回定義されているシンボルが見つかりました。
1>プロジェクト "gamegame.vcxproj" のビルドが終了しました -- 失敗。
========== ビルド: 0 正常終了、1 失敗、0 更新不要、0 スキップ ==========

**デフォルトの名無しさん** · 2020/10/22(木) 16:43:57.19

>>252
エラーが出る度に頭空っぽのまま教えてくださいって繰り返すのか？
teratailで指摘されてる通り、まずは#includeとかヘッダファイルがどういうもので何のためにあるのか、何故エラーになるのか、解決するにはどうするか、入門サイトなり入門書なりで勉強してきなよ。
それで分からないことが出てきたら、またここで具体的な質問をすれば誰かが回答してくれると思うぞ。

**デフォルトの名無しさん** · 2020/10/22(木) 16:48:32.19

たびたびすみません！
control.hの中にお手本にはない同じ文列が２つあったので一つ消すと
エラーがだいぶ減りました！

1>------ ビルド開始: プロジェクト: gamegame, 構成: Release Win32 ------
1>control.cpp
1>C:\Users\rikua\source\repos\gamegame\control.cpp(11,1): error C2600: 'CONTROL::~CONTROL': コンパイラで生成された特殊メンバー関数を定義できません (クラスで最初に宣言されなければなりません)
1>プロジェクト "gamegame.vcxproj" のビルドが終了しました -- 失敗。
========== ビルド: 0 正常終了、1 失敗、0 更新不要、0 スキップ ==========

**デフォルトの名無しさん** · 2020/10/22(木) 16:50:27.84

>>254
ありがとうございます！
もう少し勉強してみてまた質問したいと思います。

**デフォルトの名無しさん** · 2020/10/22(木) 16:51:09.80

ヘッダのインクルード順に依存するようなプログラムになってるとか？

**デフォルトの名無しさん** · 2020/10/22(木) 17:07:43.70

C言語、C++の基本のヘッダファイルの役割は理解していますか？
もし判らないのであれば、そちらの勉強を先にした方がいいと思います。
参考にしているサイトも、
> このページで学習するには、
> C言語とC++についての知識がある程度必要になります。
> 心配な方はトップページなどから、該当言語のページを選んで基礎を学んできてください。
と前置きがありますよ。
基本を理解せずに始めても、何度も同じような質問を繰り返して遠回りになるだけです。

>>246
有賀豚

**デフォルトの名無しさん** · 2020/10/22(木) 17:08:41.93

>>253 の原因は
player.c の中に back.c の中身がそのままコピーで繰り返されてるから

**デフォルトの名無しさん** · 2020/10/22(木) 17:09:40.03

>>257
それ以前の問題
こんなソース描いてる香具師が同じプロジェクトに居たら殴る

**デフォルトの名無しさん** · 2020/10/22(木) 23:26:21.71

初心者歓迎のスレでそんなきつい言い方するな

**デフォルトの名無しさん** · 2020/10/23(金) 00:01:21.09

引っかかってるところよりずっと前で理解してないパターンは面倒くさい。
前提部分からの説明が必要だがそれを学ぶ気がないからこそこういう質問になるわけで、
親切に答えても徒労に終わるパターンなのが見えてる。

**デフォルトの名無しさん** · 2020/10/23(金) 04:26:48.30

Java から C++ に流れてきた者ですが、Javaのインターフェース的なものは
多重継承で実現、ですかね?

class Rectangle : pubilc Polygon {...}; みたいなクラス（他にも Polygon を拡張した
Triangle とか Hexagon とかあるとします）で面積を返す area() の実装を要請して、
そういうオブジェクトだけに適用できるメソッドを定義したい場合、
class Area { virtual int area() = 0; }
class Rectangle : public Sharpe, public Area { int area(); ..}; 的な?
その後 doSomething(Area& area); といった呼び出しでこれらのオブジェクトを引数に?

この場合 Polygon に area() を追加する手もありますが、Polygon でない図形のクラス
でも area() の実装を要請する場合もあるならば、多重継承ということになるのかなと。
多重継承以外でも何かありますでしょうか。

**デフォルトの名無しさん** · 2020/10/23(金) 10:02:28.57

>>263
仮想関数は動的ポリモーフィズムのためにある。
(最適化で消えることもあるけど) 実行時にディスパッチする仕組みなので、
使わないで済むならその方がいい。

つまり、その例なら doSomething をテンプレートにして型に制約 (メタ関数かコンセプトで) を付ける方が好ましい。

**デフォルトの名無しさん** · 2020/10/23(金) 11:00:50.00

>>263
あまり深く考えずにセオリーを真似るだけ

More C++ Idioms/インタフェースクラス(Interface Class) - Wikibooks
https://ja.wikibooks.org/wiki/More_C%2B%2B_Idioms/%E3%82%A4%E3%83%B3%E3%82%BF%E3%83%95%E3%82%A7%E3%83%BC%E3%82%B9%E3%82%AF%E3%83%A9%E3%82%B9(Interface_Class)

**デフォルトの名無しさん** · 2020/10/23(金) 17:26:34.17

>>262
>学ぶ気がない
>親切に答えても徒労に終わる

触っちゃいけない質問者って臭いで判るよな

**デフォルトの名無しさん** · 2020/10/23(金) 17:36:09.55

いきなりどこかのサイトのソース持ってきてコンパイルできません系が怖い

**デフォルトの名無しさん** · 2020/10/24(土) 01:05:43.17

オンラインゲームとかでも初心者狩りとかしてそうｗ
学ぶ気がないとか他人に原因を押し付けてるけど

**デフォルトの名無しさん** · 2020/10/24(土) 02:31:27.76

じゃあこのスレでインクルードやら宣言やらの仕組みから解説しろってのかい？
それはさすがに入門書のひとつでも読んでくれなきゃ困るよ。

単純に説明の分量的にもね。

**デフォルトの名無しさん** · 2020/10/24(土) 06:43:17.73

別にインじゃねえの
答えたくなきゃ答えなきゃいいだけだし

**デフォルトの名無しさん** · 2020/10/24(土) 09:29:17.11

マジでそれ
答えたくないならスルーすればいいのに余計な事を言うのが害悪

**デフォルトの名無しさん** · 2020/10/24(土) 09:38:28.92

>>264
テンプレートですか、C++っぽいですね。メタ関数とか、いろいろ自分には新しいトピックが。
area() を実装しているかどうかは... is_callable とかでしょうか。
最悪制約をつけなくても area() がなければコンパイル時にテンプレート展開でエラーに
なって、駄目なことはわかることはわかるんですよね?

>>265
それはルートクラスの設計の話に見えるのですが、今回の場合はもう既に
あるクラスの場合の話でして。その場合はインターフェースクラスを多重継承する
のかな? というのが元の質問です。

**デフォルトの名無しさん** · 2020/10/24(土) 10:35:52.36

>>272
まあ仮想関数使って多重継承でいいんじゃね
機能的には必要十分かと

それより設計上の問題として
Polygon : public Area｛｝
という構造が適切なのかどうなのか疑問

分かりやすさとしては
Rectangle : public Polygon｛｝,Triangle: public Polygon｛｝・・・
という単なる形状のみの継承クラスと
Graph ｛Polygon poly; int area();・・・｝
という別の目的で機能するクラスに分けたほうが整理されてる気がする

**デフォルトの名無しさん** · 2020/10/24(土) 11:55:33.45

>>272
制約を付けなくてもテンプレートを展開してダメだったらエラーになることは保証される。
まあ当たり前っていうかそうせざるを得ないもんね。

でも「(制約を満たさないので) マッチに失敗した」と「マッチして展開した結果にエラーがあった」は意味が違っていて、
結果的に失敗させることを意図するならなるべく前者で失敗させた方が使い勝手はいい。

**デフォルトの名無しさん** · 2020/10/24(土) 15:57:20.05

答えたくなきゃ答えなきゃいいし
余計な事を言いたきゃ言えばいいよ
ここをどこだと思ってんだよｗ

**デフォルトの名無しさん** · 2020/10/24(土) 22:10:51.67

初心者歓迎のスレで初心者叩くなら出てけよ

**デフォルトの名無しさん** · 2020/10/25(日) 08:54:37.38

>>276
は？学ぶ気がないことが露骨なやつは嫌だよねって話をしてるんだろうが。
初心者叩きとか話をすり替えるなよカス

**デフォルトの名無しさん** · 2020/10/25(日) 09:14:07.10

>>276
初心者で一括りにしたら「やる気のある」初心者に失礼

**デフォルトの名無しさん** · 2020/10/25(日) 11:23:24.95

こいつには質問しても得るものないから目合わせないでおこう
と思うやつに限ってこういう反応してくるんだよなｗ
>>277

**デフォルトの名無しさん** · 2020/10/25(日) 11:49:00.98

>>277
>学ぶ気がないことが露骨なやつ

ほんとこれにつきる

**デフォルトの名無しさん** · 2020/10/25(日) 18:00:47.89

リンカーの話とかもここでおkですか?　とりあえずlinux系で、

動的ライブラリlibA.soが、別のライブラりlibB.soに依存しているときには,
libA.so作成時、libA.soにlibB.soをリンクしておけば依存性が解決されますよね?
（libA.soを使うプログラム作成時には、libA.soだけ指定すればよい）

諸般の事情で静的ライブラリlibC.aがあり、これがlibD.soに依存するのですが、
この場合、libC.aにlibD.soをリンクしてlibC.aを生成することはできない? ですか?
libD.soは、libC.aを使ったプログラムの作成時にリンク、ということでおkですかね?
これが正解だと、静的と動的で依存性を解決する場所(?)が違うんだなあと。

**デフォルトの名無しさん** · 2020/10/25(日) 19:43:38.96

ふと思いましたが、前者の場合、libA.soにあえてlibB.soをリンクしないで生成、
プログラムにlibA.soをリンクするときにlibB.soを指定する方法もありますよね。

そうすると依存性的には静的ライブラリと同じになる。
でも普通はあえてそんなリンクにはしないのかな。

**デフォルトの名無しさん** · 2020/10/26(月) 15:01:51.11

libD.a は行方不明か？

**デフォルトの名無しさん** · 2020/10/26(月) 19:57:11.86

しらない🤢

**デフォルトの名無しさん** · 2020/10/26(月) 20:00:25.63

ビタミンCならない

**デフォルトの名無しさん** · 2020/10/27(火) 05:52:06.98

http://codepad.org/qCDIHh55
https://ideone.com/uTuF1W
上二つは正常に見えます
ところが
https://ideone.com/lWvw1n
暴走しますω
なぜでしょう？

**デフォルトの名無しさん** · 2020/10/27(火) 08:29:34.74

virtual int d(){cout<<"Ad\n";}
にreturn文ないのが未定義動作でその時の振る舞いはコンパイラの実装依存なのかな
virtual int d(){cout<<"Ad\n";return 0;}
で正常に動いた

**デフォルトの名無しさん** · 2020/10/27(火) 08:37:11.64

ていうかなんでこの手のコンパイルが通るようになってるんだろうね
そっちが気になるわ

**デフォルトの名無しさん** · 2020/10/27(火) 08:56:02.30

gcc のバグ？それとも ideone のバグ？

**デフォルトの名無しさん** · 2020/10/27(火) 09:14:57.09

http://codepad.org/qCDIHh55
の件で
delete b; は B:d, ~B, A:d, ~A と呼ばれる様ですが
virtual int d() はポリモフィズムで
B:d, ~B, B:d, ~A と呼ばれるのが正しい気がするのですが
デストラクタ内から呼ばれるときは特別なのでしょうか？

**デフォルトの名無しさん** · 2020/10/27(火) 09:57:24.74

>>282
libD.aはないですw libD.soです。

なんというか、静的リンクって面倒だっなって思ったのと、自分の認識が正しいかを
確認したかったのですが。

プログラムにlibA.soをリンクするときは、libA.soだけ指定すればいい。llibB.soの依存性は
リンカーが解決してくれる（libA.soにlibB.soがリンクされているので）。

一方プログラムにlibC.aをリンクするときは、libD.soも指定しないとlibD.so内の関数が
見つからないエラーになる。かといってlibA.soのように依存するライブラリーを前もって
リンクしておくわけにはいかない。

で正しいですよね?

**デフォルトの名無しさん** · 2020/10/27(火) 10:09:47.62

~B の後　ポリモフィズムで実体のない B:d() を呼び出すのは微妙な感じ
B:d()　内部で Bにまつわるメンバを操作を考えると
すでに生命期間切れてるのにアクセスすることにもなる

**デフォルトの名無しさん** · 2020/10/27(火) 10:11:51.64

違いますよ？

**デフォルトの名無しさん** · 2020/10/27(火) 10:42:51.63

>>290
コンストラクタ、デストラクタ内では仮想関数はオーバーライド解決されない、ポリモーフィズムは効かないとはっきり決まっておる。

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/27(火) 10:47:51.31

>>290
抽象クラスを端的に言えば「実行時の型 (実装) で処理する」ための仕組みなのであって、
delete b; は *b の型が B のつもりで解体するってだけ。

**デフォルトの名無しさん** · 2020/10/27(火) 11:47:49.35

はちみつはいつもずれてるな

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/27(火) 12:05:37.93

そうか。

**デフォルトの名無しさん** · 2020/10/27(火) 13:39:09.37

~B::B()の内部で~A::A()が呼び出しされてるだけだから
B::d() -> B::~B() -> A::d() -> A::~A()
で正しいんじゃないの~A::A()を普通に呼び出して
B::d()が呼ばれたら怖くない？

**デフォルトの名無しさん** · 2020/10/27(火) 14:47:50.22

いくつかのクラス (A, B, C) をまとめて class X に対して friend にしたいとき
class X 内で
friend A, B, C;
と描くとエラーになります
friend A;
friend B;
friend C;
と描くしかないですか？

あと前方参照のために
class A, B, C;
みたいに並べるのもだめっぽいのですが
他に良い描きかたあるんですか？

**デフォルトの名無しさん** · 2020/10/27(火) 15:08:50.29

なぜ後者を良くない書き方だと思った？
その書き方で良いじゃん

**デフォルトの名無しさん** · 2020/10/28(水) 09:57:07.45

>>290
>>294
確認出来ました。
http://codepad.org/bmCXYMNU
output の 9行目が Ad なので確信です。

**デフォルトの名無しさん** · 2020/10/28(水) 10:12:29.14

僕は友達いないのですがfriend 0(;´Д⊂)と書くべきでしょうか？

**デフォルトの名無しさん** · 2020/10/28(水) 10:35:22.84

0は識別子にできないので0にすら見離されてるな

**デフォルトの名無しさん** · 2020/10/28(水) 11:08:09.05

>>302
あなたが誰かを勝手にfriendと宣言しても問題ありません。
ただ相手があなたにアクセスしないだけなので、現状と何ら変化はありません。

**デフォルトの名無しさん** · 2020/10/28(水) 11:38:33.08

こんなこといいな出来たらいいな
#define A(X, Y, ...) X Y ; A(X, __VA_ARGS__)
A(class, hoge, fuga, hage)
A(friend, hoge, fuga, hage)

**デフォルトの名無しさん** · 2020/10/28(水) 16:32:57.81

>>305
念のため言っとくけど俺はhageじゃないからな

**デフォルトの名無しさん** · 2020/10/28(水) 16:47:05.58

じゃなかったとしても気にしてるのはわかった

**デフォルトの名無しさん** · 2020/10/28(水) 17:31:58.18

piyoにしてくれ…

**デフォルトの名無しさん** · 2020/10/28(水) 19:02:09.12

友達がいないからハゲなのか
ハゲだから友達がいないのか

**デフォルトの名無しさん** · 2020/10/28(水) 19:07:42.92

ボクはgrault！

**デフォルトの名無しさん** · 2020/10/29(木) 09:41:00.08

>>305
一応出来るよ
トークン連結でA_30,A_29とかあらかじめ作っといたループ部分を呼べば、だけど

**デフォルトの名無しさん** · 2020/10/29(木) 12:02:39.93

下記のようにvectorに追加するときにemplace_backを使って初期値を入れるにはどうすればよいでしょうか？

typedef struct aaa_{
char name; // charもしくはstd::stringでも可
uint64_t count;
} aaa;

std::vector<aaa> v_aaa;

char temp[] = "hoge";

// v_aaaの末尾に初期値を入れて追加。イメージ
v_aaa.emplace_back( aaa({name=temp, count=5}) )

**デフォルトの名無しさん** · 2020/10/29(木) 12:11:31.88

>>312
https://ideone.com/1qsBsM

**デフォルトの名無しさん** · 2020/10/29(木) 12:11:50.37

無理じゃね？
structにコンストラクタを用意する訳にはいかんの？

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/29(木) 15:34:56.14

>>312
https://wandbox.org/permlink/URgQ0YQVQrvZTWll

こんな感じ？

**デフォルトの名無しさん** · 2020/10/29(木) 15:38:32.83

暗黙のコピーコンストラクタか…
emplace_backの意味ねー

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/29(木) 16:06:03.93

引数を受け取るところでも RVO の適用対象になるから余計なコピーはたぶんそんなに起こらないはずだと思う。
(ちゃんと検証してないので間違ってたらごめん。)

**デフォルトの名無しさん** · 2020/10/29(木) 20:07:08.57

push_backでいいじゃん

**デフォルトの名無しさん** · 2020/10/29(木) 23:17:56.41

>>315
ありがとうございます。できました。
高速化のためにemplace_backを使っているのですが、もしかすると逆に余計なCOPYが発生している可能性もあるんですね。

**デフォルトの名無しさん** · 2020/10/29(木) 23:30:45.17

もう一つ質問です。

uint16_t num;
std::string str;

という2つの変数があった時、strを数値変換してnumに入れたいのですが、下記はどちらも間違ってはないですか？
どちらも戻り値はintとあったのですが、uint16で受けても問題ないのでしょうか？

num = std::stoi(str);
num = atoi(str.c_str());

**デフォルトの名無しさん** · 2020/10/30(金) 00:49:26.28

問題があるかどうかはstrの仕様次第
uint16_tの範囲外があり得ないなら大丈夫、あり得るなら勿論ダメ
今時のコンパイラなら大きいものから小さいものへの暗黙の変換には警告が出る
警告消したいなら明示castして

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/30(金) 09:22:29.76

>>320
int から uint16_t の変換について

変換後の型で表せない範囲の値を変換しようとしたとき、変換後の型が符号なし整数型である場合には、
変換後の型が格納可能な最大値 + 1 で modulo を取った値になることになってる。
(変換後の型が符号付である場合には未定義。)

結果の大きさが確実に uint16_t の範囲に入るという確信がある状況であればもちろん問題ないし、
そうでないときも結果が未定義ではないという意味では問題ないが、
プログラムの意図に合致するかどうかは状況による。

----

文字列を数値に変換することについて、
atoi はエラーハンドリングがちゃんとできない (変換に失敗したら 0 を返すが変換結果が 0 のときと区別できない) ので、
事前に文字列の検証が済んでいるのでなければ stoi の方が好ましくはある。

**デフォルトの名無しさん** · 2020/10/30(金) 09:49:07.73

>>322
https://timsong-cpp.github.io/cppwp/n4659/conv.integral#3
> If the destination type is signed, the value is unchanged if it can be represented in the destination type; otherwise, the value is implementation-defined.
未定義ではないね。移植性は損なわれるけど。

**デフォルトの名無しさん** · 2020/10/30(金) 11:26:35.44

>>322
>変換後の型で表せない範囲の値を変換しようとしたとき、変換後の型が符号なし整数型である場合には、
>変換後の型が格納可能な最大値 + 1 で modulo を取った値になることになってる。
>(変換後の型が符号付である場合には未定義。)

結果的にそうなってるだけで実際は上位bit(byte)棄ててるだけじゃないの

**デフォルトの名無しさん** · 2020/10/30(金) 12:42:27.15

>>324
結果的にそうならないと規格準拠はうたえないし上位を無視してやるかどうかは実装の話だから処理系による
まあ他にいい方法は思いつかないけど

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/30(金) 13:29:46.50

>>324
言語仕様で保証されている。
その仕様を実現するためにコンパイラがどうやるかは場合によるだろうけど、
単に上位を捨てることで実現しやすいと想定はしてると思う。

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/30(金) 13:31:26.19

>>325
1 の補数を使っているときにマイナスの値が絡んでくるとちょっと調整が必要になるんじゃね？

**デフォルトの名無しさん** · 2020/10/30(金) 14:26:56.09

stoiとか" -1"とか食わせると-1のintになるんでしょ
parserとかゴリゴリ実装してんのかね

**はちみつ餃子** ◆8X2XSCHEME · 2020/10/30(金) 14:36:46.44

>>328
扱えるのは整数だけだし、マイナスの他には 0 か 0x が付く場合を処理できればいいのでそんなに複雑ではないと思う。

**デフォルトの名無しさん** · 2020/10/30(金) 14:48:32.14

なるほどね

**デフォルトの名無しさん** · 2020/10/31(土) 22:42:02.96

https://ideone.com/o3N4x1

上記の44行目の「ここから～」と70行目の「ここまで」の部分で変数p_aaa、p_bbbのstr変数を書き換えるという同等の処理をしているのですが、
実際のここの部分は何十行もあるので、メンテナンス性を良くするために、この共通部分を共有できないかと考えています。
関数化すればと思いましたが、引数の構造体が別物のため関数化できません。
何か方法はありますでしょうか？

**デフォルトの名無しさん** · 2020/10/31(土) 22:51:08.31

unsigned char* common(unsigned char* arg)
{

**デフォルトの名無しさん** · 2020/10/31(土) 22:54:20.54

送信しちゃった
unsigned char* を受けて realloc し、 reallocの戻りの [9][10][11] を書き込みしつつ
reallocの戻りを返す (失敗は exit) そういう関数　common() で
　p_aaa->str = common(p_aaa->str);
　p_bbb->str = common(p_bbb->str);
こうまとめる感じ

**デフォルトの名無しさん** · 2020/10/31(土) 23:03:36.76

ideone のコードがあくまで str の操作に終始してるので >>333 になる
共通のコードが type についても同じ操作するのなら

str, type だけを単発の構造体 BASE にして AAA, BBB は BASE を継承しておく
で、関数は BASE を受ける（必要なら BASE を返す or 参照渡しで書き換え)

**デフォルトの名無しさん** · 2020/10/31(土) 23:18:42.13

offsetofで距離の値をメタ取得しメンバアクセスでなく剥いでchar*にすればいいと思うよ

**デフォルトの名無しさん** · 2020/11/01(日) 10:32:39.62

>>331
テンプレート関数を使って、コードの上では一つの関数に見せかけておくようにする

前準備の箇所も同様

hsgAWF - Online C++0x Compiler & Debugging Tool - Ideone.com
https://ideone.com/hsgAWF

**デフォルトの名無しさん** · 2020/11/01(日) 11:12:58.96

>>301
>>294
どうあがいても無理ポ
http://codepad.org/WgHKMHuJ
http://codepad.org/GYWccmnL

**331** · 2020/11/01(日) 11:33:12.85

>>334
関数にp_aaa->strを渡して値の書き換えを考えたのですが、関数内のreallocでアドレス変わるからその値を変えてもだめだ。
って思っていました。
ご提案の通りアドレス変わったものを戻り値で返してあげればいいですね。
その考えが抜けていました。

>>335
すみません、剥いでchar*にすればいいってところがよくわかりませんでした。。

>>336
今回以外のケースでも使いみちありそうなのでテンプレートについて調べてみました。
関数やclassを定義する先頭に<typename T, typename HOGE>のように可変できる独自の型を作れる。
関数、クラス使用時に関数名のあとに<unsigned char*, int>のように型を指定する。
という理解であってますか？