Need help with code

I need help with my Auto-Broadcaster plugin. the places that keep giving errors are in bold. with the } , it told me it needed another } so i added one. it then told me to delete one, and just looped around and around. the commandSender just says it cannot be resolved to a type. How do i fix this? thanks in advance.

Code:

package me.mekixx.Broadcaster;
 
import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.LineNumberReader;
import java.util.logging.Logger;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Broadcast extends JavaPlugin {
    public static Broadcast plugin;
    public final Logger logger = Logger.getLogger("Minecraft");
    public static int currentLine = 0;
    public static int tid = 0;
    public static int running = 1;
    public static long interval = 10;
 
    @Override
    public void onDisable() {
        PluginDescriptionFile pdffile = this.getDescription();
        this.logger.info(pdffile.getName() + " version " + pdffile.getVersion() + " is now disabled.");
   
    }
    @Override
    public void onEnable() {
        PluginDescriptionFile pdffile = this.getDescription();
        this.logger.info(pdffile.getName() + " version " + pdffile.getVersion() + " is now enabled.");
   
        tid = Bukkit.getScheduler().scheduleSyncRepeatingTask(this,    new Runnable() {
            public void run() {
                try {
                    broadcastMessage("plugins/Broadcast/messages.txt");
                } catch (IOException e) {
               
                }
            }
        }, 0, interval * 20);
    }
 
    public static void broadcastMessage(String fileName) throws IOException  {
        FileInputStream fs;
        fs = new FileInputStream(fileName);
        BufferedReader br = new BufferedReader(new InputStreamReader(fs));
        for(int i = 0; i < currentLine; ++i)
            br.readLine();
        String line = br.readLine();
        line = line.replaceAll("&f", ChatColor.WHITE + "");
        line = line.replaceAll("&e", ChatColor.YELLOW + "");
        line = line.replaceAll("&d", ChatColor.LIGHT_PURPLE + "");
        line = line.replaceAll("&a", ChatColor.GREEN + "");
        Bukkit.getServer().broadcastMessage(ChatColor.LIGHT_PURPLE + "[Broadcast] " + ChatColor.WHITE + line);
        LineNumberReader lnr = new LineNumberReader(new FileReader(new File(fileName)));
        lnr.skip(Long.MAX_VALUE);
        int lastLine = lnr.getLineNumber();
        if(currentLine + 1 == lastLine + 1) {
            currentLine = 0;
        } else {
            currentLine++;
        }
  }
 
    public boolean onCommand((commandSender sender, Command cmd, String commandLabel, String[] args) {
        if(commandLabel.equalsIgnoreCase("stopbroadcast")) {
            if(running == 1) {
                Bukkit.getServer().getScheduler().cancelTask(tid);
                Player player = (Player) sender;
                player.sendMessage("Cancelled broadcasts.");
                running = 0;
            } else {
                Player player = (Player) sender;
                player.sendMessage("They aren't running!");
            }
        } else if (commandLabel.equalsIgnoreCase("startbroadcast")) {
            if(running == 1) {
                Player player = (Player) sender;
                player.sendMessage("They are still running!");
            } else {
                tid = Bukkit.getScheduler().scheduleSyncRepeatingTask(this,    new Runnable() {
                    public void run() {
                        try {
                            broadcastMessage("plugins/Broadcast/messages.txt");
                        } catch (IOException e) {
                       
                        }
                    }
                }, 0, interval * 20);
                Player player = (Player) sender;
                player.sendMessage("Started broadcasts.");
                running = 1
            ;
        }
        return false;
    }
   
 
}

 

first thing i noticed was your method has two opening parentheses, lemme keep looking

 

there is no class commandSender in bukkit, notice the case of the letters

EDIT:

You never defined currentLine

EDIT 2:
at broadcastMessage you never closing the file stream

EDIT 3:
public static int running = 1;
could be better an boolean

EDIT 4:

thats not nice behavor, read this: http://forums.bukkit.org/threads/co...ouldnt-just-return-if-instanceof-player.3186/
EDIT 5:
You never returning true if the command was successfull, you should do that
EDIT 6:

Code:

line = line.replaceAll("&f", ChatColor.WHITE + "");
        line = line.replaceAll("&e", ChatColor.YELLOW + "");
        line = line.replaceAll("&d", ChatColor.LIGHT_PURPLE + "");
        line = line.replaceAll("&a", ChatColor.GREEN + "");

That can be better: line = ChatColor.translateAlternateColorCodes('&',line);

 

thanks CorrieKay and ferrybig . i have fixed it and got it working!