ある文字列の配列から大文字だけを抽出してそれを別のchar配列に保存したいのですがエラーになります。どのようにすればよいのでしょうか。
c++
1char main_array[50][100]; 2int main(){ 3 char array[5] = "bcDe"; 4 for(int i = 0; i < 5; i++){ 5 if(islower(array[i] != 0)) strcpy(main_array[0],array[i]); 6 } 7 return 0; 8}
気になる質問をクリップする
クリップした質問は、後からいつでもMYページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
回答5件
0
最終的にやりたい事やC++ならとか色々あるのですが、参考までに
char array[5] = "bcDe"; char temp_ary[5]={'\0'};//bcDe用ですが一応初期化して置こうと編集^^ for (int i = 0, j = 0; i < 5; i++) { if (isupper(array[i])) { temp_ary[j] = array[i]; ++j; temp_ary[j] = '\0'; } } printf("%s \n", temp_ary);
因みにstrcpyは文字列の処理になります。
投稿2017/05/25 21:47
編集2017/05/26 06:43総合スコア350
0
strcpyのマニュアルはご覧になったことがありますか?
http://www9.plala.or.jp/sgwr-t/lib/strcpy.html
char *strcpy(char *s1, const char *s2);
引数にはポインタを渡しますが、ご提示のコードの第二引数はarray[i]
となっています。
これはアドレスではなく、文字を指していますので間違いです。
また例えこれを&array[i]
にしたとしても、strcpyはそのアドレスからNULL文字までをコピーしますので、ご期待の動作にはなりません。
C言語において文字(文字列ではなく)を扱う場合は、strcpy関数などは使わず、直接変数間でやりとりするのが基本です。
またislower関数は小文字判定の関数なので、isupperを使用すべきです。
修正したコードを載せようと思いましたが、kyuntaさんのコードと一緒になってしまったので割愛します。
投稿2017/05/25 23:44
総合スコア16998
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
0
エラーが記載されてないのであれですが、おそらくエラーになっているのは
C
1if(islower(array[i] != 0))
の部分です。(他にエラーとなる部分が見当たらない)
islowerの引数はint型です。でも、問題の箇所はarray[i] != 0
と条件文になっています。
どうすればいいかは他の方のようにすればいいかと。
投稿2017/06/12 08:51
編集2017/06/12 08:52総合スコア3579
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
0
ご提示されたコードが意図した通りに動作しない件につきましては、ttyp03さんのおっしゃる通りですので割愛させて頂きます。
C++であれば、std::copy_if
を使う事ができます(std::begin
、std::end
はC++11以降で利用できます)。
cpp
1#include<iostream> 2#include<algorithm> 3 4char main_array[50][100]={}; 5 6int main() 7{ 8 char array[5] = "bcDe"; 9 std::copy_if(std::begin(array),std::end(array),main_array[0],static_cast<int(*)(int)>(std::isupper)); 10 std::cout << main_array[0] << std::endl; 11}
また、char 型の配列で文字列を扱うよりも、std::string
などを使う方が、メモリ管理の面などで抽象化されている事から、安全と言えます。
cpp
1#include<iostream> 2#include<algorithm> 3#include<string> 4 5int main() 6{ 7 std::string result; 8 const std::string str = "bcDe"; 9 std::copy_if(std::begin(str),std::end(str),std::back_inserter(result),static_cast<int(*)(int)>(std::isupper)); 10 std::cout << result << std::endl; 11}
投稿2017/06/11 08:57
編集2017/06/12 09:15総合スコア48
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
0
strcpyを使って作成してみました。(お勧めではありませんが参考に)
c
1#include<stdio.h> 2#include<string.h> 3char main_array[50]; 4int main(){ 5 char array[5] = "bcDe"; 6 int i,j; 7 for(i = j = 0; i < 5; i++){ 8 if(isupper(array[i])) strcpy(&main_array[j++],&array[i]); 9 main_array[j]='\0'; 10 } 11 printf("%s\n",&main_array); 12 return 0; 13} 14
投稿2017/05/26 00:26
総合スコア4070
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
あなたの回答
tips
太字
斜体
打ち消し線
見出し
引用テキストの挿入
コードの挿入
リンクの挿入
リストの挿入
番号リストの挿入
表の挿入
水平線の挿入
プレビュー
質問の解決につながる回答をしましょう。 サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。 また、読む側のことを考えた、分かりやすい文章を心がけましょう。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。