Ｃ言語複数のソースのコンパイルに関して

Question

前提・実現したいこと

あるソースコードにて指定した位置で別のソースコードの操作を走らせたいです．
現在頭を悩ませているコードを記載するのでこの場合はどう記述すればよいのかをご教授願いたいです．
開発環境はLinux OS　で　vscodeを利用し開発しています．

発生している問題

別のコードを走らせるためにどう記述すればいいのかがわかりません．

該当のソースコード

main.cpp

#include <gtkmm/application.h>
 
#include "MyWindow.h"

int main( ){
    auto app = Gtk::Application::create( );
    ClusteringDataApp::MyWindow myWindow;
     
    return app->run( myWindow );
}

以下コードの保存処理時に
MyWindow.cpp

#include "MyWindow.h"

namespace ClusteringDataApp{
	void MyWindow::on_saveButton_clicked( ){
		const std::string fileName = "newData.txt";
		std::cerr << data_.size( ) << " data is written to " << fileName << "\n";

		auto ofs = std::ofstream( fileName, std::ios::out );
		if( ofs.good( ) )
			for( const auto& pair : data_ )
				ofs << pair.first << "\t" << pair.second << "\n";
		ofs.close( );

		hide( ); 
	}
}

MyWindow.h

#ifndef MYWINDOW_INCLUDED
#define MYWINDOW_INCLUDED
#include <gtkmm/window.h>
#include <gtkmm/button.h> 
#include <gtkmm/hvbox.h> 
#include <gtkmm/filechooserdialog.h>
#include <gtkmm/stock.h>
#include <gtkmm/filefilter.h>
#include <giomm.h>

#include <iostream>
#include <vector>
#include <utility> // std::pair
#include <fstream>
#include <string>
#include <sstream>

#endif

以下のコードを走らせた後に保存処理が実行されるようにしたい．

#include <iostream>
#include <cstdlib>
#include <cmath>
#include <vector>
#include <utility>
#include <algorithm>
#include <random>

#include <Eigen/Dense>

template <typename T = double >
using dataType = std::vector< std::pair<T, T> >;

void showAll( dataType<>& );
void normalize( dataType<>& );

int main(){...
}

void showAll( dataType<>& data ){...
}

void normalize( dataType<>& data ){...
}

補足情報

譲り受けたコードをいじっていて，別で行いたい処理を記述したコードを作成したので
譲り受けたコードに搭載させることはできないかと考えたためです．
ご教授の方よろしくお願いいたします．

Comments

[asm]

なぜ、別ソースに分ける必要があるのでしょうか？　また、別ソースの方がmain関数である必要がありますか？ 最後に、動作環境(OS)は何を想定していますか？

Answer 1

まず

#include <stdio.h>
int main(void)
{
    c *= -1.0;
}

ですが、そもそもcというのが定義されていないのでだめですね。

さて、２つのソースコードを見るとmain関数が双方にあります。つまりプログラムを2つ作ることを意図しているのでしょうか？すると「ここで別ソースコードを実行」というのはプロセスを作って呼び出すとかどういうことに・・・。

とまあ意味不明な事になっているので、ちゃんとやりたいことを整理しましょう。

いきなり別ソースコードを実行を目指すのではなくて、まず関数分けをするところから始めましょう。つまり、

#include <stdio.h>

int main(void)
{
    int a = 1;
    int b = 2;
    int c = 3;

    c += a + b;

    c *= -1;

    printf("%d", c);

    return 0;
}

のc *= -1;を別関数に切り分けてみましょう。

しかし変数cはmain関数で定義されている自動変数で他の関数からは直接アクセスすることができません。選択肢は

1. 変数cをいわゆるグローバル変数(関数スコープに定義されていないstatic storageの変数)にする
2. ポインタを介してアクセスする

というものがあります。

グローバル変数にするというのは

#include <stdio.h>
static int c = 3;
void f()
{
    c *= -1;
}
int main(void)
{
    int a = 1;
    int b = 2;

    c += a + b;

    f();

    printf("%d", c);

    return 0;
}

こういうことですね。ただこの方法は変数cがどこで書き換えられるのかがわかりにくく、プログラムが大きくなるほどバグを作りやすくなるという欠点があります。

ポインタを介してアクセスするというのは

#include <stdio.h>
void f(int* c)
{
    if(c) *c *= -1;
}
int main(void)
{
    int a = 1;
    int b = 2;
    int c = 3;

    c += a + b;

    f(&c);

    printf("%d", c);

    return 0;
}

こういうことですね。ただこの方法は関数fを呼び出すときにNULLポインタが渡される可能性があるのでNULLチェックが必要になるという欠点があります。

C++を使っていいならlvalue referenceをつかうという手法があります。

#include <stdio.h>
void f(int& c)
{
    c *= -1;
}
int main()
{
    int a = 1;
    int b = 2;
    int c = 3;

    c += a + b;

    f(c);

    printf("%d", c);
}

こうして関数分けをした後に、関数を別の翻訳単位に持っていけばいいでしょう。

Answer 2

求めているのはこういう事でしょうか

// main.c というファイルはここから.
#include <stdio.h>

// 別のファイルにchange_signという関数がある事を宣言しておく.
extern int change_sign(int c);

int main(void) {
  int a = 1;
  int b = 2;
  int c = 3;
  c += a + b;
  c = change_sign(c); // other.c にある change_sign を呼び出す.
  printf("%d", c);
  return 0;
}
// main.c というファイルはここまで.

// other.c というファイルはここから.
#include <stdio.h>

int change_sign(int c) {
  return -c; // c *= -1.0;
}
// other.c というファイルはここまで.

Comments

[HiroshiWatanabe]

ぎゃー…回答書いてる間に質問内容が全然別物に変わってた～(笑)

[yoshiki0424yi]

回答有り難うございます．
元の質問内容的にはこちらで聞きたかったことはあっています．
ありがとうございます．

Answer 3

あまり一般的な技法ではありませんが、#includeを使えばこんなこともできます。

// main.c

#include <stdio.h>

int main(void)
{
    int a = 1;
    int b = 2;
    int c = 3;

    c += a + b;

#include "other.inc"

    printf("%d", c);

    return 0;
}

// other.inc

// 直接埋め込まれるので、関数などに入れずそのまま書く
c *= -1.0;

もう少しシチュエーションなどのコメントを頂ければ、もっといい方法があるかもしれません。