Функция времени иногда работает, иногда нет

Question

Функция времени иногда работает, иногда нет

1

Поэтому я пытаюсь сделать функцию времени, которая учитывается. Это было основано на том, что я видел здесь. Переменные задаются из другой функции. По истечении времени переменная finish поворачивается на 1, и она оставляет функцию. Эта функция работает иногда, а иногда она не работает, например, если я даю ей ввод 11sec, он отлично работает, но если я дам ей 1:00 мин, это не сработает. Может кто-то сказать мне, что не так с кодом.

if (time1 == 0 && time2 == 0 && time3 == 0 && time4 == 0)
//if all the time is 0 finish the sequence
    finish = 1;

if (time1 != 0) //Checking to see if the first digit is NOT at 0
    time1 = time1 - 1; //  subtract time 1 by 1
else {
    time2 = time2 - 1;  //When time1 is 0
    time1 = 9;
} //Time1 going back to it original value

if (time2 == 0 && time1 == 0) { //if time1 and time2 are 0s
    if (time3 != 0) { //The minute value (time3)
        time2 = 5;  //60 SECONDS
        time3 = time3 - 1;
        time1 = 9;
    }             
} //Put time 1 to its original value
if (time2 <= 0 && time1 <= 0 && time3 <= 0) {
    if (time4 != 0) { //The minute value (time3)
        time2 = 5; //60 SECONDS
        time3 = 9;
        time4 = time4 - 1;
        time1 = 9; 
    }
} //Put time 1 to its original value

Time4 = 3, Time3 = 2, Time2 = 1, Time1 = 0. Это будет означать, что время составляет 32:10 мин.

Bad programmer 20 апр. 2016, в 10:29

Источник

9

Вау, это довольно сложно прочитать: o Это стиль кодирования WTF?
French Boiethios 20 апр. 2016, в 07:56
0

Я программирую на нормальном С, действительно базовая логика
Bad programmer 20 апр. 2016, в 08:00
1

Я говорю о твоем форматировании кода. Выберите один стиль форматирования (например, kernel.org/doc/Documentation/CodingStyle ); Ваш код будет намного более читабельным. Здесь, кажется, ваш отступ случайный.
French Boiethios 20 апр. 2016, в 08:03
2

Логика проста для понимания, он говорит обо всем остальном :) (отступы, стиль кодирования, ... Может быть, это потому, что вы использовали копирование-вставку, но я надеюсь, что ваш реальный код "красивее", чем он. Вам легче понять, и легче найти ошибки или ошибки в хорошо написанном коде. :)
Raph Schim 20 апр. 2016, в 08:04
0

Вам нужно переформатировать код, прежде чем мы сможем вам помочь. Как есть - у вас есть непревзойденная закрывающая скобка в строке 7. Мы не можем определить, является ли это частью вашей проблемы или просто артефактом выреза и вставки, пока вы не очистите вещи.
user590028 20 апр. 2016, в 08:22
0

@RaphaelSchimchowitsch Так что я больше не ошибаюсь, ты говоришь о том, что я сделал отступ? Я спрашиваю, поэтому я не делаю ошибку снова
Bad programmer 20 апр. 2016, в 17:53

Показать ещё 4 комментария

Теги:

c++

c

mikroc

2 ответа

Ещё вопросы

Вау, это довольно сложно прочитать: o Это стиль кодирования WTF?
Я программирую на нормальном С, действительно базовая логика
Я говорю о твоем форматировании кода. Выберите один стиль форматирования (например, kernel.org/doc/Documentation/CodingStyle ); Ваш код будет намного более читабельным. Здесь, кажется, ваш отступ случайный.
Логика проста для понимания, он говорит обо всем остальном :) (отступы, стиль кодирования, ... Может быть, это потому, что вы использовали копирование-вставку, но я надеюсь, что ваш реальный код "красивее", чем он. Вам легче понять, и легче найти ошибки или ошибки в хорошо написанном коде. :)
Вам нужно переформатировать код, прежде чем мы сможем вам помочь. Как есть - у вас есть непревзойденная закрывающая скобка в строке 7. Мы не можем определить, является ли это частью вашей проблемы или просто артефактом выреза и вставки, пока вы не очистите вещи.
@RaphaelSchimchowitsch Так что я больше не ошибаюсь, ты говоришь о том, что я сделал отступ? Я спрашиваю, поэтому я не делаю ошибку снова

fluter · Answer 1 · 2016-04-20T06-13-00.000Z

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

if (time1 > 0)
    time1 -= 1;
if (time3 > 0)
    time3 -= 1;

Другая мысль, вы рассчитываете с каждой цифрой минут и секунд, почему бы просто не использовать секунды, конвертируя свое время в секунды. Например, чтобы пересчитать 1:23:

int minutes = 1;
int seconds = 23;
int timer = minutes * 60 + seconds;

// in your timer function
if (seconds == 0) {
    finish = 1;
} else if (seconds > 0) {
    seconds -= 1;
} else {
      // error
}

Таким образом, это было бы также расширяемо, что, если вы хотите обрабатывать часы, просто добавьте hours * 3600 в seconds, вы можете сделать это легко, чтобы обрабатывать дни, даже месяцы. В вашем подходе добавить те, которые приведут к слишком большому числу случаев, их почти невозможно обработать правильно.

спасибо за предложение превратить время в секунды, я применил эту логику, и она отлично работает. Еще раз ваша помощь очень ценится.
@Badprogrammer рад знать, что это помогло.
Можете ли вы указать, где я мог пойти не так, пожалуйста, min = (time4 * 10) + time3; сек = (время2 * 10) + время1; итого = (мин * 60) + сек; всего = всего - 1; // Total хранит количество секунд if (total == 0) finish = 1; // преобразование секунд обратно в мин и секунд min = total / 60; время4 = мин / 10; время3 = мин% 10; temp1 = total - min * 60; время2 = темп1 / 10; время3 = темп1% 10;
@Badprogrammer что ты пытаешься сделать?
Я пытаюсь преобразовать входные данные в секунды, а затем вычесть его на 1, а затем преобразовать его обратно в удобочитаемое время. По какой-то причине это не работает
@Badprogrammer проверьте ideone.com/2Gbcw7 вы использовали неправильные переменные.

molbdnilo · Answer 2 · 2016-04-20T07-57-00.000Z

Проблема в том, что вы сравниваете ноль после того, как изменили число на ненулевое.

Предполагая, что 1:00 кодируется как

time1 = 0
time2 = 0
time3 = 1

вы можете следовать своей логике:

if (time1 != 0) // Nope
    time1 = time1 - 1;
else { // Yes
    time2 = time2 - 1;
    time1 = 9;
}

Теперь у вас есть

time1 == 9
time2 == 0
time3 == 1

if (time2 == 0 && time1 == 0) { // Nope, time1 is 9
    if (time3 != 0) {
        time2 = 5;  
        time3 = time3 - 1;
        time1 = 9;
    }             
}

и у вас все еще есть

time1 == 9
time2 == 0
time3 == 1

и наконец

if (time2 <= 0 && time1 <= 0 && time3 <= 0) { // Nope
    if (time4 != 0) { 
        time2 = 5; 
        time3 = 9;
        time4 = time4 - 1;
        time1 = 9; 
    }
}

так что в итоге

time1 == 9
time2 == 0
time3 == 1

то есть 1:09.

Единственный раз, когда вы хотите изменить время _k, - это когда время _k-1 "пересекло" ноль.
Это можно сделать с помощью гнезда условных выражений:

if (time1 > 0 || time2 > 0 || time3 > 0 || time4 > 0)
{    
    time1 -= 1;
    if (time1 < 0)
    {
        time1 = 9;
        time2 -= 1;
        if (time2 < 0)
        {
            time2 = 5;
            time3 -= 1;
            if (time3 < 0)
            {
                // ...
            }
        }
    }
}

Ахахах, это имеет смысл. Я бы попытался отладить его сам, но, к сожалению, отладчик на MikroC не так хорош, как отладчик на Visual Studio. Еще раз спасибо за вашу помощь
@Badprogrammer Вам не нужен отладчик для подобных проблем. Вам нужно только базовое понимание хода программы и несколько минут на обдумывание.