arrays del med lige og ulige pladser hjællp

og hvad har du indtil videre ?

public static boolean isValidEan(String ean)
    {
        if (ean.length() != 13)
            return false;
       
        try
        {
            long lEan = Long.parseLong(ean);
        }
        catch (NumberFormatException caught)
        {
            return false;
        }
       
        int part0 = 0, part1 = 0;
       
        for (int i = 0; i <= 10; i += 2)
        {
            part0 += Integer.valueOf(Character.toString(ean.charAt(i)));
            part1 += Integer.valueOf(Character.toString(ean.charAt(i + 1)));
        }
       
        part0 += Integer.valueOf(Character.toString(ean.charAt(12)));
       
        part1 *= 3;
       
        if ((part0 + part1) % 10 == 0)
            return true;
        else
            return false;
    }

Kan gøres mere effektiv ved at få konverteret direkte fra char til int, men den virker ;)

public static boolean isValidEan(String ean)
    {
        if (ean.length() != 13)
            return false;
       
        try
        {
            long lEan = Long.parseLong(ean);
        }
        catch (NumberFormatException caught)
        {
            return false;
        }
       
        int part0 = 0, part1 = 0;
       
        for (int i = 0; i <= 10; i += 2)
        {
            part0 += ean.charAt(i) - '0';
            part1 += ean.charAt(i + 1) - '0';
        }
       
        part0 += (int)ean.charAt(12) - (int)'0';
       
        part1 *= 3;
       
        if ((part0 + part1) % 10 == 0)
            return true;
        else
            return false;
    }

der var lige en mere effektiv en... Overså noget virkelig dumt

Og så kan de (int) godt slettes :/

public static boolean isValidEan(String ean)
    {
        if (ean.length() != 13)
            return false;
       
        int part0 = 0, part1 = 0;
       
        for (int i = 0; i < 13; i++)
        {
            char c = ean.charAt(i);
           
            if (c < '0' || c > '9')
                return false;
           
            if (i % 2 == 0)
                part0 += c - '0';
            else
                part1 += c - '0';
        }
       
        part1 *= 3;
       
        if ((part0 + part1) % 10 == 0)
            return true;
        else
            return false;
    }

det sidste virker ikke rigtigt

hvilken ean streng bruger du på den?

public class Ean
    {
        private String numer;   
     
     
        public Ean(String enA)     
        {
            numer = enA;
        }
public boolean isValidEan()
    {
        if (numer.length() != 13)
            return false;
       
        try
        {
            long lNumer = Long.parseLong(numer);
        }
        catch (NumberFormatException caught)
        {
            return false;
        }
       
        int ulige = 0, lige = 0;
       
        for (int i = 0; i <= 10; i += 2)
        {
            ulige += Integer.valueOf(Character.toString(numer.charAt(i)));
            lige += Integer.valueOf(Character.toString(numer.charAt(i + 1)));
        }
       
        ulige += Integer.valueOf(Character.toString(numer.charAt(12)));
       
        ulige *= 3;
               
        if ((ulige + lige) % 10 == 0)
            return true;
        else
            return false;
    } 
}

public class Ean {
   
    private String _ean;
   
    public Ean(String ean) {
        _ean = ean;
    }
   
    public boolean isValidEan()
    {
        return isValidEan(_ean);
    }
   
    public static boolean isValidEan(String ean)
    {
        if (ean.length() != 13)
            return false;
     
        int part0 = 0, part1 = 0;
     
        for (int i = 0; i < 13; i++)
        {
            char c = ean.charAt(i);
         
            if (c < '0' || c > '9')
                return false;
         
            if (i % 2 == 0)
                part0 += c - '0';
            else
                part1 += c - '0';
        }
     
        part1 *= 3;
     
        if ((part0 + part1) % 10 == 0)
            return true;
        else
            return false;
    }
}