Atcoder ABC082B C#

Question

お世話になっております。
以下のC#コードでABC082Bを解こうとしたらREになりました。

どこらへんが怪しいでしょうか？
また、REになった場合、どのように検討をつければいいのでしょうか。

以上となりますがよろしくお願いいたします。

using System.Linq;
using System;

class Program
{
   static void Main(string[] args)
   {
      string s = Console.ReadLine();
      string t = Console.ReadLine();
      s = String.Concat(s.OrderBy(c => c));
      t = String.Concat(t.OrderByDescending(c => c));

      Console.WriteLine(isAlphabetically(s,t) ? "Yes" : "No");
   }

   static bool isAlphabetically(string s, string t)
   {
      bool res = false;

         if (s[0] < t[0])
         {
            res = true;
         } else if (s[0] == t[0])
         {
            isAlphabetically(s.Substring(1), t.Substring(1));
         }

         return res;
   }
}

追記

ACしたコード

using System;

class Program
{
   static void Main(string[] args)
   {
      string s = Console.ReadLine();
      string t = Console.ReadLine();
      s = String.Concat(s.OrderBy(c => c));
      t = String.Concat(t.OrderByDescending(c => c));

      Console.WriteLine(isSortedAlphabetically(s,t) ? "Yes" : "No");
   }

   static bool isSortedAlphabetically(string s, string t)
   {
      if (s[0] < t[0])
      {
         return true;
      }
      else if (s[0] == t[0])
      {
         if (s.Length == 1 && t.Length == 2) return true;
         if (s.Length == 2 && t.Length == 1) return false;
         if (s.Length == 2 && t.Length == 1) return false;
         if (s.Length == 1 && t.Length == 1) return false;
         return isSortedAlphabetically(s.Substring(1), t.Substring(1));
      }
      else
      {
         return false;
      }
   }
}

Answer 1

自分で入力を作ってローカルで実行し、どこでエラーになるか確かめてください。
ぱっと見、二つの文字列が全く同じものであったら異常終了しそうです。

Answer 2

isAlphabetically関数から再起呼び出ししていますがその戻り値を使っていません。よって、演算結果に影響するのは先頭の文字だけになります。戻り値を変数resで受けないといけないのでは？

Comments

[mofu_mofu]

ありがとうございます。見直してみたら、再帰しているメソッドをreturnや変数に格納していないことに気づきACできました。else if (s[0] == t[0])の中身がすこし不細工だなあという感覚はありますが...。