長さ5以上、英数字を含む、同じ文字を３つ以上連続で使用してはならない文字列

Question

条件・目的

・長さが5以上
・英字と数字の両方を含む必要が有る
・同じ文字を 3 つ以上連続で使用することはできない

入力される値
t

1 ≦ (t の長さ) ≦ 30
文字列 t は半角英字あるいは半角数字で構成された文字列
英字の大文字と小文字は区別する必要はない
文字列が全ての条件を満たす場合のみ"success"
それ以外は全て"fail" と出力する

作成したコード

using System;
using System.Text.RegularExpressions;

class Program
{
   
    static bool sameCharactersOverThree(string line)
    {
        int charCount = 1;
        for (int i = 0; i < line.Length - 1; i++)
        {
            char c = line[i];
            if(c == line[i + 1])
            {
                charCount++;
                if (charCount >= 3)
                {
                    return false;
                }
            }
        }
        return true;
    }

        static void Main()
        {
            string line = Console.ReadLine();
            Regex re = new Regex(@"[0-9]");
            Regex ge = new Regex(@"[A-z]");

            if (sameCharactersOverThree(line) && re.Matches(line).Count != 0 && ge.Matches(line).Count != 0 && line.Length >= 5)
            {
                    Console.WriteLine("success");
                }
                else
                {
                    Console.WriteLine("fail");
                }
            }
        }

###エラーメッセージ(変更前)
Main.cs(22,8): error CS1525: Unexpected symbol static' Main.cs(22,16): error CS1547: Keyword void' cannot be used in this context
Main.cs(22,24): error CS1525: Unexpected symbol (' Main.cs(41,0): error CS1525: Unexpected symbol }'

###追記
修正したらコードが動きました。皆さん本当にありがとうございます。エラーメッセージをしっかり読むべきでした。

Comments

[Zuishin]

paiza.io を使うからです。
Visual Studio を使いましょう。

[退会済みユーザー]

問題・課題は何なのですか？　ある環境（？）では全く動かないからそれを何とかしたい？　Visual Studio 2019 でのエラーを何とかしたい？　正規表現を知りたい？

[dodox86]

このご質問には2つのテーマがあって
1. コンパイルエラーを取る。＞正しい文法でコードを書き、とにかく動くようにする。
2. 「長さ5以上、英数字を含む、同じ文字を３つ以上連続で使用してはならない文字列」を検出する。
がありますが、質問としては1. で良いですか？

こういうご質問で往々にしてあるのが「コンパイルエラーは無くなりましたが、答えが間違っています。どうしたらよいですか？」と続くことです。コンパイルエラーがあるので何とも、ですが"aa1bb2aa3"のような文字列で「3つ以上連続する文字」を正しく検出できていないと思います。

[jing02]

はい、質問の内容は１．です。文字列が一つ与えられるのですが、それが条件を満たしているかを判定するプログラムを作成したいのです。質問が曖昧で申し訳ありません。

[Zuishin]

paiza.io を使うからです。
あと、文字を読まないから。

[jing02]

例えば、"8Mw2ehvvvvy"のような文字列では英数字を含み、５文字以上なのでこれらの条件は満たしていますが３つ以上の同じ文字を含んでいるため"fail"となります。全ての条件を満たす場合のみ"success"と表示するようにしたいです。ただ、まずはやはりコンパイルエラーをどうにかしなくてはなりません。

[Zuishin]

どこで見つけてきたんだこの変なテンプレート。

[退会済みユーザー]

> はい、質問の内容は１．です。

質問文を編集してその旨書いてください。

[Zuishin]

> Main.cs(22,8): error CS1525: Unexpected symbol static'

エラーメッセージにはここで static は使えないと、こう書いてある。
ならば質問のタイトルはこれを問うものでなければならない。

> 長さ5以上、英数字を含む、同じ文字を３つ以上連続で使用してはならない文字列

現在のタイトルはこれ。
直面している問題に何の関係もない。
だから質問の意味を問われている。

結局、初心者かどうかではなく、国語力の問題。
親切に行番号まで書いてあるのに、どこに問題があるかを把握できてない。
この程度の問題なら、中学生の初心者でも自分で余裕で直せる。

[退会済みユーザー]

日本語レベルでダメな質問ということが認識できないのかな？

Answer 1

sameCharactersOverThree 関数の「中」に static void Main() を定義しているから、

Main.cs(22,8): error CS1525: Unexpected symbol 'static void' cannot be used in this context

つまり「static void をこんなところに書くわけがない」と教えてくれています。メンバ関数を「メンバ関数としての所定の位置」に書きましょう。

Comments

[dodox86]

質問と直接関係ない話になりますが、

> メンバ関数を「メンバ関数としての所定の位置」

「メンバー関数」はC++での呼び方で、C# では一般に「メソッド」と呼ぶと思います。teratailの過去質問回答で良いものがありましたので、付記しておきます。

「メンバ変数」「メンバ関数」等の「メンバ」という名前の由来は?
https://teratail.com/questions/207907

[jing02]

ご回答ありがとうございます。

Answer 2

こんな感じでええかのう？ ちな同じやつここにも上げといたで。

using System;
using System.Text.RegularExpressions;

class Program {

  private static string check(string text) {

    // 文字列 text は 5文字以上である。
    bool check1 = text.Length >= 5;

    // 文字列 text は アルファベットを含む。
    bool check2 = Regex.IsMatch(text, "[a-zA-Z]");

    // 文字列 text は 数字を含む。
    bool check3 = Regex.IsMatch(text, "[0-9]"); 

    // 文字列 text は 同じ文字の3文字以上の繰り返しを含んでいない。
    bool check4 = !Regex.IsMatch(text, "(.)\1{2,}");      

    // 上記の全てを満たすかどうかのフラグ（すべて満たせばTrue)
    bool checkAll = check1 & check2 & check3 & check4;

    return(checkAll ? "success" : "fail");
  }

  public static void Main (string[] args) {
    Console.WriteLine(Program.check("a123"));    // 5文字未満なのでfail
    Console.WriteLine(Program.check("1234567")); // アルファベットを含まないのでfail
    Console.WriteLine(Program.check("AbcDef"));  // 数字を含まないのでfail
    Console.WriteLine(Program.check("abcZZZ12"));  // 同じ文字'Z'が３回繰り返されているのでfail
    Console.WriteLine(Program.check("A77777"));  // 同じ文字'7'が5回繰り返されているのでfail
    Console.WriteLine(Program.check("xAA99"));  // 5文字以上、数字とアルファベットを含み、同じ文字の繰り返しは3回未満なのでsuccess
  }
}

ほんで、上のコードの check(string text) を詰めて書くと、こないな感じ

  private static string check(string text) {
    return (    
      text.Length >= 5 &&
      Regex.IsMatch(text, "[a-zA-Z]") &&
      Regex.IsMatch(text, "[0-9]") &&
      !Regex.IsMatch(text, "(.)\1{2,}") ? "success" : "fail");
  }

➡ ここにも上げといたで。

Comments

[Zuishin]

質問者のレベルから言っても、もともとの問題が正規表現の練習問題には思えませんが、何かの嫌がらせですか？

[Zuishin]

仮に正規表現の練習問題だとしても、自力で解いた人にしか利益を与えない練習問題の利益を丸々抜いてしまう意味がよくわかりません。
何の役にも立たないプログラムが一つできただけです。

この質問で問われているのは、エラーの原因だと思います。

[jing02]

ご回答ありがとうございます。