C - Преобразование подстроки в целое

Question

На данный момент я получил этот простой код C, чтобы преобразовать дни, минуты и секунды только секунды:

EDITED (я понял проблему с atoi, как это исправляется право ?):.

#include <stdio.h> 
#include <stdlib.h> 

int getseconds(char * time) 
{ 
    int seconds=0, i=0; 
    char buffer[3]; 
    while (*time != '\0') 
    { 
     switch (*time) 
     { 
      case 'h': buffer[i]='\0';i=0;seconds=seconds+atoi(buffer)*3600;break; 
      case 'm': buffer[i]='\0';i=0;seconds=seconds+atoi(buffer)*60;break; 
      case 's': buffer[i]='\0';i=0;seconds=seconds+atoi(buffer);break; 
      case ' ':break; 
      default: buffer[i]=*time;i++;break; 
     } 
     time++; 
    } 
    return seconds; 
} 

int main() 
{ 

    char *time = "12h 4m 58s"; 
    int seconds = getseconds(time); 
    printf("%d",seconds); 
    return 0; 
} 

Это работает, как я хочу, но нет другого способа сделать это, не создавая больше переменных (например, C#, где я просто нужно преобразовать «инлайн» есть ли C только функции которые преобразуются в переменные, а не в «inline»)?

C# Пример:

 string time = "12h 34m 58s"; 
     int seconds = int.Parse(time.Substring(0, 2)) * 3600 + int.Parse(time.Substring(4, 2)) * 60 + int.Parse(time.Substring(8, 2)); 

Вы можете определить количество линий разницы я думаю :).

Answer 1

Вы можете использовать atoi() или strtol(), чтобы сделать эту работу. Вам не нужно сначала извлекать подстроки, потому что эти функции останавливаются на первом символе, который не может быть преобразован. Вам просто нужно принять различное представление о том, где начинается строка. Таким образом, этот код достаточно аналогичен вашему C# Например:

int getseconds(const char * time) { 
    return atoi(time) * 3600 + atoi(time + 4) * 60 + atoi(time + 8); 
} 

Конечно, это сломает на некоторых некорректных входов, но так будет C# (хотя и не все же входов).

Answer 2

sscanf() с "%n" для обнаружения конца сканирования близко, но использует больше переменных.

int getseconds2(const char * time) { 
    int h,m,s; 
    int n = 0; 
    sscanf(time, "%d h%d m%d s %n", &h,&m, &s, &n); 
    if (n == 0 || time[n] != '\0') return -1; // failure 
    return h*3600 + m *60 + s; 
} 

"%n" сохраняет количество отсканированных символов. Поскольку используется в конце формата, тестирование, чтобы проверить, отличен ли он от нуля (исходное значение) и что time, закончившийся на нулевом символе, гарантирует целостность сканирования.

@chqrlie предложил неправильное соответствие функциональности C#.

// to match C# 
    if (n == 0) return -1; // failure 

Answer 3

Намного проще решение возможно:

int getseconds(const char* time) 
{ 
    int h, m, s = 0 ; 
    char sdelim = 0 ; 
    int check = sscanf(time, "%dh %dm %d%c", &h, &m, &s, &sdelim) ; 
    if(check == 3 && sdelim = 's') 
    { 
     s = ((h * 60) + m) * 60 + s ; 
    } 

    return s ; 
}