Attempting scheduled auto Lead creation

Question

I am attempting to distribute Leads from a list on a weekly basis. I have this code which somebody was kind enough to help me with :

global class WeeklyPreOppProcessor {
{
    global void execute (SchedulableContext scx)
    {

        Group group = [Select Id, Name FROM Group 
                       WHERE Type = 'Queue' 
                       AND Name = 'Distributed_Queue'];

        List<Lead> lstLead = [SELECT Id, Name, OwnerId 
                              FROM Lead 
                              WHERE Distributed__c = false LIMIT 50];
        for(Lead leadObj:lstLead)
        {
            leadObj.Distributed__c = true;
            leadObj.OwnerId = group.Id;
        }
        update lstLead;
    }

}

But this is producing an error : "Global variable must be contained in a global class...". I don't understand this since I have the class labeled as global.

Any ideas ?

Answer 1

You must declare variables as global. Earlier I have given this code. Just made small changes.

global class WeeklyPreOppProcessor implements Schedulable
{
    global Group grp;
    global List<Lead> lstLead;
    global void execute (SchedulableContext scx)
    {

        grp = [Select Id, Name FROM Group 
                       WHERE Type = 'Queue' 
                       AND Name = 'Distributed_Queue'];

        lstLead = [SELECT Id, Name, OwnerId 
                              FROM Lead 
                              WHERE Distributed__c = false LIMIT 50];
        for(Lead leadObj:lstLead)
        {
            leadObj.Distributed__c = true;
            leadObj.OwnerId = grp.Id;
        }
        update lstLead;
    }

}

Answer 2

You have an extra opening curly bracket ({). That's forcing everything inside of it to be considered an initializer block and really confusing the poor compiler.

global class MyClass {
    {
        // stateful initializer code
    }
}

Once you remove the extra curly bracket, you'll get a new compile fail:

Identifier name is reserved: group

Once you rename that (to something like distributedQueue), your code will be good to go. Note also in your class declaration that you forgot to implement the Schedulable interface. Also note that you don't need to use global and should almost never use it unless you're working on package development. So your proper class should look like:

public class Demo implements Schedulable
{
    public void execute (SchedulableContext scx)
    {

        Group distributedQueue = [
            SELECT Name FROM Group 
            WHERE Type = 'Queue' 
            AND Name = 'Distributed_Queue'
        ];
        List<Lead> records = [
            SELECT Name, OwnerId FROM Lead
            WHERE Distributed__c = false LIMIT 50
        ];
        for(Lead record : records)
        {
            leadObj.distributed = true;
            leadObj.OwnerId = distributedQueue.Id;
        }
        update records;
    }
}

Answer 3

Let's take an example

global class updateData implements Schedulable {   
global void execute (SchedulableContext SC) {       
    DataContact dContact = new DataContact();       
    dContact.UpdateContact();       
}       

}

It looks like you are trying to execute the similar code (code mentioned above)inside the Dev Console. The Developer Console does not support creating global classes. If you are just trying to anonymously call UpdateContact, then you can simply type this into the Developer Console:

DataContact dc = new DataContact();
dc.updateData();

If you are trying to create a schedulable object to schedule calls to DataContact then you should create an Apex class with the code that you posted and then schedule it using the 'Schedule Apex' button on the Apex Classes page.