challenge 仲間はずれの判定

リストxsが渡されたときに
  • 全部の要素が同じ値である(例:[1, 1, 1, 1])、
  • 一つだけ仲間はずれがある(例:[1, 2, 1, 1, 1])、
  • その他
を識別する関数を作ってください。 また判定後に「全部の要素が同じ値」の場合にはその値、 「一つだけ仲間はずれがある」の場合にはその仲間はずれの値と多数派の値を複雑な処理なしに取得できるようにしてください。

型にうるさい言語のために:リストの中身は非負の整数だと仮定して負の値を未定義値代わりに使っても構いません。

追記:リストの長さは3以上であると仮定して構いません。(2個で異なる値の時にどちらが仲間はずれか決まらないので。) nobsunさん、noriさん、ご指摘ありがとうございます。

もっとスリム化できそうですがとりあえず。
 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
import java.util.*;
public class NakamaHazure {
    public static void main(String[] args) {
        check(Arrays.asList(1,1,1,1));    // => [1]
        check(Arrays.asList(1,2,1,1,1));    // => [2,1]
        check(Arrays.asList(1,2,2,1,1));    // => []
        check(Arrays.asList(1,2,2,1,3));    // => []
    }
    private static void check(List<Integer> list) {
        HashSet<Integer> set = new HashSet<Integer>();
        HashSet<Integer> minor = new HashSet<Integer>();;
        Integer major = null;
        for (Integer i : list) {
            if (!set.add(i)) {
                if (major != null && major != i) {
                    System.out.println("[]");
                    return;
                }
                minor.remove(i);
                major = i;
            } else {
                minor.add(i);
            }
        }
        if (set.size() == 1) {
            System.out.println("[" + set.iterator().next() + "]");
        } else if (set.size() == 2) {
            System.out.println("[" + minor.iterator().next() + "," + major + "]");
        } else {
            System.out.println("[]");
        }
    }
}

array_count_values便利ですね。あえて使わないで作ってみる事で便利さを際実感

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
$a[]= array(1,1,1,1);#1
$a[]= array(1,2,1,1,1);#2
$a[]= array(2,1,2,2);#その他
$a[]= array(1,2,2,2);#その他
$a[]= array(1,1,2,2);#その他
$a[]= array(1,1,2,3);#その他
function check($_array){
    $uniques = sizeof(array_unique($_array));
    switch($uniques){
        case(1):
            return array("仲間"=>array_pop($_array));
        case(2):
            sort($_array);
            #仲間はずれ
            if( $_array[0]!=$_array[1] ){
                return array( "仲間はずれ"=>array_shift($_array),"仲間"=>array_pop($_array) );
            }else if($_array[0]==$_array[1] && $_array[sizeof($_array)-1] != $_array[sizeof($_array)-2] ){
                return array( "仲間はずれ"=>array_pop($_array),"仲間"=>array_shift($_array) );
            }
        default:
            return "その他";
    }


}

foreach( $a as $b ){
    print_r( check($b) ).PHP_EOL;
}

Posted feedbacks - diff

間違えた。これでは値にundefinedを扱えませんでした。
 1
 2
 3
 4
 5
 6
 7
 8
 9
10
   }
-  return a == void 0 ? [b] :
-         b == void 0 ? [a] :
-         ac == bc    ? [] :
-         ac < bc     ? [b, a] : [a, b];
+  return ac == 0  ? [b] :
+         bc == 0  ? [a] :
+         ac == bc ? [] :
+         ac < bc  ? [b, a] : [a, b];
 };

g++ では typename が必要なのを忘れていた。あと、VisualC++6では、

template <typename Iterator>
vector(Iterator beg, Iterator end);

といった、vector 以外の任意のイテレータを受けるコンストラクタがないようなので、やむなく下のようにした。
 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
--- main.orig	Thu Aug 30 16:52:48 2007
+++ main.cpp	Thu Aug 30 17:05:44 2007
@@ -26,7 +26,7 @@ result<T> judge(const std::vector<T>& v)
 
     if (v.size() >= 3)
     {
-        std::vector<T>::const_iterator it = std::find_if(v.begin() + 1, v.end(), not_equal_to(v[0]));
+        typename std::vector<T>::const_iterator it = std::find_if(v.begin() + 1, v.end(), not_equal_to(v[0]));
 
         if (it == v.end())
         {
@@ -80,6 +80,10 @@ int main()
 
         std::istream_iterator<int> beg(sin), end;
 
-        solve(std::vector<int>(beg, end));
+        std::vector<int> v;
+
+        std::copy(beg, end, std::back_inserter(v));
+
+        solve(v);
     }
 }

Index

Feed

Other

Link

Pathtraq

loading...