簡單的C++函數包含失敗

Question

我想從一個「主」文件中的另一個文件包含一個函數。我遵循這種模式：

http://www.learncpp.com/cpp-tutorial/18-programs-with-multiple-files/

這裏是我的主文件，digispark.cpp：

#include <iostream> 

using namespace std; 

int send(int argc, char **argv); 

int main() 
{ 
    char* on; 
    *on = '1'; 
    char* off; 
    *off = '0'; 
    send(1,&on); 
    return 0; 
} 

這裏是我的send.cpp：

#include <stdio.h> 
#include <iostream> 
#include <string.h> 
#if defined WIN 
    #include <lusb0_usb.h> // this is libusb, see http://libusb.sourceforge.net/ 
#else 
    #include <usb.h>  // this is libusb, see http://libusb.sourceforge.net/ 
#endif 

// I've simplified the contents of send for my debugging and your aid, but the 
// complicated arguments are a part of the function that will eventually need 
// to be here. 
int send (int argc, char **argv) 
{ 

    std::cout << "Hello"; 
    return 0; 
} 

我在Ubuntu12.10米編譯使用G ++編譯器像這樣：

g++ digispark.cpp send.cpp -o digispark 

它編譯成功。

然而，當我運行程序，「你好」不上來。所以我不相信這個函數被調用。我究竟做錯了什麼？任何幫助將是偉大的！謝謝！

編輯：

如何處理這個問題：

int send(int argc, char **argv); 

int main() 
{ 
    char* on[4]; 
    on[0] = (char*)"send"; 
    on[1] = (char*)"1"; 
    char* off[4]; 
    off[0] = (char*)"send"; 
    off[1] = (char*)"0"; 
    send(2,on); 
    return 0; 
} 

對於那些你們誰是困惑，爲什麼我堅持這樣做，正如我以前說過，發送功能已建成接受char ** argv（或char * argv []）。我的觀點是試圖模仿我的主要功能。

這本來是更難以改寫，其實雲在發送功能採取不同的類型參數不僅僅是在它所想要發送的功能。感謝大家！

因此，如果這有助於任何人試圖類似的東西隨意使用吧！

Answer 1

你的問題是你不認爲這是一個。它在這裏：

char* on; 
*on = '1'; 

你宣佈char指針，但沒有對其進行初始化。然後你解除它。邦，你死了。這就是所謂的未定義行爲。一旦你調用U.B.，任何事情都可能發生。如果你幸運的話，那就是崩潰。但我猜你這次沒有幸運。

看，如果你想開始在內存中存儲的東西，你必須先分配內存。正如hetepeperfan所說，最好的方法就是使用std::string，並讓該班負責所有的分配/釋放。但是，如果由於某種原因，你認爲你必須使用C風格的字符串和指針，那就試試這個：

char on[128]; //or however much room you think you'll need. Don't know? Maybe you shoulda used std::string ... 
*on = '1'; 
*(on+1) = '\0'; //if you're using C-strings, better null terminate. 
char off[128]; 
*off = '0'; 
*(off+1) = '\0'; 
send(1,&on); 

Answer 2

好，我覺得你嘗試做一些像下面，我試圖讓它多一點在C++風格，防止使用指針，因爲他們不應該在你表現出的代碼是必要的。

digispark.cpp

#include "send.h" 

int main (int argc, char** argv){ 

    string on = "1"; 
    string off = "0"; 

    send (on); 
    send (off); 

    return 0; 
} 

send.cpp

#include <iostream> 
#include <string> 

void send(const std::string& s) { 

    std::cout << s << std::endl; 

} 

send.h

void send(const std::string& s);