0

初心者で質問の内容が足りないかもしれません。
if文において、変数等を使って以下のコードをもう少し短く書く方法は御座いますでしょうか?
ご教授のほど宜しくお願い致します。

内容としましては、別クラスから呼び出したメソッドのTitleが「空っぽ」&「nil」でない場合のif文です。宜しくお願い致します。

if (![[Data ValueWithKey:InformationTitle] isEqual:@""] &&
    ![[Data ValueWithKey:InformationTitle] isEqual:nil]) {}
15
  • 質問文では 「空っぽ」かつ「nil」でない場合 ですが、コードでは 「空っぽ」 でない かつ「nil」でない場合 が表現されています。本当の趣旨はどちらですか
    – user26045
    Commented 2018年2月5日 4:12
  • もし、コードのほうが趣旨に沿う場合、ただ1文字以上の文字列であればよく、答えは if ([Data ValueWithKey: InformationTitle].length > 0) {} のようになってしまいます
    – user26045
    Commented 2018年2月5日 4:25
  • ご返答頂き有難うございます。空であり、そしてnilでもある。という条件で御座います!この説明で宜しいでしょうか?誰が見てもわかりやすいように、カラとヌルをあえて記載する必要性を要求されているかと思います
    – kkanzaki
    Commented 2018年2月5日 4:34
  • 後から読む方が混乱するので、質問文のコードを直したほうがいいですね
    – user26045
    Commented 2018年2月5日 4:36
  • どのようにコードを直す必要がありますでしょうか?ご教授のほどお願い致します。
    – kkanzaki
    Commented 2018年2月5日 4:38

2 件の回答 2

0

こんな感じでしょうか?

emptyString = [Data ValueWithKey:InformationTitle];
if ((emptyString != nil) && (emptyString.length == 0))
{
    // do something
} //end if value is ""

もっと短く書くなら

emptyString = [Data ValueWithKey:InformationTitle];
if (!emptyString && (emptyString.length == 0))
{
    // do something
} //end if value is ""

という書き方も出来ると思います。

余談ですが、

  • クラス名は大文字から
  • 変数名は小文字から

書き始めるのが良いと思いますので、Dataという変数名は避けた方がいいと思います。

追記
質問のコメントを拝見して、回答の趣旨が間違っていることに気がつきました。
上のコードはタイトルが「空っぽ」かつ「nilでない」事を判別するコードになります。
「空っぽ」でも「nil」でもない事を判別するためには以下のコードになります。

emptyString = [Data ValueWithKey:InformationTitle];
if ((emptyString != nil) && (emptyString.length > 0))
{
    // do something
} //end if value is not empty

再追記
if文の中に入ってくれないという事で、デバッグ用に冗長化したコードを記述してみます。
以下をステップ実行して、BOOL型の2つの変数にそれぞれYES(true)が代入されるかを確認してみてください。

NSString *title = [Data ValueWithKey:InformationTitle];
BOOL isNotNil = title ? YES : NO;
BOOL isNotEmpty = (title.length > 0) ? YES : NO;
if (isNotNil && isNotEmpty)
{
     // do something
} // end if title is there
6
  • ご教授頂き有難うございます!大変勉強になりました。また、クラス名は大文字、変数名は小文字、変数名につきましても気をつけます!有難うございました!大変助かりました!
    – kkanzaki
    Commented 2018年2月5日 3:22
  • 質問の意図を誤解しており、質問者の問題が解決していないそうです。回答を改善できますか?
    – user26045
    Commented 2018年2月5日 5:31
  • ご指摘ありがとうございます。追記しました。
    – Чайка
    Commented 2018年2月5日 5:36
  • 追記ありがとうございました。上記でも上手く動かないようで御座いました。emptyStringは、エラーとなるため、NSString *emptyStringに変更したのですが、これが原因でしょうか。(emptyString != nil) のみにしても判定せず、(emptyString.length > 0 )のみにしても判定しませんでした。もし解決方法等御座いましたら、ご教授のほどお願い致します。
    – kkanzaki
    Commented 2018年2月5日 7:42
  • 質問の意図を勘違いして申し訳ありませんでした。判定しないとはどういう状況でしょうか?
    – Чайка
    Commented 2018年2月5日 7:50
0

すでに解決してるのでこれは無視してもらってもいいです。

NSString *title = [Data ValueWithKey:InformationTitle];
if( title.length != 0 ) {
    //
}

これでいいです。

解説:
Objective-Cでは nilへのメッセージパッシングは0を返します。(例外を除きます)
ですので、もしtitleがnilであれば title.length は0を返すので条件は偽となります。
titleがnilでなければ通常の文字列の長さが返ってきますので空文字列以外は真となります。
また、lengthの戻り値の型はNSUIntegerですのでマイナスになることは考慮しなくてよいです。

nilへのメッセージパッシングの戻り値の例外:
ざっくりと言うと「レジスタに入りきらない大きさの構造体に類するものを返すメソッドの場合は戻り値が不定となる。」です。
それ以外は必ず0が返ります。

気になること:

[Data ValueWithKey:InformationTitle]

これが本当にNSStringかチェックしてないけど大丈夫?

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。