C#: Избавиться от повторяющегося кода

Делается это так. Добавляем нужные символы в HashSet<char> (кстати, в него нельзя добавить одинаковые символы ни случайно, ни специально). Далее везде в коде проверяешь есть ли символ в HashSet<char>. Почитай про HashSet<T>, работает быстрее, чем обход массива, к примеру, на каждой итерации.

using System;
using System.Collections.Generic;
namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            var charSet = new HashSet<char>(new[]
            {
                ' ', '-', ',', '.', '_', ':', ';', '?', '(', ')', '!'
            });
            string text = "Hi! My name is John.";
            Console.WriteLine(text);
            for (int i = 0; i < text.Length; i++)
            {
                if (charSet.Contains(text[i]))
                {
                    Console.WriteLine(i);
                }
            }
            Console.ReadKey(true);
        }
    }
}

Я бы сделал это при помощи String.Contains() из-за наглядности

var map = " ,._:;&()!";
for (int i = 0; i < text.Length; i++) {
    if (map.Contains(text[i])) {
        ...
    }
}

Если же Вам важна производительность, все это дело можно сложить в HashSet и искать в нем

var map = " ,._:;&()!";
var set = new HashSet<char>(map);
for (int i = 0; i < text.Length; i++) {
    if (set.Contains(text[i])) {
        ...
    }
}