«

[Japo]

The space is irrelevant. But didn't it work in the end (I'm lost)?

BTW Data, kudos! What language did you use? And Play, you were really lucky Data had the free time to spare.

[Playbahnosh]

Originally Posted by Japofran

The space is irrelevant. But didn't it work in the end (I'm lost)?

BTW Data, kudos! What language did you use? And Play, you were really lucky Data had the free time to spare.

Yeah, I'm pretty fortunate, thanks to Data, all is okay now.

And yes, it worked for Data but not for me, for some reason. I tend to be a perfectionist in matters like this, so I won't make the same mistake again, and even if it worked, I wanted to know why it didn't for me. But it's kinda irrelevant now, you are right :amused:

[Data]

the only thing I can think of is the line ends being different. I did convert them to unix line ends. It shouldn't matter as I compiled the executable for windows, but maybe that doesn't convert the line ends internally afteral, or maybe it needed a switch.

I used C++ to program it, it is a quick getting the job done piece of code.

Let me post it here. Don't bother with pointing out all C-isms and other things which I could have done better. I know them myself as well.

/* Copyright Data */

#include <string>
#include <iostream>

int main (int argc, char *argv[]) {
        std::string templine;
        std::string index, timeindex, text;
        std::string index2, timeindex2, text2;

        //Get initial record:
        getline (std::cin, index);
        getline (std::cin, timeindex);
        templine = "";
        do {
                getline (std::cin, templine);
                text += templine;
                text += "\n";
        }
        while (templine != "");

        while (std::cin) {
                getline (std::cin, index2);
                getline (std::cin, timeindex2);
                templine = "";
                do {
                        getline (std::cin, templine);
                        text2 += templine;
                        text2 += "\n";
                }
                while (templine != "");

                //writeout record:
                std::cout << index << std::endl;
                std::cout << timeindex << std::endl;
                std::cout << text2;

                //move up:
                index = index2;
                timeindex = timeindex2;
                text2 = "";
        }
        return 0;
}

[Playbahnosh]

Wow! Thanks for sharing. Maybe you could upload this to sourceforge or something, maybe other people have the same problem

[_r.u.s.s.]

whole sourceforge project dedicated to skipping subtitle positinos to the next line?

[Playbahnosh]

You really can't take a joke, do you?

But a program to correct every sorts of problems with subtitles won't be a too bad idea, now you mention it. Maybe someday, when I learn to program real good, I might do something like that.

[Japo]

Originally Posted by Data

Don't bother with pointing out all C-isms and other things

Don't worry about that, first I'm not fluent at programming since I use it so little, second C is what I grew up with and although I pushed myself to learn ++, I have used C++ not at all instead of little so it's not natural in me. (And actually see no comparative advantage for small one-person projects. It may be nice to use + instead of strcat() though. LOL And C isn't certainly obsolete nor redundant, I'm not the only one to think so, whereas C++ could very well be and sure is a messy language.)

Good job, I don't think there's much more ellegant code to accomplish the same.

The only thing I can think of is why you bother to declare the namespace once and again, instead of declaring once "using namespace std" at the start of the code. And why you declare main's arguments that you aren't using instead of declaring it (void)--or () which is the same. But maybe you have reasons. And sorry if I have just pointed what you know yourself after you warned me not to.

Very well done. And I bet the problem has the compiler to blame, it's up to it to make portable code portable as promised.

[Data]

Thank you for kind remarks.

I use using namespace std; quite often, but as I published the code I made a bit more robust for copy and paste work by including the correct namespace.

I tend to forget the possibility to omit the arguments of main.
Returning a value is good practice, especially if the utility can be used in shell scripts.

My Cism were (as far as I'm concerned)

templine = "";

instead of

templine.clear()

and

templine != ""

instead of

!templine.empty()

But the Cism are actually smaller and for me at least as intuitive as the C++ things. (long live the operator overloading )

[Japo]

Originally Posted by Data

But the Cism are actually smaller and for me at least as intuitive as the C++ things.

True, maybe people who have learnt to program oriented to objects from the start don't get how all those abstractions are both added power and added burden. You have to learn the object libraries. And in some cases a more low-level code is not more clumsy nor illegible and sometimes the contrary. That's why I think of objects as a tool rather than a philosophy for the whole of all projects; and they don't really show their full power but as an interface between different programmers. Using object orientation as a way to make learning programming easier just creates bad programmers in general.

Usually you can't be sure if you can really get away with "C-isms" and the like. Something may not be in the standard and yet your compiler may support it and others won't. (It's not the case and your code is robust.) And you're not supposed to know, because information hiding is a sought effect of object orientation.

As a matter of fact using an overloaded assignment operator with a string is not a C-ism at all... Actually the robust C code to clear a string or check if it's not empty is very brief and immediately recognizable:

*templine = '\0';

while ( *templine != '\0' );

(Actually I could write just 0 instead of '\0', couldn't I? Hm better safe than sorry.) Of course templine would here be a C string (pointer to char), not a string object. I can't do this stuff with objects and I'm not supposed to, their innards are "private".

But I'm rambling...

[Playbahnosh]

Originally Posted by Japofran

Usually you can't be sure if you can really get away with "C-isms" and the like. Something may not be in the standard and yet your compiler may support it and others won't.

Oh boy, don't get me started on that one.
I have to take C and C++ classes here on the university, and it is a constant pain in the aß, as to which compiler to use. For many years, the uni used Microsoft's Visual Studio for C programming, but the license expired, and they didn't renew it. They started using Eclipse because it's free, but the course's codes, books and learning material were optimized for Visual Studio, and a great many things, what worked in VS, didn't work in Eclipse and vice versa. Just think about it, you make a project for class in C or C++, you run it and works fine, you bring it in for class and it won't work there. Who's to blame for that? Ususally us, students.:notrust: